Skip to content

Commit beb8d2a

Browse files
committed
test(server): give the integration suite headroom over CI contention
The dialog-reachability corpus sweep (440 tree-sitter parses) runs ~3.2s locally but timed out at vitest's 5s default in CI: the External + Integration + Transpile phase runs the integration, unit, and transpile projects concurrently, so a CPU-contended runner is far slower than an isolated local run. My pre-merge local test:all passed it for exactly that reason - no contention. Raise testTimeout to 30s for the whole server-integration project rather than patch the one test: every test here sweeps real external corpora, so the class shares the risk. CI already ran the full suite and only this one test crossed the line (114/115 passed under the same contention), so 30s - ~9x the local runtime - is ample, and a genuine hang still fails.
1 parent bafa93e commit beb8d2a

1 file changed

Lines changed: 6 additions & 0 deletions

File tree

server/vitest.integration.config.ts

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,5 +21,11 @@ export default defineConfig({
2121
name: "server-integration",
2222
include: ["test/integration/**/*.test.ts"],
2323
setupFiles: ["test/integration/setup.ts"],
24+
// These tests sweep real external corpora (hundreds of tree-sitter parses), and the
25+
// full suite runs them alongside the unit and transpile projects, so a CPU-contended
26+
// CI runner is far slower than a local run - the corpus reachability sweep is ~3s
27+
// locally but blew past the 5s default under contention. Give the class real headroom;
28+
// a genuine hang still fails at 30s.
29+
testTimeout: 30_000,
2430
},
2531
});

0 commit comments

Comments
 (0)