Skip to content

Commit d994eb3

Browse files
GottZclaude
andcommitted
fix(test): audit progress probe expects the post-negative-probe end state
Test-only. The unguarded negative probe (proving an UPDATE without the source predicate demonstrably overwrites a manual row) flips its sacrificial block manual -> llm-audit BY DESIGN — the later by-source assertion still expected two manual rows. v3.5.1 lesson, next variant: vet -tags=integration catches compile breaks, not test logic; the suite now ran locally against a real PG18 testcontainer (5/5 subtests green) before this push. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent 64210d0 commit d994eb3

1 file changed

Lines changed: 6 additions & 2 deletions

File tree

go/internal/store/sensitivity_integration_test.go

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -185,8 +185,12 @@ func TestSensitivityAudit_Integration(t *testing.T) {
185185
if pending != bySource["default"] {
186186
t.Errorf("pending=%d != by_source[default]=%d", pending, bySource["default"])
187187
}
188-
if bySource["llm-audit"] < 1 || bySource["manual"] < 2 {
189-
t.Errorf("by_source missing classified rows: %v", bySource)
188+
// End state of the private scope after the subtests above: defaultID
189+
// got its verdict, probeID was overwritten BY DESIGN by the unguarded
190+
// negative probe (manual → llm-audit), manualID alone stays manual.
191+
// The foreign-scope block must not appear in private counts.
192+
if bySource["llm-audit"] != 2 || bySource["manual"] != 1 || bySource["default"] != 0 {
193+
t.Errorf("by_source end state wrong (want llm-audit=2 manual=1 default=0): %v", bySource)
190194
}
191195
})
192196
}

0 commit comments

Comments
 (0)