Skip to content

Commit 22645a9

Browse files
committed
Fix test
1 parent 6f30283 commit 22645a9

File tree

1 file changed

+13
-24
lines changed

1 file changed

+13
-24
lines changed

src/api/providers/__tests__/roo.spec.ts

Lines changed: 13 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -36,26 +36,12 @@ vitest.mock("openai", () => {
3636
return {
3737
[Symbol.asyncIterator]: async function* () {
3838
yield {
39-
choices: [
40-
{
41-
delta: { content: "Test response" },
42-
index: 0,
43-
},
44-
],
39+
choices: [{ delta: { content: "Test response" }, index: 0 }],
4540
usage: null,
4641
}
4742
yield {
48-
choices: [
49-
{
50-
delta: {},
51-
index: 0,
52-
},
53-
],
54-
usage: {
55-
prompt_tokens: 10,
56-
completion_tokens: 5,
57-
total_tokens: 15,
58-
},
43+
choices: [{ delta: {}, index: 0 }],
44+
usage: { prompt_tokens: 10, completion_tokens: 5, total_tokens: 15 },
5945
}
6046
},
6147
}
@@ -73,6 +59,7 @@ const mockHasInstance = vitest.fn()
7359
// Create mock functions that we can control
7460
const mockGetSessionTokenFn = vitest.fn()
7561
const mockHasInstanceFn = vitest.fn()
62+
const mockOnFn = vitest.fn()
7663

7764
vitest.mock("@roo-code/cloud", () => ({
7865
CloudService: {
@@ -411,7 +398,10 @@ describe("RooHandler", () => {
411398
it("should handle undefined auth service gracefully", () => {
412399
mockHasInstanceFn.mockReturnValue(true)
413400
// Mock CloudService with undefined authService
414-
const originalGetter = Object.getOwnPropertyDescriptor(CloudService, "instance")?.get
401+
const originalGetSessionToken = mockGetSessionTokenFn.getMockImplementation()
402+
403+
// Temporarily make authService return undefined
404+
mockGetSessionTokenFn.mockImplementation(() => undefined)
415405

416406
try {
417407
Object.defineProperty(CloudService, "instance", {
@@ -430,12 +420,11 @@ describe("RooHandler", () => {
430420
const handler = new RooHandler(mockOptions)
431421
expect(handler).toBeInstanceOf(RooHandler)
432422
} finally {
433-
// Always restore original getter, even if test fails
434-
if (originalGetter) {
435-
Object.defineProperty(CloudService, "instance", {
436-
get: originalGetter,
437-
configurable: true,
438-
})
423+
// Restore original mock implementation
424+
if (originalGetSessionToken) {
425+
mockGetSessionTokenFn.mockImplementation(originalGetSessionToken)
426+
} else {
427+
mockGetSessionTokenFn.mockReturnValue("test-session-token")
439428
}
440429
}
441430
})

0 commit comments

Comments
 (0)