From bfd7486c1d399c38e96094a80627ed39b34a1ebc Mon Sep 17 00:00:00 2001 From: Daniel Mursa Date: Tue, 11 Feb 2025 16:22:08 +0100 Subject: [PATCH] [#309] Create migrations tests --- .../contactgegevens/tests/test_migrations.py | 62 ++++++++++ .../klantinteracties/tests/test_migrations.py | 108 ++++++++++++++++++ src/openklant/fixtures/contactgegevens.json | 10 ++ src/openklant/fixtures/klantinteracties.json | 60 ++++++++++ 4 files changed, 240 insertions(+) diff --git a/src/openklant/components/contactgegevens/tests/test_migrations.py b/src/openklant/components/contactgegevens/tests/test_migrations.py index 379cef4d..bd549a61 100644 --- a/src/openklant/components/contactgegevens/tests/test_migrations.py +++ b/src/openklant/components/contactgegevens/tests/test_migrations.py @@ -332,3 +332,65 @@ def test_ko_migration_persoon_model_wrong_code(self): self.assertEqual(records.count(), 1) self.assertEqual(records[0].adres_nummeraanduiding_id, "1234567890000001") self.assertNotEqual(records[0].adres_nummeraanduiding_id, "123456") + + +class TestNewAadresFields(BaseMigrationTest): + app = "contactgegevens" + migrate_from = "0005_alter_organisatie_adres_nummeraanduiding_id_and_more" + migrate_to = "0006_organisatie_adres_huisnummer_and_more" + + def test_ok_migration_organisatie_model(self): + + Organisatie = self.old_app_state.get_model("contactgegevens", "Organisatie") + org = Organisatie.objects.create(adres_nummeraanduiding_id="1234567890000001") + self.assertFalse(hasattr(org, "adres_straatnaam")) + self.assertFalse(hasattr(org, "adres_huisnummer")) + self.assertFalse(hasattr(org, "adres_huisnummertoevoeging")) + self.assertFalse(hasattr(org, "adres_postcode")) + self.assertFalse(hasattr(org, "adres_stad")) + + self._perform_migration() + + Organisatie = self.apps.get_model("contactgegevens", "Organisatie") + org = Organisatie.objects.get() + + self.assertTrue(hasattr(org, "adres_straatnaam")) + self.assertTrue(hasattr(org, "adres_huisnummer")) + self.assertTrue(hasattr(org, "adres_huisnummertoevoeging")) + self.assertTrue(hasattr(org, "adres_postcode")) + self.assertTrue(hasattr(org, "adres_stad")) + self.assertEqual(org.adres_straatnaam, "") + self.assertEqual(org.adres_huisnummer, "") + self.assertEqual(org.adres_huisnummertoevoeging, "") + self.assertEqual(org.adres_postcode, "") + self.assertEqual(org.adres_stad, "") + + def test_ok_migration_persoon_model(self): + + Persoon = self.old_app_state.get_model("contactgegevens", "Persoon") + + persoon = Persoon.objects.create( + adres_nummeraanduiding_id="1234567890000001", geboortedatum="1980-02-23" + ) + + self.assertFalse(hasattr(persoon, "adres_straatnaam")) + self.assertFalse(hasattr(persoon, "adres_huisnummer")) + self.assertFalse(hasattr(persoon, "adres_huisnummertoevoeging")) + self.assertFalse(hasattr(persoon, "adres_postcode")) + self.assertFalse(hasattr(persoon, "adres_stad")) + + self._perform_migration() + Persoon = self.apps.get_model("contactgegevens", "Persoon") + + persoon = Persoon.objects.get() + + self.assertTrue(hasattr(persoon, "adres_straatnaam")) + self.assertTrue(hasattr(persoon, "adres_huisnummer")) + self.assertTrue(hasattr(persoon, "adres_huisnummertoevoeging")) + self.assertTrue(hasattr(persoon, "adres_postcode")) + self.assertTrue(hasattr(persoon, "adres_stad")) + self.assertEqual(persoon.adres_straatnaam, "") + self.assertEqual(persoon.adres_huisnummer, "") + self.assertEqual(persoon.adres_huisnummertoevoeging, "") + self.assertEqual(persoon.adres_postcode, "") + self.assertEqual(persoon.adres_stad, "") diff --git a/src/openklant/components/klantinteracties/tests/test_migrations.py b/src/openklant/components/klantinteracties/tests/test_migrations.py index 0db64006..b31df416 100644 --- a/src/openklant/components/klantinteracties/tests/test_migrations.py +++ b/src/openklant/components/klantinteracties/tests/test_migrations.py @@ -437,3 +437,111 @@ def test_ko_migration_partij_model_wrong_code(self): ) self.assertNotEqual(records[0].bezoekadres_nummeraanduiding_id, "ABC") self.assertNotEqual(records[0].correspondentieadres_nummeraanduiding_id, "DEF") + + +class TestNewAadresFields(BaseMigrationTest): + app = "klantinteracties" + migrate_from = "0027_alter_betrokkene_bezoekadres_nummeraanduiding_id_and_more" + migrate_to = "0028_betrokkene_bezoekadres_huisnummer_and_more" + + def test_ok_migration_betrokkene_model(self): + + Betrokkene = self.old_app_state.get_model("klantinteracties", "Betrokkene") + Klantcontact = self.old_app_state.get_model("klantinteracties", "Klantcontact") + + betrokkene = Betrokkene.objects.create( + partij=None, + klantcontact=Klantcontact.objects.create(vertrouwelijk=False), + initiator=False, + ) + self.assertFalse(hasattr(betrokkene, "bezoekadres_straatnaam")) + self.assertFalse(hasattr(betrokkene, "bezoekadres_huisnummer")) + self.assertFalse(hasattr(betrokkene, "bezoekadres_huisnummertoevoeging")) + self.assertFalse(hasattr(betrokkene, "bezoekadres_postcode")) + self.assertFalse(hasattr(betrokkene, "bezoekadres_stad")) + self.assertFalse(hasattr(betrokkene, "correspondentieadres_straatnaam")) + self.assertFalse(hasattr(betrokkene, "correspondentieadres_huisnummer")) + self.assertFalse( + hasattr(betrokkene, "correspondentieadres_huisnummertoevoeging") + ) + self.assertFalse(hasattr(betrokkene, "correspondentieadres_postcode")) + self.assertFalse(hasattr(betrokkene, "correspondentieadres_stad")) + + self._perform_migration() + + Betrokkene = self.apps.get_model("klantinteracties", "Betrokkene") + + betrokkene = Betrokkene.objects.get() + + self.assertTrue(hasattr(betrokkene, "bezoekadres_straatnaam")) + self.assertTrue(hasattr(betrokkene, "bezoekadres_huisnummer")) + self.assertTrue(hasattr(betrokkene, "bezoekadres_huisnummertoevoeging")) + self.assertTrue(hasattr(betrokkene, "bezoekadres_postcode")) + self.assertTrue(hasattr(betrokkene, "bezoekadres_stad")) + self.assertTrue(hasattr(betrokkene, "correspondentieadres_straatnaam")) + self.assertTrue(hasattr(betrokkene, "correspondentieadres_huisnummer")) + self.assertTrue( + hasattr(betrokkene, "correspondentieadres_huisnummertoevoeging") + ) + self.assertTrue(hasattr(betrokkene, "correspondentieadres_postcode")) + self.assertTrue(hasattr(betrokkene, "correspondentieadres_stad")) + + self.assertEqual(betrokkene.bezoekadres_straatnaam, "") + self.assertEqual(betrokkene.bezoekadres_huisnummer, "") + self.assertEqual(betrokkene.bezoekadres_huisnummertoevoeging, "") + self.assertEqual(betrokkene.bezoekadres_postcode, "") + self.assertEqual(betrokkene.bezoekadres_stad, "") + self.assertEqual(betrokkene.correspondentieadres_straatnaam, "") + self.assertEqual(betrokkene.correspondentieadres_huisnummer, "") + self.assertEqual(betrokkene.correspondentieadres_huisnummertoevoeging, "") + self.assertEqual(betrokkene.correspondentieadres_postcode, "") + self.assertEqual(betrokkene.correspondentieadres_stad, "") + + def test_ok_migration_partij_model(self): + + Partij = self.old_app_state.get_model("klantinteracties", "Partij") + + partij = Partij.objects.create( + indicatie_actief=True, + bezoekadres_nummeraanduiding_id="ABC", + correspondentieadres_nummeraanduiding_id="DEF", + ) + self.assertFalse(hasattr(partij, "bezoekadres_straatnaam")) + self.assertFalse(hasattr(partij, "bezoekadres_huisnummer")) + self.assertFalse(hasattr(partij, "bezoekadres_huisnummertoevoeging")) + self.assertFalse(hasattr(partij, "bezoekadres_postcode")) + self.assertFalse(hasattr(partij, "bezoekadres_stad")) + + self.assertFalse(hasattr(partij, "correspondentieadres_straatnaam")) + self.assertFalse(hasattr(partij, "correspondentieadres_huisnummer")) + self.assertFalse(hasattr(partij, "correspondentieadres_huisnummertoevoeging")) + self.assertFalse(hasattr(partij, "correspondentieadres_postcode")) + self.assertFalse(hasattr(partij, "correspondentieadres_stad")) + + self._perform_migration() + + Partij = self.apps.get_model("klantinteracties", "Partij") + + partij = Partij.objects.get() + self.assertTrue(hasattr(partij, "bezoekadres_straatnaam")) + self.assertTrue(hasattr(partij, "bezoekadres_huisnummer")) + self.assertTrue(hasattr(partij, "bezoekadres_huisnummertoevoeging")) + self.assertTrue(hasattr(partij, "bezoekadres_postcode")) + self.assertTrue(hasattr(partij, "bezoekadres_stad")) + + self.assertTrue(hasattr(partij, "correspondentieadres_straatnaam")) + self.assertTrue(hasattr(partij, "correspondentieadres_huisnummer")) + self.assertTrue(hasattr(partij, "correspondentieadres_huisnummertoevoeging")) + self.assertTrue(hasattr(partij, "correspondentieadres_postcode")) + self.assertTrue(hasattr(partij, "correspondentieadres_stad")) + + self.assertEqual(partij.bezoekadres_straatnaam, "") + self.assertEqual(partij.bezoekadres_huisnummer, "") + self.assertEqual(partij.bezoekadres_huisnummertoevoeging, "") + self.assertEqual(partij.bezoekadres_postcode, "") + self.assertEqual(partij.bezoekadres_stad, "") + self.assertEqual(partij.correspondentieadres_straatnaam, "") + self.assertEqual(partij.correspondentieadres_huisnummer, "") + self.assertEqual(partij.correspondentieadres_huisnummertoevoeging, "") + self.assertEqual(partij.correspondentieadres_postcode, "") + self.assertEqual(partij.correspondentieadres_stad, "") diff --git a/src/openklant/fixtures/contactgegevens.json b/src/openklant/fixtures/contactgegevens.json index a2833fcd..fee5afd9 100644 --- a/src/openklant/fixtures/contactgegevens.json +++ b/src/openklant/fixtures/contactgegevens.json @@ -4,6 +4,11 @@ "pk": 1, "fields": { "adres_nummeraanduiding_id": "1234567890000001", + "adres_straatnaam":"", + "adres_huisnummer":"", + "adres_huisnummertoevoeging":"", + "adres_postcode":"", + "adres_stad":"", "adres_adresregel1": "Keizersgracht 117", "adres_adresregel2": "1015 CJ Amsterdam", "adres_adresregel3": "Noord-Holland", @@ -20,6 +25,11 @@ "pk": 1, "fields": { "adres_nummeraanduiding_id": "1234567890000002", + "adres_straatnaam":"", + "adres_huisnummer":"", + "adres_huisnummertoevoeging":"", + "adres_postcode":"", + "adres_stad":"", "adres_adresregel1": "Keizersgracht 117", "adres_adresregel2": "1015 CJ Amsterdam", "adres_adresregel3": "Noord-Holland", diff --git a/src/openklant/fixtures/klantinteracties.json b/src/openklant/fixtures/klantinteracties.json index 8c0b7be7..cc13cb78 100644 --- a/src/openklant/fixtures/klantinteracties.json +++ b/src/openklant/fixtures/klantinteracties.json @@ -100,11 +100,21 @@ "pk": 1, "fields": { "bezoekadres_nummeraanduiding_id": "", + "bezoekadres_straatnaam":"", + "bezoekadres_huisnummer":"", + "bezoekadres_huisnummertoevoeging":"", + "bezoekadres_postcode":"", + "bezoekadres_stad":"", "bezoekadres_adresregel1": "", "bezoekadres_adresregel2": "", "bezoekadres_adresregel3": "", "bezoekadres_land": "", "correspondentieadres_nummeraanduiding_id": "", + "correspondentieadres_straatnaam":"", + "correspondentieadres_huisnummer":"", + "correspondentieadres_huisnummertoevoeging":"", + "correspondentieadres_postcode":"", + "correspondentieadres_stad":"", "correspondentieadres_adresregel1": "", "correspondentieadres_adresregel2": "", "correspondentieadres_adresregel3": "", @@ -126,11 +136,21 @@ "pk": 2, "fields": { "bezoekadres_nummeraanduiding_id": "", + "bezoekadres_straatnaam":"", + "bezoekadres_huisnummer":"", + "bezoekadres_huisnummertoevoeging":"", + "bezoekadres_postcode":"", + "bezoekadres_stad":"", "bezoekadres_adresregel1": "", "bezoekadres_adresregel2": "", "bezoekadres_adresregel3": "", "bezoekadres_land": "", "correspondentieadres_nummeraanduiding_id": "", + "correspondentieadres_straatnaam":"", + "correspondentieadres_huisnummer":"", + "correspondentieadres_huisnummertoevoeging":"", + "correspondentieadres_postcode":"", + "correspondentieadres_stad":"", "correspondentieadres_adresregel1": "", "correspondentieadres_adresregel2": "", "correspondentieadres_adresregel3": "", @@ -152,11 +172,21 @@ "pk": 3, "fields": { "bezoekadres_nummeraanduiding_id": "1234567890000001", + "bezoekadres_straatnaam":"", + "bezoekadres_huisnummer":"", + "bezoekadres_huisnummertoevoeging":"", + "bezoekadres_postcode":"", + "bezoekadres_stad":"", "bezoekadres_adresregel1": "Keizersgracht 117", "bezoekadres_adresregel2": "1015 CJ Amsterdam", "bezoekadres_adresregel3": "", "bezoekadres_land": "NL", "correspondentieadres_nummeraanduiding_id": "", + "correspondentieadres_straatnaam":"", + "correspondentieadres_huisnummer":"", + "correspondentieadres_huisnummertoevoeging":"", + "correspondentieadres_postcode":"", + "correspondentieadres_stad":"", "correspondentieadres_adresregel1": "", "correspondentieadres_adresregel2": "", "correspondentieadres_adresregel3": "", @@ -203,11 +233,21 @@ "pk": 1, "fields": { "bezoekadres_nummeraanduiding_id": "", + "bezoekadres_straatnaam":"", + "bezoekadres_huisnummer":"", + "bezoekadres_huisnummertoevoeging":"", + "bezoekadres_postcode":"", + "bezoekadres_stad":"", "bezoekadres_adresregel1": "", "bezoekadres_adresregel2": "", "bezoekadres_adresregel3": "", "bezoekadres_land": "", "correspondentieadres_nummeraanduiding_id": "", + "correspondentieadres_straatnaam":"", + "correspondentieadres_huisnummer":"", + "correspondentieadres_huisnummertoevoeging":"", + "correspondentieadres_postcode":"", + "correspondentieadres_stad":"", "correspondentieadres_adresregel1": "", "correspondentieadres_adresregel2": "", "correspondentieadres_adresregel3": "", @@ -228,11 +268,21 @@ "pk": 2, "fields": { "bezoekadres_nummeraanduiding_id": "1234567890000002", + "bezoekadres_straatnaam":"", + "bezoekadres_huisnummer":"", + "bezoekadres_huisnummertoevoeging":"", + "bezoekadres_postcode":"", + "bezoekadres_stad":"", "bezoekadres_adresregel1": "Keizersgracht 117", "bezoekadres_adresregel2": "1015 CJ Amsterdam", "bezoekadres_adresregel3": "", "bezoekadres_land": "NL", "correspondentieadres_nummeraanduiding_id": "", + "correspondentieadres_straatnaam":"", + "correspondentieadres_huisnummer":"", + "correspondentieadres_huisnummertoevoeging":"", + "correspondentieadres_postcode":"", + "correspondentieadres_stad":"", "correspondentieadres_adresregel1": "", "correspondentieadres_adresregel2": "", "correspondentieadres_adresregel3": "", @@ -253,11 +303,21 @@ "pk": 3, "fields": { "bezoekadres_nummeraanduiding_id": "", + "bezoekadres_straatnaam":"", + "bezoekadres_huisnummer":"", + "bezoekadres_huisnummertoevoeging":"", + "bezoekadres_postcode":"", + "bezoekadres_stad":"", "bezoekadres_adresregel1": "", "bezoekadres_adresregel2": "", "bezoekadres_adresregel3": "", "bezoekadres_land": "", "correspondentieadres_nummeraanduiding_id": "", + "correspondentieadres_straatnaam":"", + "correspondentieadres_huisnummer":"", + "correspondentieadres_huisnummertoevoeging":"", + "correspondentieadres_postcode":"", + "correspondentieadres_stad":"", "correspondentieadres_adresregel1": "", "correspondentieadres_adresregel2": "", "correspondentieadres_adresregel3": "",