test: guard command model mock calls · openclaw/openclaw@7a9f770

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
`@@ -145,7 +145,7 @@ describe("agents bind/unbind commands", () => {`
`145`	`145`	`await agentsBindCommand({ bind: ["telegram"] }, runtime);`
`146`	`146`
`147`	`147`	`expect(writeConfigFileMock).toHaveBeenCalledTimes(1);`
`148`		`-const writtenConfig = writeConfigFileMock.mock.calls[0]?.[0] as`
	`148`	`+const writtenConfig = writeConfigFileMock.mock.calls.at(0)?.[0] as`
`149`	`149`	`\| { bindings?: unknown }`
`150`	`150`	`\| undefined;`
`151`	`151`	`expect(writtenConfig?.bindings).toStrictEqual([`
`@@ -163,7 +163,7 @@ describe("agents bind/unbind commands", () => {`
`163`	`163`	`await agentsBindCommand({ bind: ["external-chat:work"] }, runtime);`
`164`	`164`
`165`	`165`	`expect(writeConfigFileMock).toHaveBeenCalledTimes(1);`
`166`		`-const writtenConfig = writeConfigFileMock.mock.calls[0]?.[0] as`
	`166`	`+const writtenConfig = writeConfigFileMock.mock.calls.at(0)?.[0] as`
`167`	`167`	`\| { bindings?: unknown }`
`168`	`168`	`\| undefined;`
`169`	`169`	`expect(writtenConfig?.bindings).toStrictEqual([`
`@@ -192,7 +192,7 @@ describe("agents bind/unbind commands", () => {`
`192`	`192`	`await agentsUnbindCommand({ agent: "ops", all: true }, runtime);`
`193`	`193`
`194`	`194`	`expect(writeConfigFileMock).toHaveBeenCalledTimes(1);`
`195`		`-const writtenConfig = writeConfigFileMock.mock.calls[0]?.[0] as`
	`195`	`+const writtenConfig = writeConfigFileMock.mock.calls.at(0)?.[0] as`
`196`	`196`	`\| { bindings?: unknown }`
`197`	`197`	`\| undefined;`
`198`	`198`	`expect(writtenConfig?.bindings).toStrictEqual([`

Original file line number	Diff line number	Diff line change
`@@ -111,7 +111,7 @@ describe("resolveInstallableChannelPlugin", () => {`
`111`	`111`	`expect(result.plugin?.id).toBe("telegram");`
`112`	`112`	`expect(mocks.loadChannelSetupPluginRegistrySnapshotForChannel).toHaveBeenCalledTimes(1);`
`113`	`113`	`const snapshotRequest =`
`114`		`-mocks.loadChannelSetupPluginRegistrySnapshotForChannel.mock.calls[0]?.[0];`
	`114`	`+mocks.loadChannelSetupPluginRegistrySnapshotForChannel.mock.calls.at(0)?.[0];`
`115`	`115`	`expect(snapshotRequest?.channel).toBe("telegram");`
`116`	`116`	`expect(snapshotRequest?.pluginId).toBe("telegram");`
`117`	`117`	`expect(snapshotRequest?.workspaceDir).toBe("/tmp/workspace");`
`@@ -149,7 +149,7 @@ describe("resolveInstallableChannelPlugin", () => {`
`149`	`149`	`expect(result.plugin?.id).toBe("telegram");`
`150`	`150`	`expect(mocks.loadChannelSetupPluginRegistrySnapshotForChannel).toHaveBeenCalledTimes(1);`
`151`	`151`	`const snapshotRequest =`
`152`		`-mocks.loadChannelSetupPluginRegistrySnapshotForChannel.mock.calls[0]?.[0];`
	`152`	`+mocks.loadChannelSetupPluginRegistrySnapshotForChannel.mock.calls.at(0)?.[0];`
`153`	`153`	`expect(snapshotRequest?.channel).toBe("telegram");`
`154`	`154`	`expect(snapshotRequest?.pluginId).toBe("evil-telegram-shadow");`
`155`	`155`	`expect(snapshotRequest?.workspaceDir).toBe("/tmp/workspace");`
`@@ -237,7 +237,7 @@ describe("resolveInstallableChannelPlugin", () => {`
`237`	`237`	`});`
`238`	`238`
`239`	`239`	`expect(mocks.ensureChannelSetupPluginInstalled).toHaveBeenCalledTimes(1);`
`240`		`-const installRequest = mocks.ensureChannelSetupPluginInstalled.mock.calls[0]?.[0];`
	`240`	`+const installRequest = mocks.ensureChannelSetupPluginInstalled.mock.calls.at(0)?.[0];`
`241`	`241`	`expect(installRequest?.entry).toBe(catalogEntry);`
`242`	`242`	`expect(result.pluginInstalled).toBe(true);`
`243`	`243`	`});`

Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,7 @@ describe("noteSessionLockHealth", () => {`
`55`	`55`	`});`
`56`	`56`
`57`	`57`	`expect(note).toHaveBeenCalledTimes(1);`
`58`		`-const [message, title] = note.mock.calls[0] as [string, string];`
	`58`	`+const [message, title] = note.mock.calls.at(0) as [string, string];`
`59`	`59`	`expect(title).toBe("Session locks");`
`60`	`60`	`expect(message).toContain("Found 1 session lock file");`
`61`	`61`	expect(message).toContain(`pid=${process.pid} (alive)`);
`@@ -88,7 +88,7 @@ describe("noteSessionLockHealth", () => {`
`88`	`88`	`});`
`89`	`89`
`90`	`90`	`expect(note).toHaveBeenCalledTimes(1);`
`91`		`-const [message] = note.mock.calls[0] as [string, string];`
	`91`	`+const [message] = note.mock.calls.at(0) as [string, string];`
`92`	`92`	`expect(message).toContain("[removed]");`
`93`	`93`	`expect(message).toContain("Removed 1 stale session lock file");`
`94`	`94`
`@@ -114,7 +114,7 @@ describe("noteSessionLockHealth", () => {`
`114`	`114`	`});`
`115`	`115`
`116`	`116`	`expect(note).toHaveBeenCalledTimes(1);`
`117`		`-const [message] = note.mock.calls[0] as [string, string];`
	`117`	`+const [message] = note.mock.calls.at(0) as [string, string];`
`118`	`118`	`expect(message).toContain("stale=yes (non-openclaw-owner)");`
`119`	`119`	`expect(message).toContain("[removed]");`
`120`	`120`	`expect(message).toContain("Removed 1 stale session lock file");`

Original file line number	Diff line number	Diff line change
`@@ -183,7 +183,7 @@ describe("loadProviderCatalogModelsForList", () => {`
`183`	`183`	`});`
`184`	`184`
`185`	`185`	`const discoveryRequest =`
`186`		`-providerDiscoveryMocks.resolveRuntimePluginDiscoveryProviders.mock.calls[0]?.[0];`
	`186`	`+providerDiscoveryMocks.resolveRuntimePluginDiscoveryProviders.mock.calls.at(0)?.[0];`
`187`	`187`	`expect(discoveryRequest?.onlyPluginIds).toStrictEqual(["moonshot"]);`
`188`	`188`	`expect(discoveryRequest?.requireCompleteDiscoveryEntryCoverage).toBe(true);`
`189`	`189`	`expect(discoveryRequest?.discoveryEntriesOnly).toBe(true);`
`@@ -284,7 +284,7 @@ describe("loadProviderCatalogModelsForList", () => {`
`284`	`284`	`).resolves.toBe(false);`
`285`	`285`
`286`	`286`	`const discoveryRequest =`
`287`		`-providerDiscoveryMocks.resolveRuntimePluginDiscoveryProviders.mock.calls[0]?.[0];`
	`287`	`+providerDiscoveryMocks.resolveRuntimePluginDiscoveryProviders.mock.calls.at(0)?.[0];`
`288`	`288`	`expect(discoveryRequest?.onlyPluginIds).toStrictEqual(["ollama"]);`
`289`	`289`	`expect(discoveryRequest?.requireCompleteDiscoveryEntryCoverage).toBe(true);`
`290`	`290`	`expect(discoveryRequest?.discoveryEntriesOnly).toBe(true);`
`@@ -363,7 +363,7 @@ describe("loadProviderCatalogModelsForList", () => {`
`363`	`363`	`});`
`364`	`364`
`365`	`365`	`const discoveryRequest =`
`366`		`-providerDiscoveryMocks.resolveRuntimePluginDiscoveryProviders.mock.calls[0]?.[0];`
	`366`	`+providerDiscoveryMocks.resolveRuntimePluginDiscoveryProviders.mock.calls.at(0)?.[0];`
`367`	`367`	`expect(discoveryRequest?.onlyPluginIds).toStrictEqual(["bundled-demo"]);`
`368`	`368`	`expect(discoveryRequest?.includeUntrustedWorkspacePlugins).toBe(false);`
`369`	`369`	`expect(workspaceStaticCatalog).not.toHaveBeenCalled();`

Original file line number	Diff line number	Diff line change
`@@ -73,7 +73,7 @@ describe("models scan command", () => {`
`73`	`73`	`expect(mocks.loadModelsConfig).not.toHaveBeenCalled();`
`74`	`74`	`expect(mocks.resolveApiKeyForProvider).not.toHaveBeenCalled();`
`75`	`75`	`expect(mocks.scanOpenRouterModels).toHaveBeenCalledTimes(1);`
`76`		`-expect(mocks.scanOpenRouterModels.mock.calls[0]?.[0]?.probe).toBe(false);`
	`76`	`+expect(mocks.scanOpenRouterModels.mock.calls.at(0)?.[0]?.probe).toBe(false);`
`77`	`77`	`expect(runtime.lines.join("\n")).toContain("metadata only");`
`78`	`78`	`expect(runtime.lines.join("\n")).toContain("Tool");`
`79`	`79`	`expect(runtime.lines.join("\n")).toContain("skip");`
`@@ -94,7 +94,7 @@ describe("models scan command", () => {`
`94`	`94`	`cfg: {},`
`95`	`95`	`});`
`96`	`96`	`expect(mocks.scanOpenRouterModels).toHaveBeenCalledTimes(1);`
`97`		`-expect(mocks.scanOpenRouterModels.mock.calls[0]?.[0]?.probe).toBe(false);`
	`97`	`+expect(mocks.scanOpenRouterModels.mock.calls.at(0)?.[0]?.probe).toBe(false);`
`98`	`98`	`expect(runtime.lines.join("\n")).toContain("still require OPENROUTER_API_KEY");`
`99`	`99`	`});`
`100`	`100`
`@@ -112,7 +112,7 @@ describe("models scan command", () => {`
`112`	`112`	`expect(mocks.loadModelsConfig).not.toHaveBeenCalled();`
`113`	`113`	`expect(mocks.resolveApiKeyForProvider).not.toHaveBeenCalled();`
`114`	`114`	`expect(mocks.scanOpenRouterModels).toHaveBeenCalledTimes(1);`
`115`		`-const scanRequest = mocks.scanOpenRouterModels.mock.calls[0]?.[0];`
	`115`	`+const scanRequest = mocks.scanOpenRouterModels.mock.calls.at(0)?.[0];`
`116`	`116`	`expect(scanRequest?.apiKey).toBe("sk-or-test");`
`117`	`117`	`expect(scanRequest?.probe).toBe(true);`
`118`	`118`	`});`

Original file line number	Diff line number	Diff line change
`@@ -104,12 +104,12 @@ describe("collectStatusScanOverview", () => {`
`104`	`104`	`});`
`105`	`105`
`106`	`106`	`expect(mocks.callGateway).toHaveBeenCalledOnce();`
`107`		`-const gatewayRequest = mocks.callGateway.mock.calls[0]?.[0];`
	`107`	`+const gatewayRequest = mocks.callGateway.mock.calls.at(0)?.[0];`
`108`	`108`	`expect(gatewayRequest?.method).toBe("channels.status");`
`109`	`109`	`expect(gatewayRequest?.url).toBe("ws://127.0.0.1:18789");`
`110`	`110`	`expect(gatewayRequest?.token).toBe("tok");`
`111`	`111`	`expect(mocks.buildChannelsTable).toHaveBeenCalledOnce();`
`112`		`-const channelTableCall = mocks.buildChannelsTable.mock.calls[0];`
	`112`	`+const channelTableCall = mocks.buildChannelsTable.mock.calls.at(0);`
`113`	`113`	`expect(typeof channelTableCall?.[0]).toBe("object");`
`114`	`114`	`expect(channelTableCall?.[1]?.includeSetupFallbackPlugins).toBe(true);`
`115`	`115`	`expect(channelTableCall?.[1]?.showSecrets).toBe(false);`
`@@ -128,7 +128,7 @@ describe("collectStatusScanOverview", () => {`
`128`	`128`
`129`	`129`	`expect(mocks.callGateway).not.toHaveBeenCalled();`
`130`	`130`	`expect(mocks.buildChannelsTable).toHaveBeenCalledOnce();`
`131`		`-const channelTableCall = mocks.buildChannelsTable.mock.calls[0];`
	`131`	`+const channelTableCall = mocks.buildChannelsTable.mock.calls.at(0);`
`132`	`132`	`expect(typeof channelTableCall?.[0]).toBe("object");`
`133`	`133`	`expect(channelTableCall?.[1]?.includeSetupFallbackPlugins).toBe(false);`
`134`	`134`	`expect(channelTableCall?.[1]?.showSecrets).toBe(false);`

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

推荐订阅源

Recent Commits to openclaw:main