Skip to content

Commit 03351e0

Browse files
test(e2e): bump preview-render timeout to 75s under CI load
PR #172's first pull_request-triggered run hit a worker-contention timeout in metadata/format-specific/doc-format-overrides.qmd — the preview iframe didn't render within 45s on any of the 3 attempts. Same shape as the failures we saw with 4 workers (cleared by dropping to 2), but now appearing at 2 workers because the runner happened to be slower this time. 8 other smoke-all tests passed only on retry, i.e. they were right at the edge. Bump waitForPreviewRender from 45s to 75s, and the per-test ceiling from 60s to 90s so the wider preview wait can actually fire. Fast tests still pass fast; slower fixtures under contention get the breathing room they need.
1 parent 818f920 commit 03351e0

1 file changed

Lines changed: 7 additions & 3 deletions

File tree

hub-client/e2e/smoke-all.spec.ts

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,12 @@ const allTests: DiscoveredTest[] = discoverSmokeAllTests();
3535
// ---------------------------------------------------------------------------
3636

3737
test.describe('smoke-all E2E tests', () => {
38-
// Increase timeout for SASS compilation tests
39-
test.setTimeout(60000);
38+
// Increase timeout for SASS compilation tests. On a contended CI
39+
// runner (ubuntu-latest with 2 cores, 2 workers, a vite dev server,
40+
// the hub binary, and chromium per worker) the preview render can
41+
// take 30-50s for the slower fixtures. 90s leaves headroom for the
42+
// 75s waitForPreviewRender wait below to actually fire.
43+
test.setTimeout(90000);
4044

4145
for (const fixture of allTests) {
4246
const skipReason = shouldSkip(fixture.runConfig, fixture.relPath);
@@ -108,7 +112,7 @@ test.describe('smoke-all E2E tests', () => {
108112
// Wait for render (or error)
109113
if (!spec.expectsError) {
110114
await waitForPreviewRender(page, {
111-
timeout: 45000,
115+
timeout: 75000,
112116
consoleErrors,
113117
kind,
114118
});

0 commit comments

Comments
 (0)