codeceptjs 3.0.6 → 3.1.2

package/lib/helper/Puppeteer.js

@@ -129,8 +129,9 @@ const consoleLogStore = new Console();
  *    }
  * }
  * ```
+ * > Note: When connecting to remote browser `show` and specific `chrome` options (e.g. `headless` or `devtools`) are ignored.
  *
-* #### Example #5: Target URL with provided basic authentication
+ * #### Example #5: Target URL with provided basic authentication
  *
  * ```js
  * {
@@ -143,10 +144,25 @@ const consoleLogStore = new Console();
  *    }
  * }
  * ```
+ * #### Troubleshooting
  *
+ * Error Message:  `No usable sandbox!`
+ *
+ * When running Puppeteer on CI try to disable sandbox if you see that message
+ *
+ * ```
+ * helpers: {
+ *  Puppeteer: {
+ *     url: 'http://localhost',
+ *     show: false,
+ *     chrome: {
+ *       args: ['--no-sandbox', '--disable-setuid-sandbox']
+ *     }
+ *   },
+ * }
+ * ```
  *
  *
- * Note: When connecting to remote browser `show` and specific `chrome` options (e.g. `headless` or `devtools`) are ignored.
  *
  * ## Access From Helpers
  *
@@ -670,7 +686,7 @@ class Puppeteer extends Helper {
     assertElementExists(els);
 
     // Use manual mouse.move instead of .hover() so the offset can be added to the coordinates
-    const { x, y } = await els[0]._clickablePoint();
+    const { x, y } = await getClickablePoint(els[0]);
     await this.page.mouse.move(x + offsetX, y + offsetY);
     return this._waitForAction();
   }
@@ -726,7 +742,7 @@ class Puppeteer extends Helper {
       const els = await this._locate(locator);
       assertElementExists(els, locator, 'Element');
       await els[0]._scrollIntoViewIfNeeded();
-      const elementCoordinates = await els[0]._clickablePoint();
+      const elementCoordinates = await getClickablePoint(els[0]);
       await this.executeScript((x, y) => window.scrollBy(x, y), elementCoordinates.x + offsetX, elementCoordinates.y + offsetY);
     } else {
       await this.executeScript((x, y) => window.scrollTo(x, y), offsetX, offsetY);
@@ -942,7 +958,10 @@ class Puppeteer extends Helper {
    */
   async seeElement(locator) {
     let els = await this._locate(locator);
-    els = await Promise.all(els.map(el => el.boundingBox()));
+    els = (await Promise.all(els.map(el => el.boundingBox() && el))).filter(v => v);
+    // Puppeteer visibility was ignored? | Remove when Puppeteer is fixed
+    els = await Promise.all(els.map(async el => (await el.evaluate(node => window.getComputedStyle(node).visibility !== 'hidden' && window.getComputedStyle(node).display !== 'none')) && el));
+
     return empty('visible elements').negate(els.filter(v => v).fill('ELEMENT'));
   }
 
@@ -952,7 +971,10 @@ class Puppeteer extends Helper {
    */
   async dontSeeElement(locator) {
     let els = await this._locate(locator);
-    els = await Promise.all(els.map(el => el.boundingBox()));
+    els = (await Promise.all(els.map(el => el.boundingBox() && el))).filter(v => v);
+    // Puppeteer visibility was ignored? | Remove when Puppeteer is fixed
+    els = await Promise.all(els.map(async el => (await el.evaluate(node => window.getComputedStyle(node).visibility !== 'hidden' && window.getComputedStyle(node).display !== 'none')) && el));
+
     return empty('visible elements').assert(els.filter(v => v).fill('ELEMENT'));
   }
 
@@ -1239,7 +1261,7 @@ class Puppeteer extends Helper {
    * {{ react }}
    */
   async fillField(field, value) {
-    const els = await findFields.call(this, field);
+    const els = await findVisibleFields.call(this, field);
     assertElementExists(els, field, 'Field');
     const el = els[0];
     const tag = await el.getProperty('tagName').then(el => el.jsonValue());
@@ -1266,7 +1288,7 @@ class Puppeteer extends Helper {
    * {{ react }}
    */
   async appendField(field, value) {
-    const els = await findFields.call(this, field);
+    const els = await findVisibleFields.call(this, field);
     assertElementExists(els, field, 'Field');
     await els[0].press('End');
     await els[0].type(value, { delay: this.options.pressKeyDelay });
@@ -1308,7 +1330,7 @@ class Puppeteer extends Helper {
    * {{> selectOption }}
    */
   async selectOption(select, option) {
-    const els = await findFields.call(this, select);
+    const els = await findVisibleFields.call(this, select);
     assertElementExists(els, select, 'Selectable field');
     const el = els[0];
     if (await el.getProperty('tagName').then(t => t.jsonValue()) !== 'SELECT') {
@@ -1342,7 +1364,10 @@ class Puppeteer extends Helper {
    */
   async grabNumberOfVisibleElements(locator) {
     let els = await this._locate(locator);
-    els = await Promise.all(els.map(el => el.boundingBox()));
+    els = (await Promise.all(els.map(el => el.boundingBox() && el))).filter(v => v);
+    // Puppeteer visibility was ignored? | Remove when Puppeteer is fixed
+    els = await Promise.all(els.map(async el => (await el.evaluate(node => window.getComputedStyle(node).visibility !== 'hidden' && window.getComputedStyle(node).display !== 'none')) && el));
+
     return els.filter(v => v).length;
   }
 
@@ -1727,8 +1752,8 @@ class Puppeteer extends Helper {
     const src = await this._locate(locator);
     assertElementExists(src, locator, 'Slider Element');
 
-    // Note: Using private api ._clickablePoint because the .BoundingBox does not take into account iframe offsets!
-    const sliderSource = await src[0]._clickablePoint();
+    // Note: Using public api .getClickablePoint because the .BoundingBox does not take into account iframe offsets
+    const sliderSource = await getClickablePoint(src[0]);
 
     // Drag start point
     await this.page.mouse.move(sliderSource.x, sliderSource.y, { steps: 5 });
@@ -2264,7 +2289,7 @@ class Puppeteer extends Helper {
 module.exports = Puppeteer;
 
 async function findElements(matcher, locator) {
-  if (locator.react) return findReact(matcher, locator);
+  if (locator.react) return findReact(matcher.executionContext(), locator);
   locator = new Locator(locator, 'css');
   if (!locator.isXPath()) return matcher.$$(locator.simplify());
   return matcher.$x(locator.value);
@@ -2293,7 +2318,7 @@ async function proceedClick(locator, context = null, options = {}) {
 }
 
 async function findClickable(matcher, locator) {
-  if (locator.react) return findReact(matcher, locator);
+  if (locator.react) return findReact(matcher.executionContext(), locator);
   locator = new Locator(locator);
   if (!locator.isFuzzy()) return findElements.call(this, matcher, locator);
 
@@ -2376,6 +2401,12 @@ async function proceedIsChecked(assertType, option) {
   return truth(`checkable ${option}`, 'to be checked')[assertType](selected);
 }
 
+async function findVisibleFields(locator) {
+  const els = await findFields.call(this, locator);
+  const visible = await Promise.all(els.map(el => el.boundingBox()));
+  return els.filter((el, index) => visible[index]);
+}
+
 async function findFields(locator) {
   const matchedLocator = new Locator(locator);
   if (!matchedLocator.isFuzzy()) {
@@ -2406,9 +2437,9 @@ async function proceedDragAndDrop(sourceLocator, destinationLocator) {
   const dst = await this._locate(destinationLocator);
   assertElementExists(dst, destinationLocator, 'Destination Element');
 
-  // Note: Using private api ._clickablePoint becaues the .BoundingBox does not take into account iframe offsets!
-  const dragSource = await src[0]._clickablePoint();
-  const dragDestination = await dst[0]._clickablePoint();
+  // Note: Using public api .getClickablePoint becaues the .BoundingBox does not take into account iframe offsets
+  const dragSource = await getClickablePoint(src[0]);
+  const dragDestination = await getClickablePoint(dst[0]);
 
   // Drag start point
   await this.page.mouse.move(dragSource.x, dragSource.y, { steps: 5 });
@@ -2422,7 +2453,7 @@ async function proceedDragAndDrop(sourceLocator, destinationLocator) {
 }
 
 async function proceedSeeInField(assertType, field, value) {
-  const els = await findFields.call(this, field);
+  const els = await findVisibleFields.call(this, field);
   assertElementExists(els, field, 'Field');
   const el = els[0];
   const tag = await el.getProperty('tagName').then(el => el.jsonValue());
@@ -2555,6 +2586,13 @@ async function targetCreatedHandler(page) {
   }
 }
 
+// BC compatibility for Puppeteer < 10
+async function getClickablePoint(el) {
+  if (el.clickablePoint) return el.clickablePoint();
+  if (el._clickablePoint) return el._clickablePoint();
+  return null;
+}
+
 // List of key values to key definitions
 // https://github.com/GoogleChrome/puppeteer/blob/v1.20.0/lib/USKeyboardLayout.js
 const keyDefinitionMap = {

package/lib/helper/REST.js

@@ -75,6 +75,8 @@ class REST extends Helper {
    * Executes axios request
    *
    * @param {*} request
+   *
+   * @returns {Promise<*>} response
    */
   async _executeRequest(request) {
     const _debugRequest = { ...request };
@@ -143,6 +145,8 @@ class REST extends Helper {
    *
    * @param {*} url
    * @param {object} [headers={}] - the headers object to be sent. By default it is sent as an empty object
+   *
+   * @returns {Promise<*>} response
    */
   async sendGetRequest(url, headers = {}) {
     const request = {
@@ -166,6 +170,8 @@ class REST extends Helper {
    * @param {*} url
    * @param {*} [payload={}] - the payload to be sent. By default it is sent as an empty object
    * @param {object} [headers={}] - the headers object to be sent. By default it is sent as an empty object
+   *
+   * @returns {Promise<*>} response
    */
   async sendPostRequest(url, payload = {}, headers = {}) {
     const request = {
@@ -197,6 +203,8 @@ class REST extends Helper {
    * @param {string} url
    * @param {*} [payload={}] - the payload to be sent. By default it is sent as an empty object
    * @param {object} [headers={}] - the headers object to be sent. By default it is sent as an empty object
+   *
+   * @returns {Promise<*>} response
    */
   async sendPatchRequest(url, payload = {}, headers = {}) {
     const request = {
@@ -228,6 +236,8 @@ class REST extends Helper {
    * @param {string} url
    * @param {*} [payload={}] - the payload to be sent. By default it is sent as an empty object
    * @param {object} [headers={}] - the headers object to be sent. By default it is sent as an empty object
+   *
+   * @returns {Promise<*>} response
    */
   async sendPutRequest(url, payload = {}, headers = {}) {
     const request = {
@@ -254,6 +264,8 @@ class REST extends Helper {
    *
    * @param {*} url
    * @param {object} [headers={}] - the headers object to be sent. By default it is sent as an empty object
+   *
+   * @returns {Promise<*>} response
    */
   async sendDeleteRequest(url, headers = {}) {
     const request = {

package/lib/helper/WebDriver.js

@@ -396,6 +396,7 @@ class WebDriver extends Helper {
     this.isRunning = false;
     this.sessionWindows = {};
     this.activeSessionName = '';
+    this.customLocatorStrategies = config.customLocatorStrategies;
 
     this._setConfig(config);
 
@@ -503,6 +504,33 @@ class WebDriver extends Helper {
     }
   }
 
+  _lookupCustomLocator(customStrategy) {
+    if (typeof (this.customLocatorStrategies) !== 'object') {
+      return null;
+    }
+    const strategy = this.customLocatorStrategies[customStrategy];
+    return typeof (strategy) === 'function' ? strategy : null;
+  }
+
+  _isCustomLocator(locator) {
+    const locatorObj = new Locator(locator);
+    if (locatorObj.isCustom()) {
+      const customLocator = this._lookupCustomLocator(locatorObj.type);
+      if (customLocator) {
+        return true;
+      }
+      throw new Error('Please define "customLocatorStrategies" as an Object and the Locator Strategy as a "function".');
+    }
+    return false;
+  }
+
+  async _res(locator) {
+    const res = (this._isShadowLocator(locator) || this._isCustomLocator(locator))
+      ? await this._locate(locator)
+      : await this.$$(withStrictLocator(locator));
+    return res;
+  }
+
   async _startBrowser() {
     try {
       if (this.options.multiremote) {
@@ -530,9 +558,22 @@ class WebDriver extends Helper {
     await this._resizeWindowIfNeeded(this.browser, this.options.windowSize);
 
     this.$$ = this.browser.$$.bind(this.browser);
+
+    if (this._isCustomLocatorStrategyDefined()) {
+      Object.keys(this.customLocatorStrategies).forEach(async (customLocator) => {
+        this.debugSection('Weddriver', `adding custom locator strategy: ${customLocator}`);
+        const locatorFunction = this._lookupCustomLocator(customLocator);
+        this.browser.addLocatorStrategy(customLocator, locatorFunction);
+      });
+    }
+
     return this.browser;
   }
 
+  _isCustomLocatorStrategyDefined() {
+    return this.customLocatorStrategies && Object.keys(this.customLocatorStrategies).length;
+  }
+
   async _stopBrowser() {
     if (this.browser && this.isRunning) await this.browser.deleteSession();
   }
@@ -755,17 +796,34 @@ class WebDriver extends Helper {
     }
 
     if (!this.options.smartWait || !smartWait) {
+      if (this._isCustomLocator(locator)) {
+        const locatorObj = new Locator(locator);
+        return this.browser.custom$$(locatorObj.type, locatorObj.value);
+      }
+
       const els = await this.$$(withStrictLocator(locator));
       return els;
     }
 
     await this._smartWait(locator);
 
+    if (this._isCustomLocator(locator)) {
+      const locatorObj = new Locator(locator);
+      return this.browser.custom$$(locatorObj.type, locatorObj.value);
+    }
+
     const els = await this.$$(withStrictLocator(locator));
     await this.defineTimeout({ implicit: 0 });
     return els;
   }
 
+  _grabCustomLocator(locator) {
+    if (typeof locator === 'string') {
+      locator = new Locator(locator);
+    }
+    return locator.value ? locator.value : locator.custom;
+  }
+
   /**
    * Find a checkbox by providing human readable text:
    *
@@ -962,6 +1020,7 @@ class WebDriver extends Helper {
   /**
    * {{> fillField }}
    * {{ react }}
+   * {{ custom }}
    *
    */
   async fillField(field, value) {
@@ -1193,7 +1252,6 @@ class WebDriver extends Helper {
 
   /**
    * {{> grabAttributeFromAll }}
-   * Appium: can be used for apps only with several values ("contentDescription", "text", "className", "resourceId")
    */
   async grabAttributeFromAll(locator, attr) {
     const res = await this._locate(locator, true);
@@ -1204,7 +1262,6 @@ class WebDriver extends Helper {
 
   /**
    * {{> grabAttributeFrom }}
-   * Appium: can be used for apps only with several values ("contentDescription", "text", "className", "resourceId")
    */
   async grabAttributeFrom(locator, attr) {
     const attrs = await this.grabAttributeFromAll(locator, attr);
@@ -1335,7 +1392,7 @@ class WebDriver extends Helper {
    *
    */
   async seeElementInDOM(locator) {
-    const res = await this.$$(withStrictLocator(locator));
+    const res = await this._res(locator);
     return empty('elements').negate(res);
   }
 
@@ -1344,7 +1401,7 @@ class WebDriver extends Helper {
    *
    */
   async dontSeeElementInDOM(locator) {
-    const res = await this.$$(withStrictLocator(locator));
+    const res = await this._res(locator);
     return empty('elements').assert(res);
   }
 
@@ -1991,7 +2048,7 @@ class WebDriver extends Helper {
       }, aSec * 1000, `element (${new Locator(locator)}) still not enabled after ${aSec} sec`);
     }
     return this.browser.waitUntil(async () => {
-      const res = await this.$$(withStrictLocator(locator));
+      const res = await this._res(locator);
       if (!res || res.length === 0) {
         return false;
       }
@@ -2018,7 +2075,7 @@ class WebDriver extends Helper {
       }, aSec * 1000, `element (${locator}) still not present on page after ${aSec} sec`);
     }
     return this.browser.waitUntil(async () => {
-      const res = await this.$$(withStrictLocator(locator));
+      const res = await this._res(locator);
       return res && res.length;
     }, { timeout: aSec * 1000, timeoutMsg: `element (${locator}) still not present on page after ${aSec} sec` });
   }
@@ -2188,9 +2245,7 @@ class WebDriver extends Helper {
       }, aSec * 1000, `element (${new Locator(locator)}) still not visible after ${aSec} sec`);
     }
     return this.browser.waitUntil(async () => {
-      const res = (this._isShadowLocator(locator))
-        ? await this._locate(withStrictLocator(locator))
-        : await this.$$(withStrictLocator(locator));
+      const res = await this._res(locator);
       if (!res || res.length === 0) return false;
       const selected = await forEachAsync(res, async el => el.isDisplayed());
       if (Array.isArray(selected)) {
@@ -2217,7 +2272,7 @@ class WebDriver extends Helper {
       }, aSec * 1000, `The number of elements (${new Locator(locator)}) is not ${num} after ${aSec} sec`);
     }
     return this.browser.waitUntil(async () => {
-      const res = await this.$$(withStrictLocator(locator));
+      const res = await this._res(locator);
       if (!res || res.length === 0) return false;
       let selected = await forEachAsync(res, async el => el.isDisplayed());
 
@@ -2241,7 +2296,7 @@ class WebDriver extends Helper {
       }, aSec * 1000, `element (${new Locator(locator)}) still visible after ${aSec} sec`);
     }
     return this.browser.waitUntil(async () => {
-      const res = await this.$$(withStrictLocator(locator));
+      const res = await this._res(locator);
       if (!res || res.length === 0) return true;
       const selected = await forEachAsync(res, async el => el.isDisplayed());
       return !selected.length;
@@ -2262,7 +2317,7 @@ class WebDriver extends Helper {
     const aSec = sec || this.options.waitForTimeout;
     if (isWebDriver5()) {
       return this.browser.waitUntil(async () => {
-        const res = await this.$$(withStrictLocator(locator));
+        const res = await this._res(locator);
         if (!res || res.length === 0) {
           return true;
         }
@@ -2270,7 +2325,7 @@ class WebDriver extends Helper {
       }, aSec * 1000, `element (${new Locator(locator)}) still on page after ${aSec} sec`);
     }
     return this.browser.waitUntil(async () => {
-      const res = await this.$$(withStrictLocator(locator));
+      const res = await this._res(locator);
       if (!res || res.length === 0) {
         return true;
       }
@@ -2543,12 +2598,9 @@ async function proceedSee(assertType, text, context, strict = false) {
   }
 
   const smartWaitEnabled = assertType === 'assert';
-
   const res = await this._locate(withStrictLocator(context), smartWaitEnabled);
   assertElementExists(res, context);
-
   const selected = await forEachAsync(res, async el => this.browser.getElementText(getElementId(el)));
-
   if (strict) {
     if (Array.isArray(selected) && selected.length !== 0) {
       return selected.map(elText => equals(description)[assertType](text, elText));
@@ -2616,6 +2668,11 @@ async function filterAsync(array, callback) {
 
 async function findClickable(locator, locateFn) {
   locator = new Locator(locator);
+
+  if (this._isCustomLocator(locator)) {
+    return locateFn(locator.value);
+  }
+
   if (locator.isAccessibilityId() && !this.isWeb) return locateFn(locator, true);
   if (!locator.isFuzzy()) return locateFn(locator, true);
 
@@ -2637,6 +2694,10 @@ async function findClickable(locator, locateFn) {
 async function findFields(locator) {
   locator = new Locator(locator);
 
+  if (this._isCustomLocator(locator)) {
+    return this._locate(locator);
+  }
+
   if (locator.isAccessibilityId() && !this.isWeb) return this._locate(locator, true);
   if (!locator.isFuzzy()) return this._locate(locator, true);
 
@@ -2742,6 +2803,10 @@ async function findCheckable(locator, locateFn) {
   let els;
   locator = new Locator(locator);
 
+  if (this._isCustomLocator(locator)) {
+    return locateFn(locator.value);
+  }
+
   if (locator.isAccessibilityId() && !this.isWeb) return locateFn(locator, true);
   if (!locator.isFuzzy()) return locateFn(locator, true);
 
@@ -2787,6 +2852,7 @@ function getElementId(el) {
   if (el.ELEMENT) {
     return el.ELEMENT;
   }
+
   return null;
 }
 

package/lib/helper/errors/ConnectionRefused.js

@@ -1,7 +1,7 @@
 function ConnectionRefused(err) {
   this.message = "Can't connect to WebDriver.\n";
   this.message += `${err}\n\n`;
-  this.message += 'Please make sure Selenium Server (ChromeDriver or PhantomJS) is running and accessible';
+  this.message += 'Please make sure Selenium Server is running and accessible';
   this.stack = err.stack;
 }
 

package/lib/helper/extras/Popup.js

@@ -3,7 +3,7 @@
  */
 class Popup {
   constructor(popup, defaultAction) {
-    this._popup = popup || {};
+    this._popup = popup || null;
     this._actionType = '';
     this._defaultAction = defaultAction || '';
   }

package/lib/helper/extras/React.js

@@ -4,50 +4,62 @@ let resqScript;
 
 module.exports = async function findReact(matcher, locator) {
   if (!resqScript) resqScript = fs.readFileSync(require.resolve('resq'));
-  await matcher.executionContext().evaluate(resqScript.toString());
-  await matcher.executionContext().evaluate(() => window.resq.waitToLoadReact());
-  const arrayHandle = await matcher.executionContext().evaluateHandle((selector, props, state) => {
-    let elements = window.resq.resq$$(selector);
-    if (Object.keys(props).length) {
-      elements = elements.byProps(props);
-    }
-    if (Object.keys(state).length) {
-      elements = elements.byState(state);
-    }
+  await matcher.evaluate(resqScript.toString());
+  await matcher
+    .evaluate(() => window.resq.waitToLoadReact());
+  const arrayHandle = await matcher.evaluateHandle(
+    (obj) => {
+      const { selector, props, state } = obj;
 
-    if (!elements.length) {
-      return [];
-    }
+      let elements = window.resq.resq$$(selector);
+      if (Object.keys(props).length) {
+        elements = elements.byProps(props);
+      }
+      if (Object.keys(state).length) {
+        elements = elements.byState(state);
+      }
 
-    // resq returns an array of HTMLElements if the React component is a fragment
-    // this avoids having nested arrays of nodes which the driver does not understand
-    // [[div, div], [div, div]] => [div, div, div, div]
-    let nodes = [];
+      if (!elements.length) {
+        return [];
+      }
 
-    elements.forEach((element) => {
-      let { node, isFragment } = element;
+      // resq returns an array of HTMLElements if the React component is a fragment
+      // this avoids having nested arrays of nodes which the driver does not understand
+      // [[div, div], [div, div]] => [div, div, div, div]
+      let nodes = [];
 
-      if (!node) {
-        isFragment = true;
-        node = element.children;
-      }
+      elements.forEach((element) => {
+        let { node, isFragment } = element;
 
-      if (isFragment) {
-        nodes = nodes.concat(node);
-      } else {
-        nodes.push(node);
-      }
-    });
+        if (!node) {
+          isFragment = true;
+          node = element.children;
+        }
 
-    return [...nodes];
-  }, locator.react, locator.props || {}, locator.state || {}, locator.children || {}, matcher);
+        if (isFragment) {
+          nodes = nodes.concat(node);
+        } else {
+          nodes.push(node);
+        }
+      });
+
+      return [...nodes];
+    },
+    {
+      selector: locator.react,
+      props: locator.props || {},
+      state: locator.state || {},
+    },
+  );
 
   const properties = await arrayHandle.getProperties();
   await arrayHandle.dispose();
   const result = [];
   for (const property of properties.values()) {
     const elementHandle = property.asElement();
-    if (elementHandle) { result.push(elementHandle); }
+    if (elementHandle) {
+      result.push(elementHandle);
+    }
   }
 
   return result;

package/lib/interfaces/gherkin.js

@@ -83,6 +83,7 @@ module.exports = (text) => {
           for (const index in example.cells) {
             const placeholder = fields[index];
             const value = transform('gherkin.examples', example.cells[index].value);
+            example.cells[index].value = value;
             current[placeholder] = value;
             exampleSteps = exampleSteps.map((step) => {
               step = { ...step };

package/lib/listener/exit.js

@@ -18,13 +18,11 @@ module.exports = function () {
     failedTests = failedTests.filter(failed => id !== failed);
   });
 
-  event.dispatcher.on(event.all.result, () => {
+  process.on('beforeExit', (code) => {
     if (failedTests.length) {
-      process.exitCode = 1;
+      code = 1;
     }
-  });
 
-  process.on('beforeExit', (code) => {
     if (code) {
       process.exit(code);
     }

package/lib/listener/helpers.js

@@ -11,11 +11,10 @@ module.exports = function () {
 
   const runHelpersHook = (hook, param) => {
     if (store.dryRun) return;
-    Object.keys(helpers).forEach((key) => {
-      if (!helpers[key][hook]) {
-        return;
+    Object.values(helpers).forEach((helper) => {
+      if (helper[hook]) {
+        helper[hook](param);
       }
-      helpers[key][hook](param);
     });
   };
 

package/lib/locator.js

@@ -150,6 +150,13 @@ class Locator {
     return this.isFuzzy() && this.value[0] === '~';
   }
 
+  /**
+   * @returns {boolean}
+   */
+  isBasic() {
+    return this.isCSS() || this.isXPath();
+  }
+
   /**
    * @returns {string}
    */