Change .toHaveBeenCalledTimes(1) to .toHaveBeenCalledOnce()

Author: aryaemami59

packages/toolkit/src/query/tests/buildThunks.test.tsx

@@ -102,7 +102,7 @@ describe('re-triggering behavior on arg change', () => {
       expect(result.current.status).not.toBe('pending')
     })
 
-    expect(spy).toHaveBeenCalledTimes(1)
+    expect(spy).toHaveBeenCalledOnce()
 
     for (let x = 1; x < 3; x++) {
       rerender(6)
@@ -133,7 +133,7 @@ describe('re-triggering behavior on arg change', () => {
     await waitFor(() => {
       expect(result.current.status).not.toBe('pending')
     })
-    expect(spy).toHaveBeenCalledTimes(1)
+    expect(spy).toHaveBeenCalledOnce()
 
     for (let x = 1; x < 3; x++) {
       rerender({ name: 'Bob', likes: 'waffles' })
@@ -166,7 +166,7 @@ describe('re-triggering behavior on arg change', () => {
     await waitFor(() => {
       expect(result.current.status).not.toBe('pending')
     })
-    expect(spy).toHaveBeenCalledTimes(1)
+    expect(spy).toHaveBeenCalledOnce()
 
     for (let x = 1; x < 3; x++) {
       rerender({ person: { name: name + x } })
@@ -189,14 +189,14 @@ describe('re-triggering behavior on arg change', () => {
     await waitFor(() => {
       expect(result.current.status).not.toBe('pending')
     })
-    expect(spy).toHaveBeenCalledTimes(1)
+    expect(spy).toHaveBeenCalledOnce()
 
     for (let x = 1; x < 3; x++) {
       rerender({ likes: 'Bananas', name: 'Tim' })
       await waitFor(() => {
         expect(result.current.status).not.toBe('pending')
       })
-      expect(spy).toHaveBeenCalledTimes(1)
+      expect(spy).toHaveBeenCalledOnce()
     }
   })
 })

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

@@ -515,7 +515,7 @@ test('updateCachedData', async () => {
             draft.value = 'TEST'
             trackCalls()
           })
-          expect(trackCalls).toHaveBeenCalledTimes(1)
+          expect(trackCalls).toHaveBeenCalledOnce()
           expect(getCacheEntry().data).toEqual({ value: 'TEST' })
 
           await cacheEntryRemoved
@@ -526,7 +526,7 @@ test('updateCachedData', async () => {
             draft.value = 'TEST2'
             trackCalls()
           })
-          expect(trackCalls).toHaveBeenCalledTimes(1)
+          expect(trackCalls).toHaveBeenCalledOnce()
           expect(getCacheEntry().data).toEqual(undefined)
 
           onCleanup()
@@ -564,15 +564,15 @@ test('dispatching further actions does not trigger another lifecycle', async ()
     }),
   })
   await storeRef.store.dispatch(extended.endpoints.injected.initiate())
-  expect(onNewCacheEntry).toHaveBeenCalledTimes(1)
+  expect(onNewCacheEntry).toHaveBeenCalledOnce()
 
   await storeRef.store.dispatch(extended.endpoints.injected.initiate())
-  expect(onNewCacheEntry).toHaveBeenCalledTimes(1)
+  expect(onNewCacheEntry).toHaveBeenCalledOnce()
 
   await storeRef.store.dispatch(
     extended.endpoints.injected.initiate(undefined, { forceRefetch: true }),
   )
-  expect(onNewCacheEntry).toHaveBeenCalledTimes(1)
+  expect(onNewCacheEntry).toHaveBeenCalledOnce()
 })
 
 test('dispatching a query initializer with `subscribe: false` does also start a lifecycle', async () => {
@@ -590,11 +590,11 @@ test('dispatching a query initializer with `subscribe: false` does also start a
   await storeRef.store.dispatch(
     extended.endpoints.injected.initiate(undefined, { subscribe: false }),
   )
-  expect(onNewCacheEntry).toHaveBeenCalledTimes(1)
+  expect(onNewCacheEntry).toHaveBeenCalledOnce()
 
   // will not be called a second time though
   await storeRef.store.dispatch(extended.endpoints.injected.initiate(undefined))
-  expect(onNewCacheEntry).toHaveBeenCalledTimes(1)
+  expect(onNewCacheEntry).toHaveBeenCalledOnce()
 })
 
 test('dispatching a mutation initializer with `track: false` does not start a lifecycle', async () => {
@@ -615,5 +615,5 @@ test('dispatching a mutation initializer with `track: false` does not start a li
   expect(onNewCacheEntry).toHaveBeenCalledTimes(0)
 
   await storeRef.store.dispatch(extended.endpoints.injected.initiate(undefined))
-  expect(onNewCacheEntry).toHaveBeenCalledTimes(1)
+  expect(onNewCacheEntry).toHaveBeenCalledOnce()
 })

packages/toolkit/src/query/tests/polling.test.tsx

@@ -45,13 +45,13 @@ describe('polling tests', () => {
       }),
     )
 
-    expect(mockBaseQuery).toHaveBeenCalledTimes(1)
+    expect(mockBaseQuery).toHaveBeenCalledOnce()
 
     storeRef.store.dispatch(api.util.resetApiState())
 
     await delay(30)
 
-    expect(mockBaseQuery).toHaveBeenCalledTimes(1)
+    expect(mockBaseQuery).toHaveBeenCalledOnce()
   })
 
   it('replaces polling interval when the subscription options are updated', async () => {

packages/toolkit/src/query/tests/queryLifecycle.test.tsx

@@ -438,9 +438,9 @@ test('query: will only start lifecycle if query is not skipped due to `condition
   const promise = storeRef.store.dispatch(
     extended.endpoints.injected.initiate('arg'),
   )
-  expect(onStart).toHaveBeenCalledTimes(1)
+  expect(onStart).toHaveBeenCalledOnce()
   storeRef.store.dispatch(extended.endpoints.injected.initiate('arg'))
-  expect(onStart).toHaveBeenCalledTimes(1)
+  expect(onStart).toHaveBeenCalledOnce()
   await promise
   storeRef.store.dispatch(
     extended.endpoints.injected.initiate('arg', { forceRefetch: true }),

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

@@ -1,6 +1,6 @@
-import { vi } from 'vitest'
 import type { BaseQueryFn, FetchBaseQueryError } from '@reduxjs/toolkit/query'
 import { createApi, retry } from '@reduxjs/toolkit/query'
+import { vi } from 'vitest'
 import { setupApiStore } from '../../tests/utils/helpers'
 
 beforeEach(() => {
@@ -223,7 +223,7 @@ describe('configuration', () => {
 
     await loopTimers(2)
 
-    expect(baseBaseQuery).toHaveBeenCalledTimes(1)
+    expect(baseBaseQuery).toHaveBeenCalledOnce()
   })
   test('calling retry.fail(error) will skip retrying and expose the error directly', async () => {
     const error = { message: 'banana' }
@@ -255,7 +255,7 @@ describe('configuration', () => {
 
     await loopTimers(2)
 
-    expect(baseBaseQuery).toHaveBeenCalledTimes(1)
+    expect(baseBaseQuery).toHaveBeenCalledOnce()
     expect(result.error).toEqual(error)
     expect(result).toEqual({
       endpointName: 'q1',
@@ -464,6 +464,6 @@ describe('configuration', () => {
     storeRef.store.dispatch(api.endpoints.q1.initiate({}))
     await loopTimers(2)
 
-    expect(baseBaseQuery).toHaveBeenCalledTimes(1)
+    expect(baseBaseQuery).toHaveBeenCalledOnce()
   })
 })

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

@@ -586,7 +586,7 @@ describe('conditional skipping of asyncThunks', () => {
     const asyncThunk = createAsyncThunk('test', payloadCreator, { condition })
     await asyncThunk(arg)(dispatch, getState, extra)
 
-    expect(condition).toHaveBeenCalledTimes(1)
+    expect(condition).toHaveBeenCalledOnce()
     expect(condition).toHaveBeenLastCalledWith(
       arg,
       expect.objectContaining({ getState, extra }),
@@ -597,7 +597,7 @@ describe('conditional skipping of asyncThunks', () => {
     const condition = () => true
     const asyncThunk = createAsyncThunk('test', payloadCreator, { condition })
     const thunkCallPromise = asyncThunk(arg)(dispatch, getState, extra)
-    expect(dispatch).toHaveBeenCalledTimes(1)
+    expect(dispatch).toHaveBeenCalledOnce()
     await thunkCallPromise
     expect(dispatch).toHaveBeenCalledTimes(2)
   })
@@ -613,7 +613,7 @@ describe('conditional skipping of asyncThunks', () => {
     const condition = () => Promise.reject()
     const asyncThunk = createAsyncThunk('test', payloadCreator, { condition })
     await asyncThunk(arg)(dispatch, getState, extra)
-    expect(dispatch).toHaveBeenCalledTimes(1)
+    expect(dispatch).toHaveBeenCalledOnce()
     expect(dispatch).toHaveBeenLastCalledWith(
       expect.objectContaining({ type: 'test/rejected' }),
     )
@@ -663,7 +663,7 @@ describe('conditional skipping of asyncThunks', () => {
     })
     await asyncThunk(arg)(dispatch, getState, extra)
 
-    expect(dispatch).toHaveBeenCalledTimes(1)
+    expect(dispatch).toHaveBeenCalledOnce()
     expect(dispatch).toHaveBeenLastCalledWith(
       expect.objectContaining({
         error: {
@@ -871,9 +871,9 @@ test('`condition` will see state changes from a synchronously invoked asyncThunk
   })
 
   store.dispatch(asyncThunk({ force: false }))
-  expect(onStart).toHaveBeenCalledTimes(1)
+  expect(onStart).toHaveBeenCalledOnce()
   store.dispatch(asyncThunk({ force: false }))
-  expect(onStart).toHaveBeenCalledTimes(1)
+  expect(onStart).toHaveBeenCalledOnce()
   store.dispatch(asyncThunk({ force: true }))
   expect(onStart).toHaveBeenCalledTimes(2)
 })

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

@@ -217,7 +217,7 @@ describe('createReducer', () => {
       expect(spy).not.toHaveBeenCalled()
 
       const initialState = dummyReducer(undefined, { type: 'dummy' })
-      expect(spy).toHaveBeenCalledTimes(1)
+      expect(spy).toHaveBeenCalledOnce()
     })
   })