[WIP] parametrize OK2 params

maykinmedia · Oct 9, 2024 · c5f4db7 · c5f4db7
1 parent 7f883b0
commit c5f4db7
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 9 deletions.
diff --git a/src/open_inwoner/openklant/services.py b/src/open_inwoner/openklant/services.py
@@ -146,8 +146,16 @@ def from_klantcontact_and_answer(
 class OpenKlant2Service:
 
     client: OpenKlant2Client
+
+    # TODO: When we hook up this service to the main application flows, these
+    # constants should be turned into configurable parameters.
     mijn_vragen_actor: uuid.UUID | None
-    MIJN_VRAGEN_KANAAL: str = "oip_mijn_vragen"
+    VRAGEN_DEFAULTS = {
+        "kanaal": "oip_mijn_vragen",
+        "oip_organisatie_naam": "Open Inwoner Platform",
+        "interne_taak_gevraagde_handeling": "Beantwoorden vraag Mijn Omgeving",
+        "interne_taak_toelichting": "Beantwoorden vraag",
+    }
 
     def __init__(
         self, client: OpenKlant2Client, mijn_vragen_actor: str | uuid.UUID | None = None
@@ -466,7 +474,7 @@ def create_question(
                 "inhoud": question,
                 "onderwerp": subject,
                 "taal": "nld",
-                "kanaal": self.MIJN_VRAGEN_KANAAL,
+                "kanaal": self.VRAGEN_DEFAULTS["kanaal"],
                 "vertrouwelijk": False,
                 "plaatsgevondenOp": timezone.now().isoformat(),
             }
@@ -479,16 +487,18 @@ def create_question(
                 "hadKlantcontact": {"uuid": klantcontact["uuid"]},
                 "initiator": True,
                 "wasPartij": {"uuid": partij["uuid"]},
-                "organisatienaam": "Open Inwoner Platform",
+                "organisatienaam": self.VRAGEN_DEFAULTS["oip_organisatie_naam"],
             }
         )
         logger.info("Created betrokkene: %s", betrokkene["uuid"])
 
         taak = self.client.interne_taak.create(
             data={
                 "aanleidinggevendKlantcontact": {"uuid": klantcontact["uuid"]},
-                "toelichting": "Beantwoorden vraag",
-                "gevraagdeHandeling": "Vraag beantwoorden in aanleiding gevend klant contact",
+                "toelichting": self.VRAGEN_DEFAULTS["interne_taak_toelichting"],
+                "gevraagdeHandeling": self.VRAGEN_DEFAULTS[
+                    "interne_taak_gevraagde_handeling"
+                ],
                 "status": "te_verwerken",
                 "toegewezenAanActor": {"uuid": str(self.mijn_vragen_actor)},
             }
@@ -513,7 +523,7 @@ def create_answer(
                 "inhoud": answer,
                 "onderwerp": question_klantcontact["onderwerp"],
                 "taal": "nld",
-                "kanaal": self.MIJN_VRAGEN_KANAAL,
+                "kanaal": self.VRAGEN_DEFAULTS["kanaal"],
                 "vertrouwelijk": False,
                 "plaatsgevondenOp": timezone.now().isoformat(),
             }
@@ -525,7 +535,7 @@ def create_answer(
                 "hadKlantcontact": {"uuid": answer_klantcontact["uuid"]},
                 "initiator": True,
                 "wasPartij": {"uuid": partij["uuid"]},
-                "organisatienaam": "Open Inwoner Platform",
+                "organisatienaam": self.VRAGEN_DEFAULTS["oip_organisatie_naam"],
             }
         )
 
@@ -555,7 +565,7 @@ def klantcontacten_for_partij(self, partij: Partij) -> Iterable[KlantContact]:
                     "hadBetrokkenen",
                     "hadBetrokkenen.wasPartij",
                 ],
-                "kanaal": self.MIJN_VRAGEN_KANAAL,
+                "kanaal": self.VRAGEN_DEFAULTS["kanaal"],
             }
         )
 

diff --git a/src/open_inwoner/openklant/tests/test_openklant2_service.py b/src/open_inwoner/openklant/tests/test_openklant2_service.py
@@ -350,7 +350,7 @@ def test_create_question(self):
         (betrokkene,) = self.service.client.betrokkene.list_iter()
         (taak,) = self.service.client.interne_taak.list_iter()
 
-        self.assertEqual(klantcontact["kanaal"], self.service.MIJN_VRAGEN_KANAAL)
+        self.assertEqual(klantcontact["kanaal"], self.service.VRAGEN_DEFAULTS["kanaal"])
         self.assertEqual(betrokkene["hadKlantcontact"]["uuid"], klantcontact["uuid"])
         self.assertEqual(betrokkene["wasPartij"]["uuid"], self.een_persoon["uuid"])
         self.assertEqual(