Skip to content

test(server): isolate port reclaim fixture identity and clock #7706

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
lidge-jun merged 11 commits into from
Oct 26, 2026
Merged

test(server): isolate port reclaim fixture identity and clock #7706

Changes from all commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
9e5a917
fix(test): promote Bun 1.4.4 with independent runner role verification
luvs01 Oct 23, 2026
1deb67d
docs: align Docker Bun comments with unified role defaults
luvs01 Oct 23, 2026
41f08ff
test: use a draining TLS origin for early upload replies
luvs01 Oct 23, 2026
a9c3d12
test: confine draining origin to buffered HTTP/1 upload
luvs01 Oct 23, 2026
f011a94
merge: align test-runner role branch with dev 2.93.0 dependency baseline
luvs01 Oct 24, 2026
8b4c34c
Merge branch 'dev' into codex/test-runner-role-verification-20261023
lidge-jun Oct 24, 2026
f091b1c
test(server): isolate port reclaim fixture identity and clock
luvs01 Oct 24, 2026
967a885
test: observe body before buffered early upload reply
luvs01 Oct 24, 2026
ba67ddb
test: close direct upload before fixture teardown
luvs01 Oct 24, 2026
226b45d
Merge commit 'ba67ddb501cc9ee2871025d8a6b812074afbdd08' into codex/77…
luvs01 Oct 24, 2026
c530d3a
Merge dev into port fixture parent preserving deterministic clock
luvs01 Oct 25, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 13 additions & 6 deletions tests/server/port-reclaim.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,8 +29,14 @@ async function reclaimWithMockClock(options: ReclaimListenPortOptions): Promise<
const clock = spyOn(Date, "now").mockImplementation(() => now);
try {
return await reclaimListenPort(10100, "127.0.0.1", {
...options, timeoutMs: 50, intervalMs: 10, scanIntervalMs: 10,
sleepMs: async () => { now += 10; },
...options,
timeoutMs: options.timeoutMs ?? 50,
intervalMs: options.intervalMs ?? 10,
scanIntervalMs: options.scanIntervalMs ?? 10,
sleepMs: async ms => {
now += ms;
await options.sleepMs?.(ms);
},
});
} finally {
clock.mockRestore();
Expand Down Expand Up @@ -880,16 +886,17 @@ describe("reclaimListenPort", () => {

test("killAllOcxOnPort kills ocx listeners absent from the allowlist snapshot", async () => {
const killed: number[] = [];
let holder = 9001;
const holderPid = process.pid === 9001 ? 9002 : 9001;
let holder = holderPid;
let available = false;
await expect(reclaimListenPort(10100, "127.0.0.1", {
await expect(reclaimWithMockClock({
timeoutMs: 200,
intervalMs: 20,
scanIntervalMs: 20,
dropTcpRows: false,
killOcxHolders: true,
killAllOcxOnPort: true,
onlyKillPids: [100], // pre-update PID — respawned child is 9001
onlyKillPids: [100], // pre-update PID — the respawned holder is deliberately not self
isAvailableFn: async () => available,
listListenPidsFn: () => (holder > 0 ? [holder] : []),
isAliveFn: pid => pid === holder,
Expand All @@ -902,7 +909,7 @@ describe("reclaimListenPort", () => {
if (holder === 0) available = true;
},
})).resolves.toBe(true);
expect(killed).toEqual([9001]);
expect(killed).toEqual([holderPid]);
});

test("foreign non-ocx claimant survives reclaim without allowlist or ocx identity", async () => {
Expand Down
Loading