fix(acl): keep the integration tests out of Stryker's solution (closes #174) #188

2026-09-28T12:33:57Z

not commented

2026-09-28 12:33:57 +00:00

closes #174

Took fix option 3, cut down: removed Acl.IntegrationTests from services/acl/Acl.slnx. Only Stryker reads that file. make build, make lint and make unit use the root register-referentie.slnx, and Dockerfile.integration targets the csproj directly, so nothing else changes. A comment in the slnx and a note in docs/runbooks/ci.md explain why the project is left out.

Verified locally (cd services/acl && dotnet stryker):

Number of tests found: 86 (was 94); the 8 tests are failing warning is gone.
Final score 90.45 %, the same as before. Tests that fail their initial run were never used to kill mutants, so the number was not depressed, only unverified. Re-baselined from this clean run: break: 90 stays (§5, never lower).

No test-first commit: this changes build config only. The check is the Stryker initial-run log above.

🤖 Generated with Claude Code

closes #174 Took fix option 3, cut down: removed `Acl.IntegrationTests` from `services/acl/Acl.slnx`. Only Stryker reads that file. `make build`, `make lint` and `make unit` use the root `register-referentie.slnx`, and `Dockerfile.integration` targets the csproj directly, so nothing else changes. A comment in the slnx and a note in `docs/runbooks/ci.md` explain why the project is left out. **Verified locally** (`cd services/acl && dotnet stryker`): - `Number of tests found: 86` (was 94); the `8 tests are failing` warning is gone. - Final score **90.45 %**, the same as before. Tests that fail their initial run were never used to kill mutants, so the number was not depressed, only unverified. Re-baselined from this clean run: `break: 90` stays (§5, never lower). No test-first commit: this changes build config only. The check is the Stryker initial-run log above. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

not added this to the Iteration 6 — Production Posture milestone 2026-09-28 12:33:57 +00:00

not added 1 commit 2026-09-28 12:33:58 +00:00

fix(acl): keep the integration tests out of Stryker's solution (refs #174 )

CI / lint (pull_request) Successful in 2m1s

Details

CI / k8s (pull_request) Successful in 15s

Details

CI / build (pull_request) Successful in 1m21s

Details

CI / unit (pull_request) Successful in 1m32s

Details

CI / frontend (pull_request) Successful in 2m38s

Details

CI / mutation (pull_request) Successful in 5m36s

Details

CI / verify-stack (pull_request) Skipped

Details

ed77de1219

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) <noreply@anthropic.com>

not added the type:bug area:acl labels 2026-09-28 12:34:02 +00:00

not merged commit 733ba71173 into main

2026-09-28 12:56:11 +00:00

not deleted branch fix/174-stryker-acl-integration

2026-09-28 12:56:15 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: eho/register-referentie#188