singularity/singularity-forge
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

fix: correct turn-status integration test assertions

Browse source 
Fixed two assertion issues in turn-status-integration.test.ts:
1. Line 52: Changed .toContain('blocked') to .toContain('blocker')
   - Reason field returns 'Agent discovered blocker—...' not 'Agent discovered blocked—...'
2. Line 225: Changed .toBe(100000 + 1) to .toBe(100000)
   - extractTurnStatus() applies trimEnd() to cleanOutput, removing trailing newline

Result: All 65 turn-status tests passing (31 parser + 34 integration)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This commit is contained in:
Mikael Hugo 2026-05-07 04:06:32 +02:00
parent ca431e7e78
commit e2b51b62fc
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/resources/extensions/sf/tests/turn-status-integration.test.ts
View file

@ -49,7 +49,7 @@ expect(result.status).toBe("blocked");
expect(result.action).toBe("pause");
expect(result.signal).toBe("SignalPause");
expect(result.markerFound).toBe(true);
expect(result.reason).toContain("blocked");
expect(result.reason).toContain("blocker");
});
it("giving_up_marker_produces_reassess_signal", () => {
@ -222,7 +222,7 @@ const output = `${longOutput}
const result = extractTurnStatus(output);
expect(result.status).toBe("complete");
expect(result.cleanOutput.length).toBe(100000 + 1); // long string + newline
expect(result.cleanOutput.length).toBe(100000); // long string + newline
});
it("multiple_markers_uses_last_one", () => {