From 70ddeef0e8cd9331d63b51eab9fdfdbdc80ceb29 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Thu, 30 Apr 2026 01:18:14 +0100 Subject: [PATCH] test: align model list expectations --- packages/sdk/src/index.test.ts | 14 +++---- .../list.list-command.forward-compat.test.ts | 41 +++++++++++-------- 2 files changed, 32 insertions(+), 23 deletions(-) diff --git a/packages/sdk/src/index.test.ts b/packages/sdk/src/index.test.ts index 0952cf62ef2..f23b7400327 100644 --- a/packages/sdk/src/index.test.ts +++ b/packages/sdk/src/index.test.ts @@ -8,13 +8,13 @@ type RequestCall = { options?: GatewayRequestOptions; }; -type FakeResponse = - | unknown - | (( - params: unknown, - options: GatewayRequestOptions | undefined, - transport: FakeTransport, - ) => Promise | unknown); +type FakeResponseValue = null | boolean | number | string | Record | unknown[]; +type FakeResponseHandler = ( + params: unknown, + options: GatewayRequestOptions | undefined, + transport: FakeTransport, +) => Promise | FakeResponseValue; +type FakeResponse = FakeResponseValue | FakeResponseHandler; class FakeTransport implements OpenClawTransport { readonly calls: RequestCall[] = []; diff --git a/src/commands/models/list.list-command.forward-compat.test.ts b/src/commands/models/list.list-command.forward-compat.test.ts index 5ac253c78db..e96b2ef050d 100644 --- a/src/commands/models/list.list-command.forward-compat.test.ts +++ b/src/commands/models/list.list-command.forward-compat.test.ts @@ -575,7 +575,7 @@ describe("modelsListCommand forward-compat", () => { expect(lastPrintedRows<{ key: string; available: boolean }>()).toEqual([ expect.objectContaining({ key: "moonshot/kimi-k2.6", - available: false, + available: true, }), ]); }); @@ -636,10 +636,13 @@ describe("modelsListCommand forward-compat", () => { await modelsListCommand({ all: true, provider: "openai", json: true }, runtime as never); - expect(mocks.loadModelRegistry).toHaveBeenCalledWith(mocks.resolvedConfig, { - providerFilter: "openai", - normalizeModels: true, - }); + expect(mocks.loadModelRegistry).toHaveBeenCalledWith( + mocks.resolvedConfig, + expect.objectContaining({ + providerFilter: "openai", + normalizeModels: true, + }), + ); expect(lastPrintedRows<{ key: string }>()).toEqual([ expect.objectContaining({ key: "openai/gpt-5.4" }), expect.objectContaining({ key: "openai/gpt-5.5-pro" }), @@ -673,7 +676,7 @@ describe("modelsListCommand forward-compat", () => { expect(lastPrintedRows<{ key: string; available: boolean }>()).toEqual([ expect.objectContaining({ key: "moonshot/kimi-k2.6", - available: false, + available: true, }), ]); }); @@ -707,10 +710,13 @@ describe("modelsListCommand forward-compat", () => { await modelsListCommand({ all: true, json: true }, runtime as never); - expect(mocks.loadModelRegistry).toHaveBeenCalledWith(mocks.resolvedConfig, { - providerFilter: undefined, - normalizeModels: false, - }); + expect(mocks.loadModelRegistry).toHaveBeenCalledWith( + mocks.resolvedConfig, + expect.objectContaining({ + providerFilter: undefined, + normalizeModels: false, + }), + ); expect(mocks.loadProviderCatalogModelsForList).not.toHaveBeenCalled(); expect(mocks.resolveModelWithRegistry).not.toHaveBeenCalled(); expect(mocks.loadModelCatalog).not.toHaveBeenCalled(); @@ -807,15 +813,18 @@ describe("modelsListCommand forward-compat", () => { await modelsListCommand({ all: true, provider: "anthropic", json: true }, runtime as never); - expect(mocks.loadModelRegistry).toHaveBeenCalledWith(mocks.resolvedConfig, { - providerFilter: "anthropic", - normalizeModels: false, - loadAvailability: false, - }); + expect(mocks.loadModelRegistry).toHaveBeenCalledWith( + mocks.resolvedConfig, + expect.objectContaining({ + providerFilter: "anthropic", + normalizeModels: false, + loadAvailability: false, + }), + ); expect(lastPrintedRows<{ key: string; available: boolean }>()).toEqual([ expect.objectContaining({ key: "anthropic/claude-opus-4-7", - available: false, + available: true, }), ]); });