From c56271cfcb76fff5ca80f499ec24ef2b4d0ae2e2 Mon Sep 17 00:00:00 2001 From: Vincent Koc Date: Tue, 15 Sep 2026 13:12:42 +0800 Subject: [PATCH] refactor(venice): remove duplicate discovery fallback fixture (#148779) --- extensions/venice/models.test.ts | 45 -------------------------------- 1 file changed, 45 deletions(-) diff --git a/extensions/venice/models.test.ts b/extensions/venice/models.test.ts index 56bea62132ec..56ea98fb06aa 100644 --- a/extensions/venice/models.test.ts +++ b/extensions/venice/models.test.ts @@ -8,33 +8,6 @@ import { afterEach, describe, expect, it, vi } from "vitest"; import { VENICE_BASE_URL, VENICE_MODEL_CATALOG, VENICE_MODEL_DISCOVERY_OPTIONS } from "./models.js"; import manifest from "./openclaw.plugin.json" with { type: "json" }; -function makeModelsResponse(id: string): Response { - return new Response( - JSON.stringify({ - data: [ - { - id, - model_spec: { - name: id, - privacy: "private", - availableContextTokens: 131072, - maxCompletionTokens: 4096, - capabilities: { - supportsReasoning: false, - supportsVision: false, - supportsFunctionCalling: true, - }, - }, - }, - ], - }), - { - status: 200, - headers: { "Content-Type": "application/json" }, - }, - ); -} - type ModelSpecOverride = { id: string; availableContextTokens?: number; @@ -390,24 +363,6 @@ describe("venice-models", () => { }, ); - it("uses the shared fallback after a transient fetch failure", async () => { - let attempts = 0; - const fetchMock = vi.fn(async () => { - attempts += 1; - if (attempts === 1) { - throw Object.assign(new TypeError("fetch failed"), { - cause: { code: "ECONNRESET", message: "socket hang up" }, - }); - } - return makeModelsResponse("zai-org-glm-4.7"); - }); - vi.stubGlobal("fetch", fetchMock as unknown as typeof fetch); - - const models = await discoverVeniceModels(); - expect(attempts).toBe(1); - expect(models.map((m) => m.id)).toEqual(VENICE_MODEL_CATALOG.map((m) => m.id)); - }); - it("uses API maxCompletionTokens for catalog models when present", async () => { const fetchMock = stubVeniceModelsFetch([ {