feat: or-2006 make kbo numbers unique in tests; fix vlk;

Informatievlaanderen · Dec 19, 2023 · c4219fd · c4219fd
1 parent e6ff56e
commit c4219fd
Show file tree

Hide file tree

Showing 15 changed files with 75 additions and 62 deletions.
diff --git a/...rios/EventsInDb/V029_VerenigingMetRechtspersoonlijkheidWerdGeregistreerd_With_All_Data.cs b/...rios/EventsInDb/V029_VerenigingMetRechtspersoonlijkheidWerdGeregistreerd_With_All_Data.cs
@@ -31,7 +31,7 @@ public V029_VerenigingMetRechtspersoonlijkheidWerdGeregistreerd_With_All_Data()
             VCode = VCode,
             Naam = Naam,
             KorteNaam = "RDZ",
-            KboNummer = "7981199887",
+            KboNummer = "7981199829",
             Rechtsvorm = Verenigingstype.VZW.Code,
         };
 

diff --git a/...EventsInDb/V030_VerenigingeMetRechtspersoonlijkheidWerdGeregistreerd_With_Invalid_Data.cs b/...EventsInDb/V030_VerenigingeMetRechtspersoonlijkheidWerdGeregistreerd_With_Invalid_Data.cs
@@ -29,7 +29,7 @@ public V030_VerenigingeMetRechtspersoonlijkheidWerdGeregistreerd_With_Invalid_Da
             VCode = VCode,
             Naam = Naam,
             KorteNaam = "RDZ",
-            KboNummer = "7981199887",
+            KboNummer = "7981199830",
         };
 
         MaatschappelijkeZetelKonNietOvergenomenWordenUitKbo = new MaatschappelijkeZetelKonNietOvergenomenWordenUitKbo(

diff --git a/...igingMetRechtspersoonlijkheidWerdGeregistreerd_With_ContactgegevenFromKbo_For_Wijzigen.cs b/...igingMetRechtspersoonlijkheidWerdGeregistreerd_With_ContactgegevenFromKbo_For_Wijzigen.cs
@@ -25,7 +25,7 @@ public V045_VerenigingMetRechtspersoonlijkheidWerdGeregistreerd_With_Contactgege
             VCode = VCode,
             Naam = Naam,
             KorteNaam = "RDZ",
-            KboNummer = "7981199887",
+            KboNummer = "7981199845",
             Rechtsvorm = Verenigingstype.VZW.Code,
         };
 

diff --git a/....Test.Admin.Api/Magda/MagdaFacade/When_Calling_GeefOnderneming/Given_A_Valid_KboNummer.cs b/....Test.Admin.Api/Magda/MagdaFacade/When_Calling_GeefOnderneming/Given_A_Valid_KboNummer.cs
@@ -43,9 +43,9 @@ public async Task Then_It_Returns_GeefOndernemingResponseBody(MagdaOptionsSectio
             onderneming?.Namen.MaatschappelijkeNamen.Should().ContainEquivalentOf(
                 new NaamOndernemingType
                 {
-                    Naam = "Vlaamse Liga tegen Kanker",
+                    Naam = "Kom op tegen Kanker",
                     Taalcode = "nl",
-                    DatumBegin = "1998-01-01",
+                    DatumBegin = "2015-10-13",
                     DatumEinde = null,
                 });
 

diff --git a/...nlijkheid/With_Kbo_Nummer_For_Supported_Rechtsvorm/With_KboNummer_For_PrivateStichting.cs b/...nlijkheid/With_Kbo_Nummer_For_Supported_Rechtsvorm/With_KboNummer_For_PrivateStichting.cs
@@ -1,4 +1,5 @@
-namespace AssociationRegistry.Test.Admin.Api.VerenigingMetRechtspersoonlijkheid.When_RegistreerVerenigingMetRechtspersoonlijkheid.With_Kbo_Nummer_For_Supported_Rechtsvorm;
+namespace AssociationRegistry.Test.Admin.Api.VerenigingMetRechtspersoonlijkheid.When_RegistreerVerenigingMetRechtspersoonlijkheid.
+    With_Kbo_Nummer_For_Supported_Rechtsvorm;
 
 using Events;
 using Fixtures;
@@ -7,33 +8,39 @@ namespace AssociationRegistry.Test.Admin.Api.VerenigingMetRechtspersoonlijkheid.
 using Vereniging;
 using Xunit;
 
-public class RegistreerPrivateStichtingSetup: RegistreerVereniginMetRechtspersoonlijkheidSetup
+public class RegistreerPrivateStichtingSetup : RegistreerVereniginMetRechtspersoonlijkheidSetup
 {
-    public RegistreerPrivateStichtingSetup(EventsInDbScenariosFixture fixture): base(fixture, "0546572531")
+    public RegistreerPrivateStichtingSetup(EventsInDbScenariosFixture fixture) : base(fixture, kboNummer: "0546572531")
     {
-
     }
 }
 
-public class With_KboNummer_For_PrivateStichting: With_KboNummer_For_Supported_Vereniging, IClassFixture<RegistreerPrivateStichtingSetup>
+public class With_KboNummer_For_PrivateStichting : With_KboNummer_For_Supported_Vereniging, IClassFixture<RegistreerPrivateStichtingSetup>
 {
-    public With_KboNummer_For_PrivateStichting(EventsInDbScenariosFixture fixture, RegistreerPrivateStichtingSetup registreerSetup) : base(fixture, registreerSetup)
+    public With_KboNummer_For_PrivateStichting(EventsInDbScenariosFixture fixture, RegistreerPrivateStichtingSetup registreerSetup) : base(
+        fixture, registreerSetup)
     {
     }
 
     [Fact]
     public void Then_it_saves_the_events()
     {
         using var session = _fixture.DocumentStore
-            .LightweightSession();
+                                    .LightweightSession();
+
+        var verenigingMetRechtspersoonlijkheidWerdGeregistreerd = session
+                                                                 .Events
+                                                                 .QueryRawEventDataOnly<
+                                                                      VerenigingMetRechtspersoonlijkheidWerdGeregistreerd>()
+                                                                 .Should().ContainSingle(
+                                                                      e => e.KboNummer == RegistreerVerenigingMetRechtspersoonlijkheidSetup
+                                                                                         .UitKboRequest.KboNummer).Subject;
 
-        var verenigingMetRechtspersoonlijkheidWerdGeregistreerd = session.Events.QueryRawEventDataOnly<VerenigingMetRechtspersoonlijkheidWerdGeregistreerd>()
-            .Should().ContainSingle(e => e.KboNummer == RegistreerVerenigingMetRechtspersoonlijkheidSetup.UitKboRequest.KboNummer).Subject;
         using (new AssertionScope())
         {
-            verenigingMetRechtspersoonlijkheidWerdGeregistreerd.Naam.Should().Be("Vlaamse Liga tegen Kanker");
+            verenigingMetRechtspersoonlijkheidWerdGeregistreerd.Naam.Should().Be("Kom op tegen Kanker");
             verenigingMetRechtspersoonlijkheidWerdGeregistreerd.KorteNaam.Should().Be("V.L.K.");
-            verenigingMetRechtspersoonlijkheidWerdGeregistreerd.Startdatum.Should().Be(new DateOnly(1989, 10, 03));
+            verenigingMetRechtspersoonlijkheidWerdGeregistreerd.Startdatum.Should().Be(new DateOnly(year: 1989, month: 10, day: 03));
 
             verenigingMetRechtspersoonlijkheidWerdGeregistreerd.Rechtsvorm.Should().Be(Verenigingstype.PrivateStichting.Code);
         }

diff --git a/...id/With_Kbo_Nummer_For_Supported_Rechtsvorm/With_KboNummer_For_StichtingVanOpenbaarNut.cs b/...id/With_Kbo_Nummer_For_Supported_Rechtsvorm/With_KboNummer_For_StichtingVanOpenbaarNut.cs
@@ -41,7 +41,7 @@ public void Then_it_saves_the_events()
 
         using (new AssertionScope())
         {
-            verenigingMetRechtspersoonlijkheidWerdGeregistreerd.Naam.Should().Be("Vlaamse Liga tegen Kanker");
+            verenigingMetRechtspersoonlijkheidWerdGeregistreerd.Naam.Should().Be("Kom op tegen Kanker");
             verenigingMetRechtspersoonlijkheidWerdGeregistreerd.KorteNaam.Should().Be("V.L.K.");
             verenigingMetRechtspersoonlijkheidWerdGeregistreerd.Startdatum.Should().Be(new DateOnly(year: 1989, month: 10, day: 03));
             verenigingMetRechtspersoonlijkheidWerdGeregistreerd.Rechtsvorm.Should().Be(Verenigingstype.StichtingVanOpenbaarNut.Code);

diff --git a/...tRechtspersoonlijkheid/With_Kbo_Nummer_For_Supported_Rechtsvorm/With_KboNummer_For_Vzw.cs b/...tRechtspersoonlijkheid/With_Kbo_Nummer_For_Supported_Rechtsvorm/With_KboNummer_For_Vzw.cs
@@ -38,7 +38,7 @@ public void Then_it_saves_the_events()
 
         using (new AssertionScope())
         {
-            verenigingMetRechtspersoonlijkheidWerdGeregistreerd.Naam.Should().Be("Vlaamse Liga tegen Kanker");
+            verenigingMetRechtspersoonlijkheidWerdGeregistreerd.Naam.Should().Be("Kom op tegen Kanker");
             verenigingMetRechtspersoonlijkheidWerdGeregistreerd.KorteNaam.Should().Be("V.L.K.");
             verenigingMetRechtspersoonlijkheidWerdGeregistreerd.Startdatum.Should().Be(new DateOnly(year: 1989, month: 10, day: 03));
             verenigingMetRechtspersoonlijkheidWerdGeregistreerd.Rechtsvorm.Should().Be(Verenigingstype.VZW.Code);

diff --git a/...rios/V015_VerenigingMetRechtspersoonlijkheidWerdGeregistreerd_With_WijzigBasisgegevens.cs b/...rios/V015_VerenigingMetRechtspersoonlijkheidWerdGeregistreerd_With_WijzigBasisgegevens.cs
@@ -8,12 +8,12 @@
 public class V015_VerenigingMetRechtspersoonlijkheidWerdGeregistreerd_With_WijzigBasisgegevens : IScenario
 {
     public readonly VerenigingMetRechtspersoonlijkheidWerdGeregistreerd VerenigingMetRechtspersoonlijkheidWerdGeregistreerd = new(
-        "V0001015",
-        "0987654321",
-        "VZW",
-        "Feesten Affligem",
+        VCode: "V0001015",
+        KboNummer: "0987654315",
+        Rechtsvorm: "VZW",
+        Naam: "Feesten Affligem",
         string.Empty,
-        null);
+        Startdatum: null);
 
     public readonly RoepnaamWerdGewijzigd RoepnaamWerdGewijzigd = new("The Affligem Party Squad");
 
@@ -30,5 +30,7 @@ public IEvent[] GetEvents()
     }
 
     public CommandMetadata GetCommandMetadata()
-        => new("OVO000001", new DateTimeOffset(year: 2023, month: 01, day: 25, hour: 0, minute: 0, second: 0, TimeSpan.Zero).ToInstant(), Guid.NewGuid());
+        => new(Initiator: "OVO000001",
+               new DateTimeOffset(year: 2023, month: 01, day: 25, hour: 0, minute: 0, second: 0, TimeSpan.Zero).ToInstant(),
+               Guid.NewGuid());
 }
diff --git a/...tionRegistry.Test.Public.Api/Fixtures/GivenEvents/Scenarios/V016_VerenigingWerdGestopt.cs b/...tionRegistry.Test.Public.Api/Fixtures/GivenEvents/Scenarios/V016_VerenigingWerdGestopt.cs
@@ -8,19 +8,19 @@
 public class V016_VerenigingWerdGestopt : IScenario
 {
     public readonly FeitelijkeVerenigingWerdGeregistreerd FeitelijkeVerenigingWerdGeregistreerd = new(
-        "V0001016",
-        "0987654321",
-        "",
-        "",
-        null,
-        new Registratiedata.Doelgroep(0, 150),
-        false,
+        VCode: "V0001016",
+        Naam: "0987654316",
+        KorteNaam: "",
+        KorteBeschrijving: "",
+        Startdatum: null,
+        new Registratiedata.Doelgroep(Minimumleeftijd: 0, Maximumleeftijd: 150),
+        IsUitgeschrevenUitPubliekeDatastroom: false,
         Array.Empty<Registratiedata.Contactgegeven>(),
         Array.Empty<Registratiedata.Locatie>(),
         Array.Empty<Registratiedata.Vertegenwoordiger>(),
         Array.Empty<Registratiedata.HoofdactiviteitVerenigingsloket>());
 
-    public readonly VerenigingWerdGestopt VerenigingWerdGestopt = new(new DateOnly(2023,09,06));
+    public readonly VerenigingWerdGestopt VerenigingWerdGestopt = new(new DateOnly(year: 2023, month: 09, day: 06));
 
     public VCode VCode
         => VCode.Create(FeitelijkeVerenigingWerdGeregistreerd.VCode);
@@ -35,5 +35,7 @@ public IEvent[] GetEvents()
     }
 
     public CommandMetadata GetCommandMetadata()
-        => new("OVO000001", new DateTimeOffset(year: 2023, month: 01, day: 25, hour: 0, minute: 0, second: 0, TimeSpan.Zero).ToInstant(), Guid.NewGuid());
+        => new(Initiator: "OVO000001",
+               new DateTimeOffset(year: 2023, month: 01, day: 25, hour: 0, minute: 0, second: 0, TimeSpan.Zero).ToInstant(),
+               Guid.NewGuid());
 }
diff --git a/...ingMetRechtspersoonlijkheidWerdGeregistreerd_With_WijzigMaatschappelijkeZetel_Scenario.cs b/...ingMetRechtspersoonlijkheidWerdGeregistreerd_With_WijzigMaatschappelijkeZetel_Scenario.cs
@@ -8,30 +8,31 @@
 public class V017_VerenigingMetRechtspersoonlijkheidWerdGeregistreerd_With_WijzigMaatschappelijkeZetel_Scenario : IScenario
 {
     public readonly VerenigingMetRechtspersoonlijkheidWerdGeregistreerd VerenigingMetRechtspersoonlijkheidWerdGeregistreerd = new(
-        "V0001017",
-        "0987654321",
-        "VZW",
-        "Feesten Affligem",
+        VCode: "V0001017",
+        KboNummer: "0987654317",
+        Rechtsvorm: "VZW",
+        Naam: "Feesten Affligem",
         string.Empty,
-        null);
+        Startdatum: null);
 
     public readonly MaatschappelijkeZetelWerdOvergenomenUitKbo MaatschappelijkeZetelWerdOvergenomenUitKbo = new(
         Locatie: new Registratiedata.Locatie(
-            1,
+            LocatieId: 1,
             Locatietype.MaatschappelijkeZetelVolgensKbo,
-            false,
+            IsPrimair: false,
             string.Empty,
             new Registratiedata.Adres(
-                "Stationsstraat",
-                "1",
-                "B",
-                "1790",
-                "Affligem",
-                "België"),
-            null
+                Straatnaam: "Stationsstraat",
+                Huisnummer: "1",
+                Busnummer: "B",
+                Postcode: "1790",
+                Gemeente: "Affligem",
+                Land: "België"),
+            AdresId: null
         ));
 
-    public readonly MaatschappelijkeZetelVolgensKBOWerdGewijzigd MaatschappelijkeZetelVolgensKBOWerdGewijzigd = new(1, "Station", true);
+    public readonly MaatschappelijkeZetelVolgensKBOWerdGewijzigd MaatschappelijkeZetelVolgensKBOWerdGewijzigd =
+        new(LocatieId: 1, Naam: "Station", IsPrimair: true);
 
     public VCode VCode
         => VCode.Create(VerenigingMetRechtspersoonlijkheidWerdGeregistreerd.VCode);
@@ -47,6 +48,7 @@ public IEvent[] GetEvents()
     }
 
     public CommandMetadata GetCommandMetadata()
-        => new("OVO000001", new DateTimeOffset(year: 2023, month: 01, day: 25, hour: 0, minute: 0, second: 0, TimeSpan.Zero).ToInstant(),
+        => new(Initiator: "OVO000001",
+               new DateTimeOffset(year: 2023, month: 01, day: 25, hour: 0, minute: 0, second: 0, TimeSpan.Zero).ToInstant(),
                Guid.NewGuid());
 }
diff --git a/wiremock/__files/GeefOndernemingResponses/0442528054.xml b/wiremock/__files/GeefOndernemingResponses/0442528054.xml
@@ -60,13 +60,13 @@
                 <Namen
                   xmlns:web="http://webservice.geefondernemingdienst-02_00.onderneming-02_00.vip.vlaanderen.be">
                   <MaatschappelijkeNamen>
-                    <MaatschappelijkeNaam DatumBegin="1998-01-01">
-                      <Naam>Vlaamse Liga tegen Kanker</Naam>
+                    <MaatschappelijkeNaam DatumBegin="2015-10-13">
+                      <Naam>Kom op tegen Kanker</Naam>
                       <Taalcode>nl</Taalcode>
                     </MaatschappelijkeNaam>
                   </MaatschappelijkeNamen>
                   <AfgekorteNamen>
-                    <AfgekorteNaam DatumBegin="1998-01-01">
+                    <AfgekorteNaam DatumBegin="2015-10-13">
                       <Naam>V.L.K.</Naam>
                       <Taalcode>nl</Taalcode>
                     </AfgekorteNaam>

diff --git a/wiremock/__files/GeefOndernemingResponses/0468831484.xml b/wiremock/__files/GeefOndernemingResponses/0468831484.xml
@@ -59,13 +59,13 @@
                 <Namen
                   xmlns:web="http://webservice.geefondernemingdienst-02_00.onderneming-02_00.vip.vlaanderen.be">
                   <MaatschappelijkeNamen>
-                    <MaatschappelijkeNaam DatumBegin="1998-01-01">
-                      <Naam>Vlaamse Liga tegen Kanker</Naam>
+                    <MaatschappelijkeNaam DatumBegin="2015-10-13">
+                      <Naam>Kom op tegen Kanker</Naam>
                       <Taalcode>nl</Taalcode>
                     </MaatschappelijkeNaam>
                   </MaatschappelijkeNamen>
                   <AfgekorteNamen>
-                    <AfgekorteNaam DatumBegin="1998-01-01">
+                    <AfgekorteNaam DatumBegin="2015-10-13">
                       <Naam>V.L.K.</Naam>
                       <Taalcode>nl</Taalcode>
                     </AfgekorteNaam>

diff --git a/wiremock/__files/GeefOndernemingResponses/0546572531.xml b/wiremock/__files/GeefOndernemingResponses/0546572531.xml
@@ -60,13 +60,13 @@
                 <Namen
                   xmlns:web="http://webservice.geefondernemingdienst-02_00.onderneming-02_00.vip.vlaanderen.be">
                   <MaatschappelijkeNamen>
-                    <MaatschappelijkeNaam DatumBegin="1998-01-01">
-                      <Naam>Vlaamse Liga tegen Kanker</Naam>
+                    <MaatschappelijkeNaam DatumBegin="2015-10-13">
+                      <Naam>Kom op tegen Kanker</Naam>
                       <Taalcode>nl</Taalcode>
                     </MaatschappelijkeNaam>
                   </MaatschappelijkeNamen>
                   <AfgekorteNamen>
-                    <AfgekorteNaam DatumBegin="1998-01-01">
+                    <AfgekorteNaam DatumBegin="2015-10-13">
                       <Naam>V.L.K.</Naam>
                       <Taalcode>nl</Taalcode>
                     </AfgekorteNaam>

diff --git a/wiremock/__files/GeefOndernemingResponses/0554790609.xml b/wiremock/__files/GeefOndernemingResponses/0554790609.xml
@@ -60,13 +60,13 @@
                 <Namen
                   xmlns:web="http://webservice.geefondernemingdienst-02_00.onderneming-02_00.vip.vlaanderen.be">
                   <MaatschappelijkeNamen>
-                    <MaatschappelijkeNaam DatumBegin="1998-01-01">
-                      <Naam>Vlaamse Liga tegen Kanker</Naam>
+                    <MaatschappelijkeNaam DatumBegin="2015-10-13">
+                      <Naam>Kom op tegen Kanker</Naam>
                       <Taalcode>nl</Taalcode>
                     </MaatschappelijkeNaam>
                   </MaatschappelijkeNamen>
                   <AfgekorteNamen>
-                    <AfgekorteNaam DatumBegin="1998-01-01">
+                    <AfgekorteNaam DatumBegin="2015-10-13">
                       <Naam>V.L.K.</Naam>
                       <Taalcode>nl</Taalcode>
                     </AfgekorteNaam>

diff --git a/wiremock/__files/GeefOndernemingResponses/0563634435.xml b/wiremock/__files/GeefOndernemingResponses/0563634435.xml
@@ -61,7 +61,7 @@
                 <Namen
                   xmlns:web="http://webservice.geefondernemingdienst-02_00.onderneming-02_00.vip.vlaanderen.be">
                   <MaatschappelijkeNamen>
-                    <MaatschappelijkeNaam DatumBegin="1998-01-01">
+                    <MaatschappelijkeNaam DatumBegin="2015-10-13">
                       <Naam>Kometsoft</Naam>
                       <Taalcode>nl</Taalcode>
                     </MaatschappelijkeNaam>