codeceptjs 3.3.0-beta.5 → 3.3.2

package/docs/webapi/wait.mustache

@@ -4,4 +4,5 @@ Pauses execution for a number of seconds.
 I.wait(2); // wait 2 secs
 ```
 
-@param {number} sec number of second to wait.
+@param {number} sec number of second to wait.
+[!] returns a _promise_ which is synchronized internally by recorder

package/docs/webapi/waitForClickable.mustache

@@ -7,4 +7,5 @@ I.waitForClickable('.btn.continue', 5); // wait for 5 secs
 ```
 
 @param {CodeceptJS.LocatorOrString} locator element located by CSS|XPath|strict locator.
-@param {number} [sec] (optional, `1` by default) time in seconds to wait
+@param {number} [sec] (optional, `1` by default) time in seconds to wait
+[!] returns a _promise_ which is synchronized internally by recorder

package/docs/webapi/waitForDetached.mustache

@@ -6,4 +6,5 @@ I.waitForDetached('#popup');
 ```
 
 @param {CodeceptJS.LocatorOrString} locator element located by CSS|XPath|strict locator.
-@param {number} [sec=1] (optional, `1` by default) time in seconds to wait
+@param {number} [sec=1] (optional, `1` by default) time in seconds to wait
+[!] returns a _promise_ which is synchronized internally by recorder

package/docs/webapi/waitForElement.mustache

@@ -7,4 +7,5 @@ I.waitForElement('.btn.continue', 5); // wait for 5 secs
 ```
 
 @param {CodeceptJS.LocatorOrString} locator element located by CSS|XPath|strict locator.
-@param {number} [sec] (optional, `1` by default) time in seconds to wait
+@param {number} [sec] (optional, `1` by default) time in seconds to wait
+[!] returns a _promise_ which is synchronized internally by recorder

package/docs/webapi/waitForEnabled.mustache

@@ -2,4 +2,5 @@ Waits for element to become enabled (by default waits for 1sec).
 Element can be located by CSS or XPath.
 
 @param {CodeceptJS.LocatorOrString} locator element located by CSS|XPath|strict locator.
-@param {number} [sec=1] (optional) time in seconds to wait, 1 by default.
+@param {number} [sec=1] (optional) time in seconds to wait, 1 by default.
+[!] returns a _promise_ which is synchronized internally by recorder

package/docs/webapi/waitForFunction.mustache

@@ -14,3 +14,4 @@ I.waitForFunction((count) => window.requests == count, [3], 5) // pass args and
 @param {string|function} fn to be executed in browser context.
 @param {any[]|number} [argsOrSec] (optional, `1` by default) arguments for function or seconds.
 @param {number} [sec] (optional, `1` by default) time in seconds to wait
+[!] returns a _promise_ which is synchronized internally by recorder

package/docs/webapi/waitForInvisible.mustache

@@ -6,4 +6,5 @@ I.waitForInvisible('#popup');
 ```
 
 @param {CodeceptJS.LocatorOrString} locator element located by CSS|XPath|strict locator.
-@param {number} [sec=1] (optional, `1` by default) time in seconds to wait
+@param {number} [sec=1] (optional, `1` by default) time in seconds to wait
+[!] returns a _promise_ which is synchronized internally by recorder

package/docs/webapi/waitForText.mustache

@@ -9,4 +9,5 @@ I.waitForText('Thank you, form has been submitted', 5, '#modal');
 
 @param {string }text to wait for.
 @param {number} [sec=1] (optional, `1` by default) time in seconds to wait
-@param {CodeceptJS.LocatorOrString} [context] (optional) element located by CSS|XPath|strict locator.
+@param {CodeceptJS.LocatorOrString} [context] (optional) element located by CSS|XPath|strict locator.
+[!] returns a _promise_ which is synchronized internally by recorder

package/docs/webapi/waitForValue.mustache

@@ -7,3 +7,4 @@ I.waitForValue('//input', "GoodValue");
 @param {LocatorOrString} field input field.
 @param {string }value expected value.
 @param {number} [sec=1] (optional, `1` by default) time in seconds to wait
+[!] returns a _promise_ which is synchronized internally by recorder

package/docs/webapi/waitForVisible.mustache

@@ -7,3 +7,4 @@ I.waitForVisible('#popup');
 
 @param {CodeceptJS.LocatorOrString} locator element located by CSS|XPath|strict locator.
 @param {number} [sec=1] (optional, `1` by default) time in seconds to wait
+[!] returns a _promise_ which is synchronized internally by recorder

package/docs/webapi/waitInUrl.mustache

@@ -5,4 +5,5 @@ I.waitInUrl('/info', 2);
 ```
 
 @param {string} urlPart value to check.
-@param {number} [sec=1] (optional, `1` by default) time in seconds to wait
+@param {number} [sec=1] (optional, `1` by default) time in seconds to wait
+[!] returns a _promise_ which is synchronized internally by recorder

package/docs/webapi/waitNumberOfVisibleElements.mustache

@@ -6,4 +6,5 @@ I.waitNumberOfVisibleElements('a', 3);
 
 @param {CodeceptJS.LocatorOrString} locator element located by CSS|XPath|strict locator.
 @param {number} num number of elements.
-@param {number} [sec=1] (optional, `1` by default) time in seconds to wait
+@param {number} [sec=1] (optional, `1` by default) time in seconds to wait
+[!] returns a _promise_ which is synchronized internally by recorder

package/docs/webapi/waitToHide.mustache

@@ -6,4 +6,5 @@ I.waitToHide('#popup');
 ```
 
 @param {CodeceptJS.LocatorOrString} locator element located by CSS|XPath|strict locator.
-@param {number} [sec=1] (optional, `1` by default) time in seconds to wait
+@param {number} [sec=1] (optional, `1` by default) time in seconds to wait
+[!] returns a _promise_ which is synchronized internally by recorder

package/docs/webapi/waitUrlEquals.mustache

@@ -6,4 +6,5 @@ I.waitUrlEquals('http://127.0.0.1:8000/info');
 ```
 
 @param {string} urlPart value to check.
-@param {number} [sec=1] (optional, `1` by default) time in seconds to wait
+@param {number} [sec=1] (optional, `1` by default) time in seconds to wait
+[!] returns a _promise_ which is synchronized internally by recorder

package/lib/command/definitions.js

@@ -146,10 +146,19 @@ module.exports = function (genPath, options) {
     }
   }
 
+  let autoLogin;
+  if (config.plugins.autoLogin) {
+    autoLogin = config.plugins.autoLogin.inject;
+  }
+
   const supportObject = new Map();
   supportObject.set('I', 'I');
   supportObject.set('current', 'any');
 
+  if (autoLogin) {
+    supportObject.set(autoLogin, 'any');
+  }
+
   if (customHelpers.length > 0) {
     hasCustomHelper = true;
   }

package/lib/command/run.js

@@ -23,8 +23,8 @@ module.exports = async function (test, options) {
   try {
     codecept.init(testRoot);
     await codecept.bootstrap();
-    codecept.loadTests();
-    await codecept.run(test);
+    codecept.loadTests(test);
+    await codecept.run();
   } catch (err) {
     printError(err);
     process.exitCode = 1;

package/lib/command/workers/runTests.js

@@ -115,6 +115,8 @@ function initializeListeners() {
     }
 
     return {
+      opts: test.opts || {},
+      tags: test.tags || [],
       id: test.id,
       workerIndex,
       retries: test._retries,
@@ -126,6 +128,38 @@ function initializeListeners() {
     };
   }
 
+  function simplifyStep(step, err = null) {
+    step = { ...step };
+
+    if (step.startTime && !step.duration) {
+      const end = new Date();
+      step.duration = end - step.startTime;
+    }
+
+    if (step.err) {
+      err = simplifyError(step.err);
+      step.status = 'failed';
+    } else if (err) {
+      err = simplifyError(err);
+      step.status = 'failed';
+    }
+
+    const parent = {};
+    if (step.metaStep) {
+      parent.title = step.metaStep.actor;
+    }
+    return {
+      opts: step.opts || {},
+      workerIndex,
+      title: step.name,
+      status: step.status,
+      duration: step.duration || 0,
+      err,
+      parent,
+      test: simplifyTest(step.test),
+    };
+  }
+
   collectStats();
   // suite
   event.dispatcher.on(event.suite.before, suite => sendToParentThread({ event: event.suite.before, workerIndex, data: simplifyTest(suite) }));
@@ -144,6 +178,12 @@ function initializeListeners() {
   event.dispatcher.on(event.test.started, test => sendToParentThread({ event: event.test.started, workerIndex, data: simplifyTest(test) }));
   event.dispatcher.on(event.test.skipped, test => sendToParentThread({ event: event.test.skipped, workerIndex, data: simplifyTest(test) }));
 
+  // steps
+  event.dispatcher.on(event.step.finished, step => sendToParentThread({ event: event.step.finished, workerIndex, data: simplifyStep(step) }));
+  event.dispatcher.on(event.step.started, step => sendToParentThread({ event: event.step.started, workerIndex, data: simplifyStep(step) }));
+  event.dispatcher.on(event.step.passed, step => sendToParentThread({ event: event.step.passed, workerIndex, data: simplifyStep(step) }));
+  event.dispatcher.on(event.step.failed, step => sendToParentThread({ event: event.step.failed, workerIndex, data: simplifyStep(step) }));
+
   event.dispatcher.on(event.hook.failed, (test, err) => sendToParentThread({ event: event.hook.failed, workerIndex, data: simplifyTest(test, err) }));
   event.dispatcher.on(event.hook.passed, (test, err) => sendToParentThread({ event: event.hook.passed, workerIndex, data: simplifyTest(test, err) }));
   event.dispatcher.on(event.all.failures, (data) => sendToParentThread({ event: event.all.failures, workerIndex, data }));

package/lib/helper/ApiDataFactory.js

@@ -36,7 +36,7 @@ const REST = require('./REST');
  * Install [Rosie](https://github.com/rosiejs/rosie) and [Faker](https://www.npmjs.com/package/faker) libraries.
  *
  * ```sh
- * npm i rosie faker --save-dev
+ * npm i rosie @faker-js/faker --save-dev
  * ```
  *
  * Create a factory file for a resource.
@@ -46,8 +46,8 @@ const REST = require('./REST');
  * ```js
  * // tests/factories/posts.js
  *
- * var Factory = require('rosie').Factory;
- * var faker = require('faker');
+ * const Factory = require('rosie').Factory;
+ * const faker = require('@faker-js/faker');
  *
  * module.exports = new Factory()
  *    // no need to set id, it will be set by REST API
@@ -264,6 +264,7 @@ class ApiDataFactory extends Helper {
    * @param {*} factory factory to use
    * @param {*} params predefined parameters
    * @param {*} options options for programmatically generate the attributes
+   * @returns {Promise<*>}
    */
   have(factory, params, options) {
     const item = this._createItem(factory, params, options);

package/lib/helper/Appium.js

@@ -124,6 +124,7 @@ const webRoot = 'body';
 class Appium extends Webdriver {
   /**
    * Appium Special Methods for Mobile only
+   * @augments WebDriver
    */
 
   constructor(config) {
@@ -331,7 +332,6 @@ class Appium extends Webdriver {
    *
    * @param {*} caps
    * @param {*} fn
-   * @return {Promise<any>}
    */
   async runOnIOS(caps, fn) {
     if (this.platform !== 'ios') return;
@@ -374,7 +374,6 @@ class Appium extends Webdriver {
    *
    * @param {*} caps
    * @param {*} fn
-   * @return {Promise<any>}
    */
   async runOnAndroid(caps, fn) {
     if (this.platform !== 'android') return;
@@ -395,7 +394,6 @@ class Appium extends Webdriver {
    * ```
    *
    * @param {*} fn
-   * @return {Promise<any>}
    */
   /* eslint-disable */
   async runInWeb(fn) {
@@ -599,7 +597,6 @@ class Appium extends Webdriver {
    * ```
    *
    * @param {'LANDSCAPE'|'PORTRAIT'} orientation LANDSCAPE or PORTRAIT
-   * @return {Promise<any>}
    *
    * Appium: support Android and iOS
    */
@@ -728,7 +725,6 @@ class Appium extends Webdriver {
    * Switch to the specified context.
    *
    * @param {*} context the context to switch to
-
    */
   async _switchToContext(context) {
     return this.browser.switchContext(context);
@@ -774,7 +770,7 @@ class Appium extends Webdriver {
    * // or set context explicitly
    * I.switchToNative('SOME_OTHER_CONTEXT');
    * ```
-   * @param {*} context
+   * @param {*} [context]
    * @return {Promise<void>}
    */
   async switchToNative(context = null) {
@@ -834,7 +830,6 @@ class Appium extends Webdriver {
    * ```
    *
    * @param {object} settings object
-   * @return {Promise<any>}
    *
    * Appium: support Android and iOS
    */

package/lib/helper/GraphQL.js

@@ -112,15 +112,15 @@ class GraphQL extends Helper {
    *
    * @param {object} operation
    * @param {object} headers
+   * @return {object} graphQLRequest
    */
   _prepareGraphQLRequest(operation, headers) {
-    const request = {
+    return {
       baseURL: this.options.endpoint,
       method: 'POST',
       data: operation,
       headers,
     };
-    return request;
   }
 
   /**
@@ -142,6 +142,7 @@ class GraphQL extends Helper {
    * @param {object} variables that may go along with the query
    * @param {object} options are additional query options
    * @param {object} headers
+   * @return Promise<any>
    */
   async sendQuery(query, variables, options = {}, headers = {}) {
     if (typeof query !== 'string') {
@@ -181,6 +182,7 @@ class GraphQL extends Helper {
    * @param {object} variables that may go along with the mutation
    * @param {object} options are additional query options
    * @param {object} headers
+   * @return Promise<any>
    */
   async sendMutation(mutation, variables, options = {}, headers = {}) {
     if (typeof mutation !== 'string') {

package/lib/helper/GraphQLDataFactory.js

@@ -36,7 +36,7 @@ const GraphQL = require('./GraphQL');
  * Install [Rosie](https://github.com/rosiejs/rosie) and [Faker](https://www.npmjs.com/package/faker) libraries.
  *
  * ```sh
- * npm i rosie faker --save-dev
+ * npm i rosie @faker-js/faker --save-dev
  * ```
  *
  * Create a factory file for a resource.
@@ -46,8 +46,8 @@ const GraphQL = require('./GraphQL');
  * ```js
  * // tests/factories/users.js
  *
- * var Factory = require('rosie').Factory;
- * var faker = require('faker');
+ * const Factory = require('rosie').Factory;
+ * const faker = require('@faker-js/faker');
  *
  * // Used with a constructor function passed to Factory, so that the final build
  * // object matches the necessary pattern to be sent as the variables object.

package/lib/helper/JSONResponse.js

@@ -218,7 +218,7 @@ class JSONResponse extends Helper {
    * Use it to perform custom checks of response data
    *
    * ```js
-   * I.seeResponseValidByCallback({ data, status, expect } => {
+   * I.seeResponseValidByCallback(({ data, status, expect }) => {
    *   expect(status).to.eql(200);
    *   expect(data).keys.to.include(['user', 'company']);
    * });

package/lib/helper/Playwright.js

@@ -92,6 +92,7 @@ const { createValueEngine, createDisabledEngine } = require('./extras/Playwright
  * * `waitForTimeout`: (optional) default wait* timeout in ms. Default: 1000.
  * * `basicAuth`: (optional) the basic authentication to pass to base url. Example: {username: 'username', password: 'password'}
  * * `windowSize`: (optional) default window size. Set a dimension like `640x480`.
+ * * `colorScheme`: (optional) default color scheme. Possible values: `dark` | `light` | `no-preference`.
  * * `userAgent`: (optional) user-agent string.
  * * `locale`: (optional) locale string. Example: 'en-GB', 'de-DE', 'fr-FR', ...
  * * `manualStart`: (optional, default: false) - do not start browser before a test, start it manually inside a helper with `this.helpers["Playwright"]._startBrowser()`.
@@ -212,7 +213,7 @@ const { createValueEngine, createDisabledEngine } = require('./extras/Playwright
  * }
  * ```
  *
- * #### Example #7: Launch test with a specifc user locale
+ * #### Example #7: Launch test with a specific user locale
  *
  * ```js
  * {
@@ -225,6 +226,19 @@ const { createValueEngine, createDisabledEngine } = require('./extras/Playwright
  * }
  * ```
  *
+ * * #### Example #8: Launch test with a specific color scheme
+ *
+ * ```js
+ * {
+ *  helpers: {
+ *   Playwright : {
+ *     url: "http://localhost",
+ *     colorScheme: "dark",
+ *   }
+ *  }
+ * }
+ * ```
+ *
  * Note: When connecting to remote browser `show` and specific `chrome` options (e.g. `headless` or `devtools`) are ignored.
  *
  * ## Access From Helpers
@@ -273,7 +287,7 @@ class Playwright extends Helper {
       waitForAction: 100,
       waitForTimeout: 1000,
       pressKeyDelay: 10,
-      timeout: 1000,
+      timeout: 5000,
       fullPageScreenshots: false,
       disableScreenshots: false,
       ignoreLog: ['warning', 'log'],
@@ -411,6 +425,7 @@ class Playwright extends Helper {
       if (this.storageState) contextOptions.storageState = this.storageState;
       if (this.options.userAgent) contextOptions.userAgent = this.options.userAgent;
       if (this.options.locale) contextOptions.locale = this.options.locale;
+      if (this.options.colorScheme) contextOptions.colorScheme = this.options.colorScheme;
       if (!this.browserContext || !restartsSession()) {
         this.browserContext = await this.browser.newContext(contextOptions); // Adding the HTTPSError ignore in the context so that we can ignore those errors
       }
@@ -538,7 +553,7 @@ class Playwright extends Helper {
   * ```
   *
   * @param {string} description used to show in logs.
-  * @param {function} fn async function that executed with Playwright helper as argument
+  * @param {function} fn async function that executed with Playwright helper as argumen
   */
   usePlaywrightTo(description, fn) {
     return this._useTo(...arguments);
@@ -851,7 +866,7 @@ class Playwright extends Helper {
   /**
    * {{> dragAndDrop }}
    *
-   * [Additional options](https://playwright.dev/docs/api/class-page#page-drag-and-drop) can be passed as 3rd argument.
+   * @param {any} [options] [Additional options](https://playwright.dev/docs/api/class-page#page-drag-and-drop) can be passed as 3rd argument.
    *
    * ```js
    * // specify coordinates for source position
@@ -886,7 +901,7 @@ class Playwright extends Helper {
   /**
    * {{> scrollPageToBottom }}
    */
-  scrollPageToBottom() {
+  async scrollPageToBottom() {
     return this.executeScript(() => {
       const body = document.body;
       const html = document.documentElement;
@@ -972,8 +987,6 @@ class Playwright extends Helper {
    * ```js
    * const elements = await this.helpers['Playwright']._locate({name: 'password'});
    * ```
-   *
-   *
    */
   async _locate(locator) {
     const context = await this.context || await this._getContext();
@@ -1186,6 +1199,7 @@ class Playwright extends Helper {
    * ```
    *
    * @param {string} [fileName] set filename for downloaded file
+   * @return {Promise<void>}
    */
   async handleDownloads(fileName = 'downloads') {
     this.page.waitForEvent('download').then(async (download) => {
@@ -1204,7 +1218,7 @@ class Playwright extends Helper {
   /**
    * {{> click }}
    *
-   * [Additional options](https://playwright.dev/docs/api/class-page#page-click) for click available as 3rd argument.
+   * @param {any} [opts] [Additional options](https://playwright.dev/docs/api/class-page#page-click) for click available as 3rd argument.
    *
    * Examples:
    *
@@ -1669,7 +1683,6 @@ class Playwright extends Helper {
    *
    * @param {string|function} fn function to be executed in browser context.
    * @param {any} [arg] optional argument to pass to the function
-   * @return {Promise<any>}
    */
   async executeScript(fn, arg) {
     let context = this.page;
@@ -1992,7 +2005,7 @@ class Playwright extends Helper {
       test.artifacts = {};
     }
 
-    if (this.options.recordVideo && this.page.video()) {
+    if (this.options.recordVideo && this.page && this.page.video()) {
       test.artifacts.video = await this.page.video().path();
     }
 
@@ -2004,7 +2017,7 @@ class Playwright extends Helper {
   }
 
   async _passed(test) {
-    if (this.options.recordVideo && this.page.video()) {
+    if (this.options.recordVideo && this.page && this.page.video()) {
       if (this.options.keepVideoForPassedTests) {
         test.artifacts.video = await this.page.video().path();
       } else {
@@ -2427,9 +2440,8 @@ class Playwright extends Helper {
   * ```
   * This method allows intercepting and mocking requests & responses. [Learn more about it](https://playwright.dev/docs/network#handle-requests)
   *
-  * @param {string} [url] URL, regex or pattern for to match URL
-  * @param {function} [handler] a function to process request
-  *
+  * @param {string|RegExp} [url] URL, regex or pattern for to match URL
+  * @param {function} [handler] a function to process reques
   */
   async mockRoute(url, handler) {
     return this.browserContext.route(...arguments);
@@ -2444,9 +2456,8 @@ class Playwright extends Helper {
   * ```
   * If no handler is passed, all mock requests for the rote are disabled.
   *
-  * @param {string} [url] URL, regex or pattern for to match URL
-  * @param {function} [handler] a function to process request
-  *
+  * @param {string|RegExp} [url] URL, regex or pattern for to match URL
+  * @param {function} [handler] a function to process reques
   */
   async stopMockingRoute(url, handler) {
     return this.browserContext.unroute(...arguments);

package/lib/helper/REST.js

@@ -87,7 +87,7 @@ class REST extends Helper {
    * @param {object} headers headers list
    */
   haveRequestHeaders(headers) {
-    this.headers = { ...headers, ...this.headers };
+    this.headers = { ...this.headers, ...headers };
   }
 
   /**
@@ -98,7 +98,7 @@ class REST extends Helper {
    * I.amBearerAuthenticated(secret('heregoestoken'))
    * ```
    *
-   * @param {string} accessToken  Bearer access token
+   * @param {string | CodeceptJS.Secret} accessToken  Bearer access token
    */
   amBearerAuthenticated(accessToken) {
     this.haveRequestHeaders({ Authorization: `Bearer ${accessToken}` });
@@ -112,6 +112,9 @@ class REST extends Helper {
    * @returns {Promise<*>} response
    */
   async _executeRequest(request) {
+    // Add custom headers. They can be set by amBearerAuthenticated() or haveRequestHeaders()
+    request.headers = { ...this.headers, ...request.headers };
+
     const _debugRequest = { ...request };
     this.axios.defaults.timeout = request.timeout || this.options.timeout;
 

package/lib/helper/WebDriver.js

@@ -2551,6 +2551,8 @@ class WebDriver extends Helper {
 
   /**
    * Placeholder for ~ locator only test case write once run on both Appium and WebDriver.
+   * @param {*} caps
+   * @param {*} fn
    */
   /* eslint-disable */
   runOnIOS(caps, fn) {
@@ -2558,6 +2560,8 @@ class WebDriver extends Helper {
 
   /**
    * Placeholder for ~ locator only test case write once run on both Appium and WebDriver.
+   * @param {*} caps
+   * @param {*} fn
    */
   runOnAndroid(caps, fn) {
   }

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;
       }
     });