replace hardcoded keys with constants

Author: slytechnical

src/api/providers/fetchers/__tests__/litellm.test.ts

@@ -6,6 +6,8 @@ import { COMPUTER_USE_MODELS } from "../../../../shared/api"
 jest.mock("axios")
 const mockedAxios = axios as jest.Mocked<typeof axios>
 
+const DUMMY_INVALID_KEY = "invalid-key-for-testing"
+
 describe("getLiteLLMModels", () => {
 	beforeEach(() => {
 		jest.clearAllMocks()
@@ -165,7 +167,7 @@ describe("getLiteLLMModels", () => {
 		mockedAxios.isAxiosError.mockReturnValue(true)
 		mockedAxios.get.mockRejectedValue(axiosError)
 
-		await expect(getLiteLLMModels("invalid-key", "http://localhost:4000")).rejects.toThrow(
+		await expect(getLiteLLMModels(DUMMY_INVALID_KEY, "http://localhost:4000")).rejects.toThrow(
 			"Failed to fetch LiteLLM models: 401 Unauthorized. Check base URL and API key.",
 		)
 	})

src/api/providers/fetchers/__tests__/modelCache.test.ts

@@ -34,6 +34,9 @@ const mockGetRequestyModels = getRequestyModels as jest.MockedFunction<typeof ge
 const mockGetGlamaModels = getGlamaModels as jest.MockedFunction<typeof getGlamaModels>
 const mockGetUnboundModels = getUnboundModels as jest.MockedFunction<typeof getUnboundModels>
 
+const DUMMY_REQUESTY_KEY = "requesty-key-for-testing"
+const DUMMY_UNBOUND_KEY = "unbound-key-for-testing"
+
 describe("getModels with new GetModelsOptions", () => {
 	beforeEach(() => {
 		jest.clearAllMocks()
@@ -88,9 +91,9 @@ describe("getModels with new GetModelsOptions", () => {
 		}
 		mockGetRequestyModels.mockResolvedValue(mockModels)
 
-		const result = await getModels({ provider: "requesty", apiKey: "requesty-key" })
+		const result = await getModels({ provider: "requesty", apiKey: DUMMY_REQUESTY_KEY })
 
-		expect(mockGetRequestyModels).toHaveBeenCalledWith("requesty-key")
+		expect(mockGetRequestyModels).toHaveBeenCalledWith(DUMMY_REQUESTY_KEY)
 		expect(result).toEqual(mockModels)
 	})
 
@@ -122,9 +125,9 @@ describe("getModels with new GetModelsOptions", () => {
 		}
 		mockGetUnboundModels.mockResolvedValue(mockModels)
 
-		const result = await getModels({ provider: "unbound", apiKey: "unbound-key" })
+		const result = await getModels({ provider: "unbound", apiKey: DUMMY_UNBOUND_KEY })
 
-		expect(mockGetUnboundModels).toHaveBeenCalledWith("unbound-key")
+		expect(mockGetUnboundModels).toHaveBeenCalledWith(DUMMY_UNBOUND_KEY)
 		expect(result).toEqual(mockModels)
 	})