From bf3523641913a3f7b83072d645d6c0e8a3e4a8bf Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Sat, 9 May 2026 07:13:48 +0100 Subject: [PATCH] test: tighten agents list provider assertions --- src/commands/agents.commands.list.test.ts | 33 ++++++++++++----------- 1 file changed, 17 insertions(+), 16 deletions(-) diff --git a/src/commands/agents.commands.list.test.ts b/src/commands/agents.commands.list.test.ts index 3e7ae7e4243..0b51e7b1aa4 100644 --- a/src/commands/agents.commands.list.test.ts +++ b/src/commands/agents.commands.list.test.ts @@ -87,30 +87,31 @@ describe("agentsListCommand", () => { it("keeps provider details available for JSON callers that request bindings", async () => { const runtime = createRuntime(); + const cfg = createConfig(); + const providerStatus = new Map(); + requireValidConfigMock.mockResolvedValueOnce(cfg); + buildProviderStatusIndexMock.mockResolvedValueOnce(providerStatus); await agentsListCommand({ json: true, bindings: true }, runtime); expect(buildProviderStatusIndexMock).toHaveBeenCalledOnce(); expect(buildProviderSummaryMetadataIndexMock).toHaveBeenCalledOnce(); expect(summarizeBindingsMock).toHaveBeenCalledWith( - expect.objectContaining({ agents: expect.any(Object) }), - [expect.objectContaining({ agentId: "main" })], + cfg, + cfg.bindings, providerSummaryMetadataMock, ); - expect(listProvidersForAgentMock).toHaveBeenCalledWith( - expect.objectContaining({ - cfg: expect.objectContaining({ agents: expect.any(Object) }), - bindings: [expect.objectContaining({ agentId: "main" })], - providerMetadata: providerSummaryMetadataMock, - }), - ); - expect(runtime.json[0]).toEqual([ - expect.objectContaining({ - id: "main", - routes: ["Telegram default"], - providers: ["Telegram default: configured"], - }), - ]); + expect(listProvidersForAgentMock).toHaveBeenCalledWith({ + summaryIsDefault: true, + cfg, + bindings: cfg.bindings, + providerStatus, + providerMetadata: providerSummaryMetadataMock, + }); + const [summary] = runtime.json[0] as Array>; + expect(summary?.id).toBe("main"); + expect(summary?.routes).toEqual(["Telegram default"]); + expect(summary?.providers).toEqual(["Telegram default: configured"]); }); it("keeps human output enriched from read-only provider metadata", async () => {