test(uninstall): 30s budgets for the two subprocess-heavy behavioral tests

Both tests spawn the copied uninstaller, which itself runs several
settings-hook bun -e children (the lock-contention one also waits out a
300ms give-up per call). On a loaded box those cold starts blow bun's
default 5s per-test timeout, and a timeout kill reports as a bare fail
with no assertion diff — observed at 5.6-8.5s under load avg 25+.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This commit is contained in:
Garry Tan
2026-08-18 17:05:55 -07:00
co-authored by Claude Fable 5
parent 7a88125a8b
commit 8acb268da3
+7 -2
View File
@@ -296,7 +296,9 @@ describe('hook cleanup runs before the install root is deleted', () => {
} finally {
fs.rmSync(tmp, { recursive: true, force: true });
}
});
// 30s: the copied uninstaller spawns several bun -e children; on a loaded
// box their cold starts blow bun's default 5s per-test timeout.
}, 30000);
});
describe('hook cleanup under lock contention is loud, never silent (review-army)', () => {
@@ -347,5 +349,8 @@ describe('hook cleanup under lock contention is loud, never silent (review-army)
} finally {
fs.rmSync(tmp, { recursive: true, force: true });
}
});
// 30s: several settings-hook calls each wait out the 300ms lock give-up,
// and their bun -e cold starts stack up under load — the default 5s
// per-test budget is too tight on a busy box.
}, 30000);
});