test(dispatch): reset-test van de inspect-grens deterministisch #106

Merged

janpeter merged 1 commit from test/t1973-deterministic-reset into master

2026-10-03 18:17:29 +02:00

janpeter commented

2026-10-03 18:14:17 +02:00

Owner

Waarom

WARNING uit de review van scrum4me-docker#105: de reset-test van de inspect-grens gebruikte echte timers met maar ~25 ms marge tot de grens van 50 ms. Onder CPU-belasting kon een vertraagde inspect de juiste implementatie toch uncertain laten geven, zodat de test flaky zou worden.

Wat

Een bestuurbare klok via vi.spyOn(Date,'now'): elke inspect zet de klok 30 ms verder, los van de belasting van de machine. Elke unknown-episode duurt 30 ms (onder de 50). Zonder reset zou de tweede episode 120 ms na de eerste vallen (boven de 50). De test wacht niet meer echt.

Verificatie

Supervisor-suite: 62 van 62.
Mutatie zonder de reset na running: de test faalt.
10 keer achter elkaar gedraaid: 10 van 10 groen.

Alleen een test.

🤖 Generated with Claude Code

## Waarom WARNING uit de review van scrum4me-docker#105: de reset-test van de inspect-grens gebruikte echte timers met maar ~25 ms marge tot de grens van 50 ms. Onder CPU-belasting kon een vertraagde inspect de juiste implementatie toch `uncertain` laten geven, zodat de test flaky zou worden. ## Wat Een bestuurbare klok via `vi.spyOn(Date,'now')`: elke inspect zet de klok 30 ms verder, los van de belasting van de machine. Elke `unknown`-episode duurt 30 ms (onder de 50). Zonder reset zou de tweede episode 120 ms na de eerste vallen (boven de 50). De test wacht niet meer echt. ## Verificatie - Supervisor-suite: 62 van 62. - Mutatie zonder de reset na `running`: de test faalt. - 10 keer achter elkaar gedraaid: 10 van 10 groen. Alleen een test. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

janpeter added 1 commit

2026-10-03 18:14:17 +02:00

test(dispatch): reset-test van de inspect-grens deterministisch (review #105 )

CI / Compose config (pull_request) Successful in 9s

Details

CI / Build-arg coverage (pull_request) Successful in 9s

Details

CI / Docker build (pull_request) Successful in 1m52s

Details

81b25157a6

De test gebruikte echte timers met ~25 ms marge tot de grens van 50 ms en kon
onder CPU-belasting flaky worden. Nu een bestuurbare klok (Date.now): elke
inspect zet hem 30 ms verder, los van de machinebelasting. Zonder de reset
faalt de test nog steeds (mutatie gecontroleerd); 10/10 stabiel.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

s4m-codex-reviewer reviewed

2026-10-03 18:17:05 +02:00

s4m-codex-reviewer left a comment

COMMENT

Findings:

INFO — __tests__/dispatch-supervisor.test.ts:535: De wijziging is inhoudelijk correct bij statische review: de twee unknown-episodes duren elk 30 ms, de tweede zou zonder reset de grens van 50 ms overschrijden, en finally herstelt Date.now. Geen blokkerende codebevindingen.
INFO — __tests__/dispatch-supervisor.test.ts:543: Uitvoeringsverificatie blijft onbevestigd. Vitest kan in deze reviewomgeving niet starten doordat /tmp uitvoer/native-module-loading blokkeert. De dispatch-typecheck faalt onder meer door ontbrekende vendor/scrum4me-shared-modules in de clone; deze fouten zijn niet aan deze testwijziging toegeschreven. Daarom veilig verdict COMMENT.

De diff tegen master op commit 81b25157a60f7ecc2559b621a00923991789a78e bevat uitsluitend deze testwijziging; git diff --check is groen. De beperkte wijziging past bij de productstandaarden voor minimale wijzigingen en regressiedekking. Een aanvullende productdoc is voor deze testcorrectie niet nodig.

geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

## COMMENT Findings: - **INFO — `__tests__/dispatch-supervisor.test.ts:535`**: De wijziging is inhoudelijk correct bij statische review: de twee unknown-episodes duren elk 30 ms, de tweede zou zonder reset de grens van 50 ms overschrijden, en `finally` herstelt `Date.now`. Geen blokkerende codebevindingen. - **INFO — `__tests__/dispatch-supervisor.test.ts:543`**: Uitvoeringsverificatie blijft onbevestigd. Vitest kan in deze reviewomgeving niet starten doordat `/tmp` uitvoer/native-module-loading blokkeert. De dispatch-typecheck faalt onder meer door ontbrekende `vendor/scrum4me-shared`-modules in de clone; deze fouten zijn niet aan deze testwijziging toegeschreven. Daarom veilig verdict COMMENT. De diff tegen master op commit `81b25157a60f7ecc2559b621a00923991789a78e` bevat uitsluitend deze testwijziging; `git diff --check` is groen. De beperkte wijziging past bij de productstandaarden voor minimale wijzigingen en regressiedekking. Een aanvullende productdoc is voor deze testcorrectie niet nodig. geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

janpeter merged commit 53d935b9ec into master

2026-10-03 18:17:29 +02:00

janpeter referenced this pull request from a commit

2026-10-03 18:17:29 +02:00

Merge pull request 'test(dispatch): reset-test van de inspect-grens deterministisch' (#106)