From 9107007954deb27b8a94db0ead4ff75417875ee2 Mon Sep 17 00:00:00 2001 From: Trevin Chow Date: Fri, 21 Aug 2026 15:59:16 -0700 Subject: [PATCH] test(cross-model): give the subprocess-heavy route suites a 30s per-test ceiling (#1510) --- tests/skills/ce-code-review-cross-model-routes.test.ts | 6 +++++- tests/skills/ce-doc-review-cross-model-routes.test.ts | 6 +++++- 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/tests/skills/ce-code-review-cross-model-routes.test.ts b/tests/skills/ce-code-review-cross-model-routes.test.ts index 1810ab6d..79ea6e54 100644 --- a/tests/skills/ce-code-review-cross-model-routes.test.ts +++ b/tests/skills/ce-code-review-cross-model-routes.test.ts @@ -1,4 +1,4 @@ -import { afterAll, describe, expect, test } from "bun:test" +import { afterAll, describe, expect, setDefaultTimeout, test } from "bun:test" import { spawnSync } from "node:child_process" import { mkdtempSync, @@ -14,6 +14,10 @@ import { import { devNull, tmpdir } from "node:os" import path from "node:path" +// These tests spawn bash/python/git subprocesses; on a loaded CI runner they cross the 5s default +// (2026-08-21, PR #1508: three different tests timed out across two reruns with no related change). +setDefaultTimeout(30_000) + const tempRoots: string[] = [] function mkTempRoot(prefix: string): string { const dir = mkdtempSync(path.join(tmpdir(), prefix)) diff --git a/tests/skills/ce-doc-review-cross-model-routes.test.ts b/tests/skills/ce-doc-review-cross-model-routes.test.ts index 7216d6a4..8b67e8c6 100644 --- a/tests/skills/ce-doc-review-cross-model-routes.test.ts +++ b/tests/skills/ce-doc-review-cross-model-routes.test.ts @@ -1,4 +1,4 @@ -import { afterAll, describe, expect, test } from "bun:test" +import { afterAll, describe, expect, setDefaultTimeout, test } from "bun:test" import { spawnSync } from "node:child_process" import { mkdtempSync, @@ -14,6 +14,10 @@ import { import { tmpdir } from "node:os" import path from "node:path" +// These tests spawn bash/python/git subprocesses; on a loaded CI runner they cross the 5s default +// (2026-08-21, PR #1508: three different tests timed out across two reruns with no related change). +setDefaultTimeout(30_000) + // Every temp root we create, torn down after the suite so runs don't leak dirs. const tempRoots: string[] = [] function mkTempRoot(prefix: string): string {