correct fails tests to skip

sourcegraph · Jan 28, 2025 · e54f4f2 · e54f4f2
1 parent f622dee
commit e54f4f2
Showing 1 changed file with 21 additions and 25 deletions.
diff --git a/vscode/src/chat/chat-view/ChatController.test.ts b/vscode/src/chat/chat-view/ChatController.test.ts
@@ -87,33 +87,29 @@ describe('ChatController', () => {
         })
     })
 
-    test.fails(
-        'verifies interactionId is passed through chat requests',
-        async () => {
-            const mockRequestID = '0'
-            mockContextRetriever.retrieveContext.mockResolvedValue([])
+    test.skip('verifies interactionId is passed through chat requests', async () => {
+        const mockRequestID = '0'
+        mockContextRetriever.retrieveContext.mockResolvedValue([])
 
-            await chatController.handleUserMessage({
-                requestID: mockRequestID,
-                inputText: ps`Test input`,
-                mentions: [],
-                editorState: null,
-                signal: new AbortController().signal,
-                source: 'chat',
-            })
-            await vi.runOnlyPendingTimersAsync()
+        await chatController.handleUserMessage({
+            requestID: mockRequestID,
+            inputText: ps`Test input`,
+            mentions: [],
+            editorState: null,
+            signal: new AbortController().signal,
+            source: 'chat',
+        })
+        await vi.runOnlyPendingTimersAsync()
 
-            expect(mockChatClient.chat).toHaveBeenCalledWith(
-                expect.any(Array),
-                expect.any(Object),
-                expect.any(AbortSignal),
-                mockRequestID
-            )
-        },
-        1500
-    )
+        expect(mockChatClient.chat).toHaveBeenCalledWith(
+            expect.any(Array),
+            expect.any(Object),
+            expect.any(AbortSignal),
+            mockRequestID
+        )
+    }, 1500)
 
-    test.fails('send, followup, and edit', { timeout: 1500 }, async () => {
+    test.skip('send, followup, and edit', { timeout: 1500 }, async () => {
         const postMessageSpy = vi
             .spyOn(chatController as any, 'postMessage')
             .mockImplementation(() => {})
@@ -397,7 +393,7 @@ describe('ChatController', () => {
         })
     })
 
-    test.fails('send error', async () => {
+    test.skip('send error', async () => {
         const postMessageSpy = vi
             .spyOn(chatController as any, 'postMessage')
             .mockImplementation(() => {})