codeceptjs 3.3.0-beta.4 → 3.3.1

package/lib/interfaces/bdd.js

@@ -1,4 +1,5 @@
 const { CucumberExpression, ParameterTypeRegistry } = require('cucumber-expressions');
+const Config = require('../config');
 
 let steps = {};
 
@@ -9,7 +10,11 @@ const STACK_POSITION = 2;
  * @param {*} fn
  */
 const addStep = (step, fn) => {
+  const avoidDuplicateSteps = Config.get('gherkin', {}).avoidDuplicateSteps || false;
   const stack = (new Error()).stack;
+  if (avoidDuplicateSteps && steps[step]) {
+    throw new Error(`Step '${step}' is already defined`);
+  }
   steps[step] = fn;
   fn.line = stack && stack.split('\n')[STACK_POSITION];
   if (fn.line) {

package/lib/listener/steps.js

@@ -69,6 +69,7 @@ module.exports = function () {
   event.dispatcher.on(event.step.started, (step) => {
     if (store.debugMode) return;
     step.startedAt = +new Date();
+    step.test = currentTest;
     if (currentHook && Array.isArray(currentHook.steps)) {
       return currentHook.steps.push(step);
     }

package/lib/plugin/fakerTransform.js

@@ -1,4 +1,4 @@
-const faker = require('faker');
+const faker = require('@faker-js/faker');
 const transform = require('../transform');
 
 /**

package/lib/plugin/stepByStepReport.js

@@ -142,18 +142,20 @@ module.exports = function (config) {
     if (step.metaStep && step.metaStep.name === 'BeforeSuite') return;
 
     const fileName = `${pad.substring(0, pad.length - stepNum.toString().length) + stepNum.toString()}.png`;
+    if (step.status === 'failed') {
+      scenarioFailed = true;
+    }
+    stepNum++;
+    slides[fileName] = step;
     try {
-      if (step.status === 'failed') {
-        scenarioFailed = true;
-      }
-      stepNum++;
-      slides[fileName] = step;
       await helper.saveScreenshot(path.relative(reportDir, path.join(dir, fileName)), config.fullPageScreenshots);
     } catch (err) {
       output.plugin(`Can't save step screenshot: ${err}`);
       error = err;
+      return;
+    } finally {
+      savedStep = step;
     }
-    savedStep = step;
 
     if (!currentTest.artifacts.screenshots) currentTest.artifacts.screenshots = [];
     // added attachments to test

package/lib/workers.js

@@ -358,6 +358,18 @@ class Workers extends EventEmitter {
         case event.test.after:
           this.emit(event.test.after, repackTest(message.data));
           break;
+        case event.step.finished:
+          this.emit(event.step.finished, message.data);
+          break;
+        case event.step.started:
+          this.emit(event.step.started, message.data);
+          break;
+        case event.step.passed:
+          this.emit(event.step.passed, message.data);
+          break;
+        case event.step.failed:
+          this.emit(event.step.failed, message.data);
+          break;
       }
     });
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codeceptjs",
-  "version": "3.3.0-beta.4",
+  "version": "3.3.1",
   "description": "Supercharged End 2 End Testing Framework for NodeJS",
   "keywords": [
     "acceptance",
@@ -59,6 +59,7 @@
     "allure-js-commons": "^1.3.2",
     "arrify": "^2.0.1",
     "axios": "^0.21.4",
+    "chai": "^4.3.6",
     "chai-deep-match": "^1.2.1",
     "chalk": "^4.1.2",
     "commander": "^2.20.3",
@@ -91,6 +92,7 @@
   "devDependencies": {
     "@codeceptjs/detox-helper": "^1.0.2",
     "@codeceptjs/mock-request": "^0.3.1",
+    "@faker-js/faker": "^5.5.3",
     "@pollyjs/adapter-puppeteer": "^5.1.0",
     "@pollyjs/core": "^5.1.0",
     "@types/inquirer": "^0.0.35",
@@ -111,7 +113,6 @@
     "eslint-plugin-mocha": "^6.3.0",
     "expect": "^26.6.2",
     "express": "^4.17.2",
-    "faker": "5.5.3",
     "form-data": "^3.0.1",
     "graphql": "^14.6.0",
     "husky": "^4.3.8",
@@ -144,7 +145,7 @@
     "npm": ">=5.6.0"
   },
   "peerDependencies": {
-    "faker": "^5.5.1"
+    "@faker-js/faker": "^5.5.3"
   },
   "es6": true,
   "husky": {