test: silence error console in tests where known errors are thrown

Author: TkDodo

packages/react-query/src/__tests__/useQuery.test.tsx

@@ -2888,6 +2888,9 @@ describe('useQuery', () => {
   })
 
   it('should set status to error if queryFn throws', async () => {
+    const consoleMock = vi
+      .spyOn(console, 'error')
+      .mockImplementation(() => undefined)
     const key = queryKey()
 
     function Page() {
@@ -2911,6 +2914,8 @@ describe('useQuery', () => {
 
     await waitFor(() => rendered.getByText('error'))
     await waitFor(() => rendered.getByText('Error test jaylen'))
+
+    consoleMock.mockRestore()
   })
 
   it('should throw error if queryFn throws and throwOnError is in use', async () => {

packages/solid-query/src/__tests__/createMutation.test.tsx

@@ -64,6 +64,10 @@ describe('createMutation', () => {
   })
 
   it('should be able to reset `error`', async () => {
+    const consoleMock = vi
+      .spyOn(console, 'error')
+      .mockImplementation(() => undefined)
+
     function Page() {
       const mutation = createMutation<string, Error>(() => ({
         mutationFn: () => {
@@ -105,6 +109,8 @@ describe('createMutation', () => {
     await waitFor(() => {
       expect(screen.queryByRole('heading')).toBeNull()
     })
+
+    consoleMock.mockRestore()
   })
 
   it('should be able to call `onSuccess` and `onSettled` after each successful mutate', async () => {
@@ -781,6 +787,10 @@ describe('createMutation', () => {
   })
 
   it('should be able to throw an error when throwOnError is set to true', async () => {
+    const consoleMock = vi
+      .spyOn(console, 'error')
+      .mockImplementation(() => undefined)
+
     function Page() {
       const mutation = createMutation<string, Error>(() => ({
         mutationFn: () => {
@@ -817,9 +827,15 @@ describe('createMutation', () => {
     await waitFor(() => {
       expect(screen.queryByText('error')).not.toBeNull()
     })
+
+    consoleMock.mockRestore()
   })
 
   it('should be able to throw an error when throwOnError is a function that returns true', async () => {
+    const consoleMock = vi
+      .spyOn(console, 'error')
+      .mockImplementation(() => undefined)
+
     let boundary = false
     function Page() {
       const mutation = createMutation<string, Error>(() => ({
@@ -867,6 +883,8 @@ describe('createMutation', () => {
     await waitFor(() => {
       expect(screen.queryByText('error boundary')).not.toBeNull()
     })
+
+    consoleMock.mockRestore()
   })
 
   it('should pass meta to mutation', async () => {

packages/solid-query/src/__tests__/createQuery.test.tsx

@@ -2446,6 +2446,10 @@ describe('createQuery', () => {
   it('should set status to error if queryFn throws', async () => {
     const key = queryKey()
 
+    const consoleMock = vi
+      .spyOn(console, 'error')
+      .mockImplementation(() => undefined)
+
     function Page() {
       const state = createQuery(() => ({
         queryKey: key,
@@ -2471,11 +2475,17 @@ describe('createQuery', () => {
 
     await waitFor(() => screen.getByText('error'))
     await waitFor(() => screen.getByText('Error test jaylen'))
+
+    consoleMock.mockRestore()
   })
 
   it('should throw error if queryFn throws and throwOnError is in use', async () => {
     const key = queryKey()
 
+    const consoleMock = vi
+      .spyOn(console, 'error')
+      .mockImplementation(() => undefined)
+
     function Page() {
       const state = createQuery(() => ({
         queryKey: key,
@@ -2501,6 +2511,8 @@ describe('createQuery', () => {
     ))
 
     await waitFor(() => screen.getByText('error boundary'))
+
+    consoleMock.mockRestore()
   })
 
   it('should update with data if we observe no properties and throwOnError', async () => {

packages/solid-query/src/__tests__/suspense.test.tsx

@@ -453,6 +453,10 @@ describe("useQuery's in Suspense mode", () => {
   it('should throw errors to the error boundary by default', async () => {
     const key = queryKey()
 
+    const consoleMock = vi
+      .spyOn(console, 'error')
+      .mockImplementation(() => undefined)
+
     function Page() {
       const state = createQuery(() => ({
         queryKey: key,
@@ -497,6 +501,8 @@ describe("useQuery's in Suspense mode", () => {
 
     await waitFor(() => screen.getByText('Loading...'))
     await waitFor(() => screen.getByText('error boundary'))
+
+    consoleMock.mockRestore()
   })
 
   it('should not throw errors to the error boundary when throwOnError: false', async () => {
@@ -551,6 +557,10 @@ describe("useQuery's in Suspense mode", () => {
   it('should throw errors to the error boundary when a throwOnError function returns true', async () => {
     const key = queryKey()
 
+    const consoleMock = vi
+      .spyOn(console, 'error')
+      .mockImplementation(() => undefined)
+
     function Page() {
       const state = createQuery(() => ({
         queryKey: key,
@@ -595,6 +605,8 @@ describe("useQuery's in Suspense mode", () => {
 
     await waitFor(() => screen.getByText('Loading...'))
     await waitFor(() => screen.getByText('error boundary'))
+
+    consoleMock.mockRestore()
   })
 
   it('should not throw errors to the error boundary when a throwOnError function returns false', async () => {
@@ -696,6 +708,10 @@ describe("useQuery's in Suspense mode", () => {
   it('should error catched in error boundary without infinite loop', async () => {
     const key = queryKey()
 
+    const consoleMock = vi
+      .spyOn(console, 'error')
+      .mockImplementation(() => undefined)
+
     let succeed = true
 
     function Page() {
@@ -756,11 +772,17 @@ describe("useQuery's in Suspense mode", () => {
     fireEvent.click(screen.getByLabelText('fail'))
     // render error boundary fallback (error boundary)
     await waitFor(() => screen.getByText('error boundary'))
+
+    consoleMock.mockRestore()
   })
 
   it('should error catched in error boundary without infinite loop when query keys changed', async () => {
     let succeed = true
 
+    const consoleMock = vi
+      .spyOn(console, 'error')
+      .mockImplementation(() => undefined)
+
     function Page() {
       const [key, setKey] = createSignal(0)
 
@@ -814,9 +836,15 @@ describe("useQuery's in Suspense mode", () => {
     fireEvent.click(screen.getByLabelText('fail'))
     // render error boundary fallback (error boundary)
     await waitFor(() => screen.getByText('error boundary'))
+
+    consoleMock.mockRestore()
   })
 
   it('should error catched in error boundary without infinite loop when enabled changed', async () => {
+    const consoleMock = vi
+      .spyOn(console, 'error')
+      .mockImplementation(() => undefined)
+
     function Page() {
       const queryKeys = '1'
       const [enabled, setEnabled] = createSignal(false)
@@ -874,6 +902,8 @@ describe("useQuery's in Suspense mode", () => {
 
     // render error boundary fallback (error boundary)
     await waitFor(() => screen.getByText('error boundary'))
+
+    consoleMock.mockRestore()
   })
 
   it('should render the correct amount of times in Suspense mode when gcTime is set to 0', async () => {