fix: or-2334 use type instead of anonymous type for historiek

Informatievlaanderen · Dec 13, 2024 · 282a421 · 282a421
1 parent a0520ad
commit 282a421
Show file tree

Hide file tree

Showing 3 changed files with 100 additions and 11 deletions.
diff --git a/...Registry.Admin.ProjectionHost/Projections/Historiek/BeheerVerenigingHistoriekProjector.cs b/...Registry.Admin.ProjectionHost/Projections/Historiek/BeheerVerenigingHistoriekProjector.cs
@@ -656,11 +656,7 @@ public static void Apply(IEvent<VerenigingWerdGemarkeerdAlsDubbelVan> @event, Be
     {
         AddHistoriekEntry(
             @event,
-            new
-            {
-                VCode = @event.Data.VCode,
-                VCodeAuthentiekeVereniging = @event.Data.VCodeAuthentiekeVereniging,
-            },
+            @event.Data,
             document,
             $"Vereniging werd gemarkeerd als dubbel van {@event.Data.VCodeAuthentiekeVereniging}."
         );
@@ -670,13 +666,9 @@ public static void Apply(IEvent<VerenigingAanvaarddeDubbeleVereniging> @event, B
     {
         AddHistoriekEntry(
             @event,
-            new
-            {
-                VCode = @event.Data.VCode,
-                VCodeDubbeleVereniging = @event.Data.VCodeDubbeleVereniging,
-            },
+            @event.Data,
             document,
-            $"Vereniging {@event.Data.VCodeDubbeleVereniging} werd toegevoegd als dubbel."
+            $"Vereniging aanvaardde dubbele vereniging {@event.Data.VCodeDubbeleVereniging}."
         );
     }
 

diff --git a/test/AssociationRegistry.Test.E2E/Framework/Mappers/HistoriekGebeurtenisMapper.cs b/test/AssociationRegistry.Test.E2E/Framework/Mappers/HistoriekGebeurtenisMapper.cs
@@ -520,4 +520,18 @@ public static HistoriekGebeurtenisResponse LidmaatschapWerdGewijzigd(WijzigLidma
             Initiator = "OVO002949",
             Tijdstip = "2024-07-30T11:08:05Z",
         };
+
+    public static HistoriekGebeurtenisResponse? VerenigingAanvaarddeDubbeleVereniging(MarkeerAlsDubbelVanRequest request, string vCode)
+        => new()
+        {
+            Beschrijving = $"Vereniging aanvaardde dubbele vereniging {vCode}.",
+            Gebeurtenis = nameof(Events.VerenigingAanvaarddeDubbeleVereniging),
+            Data = new
+            {
+                VCode = request.IsDubbelVan,
+                VCodeDubbeleVereniging = vCode,
+            },
+            Initiator = "OVO002949",
+            Tijdstip = "2024-07-30T11:08:05Z",
+        };
 }
diff --git a/...Markeer_Als_Dubbel_Van/Beheer/HistoriekAuthentiekeVereniging/Returns_HistoriekResponse.cs b/...Markeer_Als_Dubbel_Van/Beheer/HistoriekAuthentiekeVereniging/Returns_HistoriekResponse.cs
@@ -0,0 +1,83 @@
+namespace AssociationRegistry.Test.E2E.When_Markeer_Als_Dubbel_Van.Beheer.HistoriekAuthentiekeVereniging;
+
+using AssociationRegistry.Admin.Api.Verenigingen.Dubbels.FeitelijkeVereniging.MarkeerAlsDubbelVan.RequestModels;
+using AssociationRegistry.Admin.Api.Verenigingen.Historiek.ResponseModels;
+using AssociationRegistry.Events;
+using AssociationRegistry.Test.E2E.Framework.AlbaHost;
+using AssociationRegistry.Test.E2E.Framework.ApiSetup;
+using AssociationRegistry.Test.E2E.Framework.Comparison;
+using AssociationRegistry.Test.E2E.Framework.Mappers;
+using AssociationRegistry.Test.E2E.Framework.TestClasses;
+using KellermanSoftware.CompareNetObjects;
+using Xunit;
+using Xunit.Abstractions;
+
+[Collection(FullBlownApiCollection.Name)]
+public class Returns_Historiek : End2EndTest<MarkeerAlsDubbelVanContext, MarkeerAlsDubbelVanRequest, HistoriekResponse>
+{
+    private readonly ITestOutputHelper _helper;
+
+    public override Func<IApiSetup, HistoriekResponse> GetResponse
+        => setup => setup.AdminApiHost.GetBeheerHistoriek(TestContext.Scenario.AndereFeitelijkeVerenigingWerdGeregistreerd.VCode);
+
+    public Returns_Historiek(MarkeerAlsDubbelVanContext testContext, ITestOutputHelper helper) : base(testContext)
+    {
+        _helper = helper;
+    }
+
+    [Fact]
+    public async Task With_VCode()
+    {
+        var tryCounter = 0;
+
+        while (tryCounter < 20)
+        {
+            ++tryCounter;
+            await Task.Delay(500);
+
+            _helper.WriteLine($"Looking for CorresponderendeVCodes (try {tryCounter})...");
+            if (Response.VCode is not null)
+            {
+                _helper.WriteLine("Found it!");
+                break;
+            }
+
+            _helper.WriteLine("Did not find any CorresponderendeVCodes.");
+        }
+
+        Response.VCode.ShouldCompare(TestContext.Scenario.AndereFeitelijkeVerenigingWerdGeregistreerd.VCode);
+    }
+
+    [Fact]
+    public void With_Context()
+    {
+        Response.Context.ShouldCompare("http://127.0.0.1:11003/v1/contexten/beheer/historiek-vereniging-context.json");
+    }
+
+    [Fact]
+    public async Task With_All_Gebeurtenissen()
+    {
+        var gebeurtenis =
+            Response.Gebeurtenissen.SingleOrDefault(x => x.Gebeurtenis == nameof(VerenigingAanvaarddeDubbeleVereniging));
+
+        var tryCounter = 0;
+
+        while (tryCounter < 20)
+        {
+            ++tryCounter;
+            await Task.Delay(500);
+
+            _helper.WriteLine($"Looking for CorresponderendeVCodes (try {tryCounter})...");
+            if (gebeurtenis is not null)
+            {
+                _helper.WriteLine("Found it!");
+                break;
+            }
+
+            _helper.WriteLine("Did not find any CorresponderendeVCodes.");
+        }
+
+        gebeurtenis.ShouldCompare(HistoriekGebeurtenisMapper.VerenigingAanvaarddeDubbeleVereniging(TestContext.Request, TestContext.VCode),
+                                        compareConfig: HistoriekComparisonConfig.Instance);
+    }
+}