From ed77de1219a33c97e78e0ddb8cdb6493ad8d5527 Mon Sep 17 00:00:00 2001 From: Niek Otten Date: Mon, 28 Sep 2026 14:33:55 +0200 Subject: [PATCH] fix(acl): keep the integration tests out of Stryker's solution (refs #174) Stryker runs every test project in Acl.slnx regardless of test-projects, so Acl.IntegrationTests (needs a live OpenZaak) failed 8 tests in every initial run. Acl.slnx is read only by Stryker; the root solution still builds it. Re-measured with a green initial run: 90.45 %, so break: 90 stands. Co-Authored-By: Claude Opus 5.5 (1M context) --- docs/runbooks/ci.md | 3 +++ services/acl/Acl.slnx | 4 +++- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/docs/runbooks/ci.md b/docs/runbooks/ci.md index 44ba84f..26ded11 100644 --- a/docs/runbooks/ci.md +++ b/docs/runbooks/ci.md @@ -60,6 +60,9 @@ make mutation # dotnet tool restore + dotnet stryker on the ACL Config lives in [`services/acl/stryker-config.json`](../../services/acl/stryker-config.json). It runs in **solution mode** against `Acl.slnx`, mutating the two projects under test (`Acl.Application`, `Acl.Infrastructure`); `Acl.Api` has no tests and is skipped. +`Acl.slnx` leaves out `Acl.IntegrationTests`: it needs a live OpenZaak, and Stryker +runs every test project in the solution, so keeping it in makes 8 tests fail in the +initial run (#174). **Baseline (the ratchet):** the ACL is the first service with branching logic, so it sets the repo-wide baseline. Observed score **95%**; enforced `break` threshold **90%** diff --git a/services/acl/Acl.slnx b/services/acl/Acl.slnx index aa02b3b..20c3c10 100644 --- a/services/acl/Acl.slnx +++ b/services/acl/Acl.slnx @@ -1,7 +1,9 @@ + -