Skip to content

Commit

Permalink
fix: or-2596 make sure gemeentenaam serializes only to its value
Browse files Browse the repository at this point in the history
  • Loading branch information
emalfroy committed Dec 4, 2024
1 parent 9a67238 commit 400fbfb
Show file tree
Hide file tree
Showing 5 changed files with 28 additions and 12 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,8 @@ public static Postnamen FromValues(params string[] values)

public record Postnaam
{
public string Value { get; init; }

private Postnaam(string Value)
{
this.Value = Value;
Expand All @@ -54,7 +56,8 @@ public static Postnaam FromValue(string postnaam)
public static implicit operator string(Postnaam postnaam)
=> postnaam.Value;

public string Value { get; init; }
public override string ToString()
=> Value;

public void Deconstruct(out string Value)
{
Expand Down
9 changes: 0 additions & 9 deletions src/AssociationRegistry/Vereniging/Locaties/Adres.cs
Original file line number Diff line number Diff line change
Expand Up @@ -72,12 +72,3 @@ 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());
}
15 changes: 15 additions & 0 deletions src/AssociationRegistry/Vereniging/Locaties/Gemeentenaam.cs
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
namespace AssociationRegistry.Vereniging;

using Events;

public record Gemeentenaam(string Naam)
{
public static Gemeentenaam Hydrate(string gemeente)
=> new(gemeente);

public static Gemeentenaam FromVerrijkteGemeentenaam(VerrijkteGemeentenaam gemeentenaam)
=> new(gemeentenaam.Format());

public override string ToString()
=> Naam;
}
2 changes: 1 addition & 1 deletion src/AssociationRegistry/Vereniging/VerenigingOfAnyKind.cs
Original file line number Diff line number Diff line change
Expand Up @@ -355,8 +355,8 @@ private IEvent GetAdresMatchExceptionEvent(
{
IEvent @event = ex.StatusCode switch
{
//TODO: is this correct?
HttpStatusCode.NotFound => AdresWerdNietGevondenInAdressenregister.From(VCode, locatieVoorTeMatchenAdres),

_ => new AdresKonNietOvergenomenWordenUitAdressenregister(VCode, locatieId, locatieVoorTeMatchenAdres.Adres.ToAdresString(),
GetExceptionMessage(ex.StatusCode)),
};
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -40,12 +40,19 @@ public async Task Then_AdresKonNietOvergenomenWordenUitAdressenregister()
new VerenigingState()
.Apply(feitelijkeVerenigingWerdGeregistreerd));

await vereniging.ProbeerAdresTeMatchen(grarClient.Object, feitelijkeVerenigingWerdGeregistreerd.Locaties.First().LocatieId,
var locatie = feitelijkeVerenigingWerdGeregistreerd.Locaties.First();

await vereniging.ProbeerAdresTeMatchen(grarClient.Object, locatie.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);
}
}

0 comments on commit 400fbfb

Please sign in to comment.