Commit f6d487b
committed
fix(spec): set testTimeout 60s for this package's vitest run (#4850)
packages/spec/vitest.config.ts never set testTimeout, so every case ran under
vitest's 5000ms default. Twelve tests load the TypeScript compiler in-case and
type-resolve the whole export surface (ts.createProgram + getTypeChecker),
which is seconds of work by construction.
Measured on an idle runner the slowest such case is 3.4s against a 5000ms
budget — green on a PR branch, too thin on a merge-queue runner building
several PRs at once. Five failures in one night, all inside the queue, each
evicting an unrelated PR.
Set at the config layer so all twelve are covered, and so a thirteenth is
covered on arrival — PR #4506 set the same 60s value case-by-case and only
covered the ones red at the time.
Stop-the-bleeding only; the underlying per-run TypeScript compilation cost
stays tracked in #4796.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015Br2xsJsczFsTR9bvbh2Ny1 parent 799d098 commit f6d487b
2 files changed
Lines changed: 44 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
10 | 16 | | |
11 | 17 | | |
12 | 18 | | |
| |||
0 commit comments