test: guard extension runtime mock calls · openclaw/openc...

Recent Commits to openclaw:main

monitor-inbox.blocks-messages-from-unauthorized-senders-not-allo · 2026-05-12 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -74,7 +74,7 @@ describe("loginOpenAICodexDeviceCode", () => {`
`74`	`74`	`await vi.advanceTimersByTimeAsync(1);`
`75`	`75`	`const credentials = await credentialsPromise;`
`76`	`76`
`77`		`-const userCodeRequest = fetchMock.mock.calls[0];`
	`77`	`+const userCodeRequest = fetchMock.mock.calls.at(0);`
`78`	`78`	`expect(userCodeRequest?.[0]).toBe("https://auth.openai.com/api/accounts/deviceauth/usercode");`
`79`	`79`	`expect(userCodeRequest?.[1]?.method).toBe("POST");`
`80`	`80`	`expect(userCodeRequest?.[1]?.headers).toEqual({`
`@@ -84,7 +84,7 @@ describe("loginOpenAICodexDeviceCode", () => {`
`84`	`84`	`"User-Agent": "openclaw/2026.3.22",`
`85`	`85`	`});`
`86`	`86`
`87`		`-const deviceTokenRequest = fetchMock.mock.calls[1];`
	`87`	`+const deviceTokenRequest = fetchMock.mock.calls.at(1);`
`88`	`88`	`expect(deviceTokenRequest?.[0]).toBe("https://auth.openai.com/api/accounts/deviceauth/token");`
`89`	`89`	`expect(deviceTokenRequest?.[1]?.method).toBe("POST");`
`90`	`90`	`expect(deviceTokenRequest?.[1]?.headers).toEqual({`
`@@ -94,7 +94,7 @@ describe("loginOpenAICodexDeviceCode", () => {`
`94`	`94`	`"User-Agent": "openclaw/2026.3.22",`
`95`	`95`	`});`
`96`	`96`
`97`		`-const oauthTokenRequest = fetchMock.mock.calls[3];`
	`97`	`+const oauthTokenRequest = fetchMock.mock.calls.at(3);`
`98`	`98`	`expect(oauthTokenRequest?.[0]).toBe("https://auth.openai.com/oauth/token");`
`99`	`99`	`expect(oauthTokenRequest?.[1]?.method).toBe("POST");`
`100`	`100`	`expect(oauthTokenRequest?.[1]?.headers).toEqual({`

Original file line number	Diff line number	Diff line change
`@@ -457,16 +457,16 @@ describe("buildOpenAIRealtimeVoiceProvider", () => {`
`457`	`457`	`instructions: "Be concise.",`
`458`	`458`	`});`
`459`	`459`
`460`		`-expect(execFileSyncMock.mock.calls[0]?.[0]).toBe("/usr/bin/security");`
`461`		`-expect(execFileSyncMock.mock.calls[0]?.[1]).toEqual([`
	`460`	`+expect(execFileSyncMock.mock.calls.at(0)?.[0]).toBe("/usr/bin/security");`
	`461`	`+expect(execFileSyncMock.mock.calls.at(0)?.[1]).toEqual([`
`462`	`462`	`"find-generic-password",`
`463`	`463`	`"-s",`
`464`	`464`	`"openclaw",`
`465`	`465`	`"-a",`
`466`	`466`	`"OPENAI_REALTIME_BROWSER_TEST",`
`467`	`467`	`"-w",`
`468`	`468`	`]);`
`469`		`-expectRecordFields(execFileSyncMock.mock.calls[0]?.[2], "security command options", {`
	`469`	`+expectRecordFields(execFileSyncMock.mock.calls.at(0)?.[2], "security command options", {`
`470`	`470`	`encoding: "utf8",`
`471`	`471`	`timeout: 5000,`
`472`	`472`	`});`

Original file line number	Diff line number	Diff line change
`@@ -78,11 +78,11 @@ describe("handleQaInbound", () => {`
`78`	`78`	`);`
`79`	`79`
`80`	`80`	`expect(runtime.channel.turn.runAssembled).toHaveBeenCalledTimes(1);`
`81`		`-expect(vi.mocked(runtime.channel.turn.runAssembled).mock.calls[0]?.[0].replyPipeline).toEqual(`
`82`		`-{},`
`83`		`-);`
`84`	`81`	`expect(`
`85`		`-vi.mocked(runtime.channel.turn.runAssembled).mock.calls[0]?.[0].ctxPayload.WasMentioned,`
	`82`	`+vi.mocked(runtime.channel.turn.runAssembled).mock.calls.at(0)?.[0].replyPipeline,`
	`83`	`+).toEqual({});`
	`84`	`+expect(`
	`85`	`+vi.mocked(runtime.channel.turn.runAssembled).mock.calls.at(0)?.[0].ctxPayload.WasMentioned,`
`86`	`86`	`).toBe(true);`
`87`	`87`	`});`
`88`	`88`
`@@ -114,7 +114,9 @@ describe("handleQaInbound", () => {`
`114`	`114`	`);`
`115`	`115`
`116`	`116`	`expect(runtime.channel.turn.runAssembled).toHaveBeenCalledTimes(1);`
`117`		`-const ctxPayload = vi.mocked(runtime.channel.turn.runAssembled).mock.calls[0]?.[0].ctxPayload;`
	`117`	`+const ctxPayload = vi`
	`118`	`+.mocked(runtime.channel.turn.runAssembled)`
	`119`	`+.mock.calls.at(0)?.[0].ctxPayload;`
`118`	`120`	`expect(ctxPayload?.CommandAuthorized).toBe(true);`
`119`	`121`	`expect(ctxPayload?.SenderId).toBe("alice");`
`120`	`122`	`});`

Original file line number	Diff line number	Diff line change
`@@ -81,7 +81,7 @@ describe("startQaLiveLaneGateway", () => {`
`81`	`81`	`});`
`82`	`82`
`83`	`83`	`expect(startQaProviderServer).toHaveBeenCalledWith("mock-openai");`
`84`		`-const gatewayOptions = startQaGatewayChild.mock.calls[0]?.[0] as`
	`84`	`+const gatewayOptions = startQaGatewayChild.mock.calls.at(0)?.[0] as`
`85`	`85`	`\| { providerBaseUrl?: string; providerMode?: string; transportBaseUrl?: string }`
`86`	`86`	`\| undefined;`
`87`	`87`	`expect(gatewayOptions?.transportBaseUrl).toBe("http://127.0.0.1:43123");`
`@@ -104,7 +104,7 @@ describe("startQaLiveLaneGateway", () => {`
`104`	`104`	`controlUiEnabled: false,`
`105`	`105`	`});`
`106`	`106`
`107`		`-const [{ mutateConfig }] = startQaGatewayChild.mock.calls[0] ?? [];`
	`107`	`+const [{ mutateConfig }] = startQaGatewayChild.mock.calls.at(0) ?? [];`
`108`	`108`	`if (!mutateConfig) {`
`109`	`109`	`throw new Error("expected gateway config mutator");`
`110`	`110`	`}`
`@@ -173,7 +173,7 @@ describe("startQaLiveLaneGateway", () => {`
`173`	`173`	`});`
`174`	`174`
`175`	`175`	`expect(startQaProviderServer).toHaveBeenCalledWith("live-frontier");`
`176`		`-const gatewayOptions = startQaGatewayChild.mock.calls[0]?.[0] as`
	`176`	`+const gatewayOptions = startQaGatewayChild.mock.calls.at(0)?.[0] as`
`177`	`177`	`\| { providerBaseUrl?: string; providerMode?: string; transportBaseUrl?: string }`
`178`	`178`	`\| undefined;`
`179`	`179`	`expect(gatewayOptions?.transportBaseUrl).toBe("http://127.0.0.1:43123");`

Original file line number	Diff line number	Diff line change
`@@ -94,7 +94,7 @@ describe("qa suite runtime agent process helpers", () => {`
`94`	`94`	`child.emit("exit", 0);`
`95`	`95`
`96`	`96`	`await expect(pending).resolves.toBe("ok");`
`97`		`-const spawnCall = spawnMock.mock.calls[0];`
	`97`	`+const spawnCall = spawnMock.mock.calls.at(0);`
`98`	`98`	`expect(spawnCall?.[0]).toBe("/usr/bin/node");`
`99`	`99`	`expect(spawnCall?.[1]).toEqual([path.join("/repo", "dist", "index.js"), "qa", "suite"]);`
`100`	`100`	`expect((spawnCall?.[2] as { cwd?: string; env?: unknown } \| undefined)?.cwd).toBe(`
`@@ -132,7 +132,7 @@ describe("qa suite runtime agent process helpers", () => {`
`132`	`132`	`child.emit("exit", 0);`
`133`	`133`
`134`	`134`	`await expect(pending).resolves.toBe("ok");`
`135`		`-const spawnCall = spawnMock.mock.calls[0];`
	`135`	`+const spawnCall = spawnMock.mock.calls.at(0);`
`136`	`136`	`expect(spawnCall?.[0]).toBe("/usr/bin/node");`
`137`	`137`	`expect(spawnCall?.[1]).toEqual([`
`138`	`138`	`path.join("/repo", "dist", "index.js"),`
`@@ -253,7 +253,7 @@ describe("qa suite runtime agent process helpers", () => {`
`253`	`253`	`message: "hello",`
`254`	`254`	`}),`
`255`	`255`	`).resolves.toEqual({ runId: "run-1" });`
`256`		`-const gatewayArgs = gatewayCall.mock.calls[0] as unknown as`
	`256`	`+const gatewayArgs = gatewayCall.mock.calls.at(0) as unknown as`
`257`	`257`	`\| [string, unknown, unknown]`
`258`	`258`	`\| undefined;`
`259`	`259`	`expect(gatewayArgs?.[0]).toBe("agent");`

Original file line number	Diff line number	Diff line change
`@@ -377,7 +377,10 @@ describe("containerSendMessage", () => {`
`377`	`377`	`textStyles: [{ start: 0, length: 4, style: "BOLD" }],`
`378`	`378`	`});`
`379`	`379`
`380`		`-const callArgs = mockFetch.mock.calls[0];`
	`380`	`+const callArgs = mockFetch.mock.calls.at(0);`
	`381`	`+if (callArgs === undefined) {`
	`382`	`+throw new Error("expected container fetch call");`
	`383`	`+}`
`381`	`384`	`const body = JSON.parse(callArgs[1].body);`
`382`	`385`	`expect(body.message).toBe("Bold \\* not italic");`
`383`	`386`	`});`
`@@ -397,7 +400,10 @@ describe("containerSendMessage", () => {`
`397`	`400`	`textStyles: [{ start: 0, length: 4, style: "BOLD" }],`
`398`	`401`	`});`
`399`	`402`
`400`		`-const callArgs = mockFetch.mock.calls[0];`
	`403`	`+const callArgs = mockFetch.mock.calls.at(0);`
	`404`	`+if (callArgs === undefined) {`
	`405`	`+throw new Error("expected container fetch call");`
	`406`	`+}`
`401`	`407`	`const body = JSON.parse(callArgs[1].body);`
`402`	`408`	`expect(body.message).toBe("Bold C:\\Temp\\file and /foo\\bar/");`
`403`	`409`	`});`
`@@ -427,7 +433,10 @@ describe("containerSendMessage", () => {`
`427`	`433`	`attachments: [tmpFile],`
`428`	`434`	`});`
`429`	`435`
`430`		`-const callArgs = mockFetch.mock.calls[0];`
	`436`	`+const callArgs = mockFetch.mock.calls.at(0);`
	`437`	`+if (callArgs === undefined) {`
	`438`	`+throw new Error("expected container fetch call");`
	`439`	`+}`
`431`	`440`	`const body = JSON.parse(callArgs[1].body);`
`432`	`441`	`expect(body.attachments).toBeUndefined();`
`433`	`442`	`if (!Array.isArray(body.base64_attachments)) {`

Original file line number	Diff line number	Diff line change
`@@ -234,15 +234,15 @@ describe("telegramPlugin outbound attachments", () => {`
`234`	`234`	`to: "12345",`
`235`	`235`	`text: "hi boss",`
`236`	`236`	`});`
`237`		`-expect(sendMessageTelegram.mock.calls[0]?.[2]).not.toHaveProperty("textMode");`
	`237`	`+expect(sendMessageTelegram.mock.calls.at(0)?.[2]).not.toHaveProperty("textMode");`
`238`	`238`
`239`	`239`	`await sendText({`
`240`	`240`	`cfg: createTelegramConfig(),`
`241`	`241`	`to: "12345",`
`242`	`242`	`text: "<b>hi boss</b>",`
`243`	`243`	`formatting: { parseMode: "HTML" },`
`244`	`244`	`});`
`245`		`-expect(sendMessageTelegram.mock.calls[1]?.[2]?.textMode).toBe("html");`
	`245`	`+expect(sendMessageTelegram.mock.calls.at(1)?.[2]?.textMode).toBe("html");`
`246`	`246`	`});`
`247`	`247`
`248`	`248`	`it("preserves explicit HTML parse mode for payload media captions", async () => {`
`@@ -264,6 +264,6 @@ describe("telegramPlugin outbound attachments", () => {`
`264`	`264`	`formatting: { parseMode: "HTML" },`
`265`	`265`	`});`
`266`	`266`
`267`		`-expect(sendMessageTelegram.mock.calls[0]?.[2]?.textMode).toBe("html");`
	`267`	`+expect(sendMessageTelegram.mock.calls.at(0)?.[2]?.textMode).toBe("html");`
`268`	`268`	`});`
`269`	`269`	`});`

Original file line number	Diff line number	Diff line change
`@@ -41,12 +41,12 @@ describe("vydra video-generation provider", () => {`
`41`	`41`	`cfg: {},`
`42`	`42`	`});`
`43`	`43`
`44`		`-const createCall = fetchMock.mock.calls[0];`
	`44`	`+const createCall = fetchMock.mock.calls.at(0);`
`45`	`45`	`expect(createCall?.[0]).toBe("https://www.vydra.ai/api/v1/models/veo3");`
`46`	`46`	`const createInit = createCall?.[1] as { method?: string; body?: unknown } \| undefined;`
`47`	`47`	`expect(createInit?.method).toBe("POST");`
`48`	`48`	`expect(createInit?.body).toBe(JSON.stringify({ prompt: "tiny city at sunrise" }));`
`49`		`-const pollCall = fetchMock.mock.calls[1];`
	`49`	`+const pollCall = fetchMock.mock.calls.at(1);`
`50`	`50`	`expect(pollCall?.[0]).toBe("https://www.vydra.ai/api/v1/jobs/job-123");`
`51`	`51`	`const pollInit = pollCall?.[1] as { method?: string } \| undefined;`
`52`	`52`	`expect(pollInit?.method).toBe("GET");`
`@@ -101,7 +101,7 @@ describe("vydra video-generation provider", () => {`
`101`	`101`	`inputImages: [{ url: "https://example.com/reference.png" }],`
`102`	`102`	`});`
`103`	`103`
`104`		`-const createCall = fetchMock.mock.calls[0];`
	`104`	`+const createCall = fetchMock.mock.calls.at(0);`
`105`	`105`	`expect(createCall?.[0]).toBe("https://www.vydra.ai/api/v1/models/kling");`
`106`	`106`	`const createInit = createCall?.[1] as { method?: string; body?: unknown } \| undefined;`
`107`	`107`	`expect(createInit?.method).toBe("POST");`

Original file line number	Diff line number	Diff line change
`@@ -136,11 +136,11 @@ describe("loginWeb coverage", () => {`
`136`	`136`	`await loginWeb(false, waitForWaConnectionMock as never, runtime);`
`137`	`137`
`138`	`138`	`expect(createWaSocketMock).toHaveBeenCalledTimes(2);`
`139`		`-expect(createWaSocketMock.mock.calls[0]?.[0]).toBe(false);`
`140`		`-const initialOpts = createWaSocketMock.mock.calls[0]?.[2] as`
	`139`	`+expect(createWaSocketMock.mock.calls.at(0)?.[0]).toBe(false);`
	`140`	`+const initialOpts = createWaSocketMock.mock.calls.at(0)?.[2] as`
`141`	`141`	`\| { onQr?: (qr: string) => void }`
`142`	`142`	`\| undefined;`
`143`		`-const restartOpts = createWaSocketMock.mock.calls[1]?.[2] as`
	`143`	`+const restartOpts = createWaSocketMock.mock.calls.at(1)?.[2] as`
`144`	`144`	`\| { onQr?: (qr: string) => void }`
`145`	`145`	`\| undefined;`
`146`	`146`	`expect(initialOpts?.onQr).toBe(restartOpts?.onQr);`

Original file line number	Diff line number	Diff line change
`@@ -252,7 +252,7 @@ describe("web monitor inbox", () => {`
`252`	`252`	`await settleInboundWork();`
`253`	`253`
`254`	`254`	`expect(onMessage).toHaveBeenCalledTimes(1);`
`255`		`-const payload = onMessage.mock.calls[0][0];`
	`255`	`+const payload = onMessage.mock.calls.at(0)?.[0];`
`256`	`256`	`expect(payload.chatType).toBe("group");`
`257`	`257`	`expect(payload.senderE164).toBe("+999");`
`258`	`258`
`@@ -340,7 +340,7 @@ describe("web monitor inbox", () => {`
`340`	`340`
`341`	`341`	`// Should call onMessage because sender is in groupAllowFrom`
`342`	`342`	`expect(onMessage).toHaveBeenCalledTimes(1);`
`343`		`-const payload = onMessage.mock.calls[0][0];`
	`343`	`+const payload = onMessage.mock.calls.at(0)?.[0];`
`344`	`344`	`expect(payload.chatType).toBe("group");`
`345`	`345`	`expect(payload.senderE164).toBe("+15551234567");`
`346`	`346`
`@@ -374,7 +374,7 @@ describe("web monitor inbox", () => {`
`374`	`374`
`375`	`375`	`// Should call onMessage because wildcard allows all senders`
`376`	`376`	`expect(onMessage).toHaveBeenCalledTimes(1);`
`377`		`-const payload = onMessage.mock.calls[0][0];`
	`377`	`+const payload = onMessage.mock.calls.at(0)?.[0];`
`378`	`378`	`expect(payload.chatType).toBe("group");`
`379`	`379`
`380`	`380`	`await listener.close();`

此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。原文来自 — 版权归原作者所有。

推荐订阅源

Recent Commits to openclaw:main