Fix no-empty related problems

Author: aryaemami59

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

@@ -3,17 +3,17 @@ import {
   configureStore,
   createAsyncThunk,
   createReducer,
-  unwrapResult,
   miniSerializeError,
+  unwrapResult,
 } from '@reduxjs/toolkit'
 import { vi } from 'vitest'
 
+import { delay } from '@internal/utils'
 import {
   createConsole,
   getLog,
   mockConsole,
 } from 'console-testing-library/pure'
-import { delay } from '@internal/utils'
 
 declare global {
   interface Window {
@@ -131,7 +131,9 @@ describe('createAsyncThunk', () => {
 
     try {
       await thunkFunction(dispatch, () => {}, undefined)
-    } catch (e) {}
+    } catch (e) {
+      /* empty */
+    }
 
     expect(dispatch).toHaveBeenNthCalledWith(
       1,
@@ -167,7 +169,9 @@ describe('createAsyncThunk', () => {
 
     try {
       await thunkFunction(dispatch, () => {}, undefined)
-    } catch (e) {}
+    } catch (e) {
+      /* empty */
+    }
 
     expect(dispatch).toHaveBeenNthCalledWith(
       1,
@@ -206,7 +210,9 @@ describe('createAsyncThunk', () => {
 
     try {
       await thunkFunction(dispatch, () => {}, undefined)
-    } catch (e) {}
+    } catch (e) {
+      /* empty */
+    }
 
     expect(dispatch).toHaveBeenNthCalledWith(
       1,
@@ -251,7 +257,9 @@ describe('createAsyncThunk', () => {
 
     try {
       await thunkFunction(dispatch, () => {}, undefined)
-    } catch (e) {}
+    } catch (e) {
+      /* empty */
+    }
 
     expect(dispatch).toHaveBeenNthCalledWith(
       1,
@@ -296,7 +304,9 @@ describe('createAsyncThunk', () => {
 
     try {
       await thunkFunction(dispatch, () => {}, undefined)
-    } catch (e) {}
+    } catch (e) {
+      /* empty */
+    }
 
     expect(dispatch).toHaveBeenNthCalledWith(
       1,
@@ -350,7 +360,9 @@ describe('createAsyncThunk', () => {
 
     try {
       await thunkFunction(dispatch, () => {}, undefined)
-    } catch (e) {}
+    } catch (e) {
+      /* empty */
+    }
 
     expect(dispatch).toHaveBeenNthCalledWith(
       1,
@@ -640,7 +652,9 @@ describe('conditional skipping of asyncThunks', () => {
       const thunkPromise = asyncThunk(arg)(dispatch, getState, extra)
       thunkPromise.abort()
       await thunkPromise
-    } catch (err) {}
+    } catch (err) {
+      /* empty */
+    }
     expect(dispatch).toHaveBeenCalledTimes(0)
   })
 
@@ -969,6 +983,7 @@ describe('meta', () => {
     })
 
     if (ret.meta.requestStatus === 'rejected' && ret.meta.rejectedWithValue) {
+      /* empty */
     } else {
       // could be caused by a `throw`, `abort()` or `condition` - no `rejectedMeta` in that case
       // @ts-expect-error

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

@@ -1,8 +1,8 @@
 import type {
-  Store,
-  MiddlewareAPI,
   ImmutableStateInvariantMiddlewareOptions,
   Middleware,
+  MiddlewareAPI,
+  Store,
 } from '@reduxjs/toolkit'
 import {
   createImmutableStateInvariantMiddleware,
@@ -11,9 +11,9 @@ import {
 
 import { trackForMutations } from '@internal/immutableStateInvariantMiddleware'
 import {
-  mockConsole,
   createConsole,
   getLog,
+  mockConsole,
 } from 'console-testing-library/pure'
 
 type MWNext = Parameters<ReturnType<Middleware>>[0]
@@ -161,7 +161,9 @@ describe('createImmutableStateInvariantMiddleware', () => {
   it('Should not print a warning if "next" takes too long', () => {
     const next: MWNext = (action) => {
       const started = Date.now()
-      while (Date.now() - started < 8) {}
+      while (Date.now() - started < 8) {
+        /* empty */
+      }
       return action
     }
 

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

@@ -110,12 +110,12 @@ describe('serializableStateInvariantMiddleware', () => {
     store.dispatch(dispatchedAction)
 
     expect(getLog().log).toMatchInlineSnapshot(`
-      "A non-serializable value was detected in an action, in the path: \`payload\`. Value: Symbol(SOME_CONSTANT) 
+      "A non-serializable value was detected in an action, in the path: \`payload\`. Value: Symbol(SOME_CONSTANT)
       Take a look at the logic that dispatched this action:  Object {
         "payload": Symbol(SOME_CONSTANT),
         "type": "an-action",
-      } 
-      (See https://redux.js.org/faq/actions#why-should-type-be-a-string-or-at-least-serializable-why-should-my-action-types-be-constants) 
+      }
+      (See https://redux.js.org/faq/actions#why-should-type-be-a-string-or-at-least-serializable-why-should-my-action-types-be-constants)
       (To allow non-serializable values see: https://redux-toolkit.js.org/usage/usage-guide#working-with-non-serializable-data)"
     `)
   })
@@ -154,7 +154,7 @@ describe('serializableStateInvariantMiddleware', () => {
     store.dispatch({ type: ACTION_TYPE })
 
     expect(getLog().log).toMatchInlineSnapshot(`
-      "A non-serializable value was detected in the state, in the path: \`testSlice.a\`. Value: Map {} 
+      "A non-serializable value was detected in the state, in the path: \`testSlice.a\`. Value: Map {}
       Take a look at the reducer(s) handling this action type: TEST_ACTION.
       (See https://redux.js.org/faq/organizing-state#can-i-put-functions-promises-or-other-non-serializable-items-in-my-store-state)"
     `)
@@ -215,7 +215,7 @@ describe('serializableStateInvariantMiddleware', () => {
 
       // since default options are used, the `entries` function in `serializableObject` will cause the error
       expect(getLog().log).toMatchInlineSnapshot(`
-        "A non-serializable value was detected in the state, in the path: \`testSlice.a.entries\`. Value: [Function entries] 
+        "A non-serializable value was detected in the state, in the path: \`testSlice.a.entries\`. Value: [Function entries]
         Take a look at the reducer(s) handling this action type: TEST_ACTION.
         (See https://redux.js.org/faq/organizing-state#can-i-put-functions-promises-or-other-non-serializable-items-in-my-store-state)"
       `)
@@ -262,7 +262,7 @@ describe('serializableStateInvariantMiddleware', () => {
 
       // error reported is from a nested class instance, rather than the `entries` function `serializableObject`
       expect(getLog().log).toMatchInlineSnapshot(`
-        "A non-serializable value was detected in the state, in the path: \`testSlice.a.third.bad-map-instance\`. Value: Map {} 
+        "A non-serializable value was detected in the state, in the path: \`testSlice.a.third.bad-map-instance\`. Value: Map {}
         Take a look at the reducer(s) handling this action type: TEST_ACTION.
         (See https://redux.js.org/faq/organizing-state#can-i-put-functions-promises-or-other-non-serializable-items-in-my-store-state)"
       `)
@@ -367,14 +367,14 @@ describe('serializableStateInvariantMiddleware', () => {
       }).dispatch({ type: 'test', meta: { arg: nonSerializableValue } })
 
       expect(getLog().log).toMatchInlineSnapshot(`
-        "A non-serializable value was detected in an action, in the path: \`meta.arg\`. Value: Map {} 
+        "A non-serializable value was detected in an action, in the path: \`meta.arg\`. Value: Map {}
         Take a look at the logic that dispatched this action:  Object {
           "meta": Object {
             "arg": Map {},
           },
           "type": "test",
-        } 
-        (See https://redux.js.org/faq/actions#why-should-type-be-a-string-or-at-least-serializable-why-should-my-action-types-be-constants) 
+        }
+        (See https://redux.js.org/faq/actions#why-should-type-be-a-string-or-at-least-serializable-why-should-my-action-types-be-constants)
         (To allow non-serializable values see: https://redux-toolkit.js.org/usage/usage-guide#working-with-non-serializable-data)"
       `)
     })
@@ -499,7 +499,7 @@ describe('serializableStateInvariantMiddleware', () => {
 
     // testSlice.b.d was not covered in ignoredPaths, so will still log the error
     expect(getLog().log).toMatchInlineSnapshot(`
-      "A non-serializable value was detected in the state, in the path: \`testSlice.b.d\`. Value: Map {} 
+      "A non-serializable value was detected in the state, in the path: \`testSlice.b.d\`. Value: Map {}
       Take a look at the reducer(s) handling this action type: TEST_ACTION.
       (See https://redux.js.org/faq/organizing-state#can-i-put-functions-promises-or-other-non-serializable-items-in-my-store-state)"
     `)
@@ -587,7 +587,9 @@ describe('serializableStateInvariantMiddleware', () => {
   it('Should not print a warning if "reducer" takes too long', () => {
     const reducer: Reducer = (state = 42, action) => {
       const started = Date.now()
-      while (Date.now() - started < 8) {}
+      while (Date.now() - started < 8) {
+        /* empty */
+      }
       return state
     }