changed polling test name to represent actual behaviour

reduxjs · Jan 24, 2024 · b9d37c3 · b9d37c3
1 parent 65b448f
commit b9d37c3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/packages/toolkit/src/query/core/buildMiddleware/polling.ts b/packages/toolkit/src/query/core/buildMiddleware/polling.ts
@@ -135,9 +135,7 @@ export const buildPollingHandler: InternalHandlerBuilder = ({
           subscribers[key].pollingInterval!,
           lowestPollingInterval
         )
-        // if (!skipPollOnFocusLost) {
-          skipPollOnFocusLost = subscribers[key].skipPollOnFocusLost
-        // }
+        skipPollOnFocusLost = subscribers[key].skipPollOnFocusLost
       }
     }
 

diff --git a/packages/toolkit/src/query/tests/polling.test.tsx b/packages/toolkit/src/query/tests/polling.test.tsx
@@ -4,7 +4,6 @@ import { setupApiStore } from './helpers'
 import type { SubscriptionSelectors } from '../core/buildMiddleware/types'
 import { createListenerMiddleware } from '@reduxjs/toolkit'
 
-
 const mockBaseQuery = vi
   .fn()
   .mockImplementation((args: any) => ({ data: args }))
@@ -167,7 +166,7 @@ describe('polling tests', () => {
     storeListenerRef.store.dispatch(api.util.resetApiState())
   })
 
-  it('respects skipPollOnFocusLost if any subscription is true', async () => {
+  it('respects skipPollOnFocusLost of the most recent mounted subscription', async () => {
     const listenerMiddleware = createListenerMiddleware()
     const storeListenerRef = setupApiStore(api, undefined, {
       middleware: {