test(cli): add mock logger to fetchUsageData test to isolate from side effects

brandonkachen · brandonkachen · commit 0663d9966799 · 2025-12-05T16:43:57.000-08:00
diff --git a/cli/src/hooks/__tests__/use-usage-query.test.ts b/cli/src/hooks/__tests__/use-usage-query.test.ts
@@ -57,8 +57,9 @@ describe('fetchUsageData', () => {
 
   test('should throw error on failed request', async () => {
     globalThis.fetch = mock(async () => new Response('Error', { status: 500 })) as unknown as typeof fetch
+    const mockLogger = { error: mock(() => {}), warn: mock(() => {}), info: mock(() => {}), debug: mock(() => {}) }
 
-    await expect(fetchUsageData({ authToken: 'test-token' })).rejects.toThrow(
+    await expect(fetchUsageData({ authToken: 'test-token', logger: mockLogger as any })).rejects.toThrow(
       'Failed to fetch usage: 500',
     )
   })

Original file line number	Diff line number	Diff line change
`@@ -57,8 +57,9 @@ describe('fetchUsageData', () => {`
`57`	`57`
`58`	`58`	`test('should throw error on failed request', async () => {`
`59`	`59`	`globalThis.fetch = mock(async () => new Response('Error', { status: 500 })) as unknown as typeof fetch`
	`60`	`+ const mockLogger = { error: mock(() => {}), warn: mock(() => {}), info: mock(() => {}), debug: mock(() => {}) }`
`60`	`61`
`61`		`- await expect(fetchUsageData({ authToken: 'test-token' })).rejects.toThrow(`
	`62`	`+ await expect(fetchUsageData({ authToken: 'test-token', logger: mockLogger as any })).rejects.toThrow(`
`62`	`63`	`'Failed to fetch usage: 500',`
`63`	`64`	`)`
`64`	`65`	`})`