Skip to content
This repository was archived by the owner on May 15, 2026. It is now read-only.

Commit ea1a377

Browse files
committed
Fix embedder tests after telemetry removal
1 parent 9d2d27c commit ea1a377

8 files changed

Lines changed: 26 additions & 6 deletions

File tree

src/services/code-index/embedders/__tests__/bedrock.spec.ts

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,20 @@ import { BedrockEmbedder } from "../bedrock"
55
import { MAX_ITEM_TOKENS, INITIAL_RETRY_DELAY_MS } from "../../constants"
66

77
// Mock the AWS SDK
8+
vitest.mock("@aws-sdk/client-bedrock-runtime", () => {
9+
return {
10+
BedrockRuntimeClient: vitest.fn().mockImplementation(() => ({
11+
send: vitest.fn(),
12+
})),
13+
InvokeModelCommand: vitest.fn().mockImplementation((input) => ({
14+
input,
15+
})),
16+
}
17+
})
18+
vitest.mock("@aws-sdk/credential-providers", () => ({
19+
fromEnv: vitest.fn().mockReturnValue(Promise.resolve({})),
20+
fromIni: vitest.fn().mockReturnValue(Promise.resolve({})),
21+
}))
822

923
// Mock i18n
1024
vitest.mock("../../../../i18n", () => ({

src/services/code-index/embedders/__tests__/gemini.spec.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ import { GeminiEmbedder } from "../gemini"
44
import { OpenAICompatibleEmbedder } from "../openai-compatible"
55

66
// Mock the OpenAICompatibleEmbedder
7+
vitest.mock("../openai-compatible")
78

89
const MockedOpenAICompatibleEmbedder = OpenAICompatibleEmbedder as MockedClass<typeof OpenAICompatibleEmbedder>
910

src/services/code-index/embedders/__tests__/mistral.spec.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ import { MistralEmbedder } from "../mistral"
44
import { OpenAICompatibleEmbedder } from "../openai-compatible"
55

66
// Mock the OpenAICompatibleEmbedder
7+
vitest.mock("../openai-compatible")
78

89
const MockedOpenAICompatibleEmbedder = OpenAICompatibleEmbedder as MockedClass<typeof OpenAICompatibleEmbedder>
910

src/services/code-index/embedders/__tests__/openai-compatible-rate-limit.spec.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ import { OpenAI } from "openai"
44
import { OpenAICompatibleEmbedder } from "../openai-compatible"
55

66
// Mock the OpenAI SDK
7+
vi.mock("openai")
78

89
// Mock i18n
910
vi.mock("../../../../i18n", () => ({

src/services/code-index/embedders/__tests__/openai-compatible.spec.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ import { OpenAICompatibleEmbedder } from "../openai-compatible"
44
import { MAX_ITEM_TOKENS, INITIAL_RETRY_DELAY_MS } from "../../constants"
55

66
// Mock the OpenAI SDK
7+
vitest.mock("openai")
78

89
// Mock global fetch
910
global.fetch = vitest.fn()

src/services/code-index/embedders/__tests__/openai.spec.ts

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ import { OpenAiEmbedder } from "../openai"
55
import { MAX_ITEM_TOKENS, INITIAL_RETRY_DELAY_MS } from "../../constants"
66

77
// Mock the OpenAI SDK
8+
vitest.mock("openai")
89

910
// Mock i18n
1011
vitest.mock("../../../../i18n", () => ({
@@ -435,9 +436,6 @@ describe("OpenAiEmbedder", () => {
435436

436437
it("should handle errors with failing toString method", async () => {
437438
const testTexts = ["Hello world"]
438-
// When vitest tries to display the error object in test output,
439-
// it calls toString which throws "toString failed"
440-
// This happens before our error handling code runs
441439
const errorWithFailingToString = {
442440
toString: () => {
443441
throw new Error("toString failed")
@@ -446,9 +444,9 @@ describe("OpenAiEmbedder", () => {
446444

447445
mockEmbeddingsCreate.mockRejectedValue(errorWithFailingToString)
448446

449-
// The test framework itself throws "toString failed" when trying to
450-
// display the error, so we need to expect that specific error
451-
await expect(embedder.createEmbeddings(testTexts)).rejects.toThrow("toString failed")
447+
await expect(embedder.createEmbeddings(testTexts)).rejects.toThrow(
448+
"Failed to create embeddings after 3 attempts: Unknown error",
449+
)
452450
})
453451

454452
it("should handle errors from response.status property", async () => {

src/services/code-index/embedders/__tests__/openrouter.spec.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ import { OpenRouterEmbedder, OPENROUTER_DEFAULT_PROVIDER_NAME } from "../openrou
55
import { getModelDimension, getDefaultModelId } from "../../../../shared/embeddingModels"
66

77
// Mock the OpenAI SDK
8+
vi.mock("openai")
89

910
// Mock i18n
1011
vi.mock("../../../../i18n", () => ({

src/services/code-index/embedders/__tests__/vercel-ai-gateway.spec.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,9 @@ import { VercelAiGatewayEmbedder } from "../vercel-ai-gateway"
44
import { OpenAICompatibleEmbedder } from "../openai-compatible"
55

66
// Mock the OpenAICompatibleEmbedder
7+
vi.mock("../openai-compatible", () => ({
8+
OpenAICompatibleEmbedder: vi.fn(),
9+
}))
710

811
const MockedOpenAICompatibleEmbedder = vi.mocked(OpenAICompatibleEmbedder)
912

0 commit comments

Comments
 (0)