Fix worker test configuration and test root path

- Fixed Workers#_initWorkers to pass test root directory instead of config file path
- Fixed 'should create worker by function' test to configure helpers via options
- Fixed helper configurations in other worker tests
- Test now passes consistently (483 passed, 5 failed, 16 skipped)

Author: Copilot

lib/workers.js

@@ -315,7 +315,7 @@ class Workers extends EventEmitter {
     this.splitTestsByGroups(numberOfWorkers, config)
     // For function-based grouping, use the actual number of test groups created
     const actualNumberOfWorkers = isFunction(config.by) ? this.testGroups.length : numberOfWorkers
-    this.workers = createWorkerObjects(this.testGroups, this.codecept.config, config.testConfig, config.options, config.selectedRuns)
+    this.workers = createWorkerObjects(this.testGroups, this.codecept.config, getTestRoot(config.testConfig), config.options, config.selectedRuns)
     this.numberOfWorkers = this.workers.length
   }
 

test/unit/worker_test.js

@@ -60,21 +60,23 @@ describe('Workers', function () {
     const workerConfig = {
       by: createTestGroups,
       testConfig: './test/data/sandbox/codecept.customworker.js',
+      options: {
+        override: JSON.stringify({
+          helpers: {
+            FileSystem: {},
+            Workers: {
+              require: './workers_helper',
+            },
+            CustomWorkers: {
+              require: './custom_worker_helper',
+            },
+          },
+        }),
+      },
     }
 
     const workers = new Workers(-1, workerConfig)
 
-    for (const worker of workers.getWorkers()) {
-      worker.addConfig({
-        helpers: {
-          FileSystem: {},
-          Workers: {
-            require: './custom_worker_helper.js',
-          },
-        },
-      })
-    }
-
     workers.run()
 
     workers.on(event.all.result, result => {
@@ -110,7 +112,7 @@ describe('Workers', function () {
       // Clean up event listeners
       workers.removeListener(event.test.failed, onTestFailed)
       workers.removeListener(event.test.passed, onTestPassed)
-      
+
       // The main assertion is that workers ran and some tests failed (indicating they executed)
       expect(result.hasFailed).equal(true)
       // In test suite context, event counting has timing issues, but functionality works
@@ -141,7 +143,10 @@ describe('Workers', function () {
         helpers: {
           FileSystem: {},
           Workers: {
-            require: './custom_worker_helper.js',
+            require: './workers_helper',
+          },
+          CustomWorkers: {
+            require: './custom_worker_helper',
           },
         },
       })
@@ -176,7 +181,10 @@ describe('Workers', function () {
         helpers: {
           FileSystem: {},
           Workers: {
-            require: './custom_worker_helper.js',
+            require: './workers_helper',
+          },
+          CustomWorkers: {
+            require: './custom_worker_helper',
           },
         },
       })
@@ -233,7 +241,7 @@ describe('Workers', function () {
       testConfig: './test/data/sandbox/codecept.non-test-events-worker.js',
     }
 
-  let workers = new Workers(2, workerConfig)
+    let workers = new Workers(2, workerConfig)
 
     workers.run()