From bf709239143d43b50d5355f378d0df5da7b37c2e Mon Sep 17 00:00:00 2001 From: Hannes Rudolph Date: Sat, 13 Dec 2025 21:41:20 -0700 Subject: [PATCH] test: update claude-code tests to reflect supportsImages and valid model IDs - Changed model ID from 'claude-sonnet-4-20250514' to 'claude-sonnet-4-5' (valid model) - Updated supportsImages expectation from false to true (Claude Code now supports images) - Updated maxTokens expectation from 64_000 to 32768 (actual model value) - Renamed test to 'should return claude-code model with correct model info' --- .../ui/hooks/__tests__/useSelectedModel.spec.ts | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/webview-ui/src/components/ui/hooks/__tests__/useSelectedModel.spec.ts b/webview-ui/src/components/ui/hooks/__tests__/useSelectedModel.spec.ts index a7824b1fa2..118184857c 100644 --- a/webview-ui/src/components/ui/hooks/__tests__/useSelectedModel.spec.ts +++ b/webview-ui/src/components/ui/hooks/__tests__/useSelectedModel.spec.ts @@ -407,7 +407,7 @@ describe("useSelectedModel", () => { }) describe("claude-code provider", () => { - it("should return claude-code model with supportsImages disabled", () => { + it("should return claude-code model with correct model info", () => { mockUseRouterModels.mockReturnValue({ data: { openrouter: {}, @@ -428,19 +428,19 @@ describe("useSelectedModel", () => { const apiConfiguration: ProviderSettings = { apiProvider: "claude-code", - apiModelId: "claude-sonnet-4-20250514", + apiModelId: "claude-sonnet-4-5", // Use valid claude-code model ID } const wrapper = createWrapper() const { result } = renderHook(() => useSelectedModel(apiConfiguration), { wrapper }) expect(result.current.provider).toBe("claude-code") - expect(result.current.id).toBe("claude-sonnet-4-20250514") + expect(result.current.id).toBe("claude-sonnet-4-5") expect(result.current.info).toBeDefined() - expect(result.current.info?.supportsImages).toBe(false) + expect(result.current.info?.supportsImages).toBe(true) // Claude Code now supports images expect(result.current.info?.supportsPromptCache).toBe(true) // Claude Code now supports prompt cache - // Verify it inherits other properties from anthropic models - expect(result.current.info?.maxTokens).toBe(64_000) + // Verify it inherits other properties from claude-code models + expect(result.current.info?.maxTokens).toBe(32768) expect(result.current.info?.contextWindow).toBe(200_000) }) @@ -473,7 +473,7 @@ describe("useSelectedModel", () => { expect(result.current.provider).toBe("claude-code") expect(result.current.id).toBe("claude-sonnet-4-5") // Default model expect(result.current.info).toBeDefined() - expect(result.current.info?.supportsImages).toBe(false) + expect(result.current.info?.supportsImages).toBe(true) // Claude Code now supports images }) })