Skip to content

Commit 700e5c3

Browse files
test(refresh-lock): shrink cross-process holder hold to 600ms
1 parent 648a2ed commit 700e5c3

2 files changed

Lines changed: 5 additions & 2 deletions

File tree

‎scripts/ci-timings.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
"scripts/test-parallel.test.ts": 2496,
66
"src/plugins/shell-guard-plugin.test.ts": 1883,
77
"src/tui/tool-execution-watchdog.test.ts": 1853,
8-
"src/auth/codex/refresh-lock.test.ts": 1840,
8+
"src/auth/codex/refresh-lock.test.ts": 930,
99
"src/tui/provider-setup.test.ts": 1562,
1010
"src/tui/url-click.test.ts": 1442,
1111
"e2e/exec-signal-finalize.test.ts": 1299,

‎src/auth/codex/refresh-lock.test.ts‎

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,10 @@ describe("codex refresh lock", () => {
164164
import.meta.url,
165165
).pathname;
166166
const lock = join(dir, "refresh.lock");
167-
const proc = Bun.spawn(["bun", "run", holderPath, lock, "1500"], {
167+
// Hold 600ms: the parent's 300ms acquisition attempt below must run
168+
// entirely while the holder still holds (2x headroom), and the parent
169+
// then waits for the holder to release, so the hold is the test's floor.
170+
const proc = Bun.spawn(["bun", "run", holderPath, lock, "600"], {
168171
stdout: "pipe",
169172
stderr: "pipe",
170173
});

0 commit comments

Comments
 (0)