Prechádzať zdrojové kódy

fix: increase bun test timeout to 30s via CLI flag

The default 5s timeout is too short for CLI subprocess tests in CI.
Tobi Lutke 3 mesiacov pred
rodič
commit
2780dfb5d0
2 zmenil súbory, kde vykonal 2 pridanie a 5 odobranie
  1. 1 1
      .github/workflows/ci.yml
  2. 1 4
      src/test-preload.ts

+ 1 - 1
.github/workflows/ci.yml

@@ -64,7 +64,7 @@ jobs:
       - run: bun install
 
       - name: Tests
-        run: bun test --preload ./src/test-preload.ts test/
+        run: bun test --timeout 30000 --preload ./src/test-preload.ts test/
         env:
           CI: true
           DYLD_LIBRARY_PATH: /opt/homebrew/opt/sqlite/lib

+ 1 - 4
src/test-preload.ts

@@ -4,12 +4,9 @@
  * Uses bun:test afterAll to properly dispose of llama.cpp Metal
  * resources before the process exits, avoiding GGML_ASSERT failures.
  */
-import { afterAll, setDefaultTimeout } from "bun:test";
+import { afterAll } from "bun:test";
 import { disposeDefaultLlamaCpp } from "./llm";
 
-// Match vitest's testTimeout (default 5s is too short for CLI subprocess tests)
-setDefaultTimeout(30_000);
-
 // Global afterAll runs after all test files complete
 afterAll(async () => {
   await disposeDefaultLlamaCpp();