惯性聚合 高效追踪和阅读你感兴趣的博客、新闻、科技资讯
阅读原文 在惯性聚合中打开

推荐订阅源

博客园 - 聂微东
Y
Y Combinator Blog
WordPress大学
WordPress大学
L
LangChain Blog
钛媒体:引领未来商业与生活新知
钛媒体:引领未来商业与生活新知
A
About on SuperTechFans
小众软件
小众软件
有赞技术团队
有赞技术团队
S
SegmentFault 最新的问题
宝玉的分享
宝玉的分享
Recent Announcements
Recent Announcements
GbyAI
GbyAI
I
InfoQ
The GitHub Blog
The GitHub Blog
让小产品的独立变现更简单 - ezindie.com
让小产品的独立变现更简单 - ezindie.com
酷 壳 – CoolShell
酷 壳 – CoolShell
罗磊的独立博客
C
Check Point Blog
V
V2EX
Apple Machine Learning Research
Apple Machine Learning Research
月光博客
月光博客
量子位
雷峰网
雷峰网
Hugging Face - Blog
Hugging Face - Blog

Recent Commits to openclaw:main

test: merge chat side-result checks · openclaw/openclaw@ddd2c2a test: merge cron history checks · openclaw/openclaw@f7eb746 test: merge responsive navigation shell checks · openclaw/openclaw@c2e4b47 docs(changelog): add codex oauth fixes · openclaw/openclaw@628e6cd test: merge navigation routing cases · openclaw/openclaw@5d8cecb Tests: mock channel registry bundled fallback · openclaw/openclaw@2b08233 Secrets: avoid broad web search discovery for single plugin config · openclaw/openclaw@a464f59 test: merge config view browser checks · openclaw/openclaw@20cf511 fix(status): align oauth health with runtime · openclaw/openclaw@eed7116 feat: add macOS screen snapshots for monitor preview (#67954) thanks … · openclaw/openclaw@f377db1 fix: report shared auth scopes in hello-ok (#67810) thanks @BunsDev · openclaw/openclaw@0b6c39b Auto-reply: avoid eager bundled route fallback · openclaw/openclaw@3ea1bf4 Tests: narrow session binding contract setup · openclaw/openclaw@54e4e16 fix(macOS): enable undo/redo in webchat composer text input (#34962) · openclaw/openclaw@00951dc Tests: speed up channel setup promotion · openclaw/openclaw@82b529a Docs: refresh agent instructions · openclaw/openclaw@5775fe2 fix(auth): serialize OAuth refresh across agents to fix #26322 (#67876) · openclaw/openclaw@8e79080 test: allow ollama public surface boundary test · openclaw/openclaw@7d4f1a6 Docs: add test performance guardrails · openclaw/openclaw@89706d3 Tests: restore context-engine usage proof · openclaw/openclaw@e4c4f95 Tests: slim context engine runtime coverage · openclaw/openclaw@74c198f ci: retry failed custom checkouts · openclaw/openclaw@0ee5baf test: trim duplicate provider auth onboarding cases · openclaw/openclaw@1ffc02e matrix: fix sessions_spawn --thread subagent session spawning (#67643) · openclaw/openclaw@1ce2596 test: reduce auth choice fixture churn · openclaw/openclaw@857b9cd test: mock health status config boundaries · openclaw/openclaw@9d5ab4a test: mock onboard config io boundary · openclaw/openclaw@299694d test: mock legacy state plugin boundaries · openclaw/openclaw@2713089 test: mock channel install boundaries · openclaw/openclaw@b945248 test: mock doctor preview channel boundaries · openclaw/openclaw@b1a3ad4
test: stabilize gateway websocket startup assertion · ope...
shakkernerd · 2026-05-09 · via Recent Commits to openclaw:main

@@ -1,131 +1,100 @@

1-

import http from "node:http";

21

import { afterEach, describe, expect, it, vi } from "vitest";

3-

import { connectGatewayClient, disconnectGatewayClient } from "./test-helpers.e2e.js";

2+

import { WebSocket } from "ws";

3+

import { createEmptyPluginRegistry } from "../plugins/registry.js";

4+

import { createGatewayRuntimeState } from "./server-runtime-state.js";

45

import { getFreePort, installGatewayTestHooks, startGatewayServer } from "./test-helpers.js";

566-

const machineNameDelay = vi.hoisted(() => {

7-

let enteredResolve = () => {};

8-

let releaseResolve = () => {};

9-

let entered = new Promise<void>((resolve) => {

10-

enteredResolve = resolve;

11-

});

12-

let release = new Promise<void>((resolve) => {

13-

releaseResolve = resolve;

14-

});

15-

return {

16-

waitUntilDelayed: async () => {

17-

await entered;

18-

},

19-

release: () => {

20-

releaseResolve();

21-

},

22-

reset: () => {

23-

entered = new Promise<void>((resolve) => {

24-

enteredResolve = resolve;

25-

});

26-

release = new Promise<void>((resolve) => {

27-

releaseResolve = resolve;

28-

});

29-

},

30-

run: async () => {

31-

enteredResolve();

32-

await release;

33-

return "test-machine";

34-

},

35-

};

36-

});

37-38-

vi.mock("../infra/machine-name.js", () => ({

39-

getMachineDisplayName: () => machineNameDelay.run(),

40-

}));

7+

type StartGatewayServer = typeof import("./test-helpers.js").startGatewayServer;

8+

type GatewayServerForTest = Awaited<ReturnType<StartGatewayServer>>;

4194210

installGatewayTestHooks({ scope: "suite" });

431112+

async function connectWebSocket(url: string): Promise<WebSocket> {

13+

const ws = new WebSocket(url);

14+

return await new Promise<WebSocket>((resolve, reject) => {

15+

const timeout = setTimeout(() => {

16+

cleanup();

17+

ws.close();

18+

reject(new Error("expected websocket connect to succeed immediately after startup"));

19+

}, 5_000);

20+

timeout.unref?.();

21+

const cleanup = () => {

22+

clearTimeout(timeout);

23+

ws.off("open", handleOpen);

24+

ws.off("error", handleError);

25+

};

26+

const handleOpen = () => {

27+

cleanup();

28+

resolve(ws);

29+

};

30+

const handleError = (err: Error) => {

31+

cleanup();

32+

reject(err);

33+

};

34+

ws.once("open", handleOpen);

35+

ws.once("error", handleError);

36+

});

37+

}

38+39+

async function disconnectWebSocket(ws: WebSocket): Promise<void> {

40+

if (ws.readyState === WebSocket.CLOSED) {

41+

return;

42+

}

43+

await new Promise<void>((resolve) => {

44+

ws.once("close", () => resolve());

45+

ws.close();

46+

});

47+

}

48+4449

afterEach(() => {

45-

machineNameDelay.release();

50+

vi.restoreAllMocks();

4651

});

47524853

describe("gateway startup websocket readiness", () => {

49-

it("does not bind the websocket port until websocket handlers are attached", async () => {

50-

machineNameDelay.reset();

51-

const previousMinimal = process.env.OPENCLAW_TEST_MINIMAL_GATEWAY;

52-

process.env.OPENCLAW_TEST_MINIMAL_GATEWAY = "0";

53-

let server: Awaited<ReturnType<typeof startGatewayServer>> | undefined;

54+

it("attaches websocket upgrade handlers before exposing the listen step", async () => {

55+

const registry = createEmptyPluginRegistry();

56+

const runtimeState = await createGatewayRuntimeState({

57+

cfg: {},

58+

bindHost: "127.0.0.1",

59+

port: 0,

60+

controlUiEnabled: false,

61+

controlUiBasePath: "/",

62+

openAiChatCompletionsEnabled: false,

63+

openResponsesEnabled: false,

64+

resolvedAuth: {} as never,

65+

getResolvedAuth: () => ({}) as never,

66+

hooksConfig: () => null,

67+

getHookClientIpConfig: () => ({}) as never,

68+

pluginRegistry: registry,

69+

deps: {} as never,

70+

log: { info: () => {}, warn: () => {} },

71+

logHooks: { info: () => {}, warn: () => {}, error: () => {}, debug: () => {} } as never,

72+

logPlugins: { info: () => {}, warn: () => {}, error: () => {}, debug: () => {} } as never,

73+

});

5474

try {

55-

const port = await getFreePort();

56-

const startup = startGatewayServer(port, {

57-

auth: { mode: "none" },

58-

});

59-

await machineNameDelay.waitUntilDelayed();

60-61-

const pendingUpgrade = await new Promise<

62-

{ kind: "error"; code?: string } | { kind: "response"; status: number; body: string }

63-

>((resolve) => {

64-

const req = http.request({

65-

host: "127.0.0.1",

66-

port,

67-

path: "/",

68-

headers: {

69-

Connection: "Upgrade",

70-

Upgrade: "websocket",

71-

"Sec-WebSocket-Key": "dGVzdC1rZXktMDEyMzQ1Ng==",

72-

"Sec-WebSocket-Version": "13",

73-

},

74-

});

75-

req.once("error", (err) => {

76-

resolve({ kind: "error", code: (err as NodeJS.ErrnoException).code });

77-

});

78-

req.once("response", (res) => {

79-

let body = "";

80-

res.setEncoding("utf8");

81-

res.on("data", (chunk) => {

82-

body += chunk;

83-

});

84-

res.once("end", () => {

85-

resolve({ kind: "response", status: res.statusCode ?? 0, body });

86-

});

87-

});

88-

req.end();

89-

});

90-91-

expect(pendingUpgrade).toEqual({ kind: "error", code: "ECONNREFUSED" });

92-93-

machineNameDelay.release();

94-

server = await startup;

75+

expect(runtimeState.httpBindHosts).toEqual([]);

76+

expect(runtimeState.httpServer.listenerCount("upgrade")).toBeGreaterThan(0);

9577

} finally {

96-

machineNameDelay.release();

97-

if (server) {

98-

await server.close();

99-

}

100-

if (previousMinimal === undefined) {

101-

delete process.env.OPENCLAW_TEST_MINIMAL_GATEWAY;

102-

} else {

103-

process.env.OPENCLAW_TEST_MINIMAL_GATEWAY = previousMinimal;

104-

}

78+

runtimeState.releasePluginRouteRegistry();

79+

runtimeState.wss.close();

10580

}

10681

});

1078210883

it("accepts an immediate websocket connection once startup resolves", async () => {

109-

machineNameDelay.reset();

11084

const previousMinimal = process.env.OPENCLAW_TEST_MINIMAL_GATEWAY;

11185

process.env.OPENCLAW_TEST_MINIMAL_GATEWAY = "0";

112-

let server: Awaited<ReturnType<typeof startGatewayServer>> | undefined;

113-

let client: Awaited<ReturnType<typeof connectGatewayClient>> | undefined;

86+

let server: GatewayServerForTest | undefined;

87+

let client: WebSocket | undefined;

11488

try {

11589

const port = await getFreePort();

116-

machineNameDelay.release();

11790

server = await startGatewayServer(port, {

11891

auth: { mode: "none" },

11992

});

12093121-

client = await connectGatewayClient({

122-

url: `ws://127.0.0.1:${port}`,

123-

timeoutMs: 5_000,

124-

timeoutMessage: "expected websocket connect to succeed immediately after startup",

125-

});

94+

client = await connectWebSocket(`ws://127.0.0.1:${port}`);

12695

} finally {

12796

if (client) {

128-

await disconnectGatewayClient(client);

97+

await disconnectWebSocket(client);

12998

}

13099

if (server) {

131100

await server.close();