From 207f5fd089b80713ca74ca2233a0d23bb482d784 Mon Sep 17 00:00:00 2001 From: Arthur Belleville Date: Sat, 16 May 2026 10:46:19 +0200 Subject: [PATCH] docs(phase-09): update validation strategy --- .planning/phases/09-etapes/09-VALIDATION.md | 34 +++++++++++++++------ 1 file changed, 25 insertions(+), 9 deletions(-) diff --git a/.planning/phases/09-etapes/09-VALIDATION.md b/.planning/phases/09-etapes/09-VALIDATION.md index 13eb3a8..1a01706 100644 --- a/.planning/phases/09-etapes/09-VALIDATION.md +++ b/.planning/phases/09-etapes/09-VALIDATION.md @@ -1,9 +1,9 @@ --- phase: 09 slug: etapes -status: draft +status: complete nyquist_compliant: true -wave_0_complete: false +wave_0_complete: true created: 2026-05-15 --- @@ -38,12 +38,12 @@ created: 2026-05-15 | Task ID | Plan | Wave | Requirement | Threat Ref | Secure Behavior | Test Type | Automated Command | File Exists | Status | |---------|------|------|-------------|------------|-----------------|-----------|-------------------|-------------|--------| -| 09-01-01 | 01 | 1 | ETAPE-01, ETAPE-06 | T-09-01 | Etapes are tablo-owned rows with no parent relationship | migration/sqlc | `cd backend && just generate && go test ./internal/web -count=1` | yes | pending | -| 09-01-02 | 01 | 1 | ETAPE-03, ETAPE-04 | T-09-02 | `tasks.etape_id` is nullable and `ON DELETE SET NULL` | migration/sqlc | `cd backend && just generate && go test ./internal/web -count=1` | yes | pending | -| 09-02-01 | 02 | 2 | ETAPE-01, ETAPE-02 | T-09-03 | Only tablo owners can create/edit/delete/reorder etapes | handler integration | `cd backend && go test ./internal/web -count=1` | yes | pending | -| 09-03-01 | 03 | 3 | ETAPE-03, ETAPE-05 | T-09-04 | Task assignment rejects etapes outside the task tablo | handler integration | `cd backend && go test ./internal/web -count=1` | yes | pending | -| 09-04-01 | 04 | 4 | ETAPE-05 | T-09-05 | Reorder preserves `etape_id` and hidden filtered tasks | regression integration | `cd backend && go test ./internal/web -count=1` | yes | pending | -| 09-04-02 | 04 | 4 | ETAPE-01..06 | - | Full backend suite remains green | full suite | `cd backend && go test ./... -count=1` | yes | pending | +| 09-01-01 | 01 | 1 | ETAPE-01, ETAPE-06 | T-09-01 | Etapes are tablo-owned rows with no parent relationship | migration/sqlc | `cd backend && go test ./internal/web -run 'TestEtapeSchemaIsOneLevelAndTaskAssignmentUnassignsOnDelete|TestEtapeCreateRendersChipAndCount' -count=1` | yes | COVERED | +| 09-01-02 | 01 | 1 | ETAPE-03, ETAPE-04 | T-09-02 | `tasks.etape_id` is nullable and `ON DELETE SET NULL` | migration/sqlc | `cd backend && go test ./internal/web -run 'TestEtapeSchemaIsOneLevelAndTaskAssignmentUnassignsOnDelete|TestEtapeDeleteUnassignsTasks' -count=1` | yes | COVERED | +| 09-02-01 | 02 | 2 | ETAPE-01, ETAPE-02 | T-09-03 | Only tablo owners can create/edit/delete/reorder etapes | handler integration | `cd backend && go test ./internal/web -run 'TestEtape(Update|Delete|Reorder|Ownership)' -count=1` | yes | COVERED | +| 09-03-01 | 03 | 3 | ETAPE-03, ETAPE-05 | T-09-04 | Task assignment rejects etapes outside the task tablo | handler integration | `cd backend && go test ./internal/web -run 'TestTask.*Etape|TestTaskAssignmentRejectsForeignEtape' -count=1` | yes | COVERED | +| 09-04-01 | 04 | 4 | ETAPE-05 | T-09-05 | Reorder preserves `etape_id` and hidden filtered tasks | regression integration | `cd backend && go test ./internal/web -run 'TestTaskReorderPreservesEtapeAssignment|TestTaskOrderPersistsWithEtapeFilter' -count=1` | yes | COVERED | +| 09-04-02 | 04 | 4 | ETAPE-01..06 | - | Full backend suite remains green | full suite | `cd backend && TEST_DATABASE_URL='postgres://xtablo:xtablo@localhost:5432/xtablo?sslmode=disable' go test ./... -count=1` | yes | COVERED | --- @@ -73,4 +73,20 @@ created: 2026-05-15 - [x] Feedback latency target documented - [x] `nyquist_compliant: true` set in frontmatter -**Approval:** pending +**Approval:** audited 2026-05-16 + +## Validation Audit 2026-05-16 + +| Metric | Count | +|--------|-------| +| Gaps found | 1 | +| Resolved | 1 | +| Escalated | 0 | + +### Gap Filled + +- Added `TestEtapeSchemaIsOneLevelAndTaskAssignmentUnassignsOnDelete` in `backend/internal/web/handlers_etapes_test.go` to verify one-level etape schema, nullable `tasks.etape_id`, and `ON DELETE SET NULL` behavior required by ETAPE-03, ETAPE-04, and ETAPE-06. + +### Final Verification + +- `cd backend && TEST_DATABASE_URL='postgres://xtablo:xtablo@localhost:5432/xtablo?sslmode=disable' go test ./internal/web -run 'TestEtapeSchemaIsOneLevelAndTaskAssignmentUnassignsOnDelete|TestEtape|TestTask.*Etape|TestTaskAssignmentRejectsForeignEtape|TestTaskReorderPreservesEtapeAssignment|TestTaskOrderPersistsWithEtapeFilter|TestTaskCreateRefreshesEtapeCounts|TestTaskDeleteRefreshesEtapeCounts' -count=1`