fix(query-core): unpause restored mutations (TanStack#9015)

* fix(query-core): unpause restored mutations

ensure that restored mutations have their isPaused state set to false when continued.

* Update packages/query-core/src/__tests__/mutations.test.tsx

---------

Co-authored-by: Dominik Dorfmeister <[email protected]>

Author: RaphaelGimenez

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

@@ -252,7 +252,7 @@ describe('mutations', () => {
     const onSettled = vi.fn()
 
     queryClient.setMutationDefaults(key, {
-      mutationFn: (text: string) => Promise.resolve(text),
+      mutationFn: (text: string) => sleep(10).then(() => text),
       onMutate,
       onSuccess,
       onSettled,
@@ -290,7 +290,23 @@ describe('mutations', () => {
       submittedAt: 1,
     })
 
-    await queryClient.resumePausedMutations()
+    void queryClient.resumePausedMutations()
+    await vi.advanceTimersByTimeAsync(0)
+
+    // check that the mutation is correctly resumed
+    expect(mutation.state).toEqual({
+      context: 'todo',
+      data: undefined,
+      error: null,
+      failureCount: 1,
+      failureReason: 'err',
+      isPaused: false,
+      status: 'pending',
+      variables: 'todo',
+      submittedAt: 1,
+    })
+
+    await vi.advanceTimersByTimeAsync(20)
 
     expect(mutation.state).toEqual({
       context: 'todo',

packages/query-core/src/mutation.ts

@@ -162,6 +162,10 @@ export class Mutation<
   }
 
   async execute(variables: TVariables): Promise<TData> {
+    const onContinue = () => {
+      this.#dispatch({ type: 'continue' })
+    }
+
     this.#retryer = createRetryer({
       fn: () => {
         if (!this.options.mutationFn) {
@@ -175,9 +179,7 @@ export class Mutation<
       onPause: () => {
         this.#dispatch({ type: 'pause' })
       },
-      onContinue: () => {
-        this.#dispatch({ type: 'continue' })
-      },
+      onContinue,
       retry: this.options.retry ?? 0,
       retryDelay: this.options.retryDelay,
       networkMode: this.options.networkMode,
@@ -188,7 +190,10 @@ export class Mutation<
     const isPaused = !this.#retryer.canStart()
 
     try {
-      if (!restored) {
+      if (restored) {
+        // Dispatch continue action to unpause restored mutation
+        onContinue()
+      } else {
         this.#dispatch({ type: 'pending', variables, isPaused })
         // Notify cache callback
         await this.#mutationCache.config.onMutate?.(