Skip to content

Commit b97c686

Browse files
committed
Define failure reporting contracts and validation artifact ownership for Workspace V2 and tool testing lanes - PR_26146_005-test-failure-reporting-contracts
1 parent 75d1340 commit b97c686

1 file changed

Lines changed: 37 additions & 0 deletions

File tree

docs/dev/PROJECT_INSTRUCTIONS.md

Lines changed: 37 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -435,6 +435,43 @@ Affected game fixtures are opt-in only.
435435

436436
Fixtures must declare required manifests and toolState inputs.
437437

438+
### Test Failure Reporting Contracts
439+
440+
Failure reports must identify the exact tool, fixture, and runtime surface for every `FAIL` or `WARN`.
441+
442+
Shared failures must identify the root shared dependency.
443+
444+
Playwright output must separate:
445+
- contract failures
446+
- runtime failures
447+
- integration failures
448+
- fixture failures
449+
450+
Screenshots, traces, and logs are owned by the tool or lane that produced them.
451+
452+
Test artifacts must be grouped by tool or lane.
453+
454+
Hidden retries and silent reruns are prohibited.
455+
456+
Flaky tests must be explicitly labeled.
457+
458+
Required reporting fields:
459+
- tool name
460+
- fixture name
461+
- manifest/toolState source
462+
- runtime surface
463+
- expected behavior
464+
- actual behavior
465+
- PASS/FAIL/WARN/SKIP
466+
467+
Batch runs continue on isolated failures when possible.
468+
469+
Unrelated tool failures must not block targeted validation lanes.
470+
471+
Integration lanes are the only allowed cross-tool blockers.
472+
473+
Reports must clearly identify whether samples were skipped.
474+
438475
Every PR must document:
439476
- whether full samples test was skipped or run
440477
- reason for decision

0 commit comments

Comments
 (0)