✨ [#120] added expand detail functionality

Author: bart-maykin

src/openklant/components/klantinteracties/api/tests/test_expand.py

@@ -41,7 +41,7 @@ def setUp(self):
             klantcontact=self.klantcontact, partij=self.partij
         )
 
-    def test_sinle_expansion(self):
+    def test_list_single_expansion(self):
         list_url = reverse("klantinteracties:klantcontact-list")
         response = self.client.get(list_url, {"expand": "had_betrokkenen"})
 
@@ -91,7 +91,7 @@ def test_sinle_expansion(self):
         self.assertEqual(expand["initiator"], self.betrokkene.initiator)
         self.assertEqual(expand["wasPartij"]["uuid"], str(self.partij.uuid))
 
-    def test_multiple_level_expansion(self):
+    def test_list_multiple_level_expansion(self):
         list_url = reverse("klantinteracties:klantcontact-list")
         response = self.client.get(
             list_url, {"expand": "had_betrokkenen,had_betrokkenen.was_partij"}
@@ -179,3 +179,140 @@ def test_multiple_level_expansion(self):
                 "land": self.partij.correspondentieadres_land,
             },
         )
+
+    def test_detail_single_expansion(self):
+        detail_url = reverse(
+            "klantinteracties:klantcontact-detail",
+            kwargs={"uuid": str(self.klantcontact.uuid)},
+        )
+        response = self.client.get(detail_url, {"expand": "had_betrokkenen"})
+
+        self.assertEqual(response.status_code, status.HTTP_200_OK)
+
+        data = response.json()
+
+        self.assertEqual(data["hadBetrokkenen"][0]["uuid"], str(self.betrokkene.uuid))
+        self.assertTrue(data["_expand"])
+        expand = data["_expand"]["hadBetrokkenen"]
+
+        self.assertEqual(
+            expand["bezoekadres"],
+            {
+                "nummeraanduidingId": self.betrokkene.bezoekadres_nummeraanduiding_id,
+                "adresregel1": self.betrokkene.bezoekadres_adresregel1,
+                "adresregel2": self.betrokkene.bezoekadres_adresregel2,
+                "adresregel3": self.betrokkene.bezoekadres_adresregel3,
+                "land": self.betrokkene.bezoekadres_land,
+            },
+        )
+        self.assertEqual(
+            expand["correspondentieadres"],
+            {
+                "nummeraanduidingId": self.betrokkene.correspondentieadres_nummeraanduiding_id,
+                "adresregel1": self.betrokkene.correspondentieadres_adresregel1,
+                "adresregel2": self.betrokkene.correspondentieadres_adresregel2,
+                "adresregel3": self.betrokkene.correspondentieadres_adresregel3,
+                "land": self.betrokkene.correspondentieadres_land,
+            },
+        )
+        self.assertEqual(
+            expand["contactnaam"],
+            {
+                "voorletters": self.betrokkene.contactnaam_voorletters,
+                "voornaam": self.betrokkene.contactnaam_voornaam,
+                "voorvoegselAchternaam": self.betrokkene.contactnaam_voorvoegsel_achternaam,
+                "achternaam": self.betrokkene.contactnaam_achternaam,
+            },
+        )
+        self.assertEqual(expand["rol"], self.betrokkene.rol)
+        self.assertEqual(expand["organisatienaam"], self.betrokkene.organisatienaam)
+        self.assertEqual(expand["initiator"], self.betrokkene.initiator)
+        self.assertEqual(expand["wasPartij"]["uuid"], str(self.partij.uuid))
+
+    def test_detail_multiple_level_expansion(self):
+        detail_url = reverse(
+            "klantinteracties:klantcontact-detail",
+            kwargs={"uuid": str(self.klantcontact.uuid)},
+        )
+        response = self.client.get(
+            detail_url, {"expand": "had_betrokkenen,had_betrokkenen.was_partij"}
+        )
+
+        self.assertEqual(response.status_code, status.HTTP_200_OK)
+
+        data = response.json()
+
+        self.assertEqual(data["hadBetrokkenen"][0]["uuid"], str(self.betrokkene.uuid))
+        self.assertTrue(data["_expand"])
+        expand = data["_expand"]["hadBetrokkenen"]
+
+        self.assertEqual(
+            expand["bezoekadres"],
+            {
+                "nummeraanduidingId": self.betrokkene.bezoekadres_nummeraanduiding_id,
+                "adresregel1": self.betrokkene.bezoekadres_adresregel1,
+                "adresregel2": self.betrokkene.bezoekadres_adresregel2,
+                "adresregel3": self.betrokkene.bezoekadres_adresregel3,
+                "land": self.betrokkene.bezoekadres_land,
+            },
+        )
+        self.assertEqual(
+            expand["correspondentieadres"],
+            {
+                "nummeraanduidingId": self.betrokkene.correspondentieadres_nummeraanduiding_id,
+                "adresregel1": self.betrokkene.correspondentieadres_adresregel1,
+                "adresregel2": self.betrokkene.correspondentieadres_adresregel2,
+                "adresregel3": self.betrokkene.correspondentieadres_adresregel3,
+                "land": self.betrokkene.correspondentieadres_land,
+            },
+        )
+        self.assertEqual(
+            expand["contactnaam"],
+            {
+                "voorletters": self.betrokkene.contactnaam_voorletters,
+                "voornaam": self.betrokkene.contactnaam_voornaam,
+                "voorvoegselAchternaam": self.betrokkene.contactnaam_voorvoegsel_achternaam,
+                "achternaam": self.betrokkene.contactnaam_achternaam,
+            },
+        )
+        self.assertEqual(expand["rol"], self.betrokkene.rol)
+        self.assertEqual(expand["organisatienaam"], self.betrokkene.organisatienaam)
+        self.assertEqual(expand["initiator"], self.betrokkene.initiator)
+        self.assertEqual(expand["wasPartij"]["uuid"], str(self.partij.uuid))
+        self.assertTrue(expand["_expand"])
+
+        # second expand
+        expand = expand["_expand"]["wasPartij"]
+
+        self.assertEqual(expand["nummer"], self.partij.nummer)
+        self.assertEqual(expand["interneNotitie"], self.partij.interne_notitie)
+        self.assertEqual(
+            expand["digitaleAdressen"][0]["uuid"], str(self.digitaal_adres.uuid)
+        )
+        self.assertEqual(expand["vertegenwoordigde"], [])
+        self.assertEqual(expand["soortPartij"], self.partij.soort_partij)
+        self.assertEqual(
+            expand["indicatieGeheimhouding"], self.partij.indicatie_geheimhouding
+        )
+        self.assertEqual(expand["voorkeurstaal"], self.partij.voorkeurstaal)
+        self.assertEqual(expand["indicatieActief"], self.partij.indicatie_actief)
+        self.assertEqual(
+            expand["bezoekadres"],
+            {
+                "nummeraanduidingId": self.partij.bezoekadres_nummeraanduiding_id,
+                "adresregel1": self.partij.bezoekadres_adresregel1,
+                "adresregel2": self.partij.bezoekadres_adresregel2,
+                "adresregel3": self.partij.bezoekadres_adresregel3,
+                "land": self.partij.bezoekadres_land,
+            },
+        )
+        self.assertEqual(
+            expand["correspondentieadres"],
+            {
+                "nummeraanduidingId": self.partij.correspondentieadres_nummeraanduiding_id,
+                "adresregel1": self.partij.correspondentieadres_adresregel1,
+                "adresregel2": self.partij.correspondentieadres_adresregel2,
+                "adresregel3": self.partij.correspondentieadres_adresregel3,
+                "land": self.partij.correspondentieadres_land,
+            },
+        )

src/openklant/components/utils/mixins.py

@@ -36,7 +36,7 @@ class ExpandMixin:
     expand_param = "expand"
 
     def include_allowed(self):
-        return self.action in ["list"]
+        return self.action in ["list", "retrieve"]
 
     def get_requested_inclusions(self, request):
         # Pull expand parameter from request body in case of _zoek operation