codeceptjs 4.0.1-beta.32 → 4.0.1-beta.34

package/lib/helper/Playwright.js

@@ -912,7 +912,7 @@ class Playwright extends Helper {
   }
 
   async _finishTest() {
-    if ((restartsSession() || restartsContext() || restartsBrowser()) && this.isRunning) {
+    if (this.isRunning) {
       try {
         await Promise.race([this._stopBrowser(), new Promise((_, reject) => setTimeout(() => reject(new Error('Test finish timeout')), 10000))])
       } catch (e) {

package/lib/listener/helpers.js

@@ -73,30 +73,18 @@ export default function () {
   })
 
   event.dispatcher.on(event.all.result, () => {
-    // Skip _finishTest for all helpers if any browser helper restarts to avoid double cleanup
-    const hasBrowserRestart = Object.values(helpers).some(helper => 
-      (helper.config && (helper.config.restart === 'browser' || helper.config.restart === 'context' || helper.config.restart === true)) || 
-      (helper.options && (helper.options.restart === 'browser' || helper.options.restart === 'context' || helper.options.restart === true))
-    )
-
     Object.keys(helpers).forEach(key => {
       const helper = helpers[key]
-      if (helper._finishTest && !hasBrowserRestart) {
+      if (helper._finishTest) {
         recorder.add(`hook ${key}._finishTest()`, () => helper._finishTest(), true, false)
       }
     })
   })
 
   event.dispatcher.on(event.all.after, () => {
-    // Skip _cleanup for all helpers if any browser helper restarts to avoid double cleanup
-    const hasBrowserRestart = Object.values(helpers).some(helper => 
-      (helper.config && (helper.config.restart === 'browser' || helper.config.restart === 'context' || helper.config.restart === true)) || 
-      (helper.options && (helper.options.restart === 'browser' || helper.options.restart === 'context' || helper.options.restart === true))
-    )
-
     Object.keys(helpers).forEach(key => {
       const helper = helpers[key]
-      if (helper._cleanup && !hasBrowserRestart) {
+      if (helper._cleanup) {
         recorder.add(`hook ${key}._cleanup()`, () => helper._cleanup(), true, false)
       }
     })

package/lib/workers.js

@@ -530,9 +530,6 @@ class Workers extends EventEmitter {
     }
 
     worker.on('message', message => {
-      if (message.workerIndex !== undefined) {
-        console.log(`[DEBUG] Received message from worker with workerIndex: ${message.workerIndex}`)
-      }
       output.process(message.workerIndex)
 
       // Handle test requests for pool mode
@@ -607,7 +604,7 @@ class Workers extends EventEmitter {
               if (!this._testStates) this._testStates = new Map()
 
               if (!this._testStates.has(uid)) {
-                this._testStates.set(uid, { states: [], lastData: data })
+                this._testStates.set(uid, { states: [], lastData: data, workerIndex: message.workerIndex })
               }
 
               const testState = this._testStates.get(uid)
@@ -677,7 +674,10 @@ class Workers extends EventEmitter {
 
     // Emit states for all tracked tests before emitting results
     if (this._testStates) {
-      for (const [uid, { states, lastData }] of this._testStates) {
+      for (const [uid, { states, lastData, workerIndex }] of this._testStates) {
+        // Set correct worker index for output
+        output.process(workerIndex)
+
         // For tests with retries configured, emit all failures + final success
         // For tests without retries, emit only final state
         const lastState = states[states.length - 1]

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codeceptjs",
-  "version": "4.0.1-beta.32",
+  "version": "4.0.1-beta.34",
   "type": "module",
   "description": "Supercharged End 2 End Testing Framework for NodeJS",
   "keywords": [
@@ -137,7 +137,7 @@
     "@apollo/server": "^5",
     "@codeceptjs/expect-helper": "^1.0.2",
     "@codeceptjs/mock-request": "0.3.1",
-    "@eslint/eslintrc": "3.3.1",
+    "@eslint/eslintrc": "3.3.3",
     "@eslint/js": "9.39.2",
     "@faker-js/faker": "9.8.0",
     "@inquirer/testing": "^3.0.3",