codeceptjs 3.7.0-beta.7 → 3.7.0-beta.8

package/lib/workers.js

@@ -12,7 +12,8 @@ const { replaceValueDeep, deepClone } = require('./utils')
 const mainConfig = require('./config')
 const output = require('./output')
 const event = require('./event')
-const { repackTestForWorkersTransport: repackTest } = require('./mocha/test')
+const { deserializeTest } = require('./mocha/test')
+const { deserializeSuite } = require('./mocha/suite')
 const recorder = require('./recorder')
 const runHook = require('./hooks')
 const WorkerStorage = require('./workerStorage')
@@ -230,17 +231,10 @@ class Workers extends EventEmitter {
     super()
     this.setMaxListeners(50)
     this.codecept = initializeCodecept(config.testConfig, config.options)
-    this.failuresLog = []
     this.errors = []
     this.numberOfWorkers = 0
     this.closedWorkers = 0
     this.workers = []
-    this.stats = {
-      passes: 0,
-      failures: 0,
-      tests: 0,
-      pending: 0,
-    }
     this.testGroups = []
 
     createOutputDir(config.testConfig)
@@ -353,8 +347,6 @@ class Workers extends EventEmitter {
   }
 
   run() {
-    this.stats.start = new Date()
-    this.stats.failedHooks = 0
     recorder.startUnlessRunning()
     event.dispatcher.emit(event.workers.before)
     process.env.RUNS_WITH_WORKERS = 'true'
@@ -380,7 +372,7 @@ class Workers extends EventEmitter {
    * @returns {Boolean}
    */
   isFailed() {
-    return (this.stats.failures || this.errors.length) > 0
+    return (Container.result().failures.length || this.errors.length) > 0
   }
 
   _listenWorkerEvents(worker) {
@@ -393,33 +385,37 @@ class Workers extends EventEmitter {
       }
 
       switch (message.event) {
-        case event.all.failures:
-          this.failuresLog = this.failuresLog.concat(message.data.failuresLog)
-          this._appendStats(message.data.stats)
+        case event.all.result:
+          // we ensure consistency of result by adding tests in the very end
+          Container.result().addFailures(message.data.failures)
+          Container.result().addStats(message.data.stats)
+          message.data.tests.forEach(test => {
+            Container.result().addTest(deserializeTest(test))
+          })
           break
         case event.suite.before:
-          this.emit(event.suite.before, repackTest(message.data))
+          this.emit(event.suite.before, deserializeSuite(message.data))
           break
         case event.test.before:
-          this.emit(event.test.before, repackTest(message.data))
+          this.emit(event.test.before, deserializeTest(message.data))
           break
         case event.test.started:
-          this.emit(event.test.started, repackTest(message.data))
+          this.emit(event.test.started, deserializeTest(message.data))
           break
         case event.test.failed:
-          this.emit(event.test.failed, repackTest(message.data))
+          this.emit(event.test.failed, deserializeTest(message.data))
           break
         case event.test.passed:
-          this.emit(event.test.passed, repackTest(message.data))
+          this.emit(event.test.passed, deserializeTest(message.data))
           break
         case event.test.skipped:
-          this.emit(event.test.skipped, repackTest(message.data))
+          this.emit(event.test.skipped, deserializeTest(message.data))
           break
         case event.test.finished:
-          this.emit(event.test.finished, repackTest(message.data))
+          this.emit(event.test.finished, deserializeTest(message.data))
           break
         case event.test.after:
-          this.emit(event.test.after, repackTest(message.data))
+          this.emit(event.test.after, deserializeTest(message.data))
           break
         case event.step.finished:
           this.emit(event.step.finished, message.data)
@@ -431,7 +427,7 @@ class Workers extends EventEmitter {
           this.emit(event.step.passed, message.data)
           break
         case event.step.failed:
-          this.emit(event.step.failed, message.data)
+          this.emit(event.step.failed, message.data, message.data.error)
           break
       }
     })
@@ -450,33 +446,26 @@ class Workers extends EventEmitter {
 
   _finishRun() {
     event.dispatcher.emit(event.workers.after, { tests: this.workers.map(worker => worker.tests) })
-    if (this.isFailed()) {
+    if (Container.result().hasFailed) {
       process.exitCode = 1
     } else {
       process.exitCode = 0
     }
-    // removed this.finishedTests because in all /lib only first argument (!this.isFailed()) is used)
-    this.emit(event.all.result, !this.isFailed())
-    this.emit('end') // internal event
-  }
 
-  _appendStats(newStats) {
-    this.stats.passes += newStats.passes
-    this.stats.failures += newStats.failures
-    this.stats.tests += newStats.tests
-    this.stats.pending += newStats.pending
-    this.stats.failedHooks += newStats.failedHooks
+    this.emit(event.all.result, Container.result())
+    event.dispatcher.emit(event.workers.result, Container.result())
+    this.emit('end') // internal event
   }
 
   printResults() {
-    this.stats.end = new Date()
-    this.stats.duration = this.stats.end - this.stats.start
+    const result = Container.result()
+    result.finish()
 
     // Reset process for logs in main thread
     output.process(null)
     output.print()
 
-    this.failuresLog = this.failuresLog
+    this.failuresLog = result.failures
       .filter(log => log.length && typeof log[1] === 'number')
       // mocha/lib/reporters/base.js
       .map(([format, num, title, message, stack], i) => [format, i + 1, title, message, stack])
@@ -487,7 +476,8 @@ class Workers extends EventEmitter {
       this.failuresLog.forEach(log => output.print(...log))
     }
 
-    output.result(this.stats.passes, this.stats.failures, this.stats.pending, ms(this.stats.duration), this.stats.failedHooks)
+    output.result(result.stats.passes, result.stats.failures, result.stats.pending, ms(result.duration), result.stats.failedHooks)
+
     process.env.RUNS_WITH_WORKERS = 'false'
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codeceptjs",
-  "version": "3.7.0-beta.7",
+  "version": "3.7.0-beta.8",
   "description": "Supercharged End 2 End Testing Framework for NodeJS",
   "keywords": [
     "acceptance",
@@ -94,7 +94,7 @@
     "figures": "3.2.0",
     "fn-args": "4.0.0",
     "fs-extra": "11.2.0",
-    "glob": ">=9.0.0",
+    "glob": "^11.0.1",
     "fuse.js": "^7.0.0",
     "html-minifier-terser": "7.2.0",
     "inquirer": "6.5.2",