Revert "fix: or-2596 make sure gemeentenaam serializes only to its va…

…lue" This reverts commit b59b050.
Informatievlaanderen · Dec 4, 2024 · 9a67238 · 9a67238
1 parent b59b050
commit 9a67238
Show file tree

Hide file tree

Showing 6 changed files with 14 additions and 63 deletions.
diff --git a/...onRegistry/Acties/GrarConsumer/HeradresseerLocaties/HeradresseerLocatiesMessageFactory.cs b/...onRegistry/Acties/GrarConsumer/HeradresseerLocaties/HeradresseerLocatiesMessageFactory.cs
diff --git a/src/AssociationRegistry/Grar/Models/PostalInfo/PostalInformationResponse.cs b/src/AssociationRegistry/Grar/Models/PostalInfo/PostalInformationResponse.cs
@@ -36,16 +36,13 @@ public static Postnamen FromValues(params string[] values)
 
 public record Postnaam
 {
-    public string Value { get; init; }
-
     private Postnaam(string Value)
     {
         this.Value = Value;
     }
 
     public static Postnaam FromGrar(Models.Postnaam postnaam)
         => new(postnaam.GeografischeNaam.Spelling);
-
     public static Postnaam FromValue(string postnaam)
     {
         if (string.IsNullOrEmpty(postnaam))
@@ -57,8 +54,7 @@ public static Postnaam FromValue(string postnaam)
     public static implicit operator string(Postnaam postnaam)
         => postnaam.Value;
 
-    public override string ToString()
-        => Value;
+    public string Value { get; init; }
 
     public void Deconstruct(out string Value)
     {

diff --git a/src/AssociationRegistry/Vereniging/Locaties/Adres.cs b/src/AssociationRegistry/Vereniging/Locaties/Adres.cs
@@ -72,3 +72,12 @@ public static Adres Hydrate(Registratiedata.AdresUitAdressenregister adres)
                   adres.Gemeente,
                   België);
 }
+
+public record Gemeentenaam(string Naam)
+{
+    public static Gemeentenaam Hydrate(string gemeente)
+        => new(gemeente);
+
+    public static Gemeentenaam FromVerrijkteGemeentenaam(VerrijkteGemeentenaam gemeentenaam)
+        => new(gemeentenaam.Format());
+}
diff --git a/src/AssociationRegistry/Vereniging/Locaties/Gemeentenaam.cs b/src/AssociationRegistry/Vereniging/Locaties/Gemeentenaam.cs
diff --git a/src/AssociationRegistry/Vereniging/VerenigingOfAnyKind.cs b/src/AssociationRegistry/Vereniging/VerenigingOfAnyKind.cs
@@ -355,11 +355,9 @@ private IEvent GetAdresMatchExceptionEvent(
     {
         IEvent @event = ex.StatusCode switch
         {
-            HttpStatusCode.NotFound =>
-                AdresWerdNietGevondenInAdressenregister.From(VCode, locatieVoorTeMatchenAdres),
-
-            _ =>
-                new AdresKonNietOvergenomenWordenUitAdressenregister(VCode, locatieId, locatieVoorTeMatchenAdres.Adres.ToAdresString(),
+            //TODO: is this correct?
+            HttpStatusCode.NotFound => AdresWerdNietGevondenInAdressenregister.From(VCode, locatieVoorTeMatchenAdres),
+            _ => new AdresKonNietOvergenomenWordenUitAdressenregister(VCode, locatieId, locatieVoorTeMatchenAdres.Adres.ToAdresString(),
                                                                       GetExceptionMessage(ex.StatusCode)),
         };
 

diff --git a/...sociationRegistry.Test/When_ProbeerAdresTeMatchen/Given_GrarClient_Returned_BadRequest.cs b/...sociationRegistry.Test/When_ProbeerAdresTeMatchen/Given_GrarClient_Returned_BadRequest.cs
@@ -7,7 +7,6 @@
 using Framework.Customizations;
 using Grar;
 using Grar.Exceptions;
-using Grar.Models;
 using Moq;
 using Resources;
 using System.Net;
@@ -41,21 +40,12 @@ public async Task Then_AdresKonNietOvergenomenWordenUitAdressenregister()
             new VerenigingState()
                .Apply(feitelijkeVerenigingWerdGeregistreerd));
 
-        var locatie = feitelijkeVerenigingWerdGeregistreerd.Locaties.First();
-
-        await vereniging.ProbeerAdresTeMatchen(grarClient.Object, locatie.LocatieId,
+        await vereniging.ProbeerAdresTeMatchen(grarClient.Object, feitelijkeVerenigingWerdGeregistreerd.Locaties.First().LocatieId,
                                                CancellationToken.None);
 
         var @event = vereniging.UncommittedEvents.OfType<AdresKonNietOvergenomenWordenUitAdressenregister>().SingleOrDefault();
 
         @event.Should().NotBeNull();
-
-        @event.Adres.Should().BeEquivalentTo($"{locatie.Adres.Straatnaam} {locatie.Adres.Huisnummer}" +
-                                             (!string.IsNullOrWhiteSpace(locatie.Adres.Busnummer)
-                                                 ? $" bus {locatie.Adres.Busnummer}"
-                                                 : string.Empty) +
-                                             $", {locatie.Adres.Postcode} {locatie.Adres.Gemeente}, {locatie.Adres.Land}");
-
         @event!.Reden.Should().Be(ExceptionMessages.AdresKonNietOvergenomenWordenBadRequest);
     }
 }