test: align model list expectations

steipete · steipete · commit 70ddeef0e8cd · 2026-04-30T01:18:18.000+01:00
diff --git 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> | unknown);
+type FakeResponseValue = null | boolean | number | string | Record<string, unknown> | unknown[];
+type FakeResponseHandler = (
+  params: unknown,
+  options: GatewayRequestOptions | undefined,
+  transport: FakeTransport,
+) => Promise<FakeResponseValue> | 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
@@ -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,
         }),
       ]);
     });
