test: guard outbound helper calls · openclaw/openclaw@7e97ddc

Recent Commits to openclaw:main

steipete · 2026-05-12 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -52,6 +52,18 @@ async function importChannelResolution(scope: string) {`
`52`	`52`	`);`
`53`	`53`	`}`
`54`	`54`
	`55`	`+function firstMockArg(mock: { mock: { calls: readonly unknown[][] } }): Record<string, unknown> {`
	`56`	`+const [call] = mock.mock.calls;`
	`57`	`+if (!call) {`
	`58`	`+throw new Error("expected mock call");`
	`59`	`+}`
	`60`	`+const [arg] = call;`
	`61`	`+if (typeof arg !== "object" \|\| arg === null \|\| Array.isArray(arg)) {`
	`62`	`+throw new Error("expected mock call arg to be an object");`
	`63`	`+}`
	`64`	`+return arg as Record<string, unknown>;`
	`65`	`+}`
	`66`	`+`
`55`	`67`	`describe("outbound channel resolution", () => {`
`56`	`68`	`beforeEach(async () => {`
`57`	`69`	`resolveDefaultAgentIdMock.mockReset();`
`@@ -142,12 +154,12 @@ describe("outbound channel resolution", () => {`
`142`	`154`	`).toBe(plugin);`
`143`	`155`	`expect(applyPluginAutoEnableMock).toHaveBeenCalledWith({ config: { channels: {} } });`
`144`	`156`	`expect(resolveRuntimePluginRegistryMock).toHaveBeenCalledOnce();`
`145`		`-const registryOptions = resolveRuntimePluginRegistryMock.mock.calls[0]?.[0];`
`146`		`-expect(registryOptions?.config).toEqual({ autoEnabled: true });`
`147`		`-expect(registryOptions?.activationSourceConfig).toEqual({ channels: {} });`
`148`		`-expect(registryOptions?.autoEnabledReasons).toEqual({});`
`149`		`-expect(registryOptions?.workspaceDir).toBe("/tmp/workspace");`
`150`		`-expect(registryOptions?.runtimeOptions).toEqual({`
	`157`	`+const registryOptions = firstMockArg(resolveRuntimePluginRegistryMock);`
	`158`	`+expect(registryOptions.config).toEqual({ autoEnabled: true });`
	`159`	`+expect(registryOptions.activationSourceConfig).toEqual({ channels: {} });`
	`160`	`+expect(registryOptions.autoEnabledReasons).toEqual({});`
	`161`	`+expect(registryOptions.workspaceDir).toBe("/tmp/workspace");`
	`162`	`+expect(registryOptions.runtimeOptions).toEqual({`
`151`	`163`	`allowGatewaySubagentBinding: true,`
`152`	`164`	`});`
`153`	`165`	`});`

Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,8 @@ function isRecord(value: unknown): value is Record<string, unknown> {`
`21`	`21`	`}`
`22`	`22`
`23`	`23`	`function readFirstPluginCall(mock: { mock: { calls: unknown[][] } }): Record<string, unknown> {`
`24`		`-const call = mock.mock.calls[0]?.[0];`
	`24`	`+const [mockCall] = mock.mock.calls;`
	`25`	`+const call = mockCall?.[0];`
`25`	`26`	`if (!isRecord(call)) {`
`26`	`27`	`throw new Error("expected plugin action call");`
`27`	`28`	`}`
`@@ -32,7 +33,8 @@ function readPluginCall(`
`32`	`33`	`mock: { mock: { calls: unknown[][] } },`
`33`	`34`	`callIndex: number,`
`34`	`35`	`): Record<string, unknown> {`
`35`		`-const call = mock.mock.calls[callIndex]?.[0];`
	`36`	`+const mockCall = mock.mock.calls[callIndex];`
	`37`	`+const call = mockCall?.[0];`
`36`	`38`	`if (!isRecord(call)) {`
`37`	`39`	throw new Error(`expected plugin action call ${callIndex}`);
`38`	`40`	`}`
`@@ -49,7 +51,8 @@ function readMockCallArg(`
`49`	`51`	`callIndex = 0,`
`50`	`52`	`argIndex = 0,`
`51`	`53`	`): Record<string, unknown> {`
`52`		`-const value = mock.mock.calls[callIndex]?.[argIndex];`
	`54`	`+const mockCall = mock.mock.calls[callIndex];`
	`55`	`+const value = mockCall?.[argIndex];`
`53`	`56`	`if (!isRecord(value)) {`
`54`	`57`	throw new Error(`expected ${label}`);
`55`	`58`	`}`

Original file line number	Diff line number	Diff line change
`@@ -9,6 +9,18 @@ import {`
`9`	`9`	`const ensureOutboundSessionEntry = vi.fn(async () => undefined);`
`10`	`10`	`const resolveOutboundSessionRoute = vi.fn();`
`11`	`11`
	`12`	`+function firstMockArg(mock: { mock: { calls: readonly unknown[][] } }): Record<string, unknown> {`
	`13`	`+const [call] = mock.mock.calls;`
	`14`	`+if (!call) {`
	`15`	`+throw new Error("expected mock call");`
	`16`	`+}`
	`17`	`+const [arg] = call;`
	`18`	`+if (typeof arg !== "object" \|\| arg === null \|\| Array.isArray(arg)) {`
	`19`	`+throw new Error("expected mock call arg to be an object");`
	`20`	`+}`
	`21`	`+return arg as Record<string, unknown>;`
	`22`	`+}`
	`23`	`+`
`12`	`24`	`const workspaceConfig = {`
`13`	`25`	`channels: {`
`14`	`26`	`workspace: {`
`@@ -181,7 +193,7 @@ describe("message action threading helpers", () => {`
`181`	`193`	`});`
`182`	`194`
`183`	`195`	`expect(resolveAutoThreadId).toHaveBeenCalledOnce();`
`184`		`-expect(resolveAutoThreadId.mock.calls[0]?.[0]?.replyToId).toBe("777");`
	`196`	`+expect(firstMockArg(resolveAutoThreadId).replyToId).toBe("777");`
`185`	`197`	`expect(resolved).toBe("thread-777");`
`186`	`198`	`expect(actionParams.threadId).toBe("thread-777");`
`187`	`199`	`});`

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

推荐订阅源

Recent Commits to openclaw:main