puter/paseo
1
0
Fork 0
mirror of https://github.com/getpaseo/paseo.git synced 2026-11-09 09:54:40 +00:00
Code Issues Packages Projects Releases Wiki Activity

test(server): reserve the old service port during respawn (#9153)

Browse Source 
The OS can reuse a released ephemeral port. Occupy it before respawn so the port-change assertion has a deterministic precondition, and verify the spawned terminal receives the refreshed port.

Co-authored-by: paseo-bot[bot] <266920839+paseo-bot[bot]@users.noreply.github.com>
This commit is contained in:
paseo-bot[bot]
2026-11-09 03:35:39 +00:00
committed by GitHub
parent fdee02bdb6
commit 1fff127f45
1 changed files with 13 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/server/src/server/worktree-bootstrap.test.ts
+13 -1
View File
@@ -1,8 +1,10 @@
import { describe, it, expect, beforeEach, afterEach } from "vitest";
import { describe, it, expect, beforeEach, afterEach, onTestFinished } from "vitest";
import { execFileSync } from "child_process";
import { mkdtempSync, realpathSync, rmSync, writeFileSync, mkdirSync } from "fs";
import { join } from "path";
import { tmpdir } from "os";
import { createServer } from "node:net";
import { promisify } from "node:util";
import {
buildWorktreeSetupDetail,
runWorktreeAutoTerminals,
@@ -943,6 +945,15 @@ describe("runWorktreeAutoTerminals", () => {
});
expect(routeStore.getRouteEntry("api--feature-respawn-service--repo.localhost")).toBeNull();
// A released ephemeral port can be allocated again. Occupy the old port so
// this journey requires a different port when the service is restarted.
const occupiedPort = createServer();
onTestFinished(() => promisify(occupiedPort.close.bind(occupiedPort))());
await new Promise<void>((resolve, reject) => {
occupiedPort.once("error", reject);
occupiedPort.listen(firstPort, "127.0.0.1", resolve);
});
const secondResult = await spawnWorkspaceScript({
repoRoot: repoDir,
workspaceId: repoDir,
@@ -962,6 +973,7 @@ describe("runWorktreeAutoTerminals", () => {
}
expect(secondPort).not.toBe(firstPort);
expect(secondPort).toEqual(expect.any(Number));
expect(createTerminalCalls[2]?.env?.PASEO_PORT).toBe(String(secondPort));
expect(createTerminalCalls[2]?.env?.PASEO_SERVICE_WORKER_PORT).toBe(String(workerPort));
expect(routeStore.getRouteEntry("api--feature-respawn-service--repo.localhost")).toMatchObject({
hostname: "api--feature-respawn-service--repo.localhost",