chore: add @typescript-eslint/require-await for notice error and remove unnecessary async

packages/query-core/eslint.config.js

@@ -2,4 +2,12 @@
 
 import rootConfig from './root.eslint.config.js'
 
-export default [...rootConfig]
+export default [
+  ...rootConfig,
+  {
+    files: ['**/__tests__/**'],
+    rules: {
+      '@typescript-eslint/require-await': 'error',
+    },
+  },
+]

packages/query-core/src/__tests__/hydration.test.tsx

@@ -136,7 +136,7 @@ describe('dehydration and rehydration', () => {
   test('should not dehydrate queries if dehydrateQueries is set to false', async () => {
     const queryCache = new QueryCache()
     const queryClient = createQueryClient({ queryCache })
-    vi.waitFor(() =>
+    await vi.waitFor(() =>
       queryClient.prefetchQuery({
         queryKey: ['string'],
         queryFn: () => fetchData('string'),

packages/query-devtools/eslint.config.js

@@ -2,4 +2,10 @@
 
 import rootConfig from './root.eslint.config.js'
 
-export default [...rootConfig]
+export default [
+  ...rootConfig,
+  {
+    files: ['**/__tests__/**'],
+    rules: { '@typescript-eslint/require-await': 'error' },
+  },
+]

packages/query-devtools/src/__tests__/utils.test.ts

@@ -4,7 +4,7 @@ import { deleteNestedDataByPath, updateNestedDataByPath } from '../utils'
 describe('Utils tests', () => {
   describe('updatedNestedDataByPath', () => {
     describe('array', () => {
-      it('should update data correctly', async () => {
+      it('should update data correctly', () => {
         const oldData = ['one', 'two', 'three']
 
         const newData = updateNestedDataByPath(oldData, ['1'], 'new')
@@ -29,7 +29,7 @@ describe('Utils tests', () => {
     })
 
     describe('object', () => {
-      it('should update data correctly', async () => {
+      it('should update data correctly', () => {
         const oldData = { title: 'Hello world', id: 1, createdAt: '2021-01-01' }
 
         const newData = updateNestedDataByPath(
@@ -58,7 +58,7 @@ describe('Utils tests', () => {
     })
 
     describe('set', () => {
-      it('should update data correctly', async () => {
+      it('should update data correctly', () => {
         const oldData = new Set([123, 321, 'hello', 'world'])
 
         const newData = updateNestedDataByPath(oldData, ['2'], 'hi')
@@ -85,7 +85,7 @@ describe('Utils tests', () => {
     })
 
     describe('map', () => {
-      it('should update data correctly', async () => {
+      it('should update data correctly', () => {
         const oldData = new Map([
           ['en', 'hello'],
           ['fr', 'bonjour'],
@@ -113,7 +113,7 @@ describe('Utils tests', () => {
     })
 
     describe('nested data', () => {
-      it('should update data correctly', async () => {
+      it('should update data correctly', () => {
         /* eslint-disable cspell/spellchecker */
         const oldData = new Map([
           [
@@ -377,7 +377,7 @@ describe('Utils tests', () => {
   })
 
   describe('deleteNestedDataByPath', () => {
-    it('should delete item from array correctly', async () => {
+    it('should delete item from array correctly', () => {
       const oldData = ['one', 'two', 'three']
 
       const newData = deleteNestedDataByPath(oldData, ['1'])
@@ -399,7 +399,7 @@ describe('Utils tests', () => {
       `)
     })
 
-    it('should delete item from object correctly', async () => {
+    it('should delete item from object correctly', () => {
       const oldData = { title: 'Hello world', id: 1, createdAt: '2021-01-01' }
 
       const newData = deleteNestedDataByPath(oldData, ['createdAt'])
@@ -472,7 +472,7 @@ describe('Utils tests', () => {
     })
 
     describe('nested data', () => {
-      it('should delete nested items correctly', async () => {
+      it('should delete nested items correctly', () => {
         /* eslint-disable cspell/spellchecker */
         const oldData = new Map([
           [

packages/query-persist-client-core/eslint.config.js

@@ -2,4 +2,10 @@
 
 import rootConfig from './root.eslint.config.js'
 
-export default [...rootConfig]
+export default [
+  ...rootConfig,
+  {
+    files: ['**/__tests__/**'],
+    rules: { '@typescript-eslint/require-await': 'error' },
+  },
+]

packages/query-persist-client-core/src/__tests__/utils.ts

@@ -17,7 +17,7 @@ export function createMockPersister(): Persister {
   let storedState: PersistedClient | undefined
 
   return {
-    async persistClient(persistClient: PersistedClient) {
+    persistClient(persistClient: PersistedClient) {
       storedState = persistClient
     },
     async restoreClient() {

packages/query-sync-storage-persister/eslint.config.js

@@ -2,4 +2,10 @@
 
 import rootConfig from './root.eslint.config.js'
 
-export default [...rootConfig]
+export default [
+  ...rootConfig,
+  {
+    files: ['**/__tests__/**'],
+    rules: { '@typescript-eslint/require-await': 'error' },
+  },
+]

packages/react-query-devtools/eslint.config.js

@@ -19,4 +19,10 @@ export default [
       'react-hooks/rules-of-hooks': 'error',
     },
   },
+  {
+    files: ['**/__tests__/**'],
+    rules: {
+      '@typescript-eslint/require-await': 'error',
+    },
+  },
 ]

packages/react-query-persist-client/eslint.config.js

@@ -19,4 +19,10 @@ export default [
       'react-hooks/rules-of-hooks': 'error',
     },
   },
+  {
+    files: ['**/__tests__/**'],
+    rules: {
+      '@typescript-eslint/require-await': 'error',
+    },
+  },
 ]

packages/react-query-persist-client/src/__tests__/PersistQueryClientProvider.test.tsx

@@ -19,7 +19,7 @@ const createMockPersister = (): Persister => {
   let storedState: PersistedClient | undefined
 
   return {
-    async persistClient(persistClient: PersistedClient) {
+    persistClient(persistClient: PersistedClient) {
       storedState = persistClient
     },
     async restoreClient() {
@@ -706,7 +706,7 @@ describe('PersistQueryClientProvider', () => {
       let storedState: PersistedClient | undefined
 
       return {
-        async persistClient(persistClient) {
+        persistClient(persistClient) {
           storedState = persistClient
         },
         async restoreClient() {