@vitest/runner 2.1.0 → 2.1.2

package/dist/index.js

@@ -1,4 +1,4 @@
-import { getSafeTimers, isObject, createDefer, isNegativeNaN, format, objDisplay, objectAttr, toArray, shuffle } from '@vitest/utils';
+import { getSafeTimers, isObject, createDefer, isNegativeNaN, format, objDisplay, objectAttr, toArray, shuffle, assertTypes } from '@vitest/utils';
 import { processError } from '@vitest/utils/error';
 export { processError } from '@vitest/utils/error';
 import { l as createChainable, a as createFileTask, c as calculateSuiteHash, s as someTasksAreOnly, i as interpretTaskModes, m as limitConcurrency, p as partitionSuiteChildren, h as hasTests, j as hasFailed } from './chunk-tasks.js';
@@ -224,7 +224,11 @@ function resolveDeps(fixtures, depSet = /* @__PURE__ */ new Set(), pendingFixtur
   return pendingFixtures;
 }
 function getUsedProps(fn) {
-  const match = fn.toString().match(/[^(]*\(([^)]*)/);
+  let fnString = fn.toString();
+  if (/__async\(this, (?:null|arguments|\[[_0-9, ]*\]), function\*/.test(fnString)) {
+    fnString = fnString.split("__async(this,")[1];
+  }
+  const match = fnString.match(/[^(]*\(([^)]*)/);
   if (!match) {
     return [];
   }
@@ -1017,6 +1021,24 @@ async function runTest(test, runner) {
       } catch (e) {
         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);
+        }
+      }
+      delete test.onFailed;
+      delete test.onFinished;
       if (test.result.state === "pass") {
         break;
       }
@@ -1027,22 +1049,6 @@ async function runTest(test, runner) {
       updateTask(test, runner);
     }
   }
-  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.fails) {
     if (test.result.state === "pass") {
       const error = processError(new Error("Expect test to fail"));
@@ -1102,13 +1108,18 @@ async function runSuite(suite, runner) {
     suite.result.state = "todo";
   } else {
     try {
-      beforeAllCleanups = await callSuiteHook(
-        suite,
-        suite,
-        "beforeAll",
-        runner,
-        [suite]
-      );
+      try {
+        beforeAllCleanups = await callSuiteHook(
+          suite,
+          suite,
+          "beforeAll",
+          runner,
+          [suite]
+        );
+      } catch (e) {
+        markTasksAsSkipped(suite, runner);
+        throw e;
+      }
       if (runner.runSuite) {
         await runner.runSuite(suite);
       } else {
@@ -1211,24 +1222,28 @@ function getDefaultHookTimeout() {
   return getRunner().config.hookTimeout;
 }
 function beforeAll(fn, timeout) {
+  assertTypes(fn, '"beforeAll" callback', ["function"]);
   return getCurrentSuite().on(
     "beforeAll",
     withTimeout(fn, timeout ?? getDefaultHookTimeout(), true)
   );
 }
 function afterAll(fn, timeout) {
+  assertTypes(fn, '"afterAll" callback', ["function"]);
   return getCurrentSuite().on(
     "afterAll",
     withTimeout(fn, timeout ?? getDefaultHookTimeout(), true)
   );
 }
 function beforeEach(fn, timeout) {
+  assertTypes(fn, '"beforeEach" callback', ["function"]);
   return getCurrentSuite().on(
     "beforeEach",
     withTimeout(withFixtures(fn), timeout ?? getDefaultHookTimeout(), true)
   );
 }
 function afterEach(fn, timeout) {
+  assertTypes(fn, '"afterEach" callback', ["function"]);
   return getCurrentSuite().on(
     "afterEach",
     withTimeout(withFixtures(fn), timeout ?? getDefaultHookTimeout(), true)
@@ -1254,6 +1269,7 @@ const onTestFinished = createTestHook(
 );
 function createTestHook(name, handler) {
   return (fn, timeout) => {
+    assertTypes(fn, `"${name}" callback`, ["function"]);
     const current = getCurrentTest();
     if (!current) {
       throw new Error(`Hook ${name}() can only be called inside a test`);

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@vitest/runner",
   "type": "module",
-  "version": "2.1.0",
+  "version": "2.1.2",
   "description": "Vitest test runner",
   "license": "MIT",
   "funding": "https://opencollective.com/vitest",
@@ -39,7 +39,7 @@
   ],
   "dependencies": {
     "pathe": "^1.1.2",
-    "@vitest/utils": "2.1.0"
+    "@vitest/utils": "2.1.2"
   },
   "scripts": {
     "build": "rimraf dist && rollup -c",