Skip to content

Commit dceac12

Browse files
authored
test(tools-pack): give packaging suites a realistic timeout (#6215)
tools/pack had no vitest config, so its suites ran on vitest's 5s default. That package shells out to real packaging tools — 7z archive and extract, pnpm pack, installer assembly — against temp directories, so 5s is close enough to their actual cost that runner speed alone decides pass/fail. The Windows launcher payload archive test failed merge-queue CI at 5007ms while passing the same tree on the pull-request run and locally. Add a package vitest config with testTimeout 20_000, matching apps/daemon, which is subprocess-heavy for the same reason. The include pattern matches the 36 existing suites exactly; test counts are unchanged.
1 parent 517f39a commit dceac12

1 file changed

Lines changed: 14 additions & 0 deletions

File tree

tools/pack/vitest.config.ts

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
import { defineConfig } from "vitest/config";
2+
3+
export default defineConfig({
4+
test: {
5+
environment: "node",
6+
include: ["tests/**/*.test.ts"],
7+
// These suites shell out to real packaging tools — 7z archive/extract,
8+
// pnpm pack, installer assembly — against temp directories. Vitest's 5s
9+
// default is close enough to their real cost that runner speed alone
10+
// decides pass/fail: the Windows launcher payload archive test failed CI
11+
// at 5007ms while passing locally and on faster runners.
12+
testTimeout: 20_000,
13+
},
14+
});

0 commit comments

Comments
 (0)