chore: add typescript-eslint/require-await rule (#9116)

Co-authored-by: MINSEONG KIM <65233075+minseong0324@users.noreply.github.com>

Author: manudeli

eslint.config.js

@@ -39,6 +39,7 @@ export default [
       ],
       '@typescript-eslint/no-empty-function': 'off',
       '@typescript-eslint/no-unsafe-function-type': 'off',
+      '@typescript-eslint/require-await': 'error',
       'no-case-declarations': 'off',
     },
   },

packages/angular-query-experimental/src/__tests__/inject-infinite-query.test-d.ts

@@ -23,7 +23,7 @@ describe('injectInfiniteQuery', () => {
     vi.useRealTimers()
   })
 
-  test('should narrow type after isSuccess', async () => {
+  test('should narrow type after isSuccess', () => {
     const query = TestBed.runInInjectionContext(() => {
       return injectInfiniteQuery(() => ({
         queryKey: ['infiniteQuery'],

packages/query-broadcast-client-experimental/src/__tests__/index.test.ts

@@ -12,15 +12,15 @@ describe('broadcastQueryClient', () => {
     queryCache = queryClient.getQueryCache()
   })
 
-  it('should subscribe to the query cache', async () => {
+  it('should subscribe to the query cache', () => {
     broadcastQueryClient({
       queryClient,
       broadcastChannel: 'test_channel',
     })
     expect(queryCache.hasListeners()).toBe(true)
   })
 
-  it('should not have any listeners after cleanup', async () => {
+  it('should not have any listeners after cleanup', () => {
     const unsubscribe = broadcastQueryClient({
       queryClient,
       broadcastChannel: 'test_channel',

packages/query-codemods/eslint.config.js

@@ -1,6 +1,5 @@
 // @ts-check
 
-import vitest from '@vitest/eslint-plugin'
 import rootConfig from './root.eslint.config.js'
 
 export default [
@@ -9,6 +8,7 @@ export default [
     rules: {
       'cspell/spellchecker': 'off',
       '@typescript-eslint/no-unnecessary-condition': 'off',
+      '@typescript-eslint/require-await': 'off',
       'import/no-duplicates': 'off',
       'import/no-unresolved': 'off',
       'import/order': 'off',

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

@@ -31,7 +31,7 @@ describe('queryObserver', () => {
     vi.useRealTimers()
   })
 
-  test('should trigger a fetch when subscribed', async () => {
+  test('should trigger a fetch when subscribed', () => {
     const key = queryKey()
     const queryFn = vi
       .fn<(...args: Array<unknown>) => string>()

packages/react-query/src/__tests__/useQuery.test-d.tsx

@@ -152,7 +152,7 @@ describe('useQuery', () => {
   )
   expectTypeOf(testFuncStyle.data).toEqualTypeOf<boolean | undefined>()
 
-  it('should return the correct states for a successful query', async () => {
+  it('should return the correct states for a successful query', () => {
     const state = useQuery<string, Error>({
       queryKey: key,
       queryFn: () => Promise.resolve('test'),

packages/react-query/src/__tests__/useQuery.test.tsx

@@ -608,7 +608,7 @@ describe('useQuery', () => {
     expect(states[1]).toMatchObject({ data: 'test' })
   })
 
-  it('should not fetch when refetchOnMount is false and data has been fetched already', async () => {
+  it('should not fetch when refetchOnMount is false and data has been fetched already', () => {
     const key = queryKey()
     const states: Array<UseQueryResult<string>> = []
 
@@ -1106,7 +1106,7 @@ describe('useQuery', () => {
     })
   })
 
-  it('should not refetch disabled query when invalidated with invalidateQueries', async () => {
+  it('should not refetch disabled query when invalidated with invalidateQueries', () => {
     const key = queryKey()
     const states: Array<UseQueryResult<number>> = []
     let count = 0