✅ [#296] Test resync behaviour

Author: SilviaAmAm

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

@@ -228,6 +228,8 @@ def test_process_list(self):
         self.assertEqual(destruction_list.status, ListStatus.deleted)
         self.assertEqual(item1.processing_status, InternalStatus.succeeded)
         self.assertEqual(item2.processing_status, InternalStatus.succeeded)
+        self.assertEqual(item1._zaak_url, "")
+        self.assertEqual(item2._zaak_url, "")
 
         self.assertFalse(
             Zaak.objects.filter(

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

@@ -1,10 +1,15 @@
 from django.test import TestCase
 
 from openarchiefbeheer.accounts.tests.factories import UserFactory
+from openarchiefbeheer.zaken.tests.factories import ZaakFactory
 
 from ..constants import ListRole
-from ..utils import process_new_assignees
-from .factories import DestructionListAssigneeFactory, DestructionListFactory
+from ..utils import process_new_assignees, resync_items_and_zaken
+from .factories import (
+    DestructionListAssigneeFactory,
+    DestructionListFactory,
+    DestructionListItemFactory,
+)
 
 
 class UtilsTest(TestCase):
@@ -29,3 +34,34 @@ def test_process_assignees(self):
         self.assertEqual(2, new_assignees.count())
         self.assertEqual(users[0].pk, new_assignees[0].user.pk)
         self.assertEqual(users[1].pk, new_assignees[1].user.pk)
+
+    def test_resync_zaken(self):
+        ZaakFactory.create(url="http://zaken.nl/1")
+        ZaakFactory.create(url="http://zaken.nl/2")
+        item1 = DestructionListItemFactory.create(_zaak_url="http://zaken.nl/1")
+        item2 = DestructionListItemFactory.create(_zaak_url="http://zaken.nl/2")
+
+        resync_items_and_zaken()
+
+        item1.refresh_from_db()
+        item2.refresh_from_db()
+
+        self.assertIsNotNone(item1.zaak)
+        self.assertEqual(item1.zaak.url, "http://zaken.nl/1")
+        self.assertIsNotNone(item2.zaak)
+        self.assertEqual(item2.zaak.url, "http://zaken.nl/2")
+
+    def test_resync_zaken_missing(self):
+        ZaakFactory.create(url="http://zaken.nl/1")
+        ZaakFactory.create(url="http://zaken.nl/2")
+        item1 = DestructionListItemFactory.create(_zaak_url="http://zaken.nl/1")
+        item2 = DestructionListItemFactory.create(_zaak_url="http://zaken.nl/3")
+
+        resync_items_and_zaken()
+
+        item1.refresh_from_db()
+        item2.refresh_from_db()
+
+        self.assertIsNotNone(item1.zaak)
+        self.assertEqual(item1.zaak.url, "http://zaken.nl/1")
+        self.assertIsNone(item2.zaak)

backend/src/openarchiefbeheer/zaken/tests/test_tasks.py

@@ -10,7 +10,7 @@
 from openarchiefbeheer.destruction.tests.factories import DestructionListItemFactory
 
 from ..models import Zaak
-from ..tasks import retrieve_and_cache_zaken_from_openzaak
+from ..tasks import resync_zaken, retrieve_and_cache_zaken_from_openzaak
 from ..utils import get_procestype
 from .factories import ZaakFactory
 
@@ -444,7 +444,7 @@ def test_expand_no_selectielijst_service(self, m):
         )
 
     @tag("gh-296")
-    def test_recaching_zaken_does_not_break_destruction_list(self, m):
+    def test_resyncing_zaken_does_not_break_destruction_list(self, m):
         item = DestructionListItemFactory.create(
             with_zaak=True,
             zaak__url="http://zaken-api.nl/zaken/api/v1/zaken/75f4c682-1e16-45ea-8f78-99b4474986ac",
@@ -479,7 +479,7 @@ def test_recaching_zaken_does_not_break_destruction_list(self, m):
             },
         )
 
-        retrieve_and_cache_zaken_from_openzaak()
+        resync_zaken()
 
         item.refresh_from_db()