Retry now checks whether potential retry counts are undefined, rather than boolean, in order to avoid filtering out 0's

Author: OliverRadini

packages/toolkit/src/query/retry.ts

@@ -83,7 +83,7 @@ const retryWithBackoff: BaseQueryEnhancer<
     5,
     ((defaultOptions as any) || EMPTY_OPTIONS).maxRetries,
     ((extraOptions as any) || EMPTY_OPTIONS).maxRetries,
-  ].filter(Boolean)
+  ].filter(x => x !== undefined)
   const [maxRetries] = possibleMaxRetries.slice(-1)
 
   const defaultRetryCondition: RetryConditionFunction = (_, __, { attempt }) =>

packages/toolkit/src/query/tests/retry.test.ts

@@ -439,6 +439,33 @@ describe('configuration', () => {
     expect(baseBaseQuery).toHaveBeenCalledTimes(4)
   })
 
+  test('Specifying maxRetries as 0 in RetryOptions prevents retries', async () => {
+    const baseBaseQuery = jest.fn<
+      ReturnType<BaseQueryFn>,
+      Parameters<BaseQueryFn>
+    >()
+    baseBaseQuery.mockResolvedValue({ error: 'rejected' })
+
+    const baseQuery = retry(baseBaseQuery, { maxRetries: 0 })
+    const api = createApi({
+      baseQuery,
+      endpoints: (build) => ({
+        q1: build.query({
+          query: () => {},
+        }),
+      }),
+    })
+
+    const storeRef = setupApiStore(api, undefined, {
+      withoutTestLifecycles: true,
+    })
+
+    storeRef.store.dispatch(api.endpoints.q1.initiate({}))
+    await loopTimers(2)
+
+    expect(baseBaseQuery).toHaveBeenCalledTimes(1)
+  });
+
   test.skip('RetryOptions only accepts one of maxRetries or retryCondition', () => {
     // @ts-expect-error Should complain if both exist at once
     const ro: RetryOptions = {