test(skills-e2e): give the Idempotency setup hook the 120s budget its siblings use (#2583)

The Idempotency beforeAll runs runSkillsCli (analyze --skills) twice, each
capped at 45s, under a 90s hook budget — exactly 2x the per-call timeout,
with no headroom for fixture creation and git init. On slow Windows CI
runners the two analyzes plus setup exceed 90s and the hook times out
('Hook timed out in 90000ms'), failing the shard before the test's own
status===null timeout tolerance can apply. Every other describe hook in
this file already uses 120s; align this one.

Co-authored-by: Claude <claude@anthropic.com>
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
This commit is contained in:
Gergő Magyar
2026-07-20 19:54:54 +01:00
committed by GitHub
co-authored by Claude Claude Fable 5
parent ecf6a94a1e
commit 8fd1f8a8d8
+7 -1
View File
@@ -2371,7 +2371,13 @@ export function createEntry(level: string, msg: string) {
});
result1 = runSkillsCli(tmpDir);
result2 = runSkillsCli(tmpDir);
}, 90000);
// 120s to match the other describe hooks in this file. This hook runs
// runSkillsCli TWICE, each capped at 45s, so a 90s budget has no headroom
// over two worst-case analyzes plus fixture setup and git init — it times
// out the *hook* on slow Windows runners (the test below already tolerates
// an individual analyze hitting its own 45s timeout via status === null,
// but a hook timeout fails before that tolerance can apply).
}, 120000);
afterAll(() => {
fs.rmSync(tmpDir, { recursive: true, force: true });