fix: process hangs when _beforeSuite helper hook throws an error (#5515)

Add recorder.catch() for suite.before and suite.after events in helpers
listener. Unlike all other hook events (test.before, test.after, etc.),
these suite-level events had no catch handler, so rejected promises from
helper hooks like Playwright._beforeSuite() were unhandled — mocha's
done callback was never called, causing the process to hang indefinitely.

Also disable store.dryRun during check command's helper setup/teardown
verification so it actually launches the browser and can detect issues
like missing Playwright browser installations.

Co-authored-by: DavertMik <davert@testomat.io>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: 3 people

lib/command/check.js

@@ -135,6 +135,7 @@ export default async function (options) {
   printCheck('plugins', checks['plugins'], Object.keys(Container.plugins()).join(', '))
 
   if (Object.keys(helpers).length) {
+    store.dryRun = false
     const suite = Container.mocha().suite
     const test = createTest('test', () => {})
     checks.setup = true
@@ -166,6 +167,7 @@ export default async function (options) {
     }
 
     printCheck('Helpers After', checks['teardown'], standardActingHelpers.some(h => Object.keys(helpers).includes(h)) ? 'Closing browser' : '')
+    store.dryRun = true
   }
 
   try {

lib/listener/helpers.js

@@ -30,11 +30,13 @@ export default function () {
   event.dispatcher.on(event.suite.before, suite => {
     // if (suite.parent) return; // only for root suite
     runAsyncHelpersHook('_beforeSuite', suite, true)
+    recorder.catch()
   })
 
   event.dispatcher.on(event.suite.after, suite => {
     // if (suite.parent) return; // only for root suite
     runAsyncHelpersHook('_afterSuite', suite, true)
+    recorder.catch()
   })
 
   event.dispatcher.on(event.test.started, test => {