Skip to content

Commit 72b3ac6

Browse files
authored
Merge pull request #3957 from reduxjs/retry-error-type
2 parents c892abd + c2cfd23 commit 72b3ac6

File tree

2 files changed

+6
-3
lines changed

2 files changed

+6
-3
lines changed

packages/toolkit/src/query/retry.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ import type {
22
BaseQueryApi,
33
BaseQueryArg,
44
BaseQueryEnhancer,
5+
BaseQueryError,
56
BaseQueryExtraOptions,
67
BaseQueryFn,
78
} from './baseQueryTypes'
@@ -31,7 +32,7 @@ async function defaultBackoff(attempt: number = 0, maxRetries: number = 5) {
3132
}
3233

3334
type RetryConditionFunction = (
34-
error: FetchBaseQueryError,
35+
error: BaseQueryError<BaseQueryFn>,
3536
args: BaseQueryArg<BaseQueryFn>,
3637
extraArgs: {
3738
attempt: number

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

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
1-
import type { BaseQueryFn } from '@reduxjs/toolkit/query'
1+
import { vi } from 'vitest'
2+
import type { BaseQueryFn, FetchBaseQueryError } from '@reduxjs/toolkit/query'
23
import { createApi, retry } from '@reduxjs/toolkit/query'
34
import { setupApiStore } from '../../tests/utils/helpers'
45

@@ -422,7 +423,8 @@ describe('configuration', () => {
422423
m1: build.mutation({
423424
query: () => ({ method: 'PUT' }),
424425
extraOptions: {
425-
retryCondition: (e) => e.data === 'hello retryCondition',
426+
retryCondition: (e) =>
427+
(e as FetchBaseQueryError).data === 'hello retryCondition',
426428
},
427429
}),
428430
}),

0 commit comments

Comments
 (0)