fix: Add .js extensions to module imports in requireModules() and fix async init in workers

- Fixed requireModules() to add .js extension for local files (resolves require parameter tests)
- Fixed Workers constructor to properly initialize codecept as async Promise
- Fixed runTests.js to properly await codecept.init() in worker threads
- Fixed codecept.timeout.obj.conf.cjs to use CommonJS syntax instead of ESM

Progress:
- All 6 require parameter tests now passing ✅
- Reduced runner test failures from 48 to 44
- Workers still have initialization issues (hanging) - needs further investigation

Author: kobenguyent

lib/codecept.js

@@ -59,6 +59,12 @@ class Codecept {
         const isLocalFile = existsSync(modulePath) || existsSync(`${modulePath}.js`)
         if (isLocalFile) {
           modulePath = resolve(modulePath)
+          // For ESM, ensure .js extension for local files
+          if (!modulePath.endsWith('.js') && !modulePath.endsWith('.mjs') && !modulePath.endsWith('.cjs')) {
+            if (existsSync(`${modulePath}.js`)) {
+              modulePath = `${modulePath}.js`
+            }
+          }
         }
         // Use dynamic import for ESM
         await import(modulePath)

lib/command/workers/runTests.js

@@ -72,26 +72,35 @@ const overrideConfigs = tryOrDefault(() => JSON.parse(options.override), {})
 // important deep merge so dynamic things e.g. functions on config are not overridden
 const config = deepMerge(getConfig(options.config || testRoot), overrideConfigs)
 
-// Load test and run
-const codecept = new Codecept(config, options)
-codecept.init(testRoot)
-codecept.loadTests()
-const mocha = container.mocha()
-
-if (poolMode) {
-  // In pool mode, don't filter tests upfront - wait for assignments
-  // We'll reload test files fresh for each test request
-} else {
-  // Legacy mode - filter tests upfront
-  filterTests()
-}
+// Declare codecept and mocha at module level so they can be accessed by functions
+let codecept
+let mocha
 
-// run tests
+// Load test and run
 ;(async function () {
-  if (poolMode) {
-    await runPoolTests()
-  } else if (mocha.suite.total()) {
-    await runTests()
+  try {
+    codecept = new Codecept(config, options)
+    await codecept.init(testRoot)
+    codecept.loadTests()
+    mocha = container.mocha()
+
+    if (poolMode) {
+      // In pool mode, don't filter tests upfront - wait for assignments
+      // We'll reload test files fresh for each test request
+    } else {
+      // Legacy mode - filter tests upfront
+      filterTests()
+    }
+
+    // run tests
+    if (poolMode) {
+      await runPoolTests()
+    } else if (mocha.suite.total()) {
+      await runTests()
+    }
+  } catch (err) {
+    console.error('Error in worker initialization:', err)
+    process.exit(1)
   }
 })()
 

lib/workers.js

@@ -236,7 +236,8 @@ class Workers extends EventEmitter {
   constructor(numberOfWorkers, config = { by: 'test' }) {
     super()
     this.setMaxListeners(50)
-    this.codecept = initializeCodecept(config.testConfig, config.options)
+    this.codeceptPromise = initializeCodecept(config.testConfig, config.options)
+    this.codecept = null
     this.options = config.options || {}
     this.errors = []
     this.numberOfWorkers = 0

test/data/sandbox/configs/timeouts/codecept.timeout.obj.conf.cjs

@@ -1,4 +1,4 @@
-export const config = {
+module.exports = {
   tests: './*_test.js',
   output: './output',
   helpers: {