[IMP]l10n_es_facturae_face: add unidad tramitadora and oficina contab…

…le checks

l10n_es_facturae_face/models/account_move.py

@@ -87,6 +87,19 @@ def _get_l10n_es_facturae_excluded_status(self):
 
     def validate_facturae_fields(self):
         super().validate_facturae_fields()
-        if not self.partner_id.organo_gestor:
+        if (
+            self.partner_id.l10n_es_facturae_sending_code == "face"
+            and not self.partner_id.organo_gestor
+        ):
             raise ValidationError(_("Organo Gestor not provided"))
+        if (
+            self.partner_id.l10n_es_facturae_sending_code == "face"
+            and not self.partner_id.unidad_tramitadora
+        ):
+            raise ValidationError(_("Unidad Tramitadora not provided"))
+        if (
+            self.partner_id.l10n_es_facturae_sending_code == "face"
+            and not self.partner_id.oficina_contable
+        ):
+            raise ValidationError(_("Oficina Contable not provided"))
         return

l10n_es_facturae_face/tests/test_facturae_face.py

@@ -227,6 +227,32 @@ def test_create_facturae_file_without_organo_gestor(self):
         with self.assertRaises(exceptions.ValidationError):
             wizard.create_facturae_file()
 
+    def test_create_facturae_file_without_unidad_tramitadora(self):
+        self._activate_certificate(self.certificate_password)
+        self.move.action_post()
+        self.move.name = "2999/99999"
+        wizard = (
+            self.env["create.facturae"]
+            .with_context(active_ids=self.move.ids, active_model="account.move")
+            .create({})
+        )
+        self.partner.unidad_tramitadora = False
+        with self.assertRaises(exceptions.ValidationError):
+            wizard.create_facturae_file()
+
+    def test_create_facturae_file_without_oficina_contable(self):
+        self._activate_certificate(self.certificate_password)
+        self.move.action_post()
+        self.move.name = "2999/99999"
+        wizard = (
+            self.env["create.facturae"]
+            .with_context(active_ids=self.move.ids, active_model="account.move")
+            .create({})
+        )
+        self.partner.oficina_contable = False
+        with self.assertRaises(exceptions.ValidationError):
+            wizard.create_facturae_file()
+
     def test_facturae_face_0(self):
         class DemoService:
             def __init__(self, value):