Fix issue with console spy inside buildHooks.test.tsx

- `.mockReset()` should not be called in spies since it calls `.mockClear()` and returns the implementation to its **initial** form. In this case it was silencing some of the `act` related warnings emitted by `@testing-library/react` which needed to be resolved since they were calling issues. So `.mockReset()` calls on spies need to be changed to `.mockRestore()` calls since `.mockRestore()` restores the implementation to its **original** form.

Author: aryaemami59

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

@@ -1,3 +1,4 @@
+import { noop } from '@internal/listenerMiddleware/utils'
 import type { SubscriptionOptions } from '@internal/query/core/apiState'
 import type { SubscriptionSelectors } from '@internal/query/core/buildMiddleware/types'
 import { server } from '@internal/query/tests/mocks/server'
@@ -32,7 +33,6 @@ import {
 import userEvent from '@testing-library/user-event'
 import { HttpResponse, http } from 'msw'
 import { useEffect, useState } from 'react'
-import type { MockInstance } from 'vitest'
 
 // Just setup a temporary in-memory counter for tests that `getIncrementedAmount`.
 // This can be used to test how many renders happen due to data changes or
@@ -959,14 +959,16 @@ describe('hooks tests', () => {
     })
 
     describe('Hook middleware requirements', () => {
-      let mock: MockInstance
+      const consoleErrorSpy = vi
+        .spyOn(console, 'error')
+        .mockImplementation(noop)
 
-      beforeEach(() => {
-        mock = vi.spyOn(console, 'error').mockImplementation(() => {})
+      afterEach(() => {
+        consoleErrorSpy.mockClear()
       })
 
-      afterEach(() => {
-        mock.mockReset()
+      afterAll(() => {
+        consoleErrorSpy.mockRestore()
       })
 
       test('Throws error if middleware is not added to the store', async () => {