@tanstack/solid-query 5.0.0-beta.9 → 5.0.0-rc.1

package/src/__tests__/createInfiniteQuery.test.tsx

@@ -33,7 +33,7 @@ import type {
 import type { Mock } from 'vitest'
 
 interface Result {
-  items: number[]
+  items: Array<number>
   nextId?: number
   prevId?: number
   ts: number
@@ -62,14 +62,14 @@ describe('useInfiniteQuery', () => {
 
   it('should return the correct states for a successful query', async () => {
     const key = queryKey()
-    const states: CreateInfiniteQueryResult<InfiniteData<number>>[] = []
+    const states: Array<CreateInfiniteQueryResult<InfiniteData<number>>> = []
 
     function Page() {
       const state = createInfiniteQuery(() => ({
         queryKey: key,
         queryFn: ({ pageParam }) => Number(pageParam),
         getNextPageParam: (lastPage) => lastPage + 1,
-        defaultPageParam: 0,
+        initialPageParam: 0,
       }))
       createRenderEffect(() => {
         states.push({ ...state })
@@ -170,7 +170,7 @@ describe('useInfiniteQuery', () => {
 
         retry: 1,
         retryDelay: 10,
-        defaultPageParam: start,
+        initialPageParam: start,
         getNextPageParam: (lastPage) => lastPage + 1,
       }))
 
@@ -199,8 +199,9 @@ describe('useInfiniteQuery', () => {
 
   it('should keep the previous data when placeholderData is set', async () => {
     const key = queryKey()
-    const states: Partial<CreateInfiniteQueryResult<InfiniteData<string>>>[] =
-      []
+    const states: Array<
+      Partial<CreateInfiniteQueryResult<InfiniteData<string>>>
+    > = []
 
     function Page() {
       const [order, setOrder] = createSignal('desc')
@@ -213,7 +214,7 @@ describe('useInfiniteQuery', () => {
         },
 
         getNextPageParam: () => 1,
-        defaultPageParam: 0,
+        initialPageParam: 0,
         placeholderData: keepPreviousData,
         notifyOnChangeProps: 'all',
       }))
@@ -304,7 +305,7 @@ describe('useInfiniteQuery', () => {
 
   it('should be able to select a part of the data', async () => {
     const key = queryKey()
-    const states: CreateInfiniteQueryResult<InfiniteData<string>>[] = []
+    const states: Array<CreateInfiniteQueryResult<InfiniteData<string>>> = []
 
     function Page() {
       const state = createInfiniteQuery(() => ({
@@ -315,7 +316,7 @@ describe('useInfiniteQuery', () => {
           pageParams: data.pageParams,
         }),
         getNextPageParam: () => undefined,
-        defaultPageParam: 0,
+        initialPageParam: 0,
       }))
       createRenderEffect(() => {
         states.push({ ...state })
@@ -344,9 +345,9 @@ describe('useInfiniteQuery', () => {
 
   it('should be able to select a new result and not cause infinite renders', async () => {
     const key = queryKey()
-    const states: CreateInfiniteQueryResult<
-      InfiniteData<{ count: number; id: number }>
-    >[] = []
+    const states: Array<
+      CreateInfiniteQueryResult<InfiniteData<{ count: number; id: number }>>
+    > = []
     let selectCalled = 0
 
     function Page() {
@@ -361,7 +362,7 @@ describe('useInfiniteQuery', () => {
           }
         },
         getNextPageParam: () => undefined,
-        defaultPageParam: 0,
+        initialPageParam: 0,
       }))
       createRenderEffect(() => {
         states.push({ ...state })
@@ -391,8 +392,9 @@ describe('useInfiniteQuery', () => {
 
   it('should be able to reverse the data', async () => {
     const key = queryKey()
-    const states: Partial<CreateInfiniteQueryResult<InfiniteData<number>>>[] =
-      []
+    const states: Array<
+      Partial<CreateInfiniteQueryResult<InfiniteData<number>>>
+    > = []
 
     function Page() {
       const state = createInfiniteQuery(() => ({
@@ -408,7 +410,7 @@ describe('useInfiniteQuery', () => {
         }),
         notifyOnChangeProps: 'all',
         getNextPageParam: () => 1,
-        defaultPageParam: 0,
+        initialPageParam: 0,
       }))
 
       createRenderEffect(
@@ -466,8 +468,9 @@ describe('useInfiniteQuery', () => {
 
   it('should be able to fetch a previous page', async () => {
     const key = queryKey()
-    const states: Partial<CreateInfiniteQueryResult<InfiniteData<number>>>[] =
-      []
+    const states: Array<
+      Partial<CreateInfiniteQueryResult<InfiniteData<number>>>
+    > = []
 
     function Page() {
       const start = 10
@@ -479,7 +482,7 @@ describe('useInfiniteQuery', () => {
         },
         getNextPageParam: (lastPage) => lastPage + 1,
         getPreviousPageParam: (firstPage) => firstPage - 1,
-        defaultPageParam: start,
+        initialPageParam: start,
         notifyOnChangeProps: 'all',
       }))
 
@@ -554,8 +557,9 @@ describe('useInfiniteQuery', () => {
 
   it('should be able to refetch when providing page params automatically', async () => {
     const key = queryKey()
-    const states: Partial<CreateInfiniteQueryResult<InfiniteData<number>>>[] =
-      []
+    const states: Array<
+      Partial<CreateInfiniteQueryResult<InfiniteData<number>>>
+    > = []
 
     function Page() {
       const state = createInfiniteQuery(() => ({
@@ -567,7 +571,7 @@ describe('useInfiniteQuery', () => {
 
         getPreviousPageParam: (firstPage) => firstPage - 1,
         getNextPageParam: (lastPage) => lastPage + 1,
-        defaultPageParam: 10,
+        initialPageParam: 10,
         notifyOnChangeProps: 'all',
       }))
 
@@ -675,8 +679,9 @@ describe('useInfiniteQuery', () => {
 
   it('should silently cancel any ongoing fetch when fetching more', async () => {
     const key = queryKey()
-    const states: Partial<CreateInfiniteQueryResult<InfiniteData<number>>>[] =
-      []
+    const states: Array<
+      Partial<CreateInfiniteQueryResult<InfiniteData<number>>>
+    > = []
 
     function Page() {
       const start = 10
@@ -688,7 +693,7 @@ describe('useInfiniteQuery', () => {
         },
 
         getNextPageParam: (lastPage) => lastPage + 1,
-        defaultPageParam: start,
+        initialPageParam: start,
         notifyOnChangeProps: 'all',
       }))
 
@@ -764,8 +769,8 @@ describe('useInfiniteQuery', () => {
   it('should silently cancel an ongoing fetchNextPage request when another fetchNextPage is invoked', async () => {
     const key = queryKey()
     const start = 10
-    const onAborts: Mock<any, any>[] = []
-    const abortListeners: Mock<any, any>[] = []
+    const onAborts: Array<Mock<any, any>> = []
+    const abortListeners: Array<Mock<any, any>> = []
     const fetchPage = vi.fn<
       [QueryFunctionContext<typeof key, number>],
       Promise<number>
@@ -786,7 +791,7 @@ describe('useInfiniteQuery', () => {
         queryKey: key,
         queryFn: fetchPage,
         getNextPageParam: (lastPage) => lastPage + 1,
-        defaultPageParam: start,
+        initialPageParam: start,
       }))
 
       createEffect(() => {
@@ -846,8 +851,8 @@ describe('useInfiniteQuery', () => {
   it('should not cancel an ongoing fetchNextPage request when another fetchNextPage is invoked if `cancelRefetch: false` is used ', async () => {
     const key = queryKey()
     const start = 10
-    const onAborts: Mock<any, any>[] = []
-    const abortListeners: Mock<any, any>[] = []
+    const onAborts: Array<Mock<any, any>> = []
+    const abortListeners: Array<Mock<any, any>> = []
     const fetchPage = vi.fn<
       [QueryFunctionContext<typeof key, number>],
       Promise<number>
@@ -868,7 +873,7 @@ describe('useInfiniteQuery', () => {
         queryKey: key,
         queryFn: fetchPage,
         getNextPageParam: (lastPage) => lastPage + 1,
-        defaultPageParam: start,
+        initialPageParam: start,
       }))
 
       createEffect(() => {
@@ -918,7 +923,7 @@ describe('useInfiniteQuery', () => {
 
   it('should keep fetching first page when not loaded yet and triggering fetch more', async () => {
     const key = queryKey()
-    const states: CreateInfiniteQueryResult<InfiniteData<number>>[] = []
+    const states: Array<CreateInfiniteQueryResult<InfiniteData<number>>> = []
 
     function Page() {
       const start = 10
@@ -930,7 +935,7 @@ describe('useInfiniteQuery', () => {
         },
 
         getNextPageParam: (lastPage) => lastPage + 1,
-        defaultPageParam: start,
+        initialPageParam: start,
         notifyOnChangeProps: 'all',
       }))
 
@@ -992,7 +997,7 @@ describe('useInfiniteQuery', () => {
         getNextPageParam: (_, allPages) => {
           return allPages.length === 4 ? undefined : allPages.length
         },
-        defaultPageParam: 0,
+        initialPageParam: 0,
       }))
 
       return null
@@ -1028,8 +1033,9 @@ describe('useInfiniteQuery', () => {
 
   it('should be able to set new pages with the query client', async () => {
     const key = queryKey()
-    const states: Partial<CreateInfiniteQueryResult<InfiniteData<number>>>[] =
-      []
+    const states: Array<
+      Partial<CreateInfiniteQueryResult<InfiniteData<number>>>
+    > = []
 
     function Page() {
       const [firstPage, setFirstPage] = createSignal(0)
@@ -1043,7 +1049,7 @@ describe('useInfiniteQuery', () => {
 
         getNextPageParam: (lastPage) => lastPage + 1,
         notifyOnChangeProps: 'all',
-        defaultPageParam: firstPage(),
+        initialPageParam: firstPage(),
       }))
 
       createRenderEffect(() => {
@@ -1123,8 +1129,9 @@ describe('useInfiniteQuery', () => {
 
   it('should only refetch the first page when initialData is provided', async () => {
     const key = queryKey()
-    const states: Partial<CreateInfiniteQueryResult<InfiniteData<number>>>[] =
-      []
+    const states: Array<
+      Partial<CreateInfiniteQueryResult<InfiniteData<number>>>
+    > = []
 
     function Page() {
       const state = createInfiniteQuery(() => ({
@@ -1136,7 +1143,7 @@ describe('useInfiniteQuery', () => {
 
         initialData: { pages: [1], pageParams: [1] },
         getNextPageParam: (lastPage) => lastPage + 1,
-        defaultPageParam: 0,
+        initialPageParam: 0,
         notifyOnChangeProps: 'all',
       }))
 
@@ -1201,13 +1208,13 @@ describe('useInfiniteQuery', () => {
 
   it('should set hasNextPage to false if getNextPageParam returns undefined', async () => {
     const key = queryKey()
-    const states: CreateInfiniteQueryResult<InfiniteData<number>>[] = []
+    const states: Array<CreateInfiniteQueryResult<InfiniteData<number>>> = []
 
     function Page() {
       const state = createInfiniteQuery(() => ({
         queryKey: key,
         queryFn: ({ pageParam }) => Number(pageParam),
-        defaultPageParam: 1,
+        initialPageParam: 1,
         getNextPageParam: () => undefined,
       }))
 
@@ -1245,13 +1252,13 @@ describe('useInfiniteQuery', () => {
 
   it('should compute hasNextPage correctly using initialData', async () => {
     const key = queryKey()
-    const states: CreateInfiniteQueryResult<InfiniteData<number>>[] = []
+    const states: Array<CreateInfiniteQueryResult<InfiniteData<number>>> = []
 
     function Page() {
       const state = createInfiniteQuery(() => ({
         queryKey: key,
         queryFn: ({ pageParam }): number => pageParam,
-        defaultPageParam: 10,
+        initialPageParam: 10,
         initialData: { pages: [10], pageParams: [10] },
         getNextPageParam: (lastPage) => (lastPage === 10 ? 11 : undefined),
       }))
@@ -1289,13 +1296,13 @@ describe('useInfiniteQuery', () => {
 
   it('should compute hasNextPage correctly for falsy getFetchMore return value using initialData', async () => {
     const key = queryKey()
-    const states: CreateInfiniteQueryResult<InfiniteData<number>>[] = []
+    const states: Array<CreateInfiniteQueryResult<InfiniteData<number>>> = []
 
     function Page() {
       const state = createInfiniteQuery(() => ({
         queryKey: key,
         queryFn: ({ pageParam }): number => pageParam,
-        defaultPageParam: 10,
+        initialPageParam: 10,
         initialData: { pages: [10], pageParams: [10] },
         getNextPageParam: () => undefined,
       }))
@@ -1333,13 +1340,13 @@ describe('useInfiniteQuery', () => {
 
   it('should not use selected data when computing hasNextPage', async () => {
     const key = queryKey()
-    const states: CreateInfiniteQueryResult<InfiniteData<string>>[] = []
+    const states: Array<CreateInfiniteQueryResult<InfiniteData<string>>> = []
 
     function Page() {
       const state = createInfiniteQuery(() => ({
         queryKey: key,
         queryFn: ({ pageParam }) => Number(pageParam),
-        defaultPageParam: 1,
+        initialPageParam: 1,
         getNextPageParam: (lastPage) => (lastPage === 1 ? 2 : undefined),
         select: (data) => ({
           pages: data.pages.map((x) => x.toString()),
@@ -1402,7 +1409,7 @@ describe('useInfiniteQuery', () => {
         queryKey: key,
         queryFn: ({ pageParam }) =>
           fetchItemsWithLimit(pageParam, fetchCountRef++),
-        defaultPageParam: 0,
+        initialPageParam: 0,
         getNextPageParam: (lastPage) => lastPage.nextId,
       }))
 
@@ -1536,7 +1543,7 @@ describe('useInfiniteQuery', () => {
             fetchCountRef++,
             pageParam === MAX || (pageParam === MAX - 1 && isRemovedLastPage()),
           ),
-        defaultPageParam: 0,
+        initialPageParam: 0,
         getNextPageParam: (lastPage) => lastPage.nextId,
       }))
 
@@ -1670,7 +1677,7 @@ describe('useInfiniteQuery', () => {
         queryKey: key,
         queryFn,
         getNextPageParam: () => undefined,
-        defaultPageParam: 0,
+        initialPageParam: 0,
       }))
       return (
         <div>
@@ -1704,7 +1711,7 @@ describe('useInfiniteQuery', () => {
           queryKey: key,
           queryFn,
           getNextPageParam: () => undefined,
-          defaultPageParam: 0,
+          initialPageParam: 0,
         }),
         () => queryClient,
       )

package/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 () => {
@@ -312,7 +318,7 @@ describe('createMutation', () => {
   })
 
   it('should be able to override the useMutation success callbacks', async () => {
-    const callbacks: string[] = []
+    const callbacks: Array<string> = []
 
     function Page() {
       const mutation = createMutation(() => ({
@@ -363,7 +369,7 @@ describe('createMutation', () => {
   })
 
   it('should be able to override the error callbacks when using mutateAsync', async () => {
-    const callbacks: string[] = []
+    const callbacks: Array<string> = []
 
     function Page() {
       const mutation = createMutation(() => ({
@@ -425,7 +431,7 @@ describe('createMutation', () => {
       },
     })
 
-    const states: CreateMutationResult<any, any, any, any>[] = []
+    const states: Array<CreateMutationResult<any, any, any, any>> = []
 
     function Page() {
       const mutation = createMutation<string, unknown, string>(() => ({
@@ -670,7 +676,7 @@ describe('createMutation', () => {
     const onlineMock = mockOnlineManagerIsOnline(false)
 
     let count = 0
-    const states: CreateMutationResult<any, any, any, any>[] = []
+    const states: Array<CreateMutationResult<any, any, any, any>> = []
 
     function Page() {
       const mutation = createMutation(() => ({
@@ -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 () => {