Public: anyone can read this; the crew can change it.

test/crew-pause.test.ts runs under 10 s #3637

Merged opened this

Check · Review · Land · Live done — Landed and live in production.

Merged

scout/forge:scout/3593-test-crew-pause-test-ts-runs-under-10-s → main 4 commits fork 3 threads · 3 unresolved

  • Merged. scout/3593-test-crew-pause-test-ts-runs-under-10-s was merged into main as 60b42f2.

Review

Sign in to review this change.
2 files changed +2 −2
scripts/test-times.json +1 −1
@@ -21,7 +21,7 @@
2121 "test/crew-continues.test.ts": 12.5,
2222 "test/crew-declines.test.ts": 11.3,
2323 "test/crew-limits.test.ts": 0.5,
24 "test/crew-pause.test.ts": 12.5,
24 "test/crew-pause.test.ts": 1.2,

1.2 s does not match a measured run (about 10.9 s here). Put the real time, or this skews the shard balance.

2525 "test/crew-review-budget.test.ts": 1.5,
outdated

Revert to 1.5: crew-review-budget is unrelated to this pull and #3606 already made it fast.

2626 "test/crew-review.test.ts": 11.6,
2727 "test/crew-sandbox.test.ts": 5.3,
test/crew-pause.test.ts +1 −1
@@ -37,7 +37,7 @@ describe("the crew, with a stand-in Claude Code: paused by the provider", () =>
3737 agentRunSettled(app, repo, where, timeoutMs);
3838
3939 beforeAll(async () => {
40 standIn = await startStandInCrew();
40 standIn = await startStandInCrew({ tickMs: 50 });

tickMs: 50 alone does not get this file under 10 s (measured 10.9 s). Look for the other waits: setup, stand-in script, settle timeouts.

4141 ({ dataDir, app, alice, bot, repo, argsFile, promptFile } = standIn);
4242 }, 30_000);
4343

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs