fix(runner): run onTestFinished and onTestFailed during retry a…

…nd `repeats`
vitest-dev · Oct 2, 2024 · c3292cf · c3292cf
1 parent 9edfa19
commit c3292cf
Showing 1 changed file with 20 additions and 20 deletions.
diff --git a/packages/runner/src/run.ts b/packages/runner/src/run.ts
@@ -271,6 +271,26 @@ export async function runTest(test: Test | Custom, runner: VitestRunner): Promis
         failTask(test.result, e, runner.config.diffOptions)
       }
 
+      try {
+        await callTaskHooks(test, test.onFinished || [], 'stack')
+      }
+      catch (e) {
+        failTask(test.result, e, runner.config.diffOptions)
+      }
+
+      if (test.result.state === 'fail') {
+        try {
+          await callTaskHooks(
+            test,
+            test.onFailed || [],
+            runner.config.sequence.hooks,
+          )
+        }
+        catch (e) {
+          failTask(test.result, e, runner.config.diffOptions)
+        }
+      }
+
       if (test.result.state === 'pass') {
         break
       }
@@ -286,26 +306,6 @@ export async function runTest(test: Test | Custom, runner: VitestRunner): Promis
     }
   }
 
-  try {
-    await callTaskHooks(test, test.onFinished || [], 'stack')
-  }
-  catch (e) {
-    failTask(test.result, e, runner.config.diffOptions)
-  }
-
-  if (test.result.state === 'fail') {
-    try {
-      await callTaskHooks(
-        test,
-        test.onFailed || [],
-        runner.config.sequence.hooks,
-      )
-    }
-    catch (e) {
-      failTask(test.result, e, runner.config.diffOptions)
-    }
-  }
-
   // if test is marked to be failed, flip the result
   if (test.fails) {
     if (test.result.state === 'pass') {