@tanstack/solid-query 4.7.1 → 4.7.2

package/LICENSE

@@ -0,0 +1,21 @@
+MIT License
+
+Copyright (c) 2021 Tanner Linsley
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tanstack/solid-query",
-  "version": "4.7.1",
+  "version": "4.7.2",
   "description": "Primitives for managing, caching and syncing asynchronous and remote data in Solid",
   "author": "tannerlinsley",
   "license": "MIT",
@@ -26,12 +26,6 @@
     "./package.json": "./package.json"
   },
   "sideEffects": false,
-  "scripts": {
-    "clean": "rm -rf ./build",
-    "test:eslint": "../../node_modules/.bin/eslint --ext .ts,.tsx ./src",
-    "test:jest": "../../node_modules/.bin/jest --config jest.config.js",
-    "test:jest:dev": "yarn test:jest --watch"
-  },
   "files": [
     "build/lib/*",
     "build/umd/*",
@@ -43,10 +37,16 @@
     "jscodeshift": "^0.13.1"
   },
   "dependencies": {
-    "@tanstack/query-core": "4.7.1"
+    "@tanstack/query-core": "4.7.2"
   },
   "peerDependencies": {
     "solid-js": "^1.5.4"
   },
-  "peerDependenciesMeta": {}
-}
+  "peerDependenciesMeta": {},
+  "scripts": {
+    "clean": "rm -rf ./build",
+    "test:eslint": "../../node_modules/.bin/eslint --ext .ts,.tsx ./src",
+    "test:jest": "../../node_modules/.bin/jest --config jest.config.js",
+    "test:jest:dev": "yarn test:jest --watch"
+  }
+}

package/src/__tests__/createInfiniteQuery.test.tsx

@@ -938,34 +938,28 @@ describe('useInfiniteQuery', () => {
     const firstCtx = fetchPage.mock.calls[callIndex]![0]
     expect(firstCtx.pageParam).toBeUndefined()
     expect(firstCtx.queryKey).toEqual(key())
-    if (typeof AbortSignal === 'function') {
-      expect(firstCtx.signal).toBeInstanceOf(AbortSignal)
-      expect(firstCtx.signal?.aborted).toBe(false)
-      expect(onAborts[callIndex]).not.toHaveBeenCalled()
-      expect(abortListeners[callIndex]).not.toHaveBeenCalled()
-    }
+    expect(firstCtx.signal).toBeInstanceOf(AbortSignal)
+    expect(firstCtx.signal?.aborted).toBe(false)
+    expect(onAborts[callIndex]).not.toHaveBeenCalled()
+    expect(abortListeners[callIndex]).not.toHaveBeenCalled()
 
     callIndex = 1
     const secondCtx = fetchPage.mock.calls[callIndex]![0]
     expect(secondCtx.pageParam).toBe(11)
     expect(secondCtx.queryKey).toEqual(key())
-    if (typeof AbortSignal === 'function') {
-      expect(secondCtx.signal).toBeInstanceOf(AbortSignal)
-      expect(secondCtx.signal?.aborted).toBe(true)
-      expect(onAborts[callIndex]).toHaveBeenCalledTimes(1)
-      expect(abortListeners[callIndex]).toHaveBeenCalledTimes(1)
-    }
+    expect(secondCtx.signal).toBeInstanceOf(AbortSignal)
+    expect(secondCtx.signal?.aborted).toBe(true)
+    expect(onAborts[callIndex]).toHaveBeenCalledTimes(1)
+    expect(abortListeners[callIndex]).toHaveBeenCalledTimes(1)
 
     callIndex = 2
     const thirdCtx = fetchPage.mock.calls[callIndex]![0]
     expect(thirdCtx.pageParam).toBe(11)
     expect(thirdCtx.queryKey).toEqual(key())
-    if (typeof AbortSignal === 'function') {
-      expect(thirdCtx.signal).toBeInstanceOf(AbortSignal)
-      expect(thirdCtx.signal?.aborted).toBe(false)
-      expect(onAborts[callIndex]).not.toHaveBeenCalled()
-      expect(abortListeners[callIndex]).not.toHaveBeenCalled()
-    }
+    expect(thirdCtx.signal).toBeInstanceOf(AbortSignal)
+    expect(thirdCtx.signal?.aborted).toBe(false)
+    expect(onAborts[callIndex]).not.toHaveBeenCalled()
+    expect(abortListeners[callIndex]).not.toHaveBeenCalled()
   })
 
   it('should not cancel an ongoing fetchNextPage request when another fetchNextPage is invoked if `cancelRefetch: false` is used ', async () => {
@@ -1024,23 +1018,19 @@ describe('useInfiniteQuery', () => {
     const firstCtx = fetchPage.mock.calls[callIndex]![0]
     expect(firstCtx.pageParam).toBeUndefined()
     expect(firstCtx.queryKey).toEqual(key())
-    if (typeof AbortSignal === 'function') {
-      expect(firstCtx.signal).toBeInstanceOf(AbortSignal)
-      expect(firstCtx.signal?.aborted).toBe(false)
-      expect(onAborts[callIndex]).not.toHaveBeenCalled()
-      expect(abortListeners[callIndex]).not.toHaveBeenCalled()
-    }
+    expect(firstCtx.signal).toBeInstanceOf(AbortSignal)
+    expect(firstCtx.signal?.aborted).toBe(false)
+    expect(onAborts[callIndex]).not.toHaveBeenCalled()
+    expect(abortListeners[callIndex]).not.toHaveBeenCalled()
 
     callIndex = 1
     const secondCtx = fetchPage.mock.calls[callIndex]![0]
     expect(secondCtx.pageParam).toBe(11)
     expect(secondCtx.queryKey).toEqual(key())
-    if (typeof AbortSignal === 'function') {
-      expect(secondCtx.signal).toBeInstanceOf(AbortSignal)
-      expect(secondCtx.signal?.aborted).toBe(false)
-      expect(onAborts[callIndex]).not.toHaveBeenCalled()
-      expect(abortListeners[callIndex]).not.toHaveBeenCalled()
-    }
+    expect(secondCtx.signal).toBeInstanceOf(AbortSignal)
+    expect(secondCtx.signal?.aborted).toBe(false)
+    expect(onAborts[callIndex]).not.toHaveBeenCalled()
+    expect(abortListeners[callIndex]).not.toHaveBeenCalled()
   })
 
   it('should keep fetching first page when not loaded yet and triggering fetch more', async () => {
@@ -1145,22 +1135,12 @@ describe('useInfiniteQuery', () => {
 
     await sleep(300)
 
-    if (typeof AbortSignal === 'function') {
-      expect(fetches).toBe(2)
-      expect(queryClient.getQueryState(key())).toMatchObject({
-        data: initialData,
-        status: 'success',
-        error: null,
-      })
-    } else {
-      // if AbortSignal is not consumed, fetches should not abort
-      expect(fetches).toBe(4)
-      expect(queryClient.getQueryState(key())).toMatchObject({
-        data: { pages: [0, 10, 20, 30], pageParams: [0, 1, 2, 3] },
-        status: 'success',
-        error: null,
-      })
-    }
+    expect(fetches).toBe(2)
+    expect(queryClient.getQueryState(key())).toMatchObject({
+      data: initialData,
+      status: 'success',
+      error: null,
+    })
   })
 
   it('should be able to override the cursor in the fetchNextPage callback', async () => {
@@ -1857,7 +1837,7 @@ describe('useInfiniteQuery', () => {
       const promise = new Promise<string>((resolve, reject) => {
         cancelFn = jest.fn(() => reject('Cancelled'))
         signal?.addEventListener('abort', cancelFn)
-        sleep(10).then(() => resolve('OK'))
+        sleep(20).then(() => resolve('OK'))
       })
 
       return promise
@@ -1882,8 +1862,6 @@ describe('useInfiniteQuery', () => {
 
     await waitFor(() => screen.getByText('off'))
 
-    if (typeof AbortSignal === 'function') {
-      expect(cancelFn).toHaveBeenCalled()
-    }
+    expect(cancelFn).toHaveBeenCalled()
   })
 })

package/src/__tests__/createQuery.test.tsx

@@ -4936,7 +4936,7 @@ describe('createQuery', () => {
       const promise = new Promise<string>((resolve, reject) => {
         cancelFn = jest.fn(() => reject('Cancelled'))
         signal?.addEventListener('abort', cancelFn)
-        sleep(10).then(() => resolve('OK'))
+        sleep(20).then(() => resolve('OK'))
       })
 
       return promise
@@ -4961,9 +4961,7 @@ describe('createQuery', () => {
 
     await waitFor(() => screen.getByText('off'))
 
-    if (typeof AbortSignal === 'function') {
-      expect(cancelFn).toHaveBeenCalled()
-    }
+    expect(cancelFn).toHaveBeenCalled()
   })
 
   it('should cancel the query if the signal was consumed and there are no more subscriptions', async () => {
@@ -5013,19 +5011,11 @@ describe('createQuery', () => {
       dataUpdateCount: 1,
     })
 
-    if (typeof AbortSignal === 'function') {
-      expect(queryCache.find([key(), 1])?.state).toMatchObject({
-        data: undefined,
-        status: 'loading',
-        fetchStatus: 'idle',
-      })
-    } else {
-      expect(queryCache.find([key(), 1])?.state).toMatchObject({
-        data: 'data 1',
-        status: 'success',
-        dataUpdateCount: 1,
-      })
-    }
+    expect(queryCache.find([key(), 1])?.state).toMatchObject({
+      data: undefined,
+      status: 'loading',
+      fetchStatus: 'idle',
+    })
 
     expect(queryCache.find([key(), 2])?.state).toMatchObject({
       data: 'data 2',
@@ -5033,19 +5023,11 @@ describe('createQuery', () => {
       dataUpdateCount: 1,
     })
 
-    if (typeof AbortSignal === 'function') {
-      expect(queryCache.find([key(), 3])?.state).toMatchObject({
-        data: undefined,
-        status: 'loading',
-        fetchStatus: 'idle',
-      })
-    } else {
-      expect(queryCache.find([key(), 3])?.state).toMatchObject({
-        data: 'data 3',
-        status: 'success',
-        dataUpdateCount: 1,
-      })
-    }
+    expect(queryCache.find([key(), 3])?.state).toMatchObject({
+      data: undefined,
+      status: 'loading',
+      fetchStatus: 'idle',
+    })
   })
 
   it('should refetch when quickly switching to a failed query', async () => {
@@ -6195,7 +6177,7 @@ describe('createQuery', () => {
         status: 'success',
       })
 
-      expect(count).toBe(typeof AbortSignal === 'function' ? 1 : 2)
+      expect(count).toBe(1)
 
       onlineMock.mockRestore()
     })