codeceptjs 4.0.2-beta.3 → 4.0.2-beta.5

package/lib/command/workers/runTests.js

@@ -19,6 +19,18 @@ const stderr = ''
 
 const { options, tests, testRoot, workerIndex, poolMode } = workerData
 
+// Global error handlers to prevent worker from hanging
+process.on('uncaughtException', (err) => {
+  console.error(`[Worker ${workerIndex}] Uncaught exception:`, err.message)
+  console.error(err.stack)
+  process.exit(1)
+})
+
+process.on('unhandledRejection', (reason, promise) => {
+  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
@@ -116,7 +128,10 @@ 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`)
     }
 
     // run tests
@@ -126,6 +141,7 @@ 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) {
@@ -140,6 +156,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 +164,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,24 +360,24 @@ function filterTests() {
   mocha.files = files
   mocha.loadFiles()
 
-  // Debug logging to help diagnose test filtering issues
-  if (options.debug || options.verbose) {
-    const allLoadedTests = [];
-    mocha.suite.eachTest(test => {
-      if (test) {
-        allLoadedTests.push({ uid: test.uid, title: test.fullTitle() });
-      }
-    });
-    
-    console.log(`[Worker ${workerIndex}] Loaded ${allLoadedTests.length} tests, expecting ${tests.length} tests`);
-    
-    const loadedUids = new Set(allLoadedTests.map(t => t.uid));
-    const missingTests = tests.filter(uid => !loadedUids.has(uid));
-    
-    if (missingTests.length > 0) {
-      console.log(`[Worker ${workerIndex}] WARNING: ${missingTests.length} assigned tests not found in loaded files`);
-      console.log(`[Worker ${workerIndex}] Missing UIDs:`, missingTests);
+  // 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)
@@ -367,13 +391,6 @@ function filterTests() {
   for (const suite of mocha.suite.suites) {
     filterSuiteTests(suite)
   }
-  
-  // Verify final test count
-  if (options.debug || options.verbose) {
-    let finalCount = 0;
-    mocha.suite.eachTest(() => finalCount++);
-    console.log(`[Worker ${workerIndex}] After filtering: ${finalCount} tests will run`);
-  }
 }
 
 function initializeListeners() {

package/lib/workers.js

@@ -542,8 +542,22 @@ 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()
+    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...`)
+        clearInterval(timeoutChecker)
+        worker.terminate()
+      }
+    }, 30000) // Check every 30 seconds
 
     worker.on('message', message => {
+      lastActivity = Date.now() // Update activity timestamp
       output.process(message.workerIndex)
 
       // Handle test requests for pool mode
@@ -660,11 +674,17 @@ class Workers extends EventEmitter {
     })
 
     worker.on('error', err => {
+      console.error(`[Main] Worker error:`, err.message || err)
       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 (this.isPoolMode) {
         // Pool mode: finish when all workers have exited and no more tests

package/package.json

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