All checks were successful
## What & why Before this change the self-service confirmation and the openbaar register showed **different** identifiers, so a citizen could not look their registration back up (#78). Now both surface the same **reference**: - **domain → ACL (write):** the domain `registrationId` is set as the zaak's `identificatie` on `POST /zaken`. - **event-subscriber → ACL (read):** the subscriber reads the zaak's `identificatie` back through the ACL (§8.1 — only the ACL talks to ZGW) via a new `POST /zaken/reference`, and stores it on the projection row **and** the `processed_notifications` replay log. - **BFF + openbaar:** the public view exposes `id/status/reference` (never bsn/naam) and searches by id or reference; the register's "Referentie" column shows the reference. Storing the reference in the replay log keeps ADR-0008's **rebuild-is-log-only** invariant intact — `/admin/rebuild` reproduces the reference without re-reading the ACL. Decision recorded in **ADR-0012**. ## Definition of Done - [x] Linked issue: #78 - [x] Tests written first; red → green per layer - [x] Unit + acceptance green (`make unit`): domain 49, acl 27, bff 20, event-subscriber 19, acceptance 7 - [x] Frontend lint + test green (`nx run-many -t lint test`) - [x] Mutation ≥ break(90): acl 100%, event-subscriber 100%, bff 100%, domain 98.41% (pre-existing FlowableWorkflowClient baseline, untouched) - [x] e2e extended: confirmation reference == register reference - [x] openapi.json + api-client regenerated (drift guard green) - [x] ADR-0012 added; demo-script note appended - [x] `Acl__BaseUrl` wired for the subscriber in compose closes #78 Reviewed-on: #79
150 lines
6.2 KiB
C#
150 lines
6.2 KiB
C#
using EventSubscriber.Application;
|
|
|
|
namespace EventSubscriber.Tests;
|
|
|
|
/// <summary>Behaviour of the projector that turns NRC notifications into projection rows.
|
|
/// The walking skeleton reacts only to a zaak being created (status INGEDIEND) and must
|
|
/// tolerate duplicate and out-of-order deliveries (CLAUDE.md §8.6).</summary>
|
|
public sealed class NotificationProjectorTests
|
|
{
|
|
private const string ZaakUrl = "http://openzaak:8000/zaken/api/v1/zaken/11111111-1111-1111-1111-111111111111";
|
|
private const string StatusUrl = "http://openzaak:8000/zaken/api/v1/statussen/22222222-2222-2222-2222-222222222222";
|
|
|
|
private readonly InMemoryNotificationLog _log = new();
|
|
private readonly InMemoryProjectionStore _store = new();
|
|
private readonly FakeAclClient _acl = new();
|
|
|
|
private NotificationProjector Projector() => new(_log, _store, _acl);
|
|
|
|
private static Notification ZaakCreated(string url = ZaakUrl)
|
|
=> new("zaken", "zaak", "create", new Uri(url));
|
|
|
|
// A status-set notification: resourceUrl is the status resource, hoofdObject is the zaak it belongs to.
|
|
private static Notification StatusSet(string zaakUrl = ZaakUrl, string statusUrl = StatusUrl)
|
|
=> new("zaken", "status", "create", new Uri(statusUrl), new Uri(zaakUrl));
|
|
|
|
[Fact]
|
|
public async Task creating_a_zaak_writes_one_row_with_status_ingediend()
|
|
{
|
|
await Projector().HandleAsync(ZaakCreated());
|
|
|
|
var entry = Assert.Single(await _store.AllAsync());
|
|
Assert.Equal("11111111-1111-1111-1111-111111111111", entry.Id);
|
|
Assert.Equal(RegistrationStatus.Ingediend, entry.Status);
|
|
// Enriched with the zaak's reference (identificatie), fetched via the ACL (#78).
|
|
Assert.Equal("REG-11111111-1111-1111-1111-111111111111", entry.Reference);
|
|
}
|
|
|
|
[Fact]
|
|
public async Task rebuild_reproduces_the_reference_without_re_reading_via_the_acl()
|
|
{
|
|
var projector = Projector();
|
|
await projector.HandleAsync(ZaakCreated());
|
|
var callsAfterProjection = _acl.CallCount;
|
|
|
|
await projector.RebuildAsync();
|
|
|
|
var entry = Assert.Single(await _store.AllAsync());
|
|
Assert.Equal("REG-11111111-1111-1111-1111-111111111111", entry.Reference);
|
|
// Rebuild replays the log (which stored the reference) — no extra ACL calls (#78, ADR-0008).
|
|
Assert.Equal(callsAfterProjection, _acl.CallCount);
|
|
}
|
|
|
|
[Fact]
|
|
public async Task replaying_the_same_notification_keeps_a_single_row()
|
|
{
|
|
var projector = Projector();
|
|
await projector.HandleAsync(ZaakCreated());
|
|
await projector.HandleAsync(ZaakCreated());
|
|
|
|
Assert.Single(await _store.AllAsync());
|
|
}
|
|
|
|
[Fact]
|
|
public async Task a_replayed_notification_never_reaches_the_projection_store()
|
|
{
|
|
var projector = Projector();
|
|
await projector.HandleAsync(ZaakCreated());
|
|
await projector.HandleAsync(ZaakCreated());
|
|
|
|
// The duplicate is dropped at the log, before the (idempotent) upsert — so the store
|
|
// is written exactly once. Row count alone can't see this; the upsert count can.
|
|
Assert.Equal(1, _store.UpsertCount);
|
|
}
|
|
|
|
[Fact]
|
|
public async Task two_different_zaken_each_get_their_own_row()
|
|
{
|
|
var projector = Projector();
|
|
await projector.HandleAsync(ZaakCreated());
|
|
await projector.HandleAsync(ZaakCreated(ZaakUrl[..^1] + "2")); // a distinct zaak url
|
|
|
|
Assert.Equal(2, (await _store.AllAsync()).Count);
|
|
}
|
|
|
|
[Theory]
|
|
[InlineData("documenten", "enkelvoudiginformatieobject", "create")] // wrong kanaal + resource
|
|
[InlineData("documenten", "zaak", "create")] // wrong kanaal only
|
|
[InlineData("zaken", "zaak", "update")] // wrong actie
|
|
[InlineData("zaken", "zaak", "destroy")] // wrong actie
|
|
[InlineData("zaken", "status", "update")] // a status change we ignore
|
|
[InlineData("zaken", "resultaat", "create")] // not a status we project
|
|
public async Task an_unrelated_notification_is_not_projected(string kanaal, string resource, string actie)
|
|
{
|
|
await Projector().HandleAsync(new Notification(kanaal, resource, actie, new Uri(ZaakUrl)));
|
|
|
|
Assert.Empty(await _store.AllAsync());
|
|
}
|
|
|
|
[Fact]
|
|
public async Task setting_a_status_projects_ingeschreven_keyed_on_the_zaak_not_the_status()
|
|
{
|
|
await Projector().HandleAsync(StatusSet());
|
|
|
|
var entry = Assert.Single(await _store.AllAsync());
|
|
// Keyed on the zaak (hoofdObject), not the status resource URL.
|
|
Assert.Equal("11111111-1111-1111-1111-111111111111", entry.Id);
|
|
Assert.Equal(RegistrationStatus.Ingeschreven, entry.Status);
|
|
}
|
|
|
|
[Fact]
|
|
public async Task approving_updates_the_existing_zaak_row_from_ingediend_to_ingeschreven()
|
|
{
|
|
var projector = Projector();
|
|
await projector.HandleAsync(ZaakCreated());
|
|
await projector.HandleAsync(StatusSet());
|
|
|
|
var entry = Assert.Single(await _store.AllAsync());
|
|
Assert.Equal("11111111-1111-1111-1111-111111111111", entry.Id);
|
|
Assert.Equal(RegistrationStatus.Ingeschreven, entry.Status);
|
|
}
|
|
|
|
[Fact]
|
|
public async Task rebuild_reproduces_the_approved_status()
|
|
{
|
|
var projector = Projector();
|
|
await projector.HandleAsync(ZaakCreated());
|
|
await projector.HandleAsync(StatusSet());
|
|
|
|
await projector.RebuildAsync();
|
|
|
|
var entry = Assert.Single(await _store.AllAsync());
|
|
Assert.Equal(RegistrationStatus.Ingeschreven, entry.Status);
|
|
}
|
|
|
|
[Fact]
|
|
public async Task rebuild_clears_stale_rows_and_repopulates_from_the_notification_log()
|
|
{
|
|
var projector = Projector();
|
|
await projector.HandleAsync(ZaakCreated());
|
|
// A stale row that is not backed by any logged notification must not survive a rebuild.
|
|
await _store.UpsertAsync(new RegisterEntry("stale-9999", RegistrationStatus.Ingediend));
|
|
|
|
await projector.RebuildAsync();
|
|
|
|
var entry = Assert.Single(await _store.AllAsync());
|
|
Assert.Equal("11111111-1111-1111-1111-111111111111", entry.Id);
|
|
Assert.Equal(RegistrationStatus.Ingediend, entry.Status);
|
|
}
|
|
}
|