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

fix(test): align auto-loop test timers with updated session timeout

Browse source 
NEW_SESSION_TIMEOUT_MS was increased from 30s to 120s in #3819, but the
session-switch guard test still ticked only 30s. The test hung, leaked
mock.timers state, and caused 7 downstream tests to hang — timing out
the entire CI build job.
This commit is contained in:
Jeremy 2026-04-09 00:28:45 -05:00
parent d8574e5669
commit b02b1b1a81
1 changed files with 12 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
src/resources/extensions/gsd/tests/auto-loop.test.ts
View file

@ -198,12 +198,14 @@ test("runUnit keeps the session-switch guard across a late newSession settlement
try {
const ctx = makeMockCtx();
const pi = makeMockPi();
const firstSession = makeMockSession({ newSessionDelayMs: 60_000 });
const secondSession = makeMockSession({ newSessionDelayMs: 60_000 });
// Use delays longer than NEW_SESSION_TIMEOUT_MS (120s) so the timeout fires
const firstSession = makeMockSession({ newSessionDelayMs: 200_000 });
const secondSession = makeMockSession({ newSessionDelayMs: 200_000 });
const firstRun = runUnit(ctx, pi, firstSession, "task", "T01", "prompt");
mock.timers.tick(30_000);
// Tick past the 120s session timeout
mock.timers.tick(121_000);
await Promise.resolve();
const firstResult = await firstRun;
@ -213,7 +215,7 @@ test("runUnit keeps the session-switch guard across a late newSession settlement
mock.timers.tick(1);
const secondRun = runUnit(ctx, pi, secondSession, "task", "T02", "prompt");
mock.timers.tick(29_999);
mock.timers.tick(100_000);
await Promise.resolve();
assert.equal(
isSessionSwitchInFlight(),
@ -221,11 +223,16 @@ test("runUnit keeps the session-switch guard across a late newSession settlement
"late settlement from the first session must not clear the newer session guard",
);
mock.timers.tick(30_001);
// Tick past the second session's timeout (121s total > 120s NEW_SESSION_TIMEOUT_MS)
mock.timers.tick(21_001);
await Promise.resolve();
const secondResult = await secondRun;
assert.equal(secondResult.status, "cancelled");
// Tick past the second session's delayed promise (200s) so .finally() fires
mock.timers.tick(80_000);
await Promise.resolve();
assert.equal(isSessionSwitchInFlight(), false, "guard should clear after the newer session settles");
} finally {
mock.timers.reset();