Change .toHaveBeenCalledTimes(0) to .not.toHaveBeenCalled()

Author: aryaemami59

packages/toolkit/src/query/tests/cacheLifecycle.test.ts

@@ -505,7 +505,7 @@ test('updateCachedData', async () => {
             draft.value = 'TEST'
             trackCalls()
           })
-          expect(trackCalls).toHaveBeenCalledTimes(0)
+          expect(trackCalls).not.toHaveBeenCalled()
           expect(getCacheEntry().data).toEqual(undefined)
 
           gotFirstValue(await cacheDataLoaded)
@@ -612,7 +612,7 @@ test('dispatching a mutation initializer with `track: false` does not start a li
   await storeRef.store.dispatch(
     extended.endpoints.injected.initiate(undefined, { track: false }),
   )
-  expect(onNewCacheEntry).toHaveBeenCalledTimes(0)
+  expect(onNewCacheEntry).not.toHaveBeenCalled()
 
   await storeRef.store.dispatch(extended.endpoints.injected.initiate(undefined))
   expect(onNewCacheEntry).toHaveBeenCalledOnce()

packages/toolkit/src/query/tests/createApi.test.ts

@@ -1017,7 +1017,7 @@ describe('custom serializeQueryArgs per endpoint', () => {
       api.endpoints.queryWithNoSerializer.initiate(99),
     )
 
-    expect(serializer1).toHaveBeenCalledTimes(0)
+    expect(serializer1).not.toHaveBeenCalled()
 
     await storeRef.store.dispatch(
       api.endpoints.queryWithCustomSerializer.initiate(42),
@@ -1046,7 +1046,7 @@ describe('custom serializeQueryArgs per endpoint', () => {
   })
 
   it('Works via injectEndpoints', async () => {
-    expect(serializer2).toHaveBeenCalledTimes(0)
+    expect(serializer2).not.toHaveBeenCalled()
 
     await storeRef.store.dispatch(
       injectedApi.endpoints.injectedQueryWithCustomSerializer.initiate(5),

packages/toolkit/src/tests/createAsyncThunk.test.ts

@@ -606,7 +606,7 @@ describe('conditional skipping of asyncThunks', () => {
     const condition = () => Promise.resolve(false)
     const asyncThunk = createAsyncThunk('test', payloadCreator, { condition })
     await asyncThunk(arg)(dispatch, getState, extra)
-    expect(dispatch).toHaveBeenCalledTimes(0)
+    expect(dispatch).not.toHaveBeenCalled()
   })
 
   test('async condition with rejected promise', async () => {
@@ -631,14 +631,14 @@ describe('conditional skipping of asyncThunks', () => {
       thunkPromise.abort()
       await thunkPromise
     } catch (err) {}
-    expect(dispatch).toHaveBeenCalledTimes(0)
+    expect(dispatch).not.toHaveBeenCalled()
   })
 
   test('rejected action is not dispatched by default', async () => {
     const asyncThunk = createAsyncThunk('test', payloadCreator, { condition })
     await asyncThunk(arg)(dispatch, getState, extra)
 
-    expect(dispatch).toHaveBeenCalledTimes(0)
+    expect(dispatch).not.toHaveBeenCalled()
   })
 
   test('does not fail when attempting to abort a canceled promise', async () => {