Skip to content

Commit d01700a

Browse files
committed
mock logger in tests
1 parent 1ae20a1 commit d01700a

File tree

2 files changed

+8
-13
lines changed

2 files changed

+8
-13
lines changed

backend/src/__tests__/main-prompt.integration.test.ts

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -17,15 +17,22 @@ import * as checkTerminalCommandModule from '../check-terminal-command'
1717
import * as requestFilesPrompt from '../find-files/request-files-prompt'
1818
import * as aisdk from '../llm-apis/vercel-ai-sdk/ai-sdk'
1919
import { mainPrompt } from '../main-prompt'
20-
import { logger } from '../util/logger'
2120
import * as websocketAction from '../websockets/websocket-action'
2221

2322
import type { PrintModeEvent } from '@codebuff/common/types/print-mode'
2423
import type { ProjectFileContext } from '@codebuff/common/util/file'
24+
import type { Logger } from '@codebuff/types/logger'
2525
import type { WebSocket } from 'ws'
2626

2727
// --- Shared Mocks & Helpers ---
2828

29+
const logger: Logger = {
30+
debug: () => {},
31+
info: () => {},
32+
warn: () => {},
33+
error: () => {},
34+
}
35+
2936
class MockWebSocket {
3037
send(msg: string) {}
3138
close() {}
@@ -107,13 +114,6 @@ describe.skip('mainPrompt (Integration)', () => {
107114
})
108115

109116
it('should delete a specified function while preserving other code', async () => {
110-
// Mock necessary non-LLM functions
111-
spyOn(logger, 'debug').mockImplementation(() => {})
112-
spyOn(logger, 'error').mockImplementation(() => {})
113-
spyOn(logger, 'info').mockImplementation(() => {})
114-
spyOn(logger, 'warn').mockImplementation(() => {})
115-
spyOn(requestFilesPrompt, 'requestRelevantFiles').mockResolvedValue([])
116-
117117
const initialContent = `import { Message } from '@codebuff/common/types/message'
118118
import { withCacheControl } from '@codebuff/common/util/messages'
119119

backend/src/__tests__/tool-call-schema.test.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
11
import { describe, it, expect, beforeEach } from 'bun:test'
22

3-
import { logger } from '../util/logger'
4-
53
import type { WebSocket } from 'ws'
64

75
describe('Backend Tool Call Schema', () => {
@@ -221,7 +219,6 @@ export async function generateMockProjectStructureAnalysis(
221219

222220
return analysis
223221
} catch (error) {
224-
logger.error({ error }, 'Project analysis failed')
225222
return `Project analysis failed: ${error instanceof Error ? error.message : error}`
226223
}
227224
}
@@ -276,7 +273,6 @@ src/utils.ts:2:import { readFileSync } from 'fs'`,
276273

277274
return analysis
278275
} catch (error) {
279-
logger.error({ error }, 'Dependency analysis failed')
280276
return `Dependency analysis failed: ${error instanceof Error ? error.message : error}`
281277
}
282278
}
@@ -318,7 +314,6 @@ export async function generateMockFileContentAnalysis(
318314

319315
return analysis
320316
} catch (error) {
321-
logger.error({ error }, 'File content analysis failed')
322317
return `File content analysis failed: ${error instanceof Error ? error.message : error}`
323318
}
324319
}

0 commit comments

Comments
 (0)