test(dispatch): HTTP-regressietest voor usage op de herstelroute #181

Merged

janpeter merged 1 commit from test/t1972-recovery-http into main

2026-10-03 18:13:57 +02:00

janpeter commented

2026-10-03 18:07:56 +02:00

Owner

Waarom

Vervolgpunt uit de review van scrum4me-mcp#180 (LOW): de nieuwe tests riepen submitResult direct aan. Een strikte route-parser zonder usage zou die tests ook laten slagen, terwijl hij het veld over HTTP weigert.

Wat

Een HTTP-regressietest in __tests__/dispatch/end-to-end.integration.test.ts. POST /attempts/recovery/result met {binding,result,usage} via de echte route en de supervisor-bearer geeft 200 en accepted, en de job-rij krijgt de usage: gpt-6.1-sol, dispatch_transcript en captured.

Verificatie

RED: met het oude strikte schema (zonder usage) geeft de route 422, en de test faalt.
GREEN: de volledige dispatch-DB-suite slaagt met 289 van 289; tsc --noEmit is schoon.

Alleen een test; de productcode is niet gewijzigd.

🤖 Generated with Claude Code

## Waarom Vervolgpunt uit de review van scrum4me-mcp#180 (LOW): de nieuwe tests riepen `submitResult` direct aan. Een strikte route-parser zonder `usage` zou die tests ook laten slagen, terwijl hij het veld over HTTP weigert. ## Wat Een HTTP-regressietest in `__tests__/dispatch/end-to-end.integration.test.ts`. `POST /attempts/recovery/result` met `{binding,result,usage}` via de echte route en de supervisor-bearer geeft 200 en `accepted`, en de job-rij krijgt de usage: `gpt-6.1-sol`, `dispatch_transcript` en `captured`. ## Verificatie - RED: met het oude strikte schema (zonder `usage`) geeft de route **422**, en de test faalt. - GREEN: de volledige dispatch-DB-suite slaagt met **289 van 289**; `tsc --noEmit` is schoon. Alleen een test; de productcode is niet gewijzigd. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

janpeter added 1 commit

2026-10-03 18:07:56 +02:00

test(dispatch): HTTP-regressietest voor usage op /attempts/recovery/result (review #180 )

CI / Final merge attestation and immutable publication (pull_request) Has been skipped

Details

CI / PR candidate (never published) (pull_request) Successful in 5m11s

Details

00c894a7de

Via de echte route en bearer: een body met usage wordt geaccepteerd en de
job-rij krijgt de usage. RED gezien met het oude strikte schema (422).

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

s4m-codex-reviewer reviewed

2026-10-03 18:12:41 +02:00

s4m-codex-reviewer left a comment

COMMENT

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

De diff op head 00c894a7ded6ef92859c3ebd753c54ac3797f1de voegt één gerichte HTTP-regressietest toe. De bestaande fixture levert een gestopte, nog niet afgeronde attempt met historische binding; de test controleert HTTP 200, accepted en persistente usage op de job. Dit dekt zowel weigering van usage door het strikte routeschema als verlies van het veld tussen route en domein. Bestaande testpatronen en cleanup worden hergebruikt. Geen blokkerende code-, architectuur- of documentatiebevindingen; deze testuitbreiding vereist geen nieuwe productdocumentatie.

Findings

INFO — tests/dispatch/end-to-end.integration.test.ts:858 — Verificatie nog niet onafhankelijk bevestigd: de CI-check CI / PR candidate staat op pending voor deze head. De PR-beschrijving meldt 289/289 dispatch-tests en een schone tsc, maar lokaal ontbreken de disposable PostgreSQL-testomgeving en Docker; de tests en typechecks zijn hier niet opnieuw uitgevoerd. De productstandaard vereist typechecking van zowel broncode als tests. Daarom voorlopig COMMENT volgens de safe-default.

# COMMENT geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. De diff op head `00c894a7ded6ef92859c3ebd753c54ac3797f1de` voegt één gerichte HTTP-regressietest toe. De bestaande fixture levert een gestopte, nog niet afgeronde attempt met historische binding; de test controleert HTTP 200, `accepted` en persistente usage op de job. Dit dekt zowel weigering van `usage` door het strikte routeschema als verlies van het veld tussen route en domein. Bestaande testpatronen en cleanup worden hergebruikt. Geen blokkerende code-, architectuur- of documentatiebevindingen; deze testuitbreiding vereist geen nieuwe productdocumentatie. ## Findings - **INFO — __tests__/dispatch/end-to-end.integration.test.ts:858** — Verificatie nog niet onafhankelijk bevestigd: de CI-check `CI / PR candidate` staat op `pending` voor deze head. De PR-beschrijving meldt 289/289 dispatch-tests en een schone `tsc`, maar lokaal ontbreken de disposable PostgreSQL-testomgeving en Docker; de tests en typechecks zijn hier niet opnieuw uitgevoerd. De productstandaard vereist typechecking van zowel broncode als tests. Daarom voorlopig `COMMENT` volgens de safe-default.

janpeter merged commit 5af973636c into main

2026-10-03 18:13:57 +02:00

janpeter referenced this pull request from a commit

2026-10-03 18:13:59 +02:00

Merge pull request 'test(dispatch): HTTP-regressietest voor usage op de herstelroute' (#181)