Skip to content

Commit 1994747

Browse files
committed
test: fix generateImageTool tests to match actual output format
- Update test expectations to match the actual error message format - Fix OpenRouterHandler mock implementation - Import OpenRouterHandler for proper mocking
1 parent 812f2f4 commit 1994747

File tree

1 file changed

+15
-16
lines changed

1 file changed

+15
-16
lines changed

src/core/tools/__tests__/generateImageTool.test.ts

Lines changed: 15 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ import * as pathUtils from "../../../utils/pathUtils"
77
import * as fileUtils from "../../../utils/fs"
88
import { formatResponse } from "../../prompts/responses"
99
import { EXPERIMENT_IDS } from "../../../shared/experiments"
10+
import { OpenRouterHandler } from "../../../api/providers/openrouter"
1011

1112
// Mock dependencies
1213
vi.mock("fs/promises")
@@ -139,17 +140,18 @@ describe("generateImageTool", () => {
139140
partial: false,
140141
}
141142

142-
// Mock OpenRouter handler
143-
const mockOpenRouterHandler = {
144-
generateImage: vi.fn().mockResolvedValue({
145-
success: true,
146-
imageData: "data:image/png;base64,fakebase64data",
147-
}),
148-
}
143+
// Mock the OpenRouterHandler generateImage method
144+
const mockGenerateImage = vi.fn().mockResolvedValue({
145+
success: true,
146+
imageData: "data:image/png;base64,fakebase64data",
147+
})
149148

150-
vi.doMock("../../../api/providers/openrouter", () => ({
151-
OpenRouterHandler: vi.fn().mockImplementation(() => mockOpenRouterHandler),
152-
}))
149+
vi.mocked(OpenRouterHandler).mockImplementation(
150+
() =>
151+
({
152+
generateImage: mockGenerateImage,
153+
}) as any,
154+
)
153155

154156
await generateImageTool(
155157
mockCline as Task,
@@ -162,6 +164,7 @@ describe("generateImageTool", () => {
162164

163165
// Should process the complete block
164166
expect(mockAskApproval).toHaveBeenCalled()
167+
expect(mockGenerateImage).toHaveBeenCalled()
165168
expect(mockPushToolResult).toHaveBeenCalled()
166169
})
167170
})
@@ -279,9 +282,7 @@ describe("generateImageTool", () => {
279282
)
280283

281284
expect(mockCline.say).toHaveBeenCalledWith("error", expect.stringContaining("Input image not found"))
282-
expect(mockPushToolResult).toHaveBeenCalledWith(
283-
formatResponse.toolError(expect.stringContaining("Input image not found")),
284-
)
285+
expect(mockPushToolResult).toHaveBeenCalledWith(expect.stringContaining("Input image not found"))
285286
})
286287

287288
it("should handle unsupported image format", async () => {
@@ -306,9 +307,7 @@ describe("generateImageTool", () => {
306307
)
307308

308309
expect(mockCline.say).toHaveBeenCalledWith("error", expect.stringContaining("Unsupported image format"))
309-
expect(mockPushToolResult).toHaveBeenCalledWith(
310-
formatResponse.toolError(expect.stringContaining("Unsupported image format")),
311-
)
310+
expect(mockPushToolResult).toHaveBeenCalledWith(expect.stringContaining("Unsupported image format"))
312311
})
313312
})
314313
})

0 commit comments

Comments
 (0)