perf(runner): skipped suite handler should not be called

Akryum · Sep 25, 2022 · e213603 · e213603
1 parent 7782736
commit e213603
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/packages/peeky-runner/src/runtime/collect-tests.ts b/packages/peeky-runner/src/runtime/collect-tests.ts
@@ -17,6 +17,8 @@ import type {
 } from '../types'
 import { toMainThread } from './message.js'
 
+const SKIP_SUITE_HANDLER = ['skip', 'todo'] as const
+
 export function setupTestCollector (ctx: Context): {
   exposed: {
     describe: DescribeFn
@@ -86,7 +88,7 @@ export function setupTestCollector (ctx: Context): {
       parentSuite.children.push(['suite', suite])
     }
     currentSuites.push(suite)
-    if (handler && flag !== 'todo') {
+    if (handler && !SKIP_SUITE_HANDLER.includes(flag)) {
       handler()
     }
     currentSuites.pop()