✅ [#296] Test new _zaak_url field

Author: SilviaAmAm

backend/src/openarchiefbeheer/destruction/tests/test_migrations.py

@@ -141,3 +141,38 @@ def test_zaak_url_repopulated(self):
         self.assertEqual(items[0].zaak_url, "http://zaken.nl/api/v1/zaken/111-111-111")
         self.assertEqual(items[1].zaak_url, "http://zaken.nl/api/v1/zaken/222-222-222")
         self.assertEqual(items[2].zaak_url, "http://zaken.nl/api/v1/zaken/333-333-333")
+
+
+class TestAddZaakUrlMigration(TestMigrations):
+    app = "destruction"
+    migrate_from = "0019_destructionlistitem__zaak_url"
+    migrate_to = "0020_auto_20240822_1113"
+
+    def setUpBeforeMigration(self, apps):
+        Zaak = apps.get_model("zaken", "Zaak")
+        DestructionListItem = apps.get_model("destruction", "DestructionListItem")
+        DestructionList = apps.get_model("destruction", "DestructionList")
+        User = apps.get_model("accounts", "User")
+
+        zaak1 = Zaak.objects.create(
+            uuid=str(uuid4()),
+            url="http://zaken.nl/api/v1/zaken/111-111-111",
+            startdatum=date(2000, 1, 1),
+            zaaktype="http://catalogue-api.nl/zaaktypen/111-111-111",
+            bronorganisatie="000000000",
+            verantwoordelijke_organisatie="000000000",
+        )
+        destruction_list = DestructionList.objects.create(
+            name="Test migration",
+            author=User.objects.create(username="recordmanager"),
+        )
+        self.item = DestructionListItem.objects.create(
+            zaak=zaak1,
+            status=ListItemStatus.suggested,
+            destruction_list_id=destruction_list.pk,
+        )
+
+    def test_added_zaak_url(self):
+        self.item.refresh_from_db()
+
+        self.assertEqual(self.item.zaak.url, self.item._zaak_url)

backend/src/openarchiefbeheer/destruction/tests/test_models.py

@@ -9,6 +9,7 @@
 
 from openarchiefbeheer.config.models import ArchiveConfig
 from openarchiefbeheer.zaken.models import Zaak
+from openarchiefbeheer.zaken.tests.factories import ZaakFactory
 
 from ..constants import InternalStatus, ListItemStatus
 from .factories import (
@@ -68,6 +69,21 @@ def test_process_deletion(self):
         with self.assertRaises(ObjectDoesNotExist):
             Zaak.objects.get(url="http://zaken.nl/api/v1/zaken/111-111-111")
 
+    def test_keeping_zaak_url_in_sync(self):
+        zaak = ZaakFactory.create(url="http://zaken.nl/1")
+        item = DestructionListItemFactory.create(
+            with_zaak=True, zaak__url="http://zaken.nl/2"
+        )
+
+        self.assertEqual(item._zaak_url, "http://zaken.nl/2")
+
+        item.zaak = zaak
+        item.save()
+
+        item.refresh_from_db()
+
+        self.assertEqual(item._zaak_url, "http://zaken.nl/1")
+
 
 class ReviewResponseTests(TestCase):
     def test_derive_status(self):