From 980fb1acfc222e9e5bd300fbbc48e130f57c8490 Mon Sep 17 00:00:00 2001 From: Garry Tan Date: Mon, 23 Mar 2026 20:34:19 -0700 Subject: [PATCH] fix: increase beforeAll timeout for browse pre-warm in CI Bun's default beforeAll timeout is 5s but Chromium launch in CI Docker can take 10-20s. Set explicit 45s timeout on the beforeAll hook. Co-Authored-By: Claude Opus 4.6 (1M context) --- test/skill-e2e-bws.test.ts | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/test/skill-e2e-bws.test.ts b/test/skill-e2e-bws.test.ts index 0f1643c5..46672217 100644 --- a/test/skill-e2e-bws.test.ts +++ b/test/skill-e2e-bws.test.ts @@ -26,9 +26,10 @@ describeIfSelected('Skill E2E tests', [ tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'skill-e2e-')); setupBrowseShims(tmpDir); - // Pre-warm the browse server so Chromium is already launched for tests + // Pre-warm the browse server so Chromium is already launched for tests. + // In CI, Chromium can take 10-20s to launch (Docker + --no-sandbox). spawnSync(browseBin, ['goto', testServer.url], { cwd: tmpDir, timeout: 30000, stdio: 'pipe' }); - }); + }, 45_000); afterAll(() => { testServer?.server?.stop();