codeceptjs 4.0.2-beta.1 → 4.0.2-beta.11

package/lib/command/run-workers.js

@@ -40,11 +40,22 @@ export default async function (workerCount, selectedRuns, options) {
 
   output.print(`CodeceptJS v${Codecept.version()} ${output.standWithUkraine()}`)
   output.print(`Running tests in ${output.styles.bold(numberOfWorkers)} workers...`)
-  output.print()
   store.hasWorkers = true
 
   const workers = new Workers(numberOfWorkers, config)
   workers.overrideConfig(overrideConfigs)
+  
+  // Show test distribution after workers are initialized
+  await workers.bootstrapAll()
+  
+  const workerObjects = workers.getWorkers()
+  output.print()
+  output.print('Test distribution:')
+  workerObjects.forEach((worker, index) => {
+    const testCount = worker.tests.length
+    output.print(`  Worker ${index + 1}: ${testCount} test${testCount !== 1 ? 's' : ''}`)
+  })
+  output.print()
 
   workers.on(event.test.failed, test => {
     output.test.failed(test)
@@ -68,7 +79,6 @@ export default async function (workerCount, selectedRuns, options) {
     if (options.verbose) {
       await getMachineInfo()
     }
-    await workers.bootstrapAll()
     await workers.run()
   } catch (err) {
     output.error(err)

package/lib/command/workers/runTests.js

@@ -19,6 +19,29 @@ const stderr = ''
 
 const { options, tests, testRoot, workerIndex, poolMode } = workerData
 
+// Global error handlers to catch critical errors but not test failures
+process.on('uncaughtException', (err) => {
+  // Don't exit on test assertion errors - those are handled by mocha
+  if (err.name === 'AssertionError' || err.message?.includes('expected')) {
+    console.error(`[Worker ${workerIndex}] Test assertion error (handled by mocha):`, err.message)
+    return
+  }
+  console.error(`[Worker ${workerIndex}] Uncaught exception:`, err.message)
+  console.error(err.stack)
+  process.exit(1)
+})
+
+process.on('unhandledRejection', (reason, promise) => {
+  // Don't exit on test-related rejections
+  const msg = reason?.message || String(reason)
+  if (msg.includes('expected') || msg.includes('AssertionError')) {
+    console.error(`[Worker ${workerIndex}] Test rejection (handled by mocha):`, msg)
+    return
+  }
+  console.error(`[Worker ${workerIndex}] Unhandled rejection:`, reason)
+  process.exit(1)
+})
+
 // hide worker output
 // In pool mode, only suppress output if debug is NOT enabled
 // In regular mode, hide result output but allow step output in verbose/debug
@@ -26,6 +49,10 @@ if (poolMode && !options.debug) {
   // In pool mode without debug, allow test names and important output but suppress verbose details
   const originalWrite = process.stdout.write
   process.stdout.write = string => {
+    // Always allow Worker logs
+    if (string.includes('[Worker')) {
+      return originalWrite.call(process.stdout, string)
+    }
     // Allow test names (✔ or ✖), Scenario Steps, failures, and important markers
     if (
       string.includes('✔') ||
@@ -45,7 +72,12 @@ if (poolMode && !options.debug) {
     return originalWrite.call(process.stdout, string)
   }
 } else if (!poolMode && !options.debug && !options.verbose) {
+  const originalWrite = process.stdout.write
   process.stdout.write = string => {
+    // Always allow Worker logs
+    if (string.includes('[Worker')) {
+      return originalWrite.call(process.stdout, string)
+    }
     stdout += string
     return true
   }
@@ -82,6 +114,8 @@ let config
 // Load test and run
 initPromise = (async function () {
   try {
+    console.log(`[Worker ${workerIndex}] Starting initialization...`)
+    
     // Import modules dynamically to avoid ES Module loader race conditions in Node 22.x
     const eventModule = await import('../../event.js')
     const containerModule = await import('../../container.js')
@@ -89,6 +123,8 @@ initPromise = (async function () {
     const coreUtilsModule = await import('../../utils.js')
     const CodeceptModule = await import('../../codecept.js')
     
+    console.log(`[Worker ${workerIndex}] Modules imported`)
+    
     event = eventModule.default
     container = containerModule.default
     getConfig = utilsModule.getConfig
@@ -98,16 +134,24 @@ initPromise = (async function () {
     
     const overrideConfigs = tryOrDefault(() => JSON.parse(options.override), {})
     
+    console.log(`[Worker ${workerIndex}] Loading config...`)
+    
     // IMPORTANT: await is required here since getConfig is async
     const baseConfig = await getConfig(options.config || testRoot)
     
+    console.log(`[Worker ${workerIndex}] Config loaded, creating Codecept...`)
+    
     // important deep merge so dynamic things e.g. functions on config are not overridden
     config = deepMerge(baseConfig, overrideConfigs)
     
     // Pass workerIndex as child option for output.process() to display worker prefix
     const optsWithChild = { ...options, child: workerIndex }
     codecept = new Codecept(config, optsWithChild)
+    
+    console.log(`[Worker ${workerIndex}] Initializing Codecept...`)
     await codecept.init(testRoot)
+    
+    console.log(`[Worker ${workerIndex}] Loading tests...`)
     codecept.loadTests()
     mocha = container.mocha()
 
@@ -116,9 +160,14 @@ initPromise = (async function () {
       // We'll reload test files fresh for each test request
     } else {
       // Legacy mode - filter tests upfront
+      console.log(`[Worker ${workerIndex}] Starting test filtering. Assigned ${tests.length} test UIDs`)
       filterTests()
+      const finalCount = mocha.suite.total()
+      console.log(`[Worker ${workerIndex}] After filtering: ${finalCount} tests to run`)
     }
 
+    console.log(`[Worker ${workerIndex}] Initialization complete, starting tests...`)
+
     // run tests
     if (poolMode) {
       await runPoolTests()
@@ -126,10 +175,12 @@ initPromise = (async function () {
       await runTests()
     } else {
       // No tests to run, close the worker
+      console.error(`[Worker ${workerIndex}] ERROR: No tests found after filtering! Assigned ${tests.length} UIDs but none matched.`)
       parentPort?.close()
     }
   } catch (err) {
-    console.error('Error in worker initialization:', err)
+    console.error(`[Worker ${workerIndex}] FATAL ERROR:`, err.message)
+    console.error(err.stack)
     process.exit(1)
   }
 })()
@@ -140,6 +191,7 @@ async function runTests() {
   try {
     await codecept.bootstrap()
   } catch (err) {
+    console.error(`[Worker ${workerIndex}] Bootstrap error:`, err.message)
     throw new Error(`Error while running bootstrap file :${err}`)
   }
   listenToParentThread()
@@ -147,8 +199,15 @@ async function runTests() {
   disablePause()
   try {
     await codecept.run()
+  } catch (err) {
+    console.error(`[Worker ${workerIndex}] Runtime error:`, err.message)
+    throw err
   } finally {
-    await codecept.teardown()
+    try {
+      await codecept.teardown()
+    } catch (err) {
+      console.error(`[Worker ${workerIndex}] Teardown error:`, err.message)
+    }
   }
 }
 
@@ -336,6 +395,26 @@ function filterTests() {
   mocha.files = files
   mocha.loadFiles()
 
+  // Collect all loaded tests for debugging
+  const allLoadedTests = [];
+  mocha.suite.eachTest(test => {
+    if (test) {
+      allLoadedTests.push({ uid: test.uid, title: test.fullTitle() });
+    }
+  });
+  
+  console.log(`[Worker ${workerIndex}] Loaded ${allLoadedTests.length} tests from ${files.length} files`);
+  console.log(`[Worker ${workerIndex}] Expecting ${tests.length} test UIDs`);
+  
+  const loadedUids = new Set(allLoadedTests.map(t => t.uid));
+  const missingTests = tests.filter(uid => !loadedUids.has(uid));
+  
+  if (missingTests.length > 0) {
+    console.error(`[Worker ${workerIndex}] ERROR: ${missingTests.length} assigned tests NOT FOUND in loaded files!`);
+    console.error(`[Worker ${workerIndex}] Missing UIDs:`, missingTests);
+    console.error(`[Worker ${workerIndex}] Available UIDs:`, Array.from(loadedUids).slice(0, 5), '...');
+  }
+
   // Recursively filter tests in all suites (including nested ones)
   const filterSuiteTests = (suite) => {
     suite.tests = suite.tests.filter(test => tests.indexOf(test.uid) >= 0)

package/lib/workers.js

@@ -370,6 +370,9 @@ class Workers extends EventEmitter {
     // If Codecept isn't initialized yet, return empty groups as a safe fallback
     if (!this.codecept) return populateGroups(numberOfWorkers)
     const files = this.codecept.testFiles
+    
+    // Create a fresh mocha instance to avoid state pollution
+    Container.createMocha(this.codecept.config.mocha || {}, this.options)
     const mocha = Container.mocha()
     mocha.files = files
     mocha.loadFiles()
@@ -384,6 +387,10 @@ class Workers extends EventEmitter {
         groupCounter++
       }
     })
+    
+    // Clean up after collecting test UIDs
+    mocha.unloadFiles()
+    
     return groups
   }
 
@@ -452,9 +459,12 @@ class Workers extends EventEmitter {
     const files = this.codecept.testFiles
     const groups = populateGroups(numberOfWorkers)
 
+    // Create a fresh mocha instance to avoid state pollution
+    Container.createMocha(this.codecept.config.mocha || {}, this.options)
     const mocha = Container.mocha()
     mocha.files = files
     mocha.loadFiles()
+    
     mocha.suite.suites.forEach(suite => {
       const i = indexOfSmallestElement(groups)
       suite.tests.forEach(test => {
@@ -463,6 +473,10 @@ class Workers extends EventEmitter {
         }
       })
     })
+    
+    // Clean up after collecting test UIDs
+    mocha.unloadFiles()
+    
     return groups
   }
 
@@ -504,8 +518,24 @@ class Workers extends EventEmitter {
       // Workers are already running, this is just a placeholder step
     })
 
+    // Add overall timeout to prevent infinite hanging
+    const overallTimeout = setTimeout(() => {
+      console.error('[Main] Overall timeout reached (10 minutes). Force terminating remaining workers...')
+      workerThreads.forEach(w => {
+        try {
+          w.terminate()
+        } catch (e) {
+          // ignore
+        }
+      })
+      this._finishRun()
+    }, 600000) // 10 minutes
+
     return new Promise(resolve => {
-      this.on('end', resolve)
+      this.on('end', () => {
+        clearTimeout(overallTimeout)
+        resolve()
+      })
     })
   }
 
@@ -528,8 +558,37 @@ class Workers extends EventEmitter {
     if (this.isPoolMode) {
       this.activeWorkers.set(worker, { available: true, workerIndex: null })
     }
+    
+    // Track last activity time to detect hanging workers
+    let lastActivity = Date.now()
+    let currentTest = null
+    const workerTimeout = 300000 // 5 minutes
+    
+    const timeoutChecker = setInterval(() => {
+      const elapsed = Date.now() - lastActivity
+      if (elapsed > workerTimeout) {
+        console.error(`[Main] Worker appears to be hanging (no activity for ${Math.floor(elapsed/1000)}s). Terminating...`)
+        if (currentTest) {
+          console.error(`[Main] Last test: ${currentTest}`)
+        }
+        clearInterval(timeoutChecker)
+        worker.terminate()
+      }
+    }, 30000) // Check every 30 seconds
 
     worker.on('message', message => {
+      lastActivity = Date.now() // Update activity timestamp
+      
+      // Track current test
+      if (message.event === event.test.started && message.data) {
+        currentTest = message.data.title || message.data.fullTitle
+        console.log(`[Worker ${message.workerIndex}] Started: ${currentTest}`)
+      }
+      
+      if (message.event === event.test.finished && message.data) {
+        console.log(`[Worker ${message.workerIndex}] Finished: ${message.data.title || currentTest}`)
+      }
+      
       output.process(message.workerIndex)
 
       // Handle test requests for pool mode
@@ -646,11 +705,27 @@ class Workers extends EventEmitter {
     })
 
     worker.on('error', err => {
+      console.error(`[Main] Worker error:`, err.message || err)
+      if (currentTest) {
+        console.error(`[Main] Failed during test: ${currentTest}`)
+      }
       this.errors.push(err)
     })
 
-    worker.on('exit', () => {
+    worker.on('exit', (code) => {
+      clearInterval(timeoutChecker)
       this.closedWorkers += 1
+      
+      if (code !== 0) {
+        console.error(`[Main] Worker exited with code ${code}`)
+        if (currentTest) {
+          console.error(`[Main] Last test running: ${currentTest}`)
+        }
+        // Mark as failed
+        process.exitCode = 1
+      }
+      
+      console.log(`[Main] Workers closed: ${this.closedWorkers}/${this.numberOfWorkers}`)
 
       if (this.isPoolMode) {
         // Pool mode: finish when all workers have exited and no more tests
@@ -665,8 +740,9 @@ class Workers extends EventEmitter {
   }
 
   _finishRun() {
+    console.log('[Main] Finishing test run...')
     event.dispatcher.emit(event.workers.after, { tests: this.workers.map(worker => worker.tests) })
-    if (Container.result().hasFailed) {
+    if (Container.result().hasFailed || this.errors.length > 0) {
       process.exitCode = 1
     } else {
       process.exitCode = 0
@@ -706,8 +782,10 @@ class Workers extends EventEmitter {
       this._testStates.clear()
     }
 
+    console.log('[Main] Emitting final results...')
     this.emit(event.all.result, Container.result())
     event.dispatcher.emit(event.workers.result, Container.result())
+    console.log('[Main] Emitting end event...')
     this.emit('end') // internal event
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codeceptjs",
-  "version": "4.0.2-beta.1",
+  "version": "4.0.2-beta.11",
   "type": "module",
   "description": "Supercharged End 2 End Testing Framework for NodeJS",
   "keywords": [