codeceptjs 3.3.8-beta.1 → 3.4.0

package/lib/helper/WebDriver.js

@@ -4,12 +4,12 @@ const assert = require('assert');
 const path = require('path');
 const fs = require('fs');
 
-const Helper = require('../helper');
+const Helper = require('@codeceptjs/helper');
 const stringIncludes = require('../assert/include').includes;
 const { urlEquals, equals } = require('../assert/equal');
 const { debug } = require('../output');
-const empty = require('../assert/empty').empty;
-const truth = require('../assert/truth').truth;
+const { empty } = require('../assert/empty');
+const { truth } = require('../assert/truth');
 const {
   xpathLocator,
   fileExists,
@@ -31,8 +31,6 @@ const Locator = require('../locator');
 const SHADOW = 'shadow';
 const webRoot = 'body';
 
-let version;
-
 /**
  * ## Configuration
  *
@@ -266,7 +264,7 @@ const config = {};
  * ```js
  * plugins: {
  *    wdio: {
-  *       enabled: true,
+ *       enabled: true,
  *        services: ['sauce'],
  *        user: ... ,// saucelabs username
  *        key: ... // saucelabs api key
@@ -294,7 +292,7 @@ const config = {};
  * ```js
  * plugins: {
  *    wdio: {
-  *       enabled: true,
+ *       enabled: true,
  *        services: ['browserstack'],
  *        user: ... ,// browserstack username
  *        key: ... // browserstack api key
@@ -386,16 +384,6 @@ class WebDriver extends Helper {
     super(config);
     webdriverio = require('webdriverio');
 
-    try {
-      version = JSON.parse(fs.readFileSync(path.join(require.resolve('webdriverio'), '/../../', 'package.json')).toString()).version;
-    } catch (err) {
-      this.debug('Can\'t detect webdriverio version, assuming webdriverio v6 is used');
-    }
-
-    if (isWebDriver5()) {
-      console.log('DEPRECATION NOTICE:');
-      console.log('You are using webdriverio v5. It is recommended to update to webdriverio@6.\nSupport of webdriverio v5 is deprecated and will be removed in CodeceptJS 3.0\n');
-    }
     // set defaults
     this.root = webRoot;
     this.isWeb = true;
@@ -661,23 +649,23 @@ class WebDriver extends Helper {
   }
 
   /**
-  * Use [webdriverio](https://webdriver.io/docs/api.html) API inside a test.
-  *
-  * First argument is a description of an action.
-  * Second argument is async function that gets this helper as parameter.
-  *
-  * { [`browser`](https://webdriver.io/docs/api.html)) } object from WebDriver API is available.
-  *
-  * ```js
-  * I.useWebDriverTo('open multiple windows', async ({ browser }) {
-  *    // create new window
-  *    await browser.newWindow('https://webdriver.io');
-  * });
-  * ```
-  *
-  * @param {string} description used to show in logs.
-  * @param {function} fn async functuion that executed with WebDriver helper as argument
-  */
+   * Use [webdriverio](https://webdriver.io/docs/api.html) API inside a test.
+   *
+   * First argument is a description of an action.
+   * Second argument is async function that gets this helper as parameter.
+   *
+   * { [`browser`](https://webdriver.io/docs/api.html)) } object from WebDriver API is available.
+   *
+   * ```js
+   * I.useWebDriverTo('open multiple windows', async ({ browser }) {
+   *    // create new window
+   *    await browser.newWindow('https://webdriver.io');
+   * });
+   * ```
+   *
+   * @param {string} description used to show in logs.
+   * @param {function} fn async functuion that executed with WebDriver helper as argument
+   */
   useWebDriverTo(description, fn) {
     return this._useTo(...arguments);
   }
@@ -1654,7 +1642,6 @@ class WebDriver extends Helper {
     const res = await this._locate(withStrictLocator(locator), true);
     assertElementExists(res, locator);
     const elem = usingFirstElement(res);
-    if (isWebDriver5()) return elem.moveTo(xOffset, yOffset);
     return elem.moveTo({ xOffset, yOffset });
   }
 
@@ -2043,19 +2030,7 @@ class WebDriver extends Helper {
    */
   async waitForEnabled(locator, sec = null) {
     const aSec = sec || this.options.waitForTimeoutInSeconds;
-    if (isWebDriver5()) {
-      return this.browser.waitUntil(async () => {
-        const res = await this.$$(withStrictLocator(locator));
-        if (!res || res.length === 0) {
-          return false;
-        }
-        const selected = await forEachAsync(res, async el => this.browser.isElementEnabled(getElementId(el)));
-        if (Array.isArray(selected)) {
-          return selected.filter(val => val === true).length > 0;
-        }
-        return selected;
-      }, aSec * 1000, `element (${new Locator(locator)}) still not enabled after ${aSec} sec`);
-    }
+
     return this.browser.waitUntil(async () => {
       const res = await this._res(locator);
       if (!res || res.length === 0) {
@@ -2077,12 +2052,7 @@ class WebDriver extends Helper {
    */
   async waitForElement(locator, sec = null) {
     const aSec = sec || this.options.waitForTimeoutInSeconds;
-    if (isWebDriver5()) {
-      return this.browser.waitUntil(async () => {
-        const res = await this.$$(withStrictLocator(locator));
-        return res && res.length;
-      }, aSec * 1000, `element (${(new Locator(locator))}) still not present on page after ${aSec} sec`);
-    }
+
     return this.browser.waitUntil(async () => {
       const res = await this._res(locator);
       return res && res.length;
@@ -2111,20 +2081,7 @@ class WebDriver extends Helper {
     const client = this.browser;
     const aSec = sec || this.options.waitForTimeoutInSeconds;
     let currUrl = '';
-    if (isWebDriver5()) {
-      return client
-        .waitUntil(function () {
-          return this.getUrl().then((res) => {
-            currUrl = decodeUrl(res);
-            return currUrl.indexOf(urlPart) > -1;
-          });
-        }, aSec * 1000).catch((e) => {
-          if (e.message.indexOf('timeout')) {
-            throw new Error(`expected url to include ${urlPart}, but found ${currUrl}`);
-          }
-          throw e;
-        });
-    }
+
     return client
       .waitUntil(function () {
         return this.getUrl().then((res) => {
@@ -2169,20 +2126,6 @@ class WebDriver extends Helper {
   async waitForText(text, sec = null, context = null) {
     const aSec = sec || this.options.waitForTimeoutInSeconds;
     const _context = context || this.root;
-    if (isWebDriver5()) {
-      return this.browser.waitUntil(
-        async () => {
-          const res = await this.$$(withStrictLocator.call(this, _context));
-          if (!res || res.length === 0) return false;
-          const selected = await forEachAsync(res, async el => this.browser.getElementText(getElementId(el)));
-          if (Array.isArray(selected)) {
-            return selected.filter(part => part.indexOf(text) >= 0).length > 0;
-          }
-          return selected.indexOf(text) >= 0;
-        }, aSec * 1000,
-        `element (${_context}) is not in DOM or there is no element(${_context}) with text "${text}" after ${aSec} sec`,
-      );
-    }
 
     return this.browser.waitUntil(
       async () => {
@@ -2206,20 +2149,7 @@ class WebDriver extends Helper {
   async waitForValue(field, value, sec = null) {
     const client = this.browser;
     const aSec = sec || this.options.waitForTimeoutInSeconds;
-    if (isWebDriver5()) {
-      return client.waitUntil(
-        async () => {
-          const res = await findFields.call(this, field);
-          if (!res || res.length === 0) return false;
-          const selected = await forEachAsync(res, async el => el.getValue());
-          if (Array.isArray(selected)) {
-            return selected.filter(part => part.indexOf(value) >= 0).length > 0;
-          }
-          return selected.indexOf(value) >= 0;
-        }, aSec * 1000,
-        `element (${field}) is not in DOM or there is no element(${field}) with value "${value}" after ${aSec} sec`,
-      );
-    }
+
     return client.waitUntil(
       async () => {
         const res = await findFields.call(this, field);
@@ -2242,17 +2172,7 @@ class WebDriver extends Helper {
    */
   async waitForVisible(locator, sec = null) {
     const aSec = sec || this.options.waitForTimeoutInSeconds;
-    if (isWebDriver5()) {
-      return this.browser.waitUntil(async () => {
-        const res = await this.$$(withStrictLocator(locator));
-        if (!res || res.length === 0) return false;
-        const selected = await forEachAsync(res, async el => el.isDisplayed());
-        if (Array.isArray(selected)) {
-          return selected.filter(val => val === true).length > 0;
-        }
-        return selected;
-      }, aSec * 1000, `element (${new Locator(locator)}) still not visible after ${aSec} sec`);
-    }
+
     return this.browser.waitUntil(async () => {
       const res = await this._res(locator);
       if (!res || res.length === 0) return false;
@@ -2269,17 +2189,7 @@ class WebDriver extends Helper {
    */
   async waitNumberOfVisibleElements(locator, num, sec = null) {
     const aSec = sec || this.options.waitForTimeoutInSeconds;
-    if (isWebDriver5()) {
-      return this.browser.waitUntil(async () => {
-        const res = await this.$$(withStrictLocator(locator));
-        if (!res || res.length === 0) return false;
-        let selected = await forEachAsync(res, async el => el.isDisplayed());
 
-        if (!Array.isArray(selected)) selected = [selected];
-        selected = selected.filter(val => val === true);
-        return selected.length === num;
-      }, aSec * 1000, `The number of elements (${new Locator(locator)}) is not ${num} after ${aSec} sec`);
-    }
     return this.browser.waitUntil(async () => {
       const res = await this._res(locator);
       if (!res || res.length === 0) return false;
@@ -2296,14 +2206,7 @@ class WebDriver extends Helper {
    */
   async waitForInvisible(locator, sec = null) {
     const aSec = sec || this.options.waitForTimeoutInSeconds;
-    if (isWebDriver5()) {
-      return this.browser.waitUntil(async () => {
-        const res = await this.$$(withStrictLocator(locator));
-        if (!res || res.length === 0) return true;
-        const selected = await forEachAsync(res, async el => el.isDisplayed());
-        return !selected.length;
-      }, aSec * 1000, `element (${new Locator(locator)}) still visible after ${aSec} sec`);
-    }
+
     return this.browser.waitUntil(async () => {
       const res = await this._res(locator);
       if (!res || res.length === 0) return true;
@@ -2324,15 +2227,7 @@ class WebDriver extends Helper {
    */
   async waitForDetached(locator, sec = null) {
     const aSec = sec || this.options.waitForTimeoutInSeconds;
-    if (isWebDriver5()) {
-      return this.browser.waitUntil(async () => {
-        const res = await this._res(locator);
-        if (!res || res.length === 0) {
-          return true;
-        }
-        return false;
-      }, aSec * 1000, `element (${new Locator(locator)}) still on page after ${aSec} sec`);
-    }
+
     return this.browser.waitUntil(async () => {
       const res = await this._res(locator);
       if (!res || res.length === 0) {
@@ -2356,9 +2251,7 @@ class WebDriver extends Helper {
     }
 
     const aSec = sec || this.options.waitForTimeoutInSeconds;
-    if (isWebDriver5()) {
-      return this.browser.waitUntil(async () => this.browser.execute(fn, ...args), aSec * 1000, '');
-    }
+
     return this.browser.waitUntil(async () => this.browser.execute(fn, ...args), { timeout: aSec * 1000, timeoutMsg: '' });
   }
 
@@ -2388,18 +2281,6 @@ class WebDriver extends Helper {
     let target;
     const current = await this.browser.getWindowHandle();
 
-    if (isWebDriver5()) {
-      await this.browser.waitUntil(async () => {
-        await this.browser.getWindowHandles().then((handles) => {
-          if (handles.indexOf(current) + num + 1 <= handles.length) {
-            target = handles[handles.indexOf(current) + num];
-          }
-        });
-        return target;
-      }, aSec * 1000, `There is no ability to switch to next tab with offset ${num}`);
-      return this.browser.switchToWindow(target);
-    }
-
     await this.browser.waitUntil(async () => {
       await this.browser.getWindowHandles().then((handles) => {
         if (handles.indexOf(current) + num + 1 <= handles.length) {
@@ -2419,18 +2300,6 @@ class WebDriver extends Helper {
     const current = await this.browser.getWindowHandle();
     let target;
 
-    if (isWebDriver5()) {
-      await this.browser.waitUntil(async () => {
-        await this.browser.getWindowHandles().then((handles) => {
-          if (handles.indexOf(current) - num > -1) {
-            target = handles[handles.indexOf(current) - num];
-          }
-        });
-        return target;
-      }, aSec * 1000, `There is no ability to switch to previous tab with offset ${num}`);
-      return this.browser.switchToWindow(target);
-    }
-
     await this.browser.waitUntil(async () => {
       await this.browser.getWindowHandles().then((handles) => {
         if (handles.indexOf(current) - num > -1) {
@@ -3018,8 +2887,4 @@ function prepareLocateFn(context) {
   };
 }
 
-function isWebDriver5() {
-  return version && version.indexOf('5') === 0;
-}
-
 module.exports = WebDriver;

package/lib/helper.js

@@ -1,4 +1,2 @@
 // helper class was moved out from this repository to allow extending from base class
-// without loading full CodeceptJS package
-if (!global.codeceptjs) global.codeceptjs = require('./index');
 module.exports = require('@codeceptjs/helper');

package/lib/interfaces/bdd.js

@@ -1,4 +1,4 @@
-const { CucumberExpression, ParameterTypeRegistry, ParameterType } = require('cucumber-expressions');
+const { CucumberExpression, ParameterTypeRegistry, ParameterType } = require('@cucumber/cucumber-expressions');
 const Config = require('../config');
 
 let steps = {};

package/lib/interfaces/featureConfig.js

@@ -19,6 +19,7 @@ class FeatureConfig {
    * Set timeout for this suite
    * @param {number} timeout
    * @returns {this}
+   * @deprecated
    */
   timeout(timeout) {
     console.log(`Feature('${this.suite.title}').timeout(${timeout}) is deprecated!`);

package/lib/interfaces/gherkin.js

@@ -1,4 +1,5 @@
-const { Parser } = require('gherkin');
+const Gherkin = require('@cucumber/gherkin');
+const Messages = require('@cucumber/messages');
 const { Context, Suite, Test } = require('mocha');
 
 const { matchStep } = require('./bdd');
@@ -8,7 +9,10 @@ const Step = require('../step');
 const DataTableArgument = require('../data/dataTableArgument');
 const transform = require('../transform');
 
-const parser = new Parser();
+const uuidFn = Messages.IdGenerator.uuid();
+const builder = new Gherkin.AstBuilder(uuidFn);
+const matcher = new Gherkin.GherkinClassicTokenMatcher();
+const parser = new Gherkin.Parser(builder, matcher);
 parser.stopAtFirstError = false;
 
 module.exports = (text, file) => {
@@ -46,14 +50,20 @@ module.exports = (text, file) => {
         step.metaStep = metaStep;
       };
       const fn = matchStep(step.text);
-      if (step.argument) {
-        step.argument.parse = () => {
-          return new DataTableArgument(step.argument);
-        };
-        fn.params.push(step.argument);
-        if (step.argument.type === 'DataTable') metaStep.comment = `\n${transformTable(step.argument)}`;
-        if (step.argument.content) metaStep.comment = `\n${step.argument.content}`;
+
+      if (step.dataTable) {
+        fn.params.push({
+          ...step.dataTable,
+          parse: () => new DataTableArgument(step.dataTable),
+        });
+        metaStep.comment = `\n${transformTable(step.dataTable)}`;
+      }
+
+      if (step.docString) {
+        fn.params.push(step.docString);
+        metaStep.comment = `\n"""\n${step.docString.content}\n"""`;
       }
+
       step.startTime = Date.now();
       step.match = fn.line;
       event.emit(event.bddStep.before, step);
@@ -74,15 +84,15 @@ module.exports = (text, file) => {
   };
 
   for (const child of ast.feature.children) {
-    if (child.type === 'Background') {
-      suite.beforeEach('Before', scenario.injected(async () => runSteps(child.steps), suite, 'before'));
+    if (child.background) {
+      suite.beforeEach('Before', scenario.injected(async () => runSteps(child.background.steps), suite, 'before'));
       continue;
     }
-    if (child.type === 'ScenarioOutline') {
-      for (const examples of child.examples) {
+    if (child.scenario && child.scenario.keyword === 'Scenario Outline') {
+      for (const examples of child.scenario.examples) {
         const fields = examples.tableHeader.cells.map(c => c.value);
         for (const example of examples.tableBody) {
-          let exampleSteps = [...child.steps];
+          let exampleSteps = [...child.scenario.steps];
           const current = {};
           for (const index in example.cells) {
             const placeholder = fields[index];
@@ -95,8 +105,8 @@ module.exports = (text, file) => {
               return step;
             });
           }
-          const tags = child.tags.map(t => t.name).concat(examples.tags.map(t => t.name));
-          const title = `${child.name} ${JSON.stringify(current)} ${tags.join(' ')}`.trim();
+          const tags = child.scenario.tags.map(t => t.name).concat(examples.tags.map(t => t.name));
+          const title = `${child.scenario.name} ${JSON.stringify(current)} ${tags.join(' ')}`.trim();
           const test = new Test(title, async () => runSteps(addExampleInTable(exampleSteps, current)));
           test.tags = suite.tags.concat(tags);
           test.file = file;
@@ -105,12 +115,15 @@ module.exports = (text, file) => {
       }
       continue;
     }
-    const tags = child.tags.map(t => t.name);
-    const title = `${child.name} ${tags.join(' ')}`.trim();
-    const test = new Test(title, async () => runSteps(child.steps));
-    test.tags = suite.tags.concat(tags);
-    test.file = file;
-    suite.addTest(scenario.test(test));
+
+    if (child.scenario) {
+      const tags = child.scenario.tags.map(t => t.name);
+      const title = `${child.scenario.name} ${tags.join(' ')}`.trim();
+      const test = new Test(title, async () => runSteps(child.scenario.steps));
+      test.tags = suite.tags.concat(tags);
+      test.file = file;
+      suite.addTest(scenario.test(test));
+    }
   }
 
   return suite;
@@ -130,9 +143,9 @@ function addExampleInTable(exampleSteps, placeholders) {
   for (const placeholder in placeholders) {
     steps.map((step) => {
       step = { ...step };
-      if (step.argument && step.argument.type === 'DataTable') {
-        for (const id in step.argument.rows) {
-          const cells = step.argument.rows[id].cells;
+      if (step.dataTable) {
+        for (const id in step.dataTable.rows) {
+          const cells = step.dataTable.rows[id].cells;
           cells.map(c => (c.value = c.value.replace(`<${placeholder}>`, placeholders[placeholder])));
         }
       }

package/lib/listener/exit.js

@@ -6,7 +6,7 @@ module.exports = function () {
   event.dispatcher.on(event.test.failed, (testOrSuite) => {
     // NOTE When an error happens in one of the hooks (BeforeAll/BeforeEach...) the event object
     // is a suite and not a test
-    const id = testOrSuite.id || (testOrSuite.ctx && testOrSuite.ctx.test.id) || 'empty';
+    const id = testOrSuite.uid || (testOrSuite.ctx && testOrSuite.ctx.test.uid) || 'empty';
     failedTests.push(id);
   });
 
@@ -14,7 +14,7 @@ module.exports = function () {
   event.dispatcher.on(event.test.passed, (testOrSuite) => {
     // NOTE When an error happens in one of the hooks (BeforeAll/BeforeEach...) the event object
     // is a suite and not a test
-    const id = testOrSuite.id || (testOrSuite.ctx && testOrSuite.ctx.test.id) || 'empty';
+    const id = testOrSuite.uid || (testOrSuite.ctx && testOrSuite.ctx.test.uid) || 'empty';
     failedTests = failedTests.filter(failed => id !== failed);
   });
 

package/lib/listener/retry.js

@@ -0,0 +1,67 @@
+const event = require('../event');
+const output = require('../output');
+const Config = require('../config');
+const { isNotSet } = require('../utils');
+
+const hooks = ['Before', 'After', 'BeforeSuite', 'AfterSuite'];
+
+module.exports = function () {
+  event.dispatcher.on(event.suite.before, (suite) => {
+    let retryConfig = Config.get('retry');
+    if (!retryConfig) return;
+
+    if (Number.isInteger(+retryConfig)) {
+      // is number
+      const retryNum = +retryConfig;
+      output.log(`Retries: ${retryNum}`);
+      suite.retries(retryNum);
+      return;
+    }
+
+    if (!Array.isArray(retryConfig)) {
+      retryConfig = [retryConfig];
+    }
+
+    for (const config of retryConfig) {
+      if (config.grep) {
+        if (!suite.title.includes(config.grep)) continue;
+      }
+
+      hooks.filter(hook => !!config[hook]).forEach((hook) => {
+        if (isNotSet(suite.opts[`retry${hook}`])) suite.opts[`retry${hook}`] = config[hook];
+      });
+
+      if (config.Feature) {
+        if (isNotSet(suite.retries())) suite.retries(config.Feature);
+      }
+
+      output.log(`Retries: ${JSON.stringify(config)}`);
+    }
+  });
+
+  event.dispatcher.on(event.test.before, (test) => {
+    let retryConfig = Config.get('retry');
+    if (!retryConfig) return;
+
+    if (Number.isInteger(+retryConfig)) {
+      return;
+    }
+
+    if (!Array.isArray(retryConfig)) {
+      retryConfig = [retryConfig];
+    }
+
+    retryConfig = retryConfig.filter(config => !!config.Scenario);
+
+    for (const config of retryConfig) {
+      if (config.grep) {
+        if (!test.fullTitle().includes(config.grep)) continue;
+      }
+
+      if (config.Scenario) {
+        if (isNotSet(test.retries())) test.retries(config.Scenario);
+        output.log(`Retries: ${config.Scenario}`);
+      }
+    }
+  });
+};

package/lib/listener/steps.js

@@ -81,6 +81,6 @@ module.exports = function () {
     if (store.debugMode) return;
     step.finishedAt = +new Date();
     if (step.startedAt) step.duration = step.finishedAt - step.startedAt;
-    debug(`Step '${step}' finished; Duration: ${step.duration}ms`);
+    debug(`Step '${step}' finished; Duration: ${step.duration || 0}ms`);
   });
 };

package/lib/listener/timeout.js

@@ -7,7 +7,7 @@ const TIMEOUT_ORDER = require('../step').TIMEOUT_ORDER;
 
 module.exports = function () {
   let timeout;
-  let timeoutStack = [];
+  let suiteTimeout = [];
   let currentTest;
   let currentTimeout;
 
@@ -17,21 +17,52 @@ module.exports = function () {
   }
 
   event.dispatcher.on(event.suite.before, (suite) => {
-    timeoutStack = [];
-    const globalTimeout = Config.get('timeout');
-    if (globalTimeout) {
-      if (globalTimeout >= 1000) {
-        console.log(`Warning: Timeout was set to ${globalTimeout}secs.\nGlobal timeout should be specified in seconds.`);
+    suiteTimeout = [];
+    let timeoutConfig = Config.get('timeout');
+
+    if (timeoutConfig) {
+      if (!Number.isNaN(+timeoutConfig)) {
+        checkForSeconds(timeoutConfig);
+        suiteTimeout.push(timeoutConfig);
+      }
+
+      if (!Array.isArray(timeoutConfig)) {
+        timeoutConfig = [timeoutConfig];
+      }
+
+      for (const config of timeoutConfig.filter(c => !!c.Feature)) {
+        if (config.grep) {
+          if (!suite.title.includes(config.grep)) continue;
+        }
+        suiteTimeout.push(config.Feature);
       }
-      timeoutStack.push(globalTimeout);
     }
-    if (suite.totalTimeout) timeoutStack.push(suite.totalTimeout);
-    output.log(`Timeouts: ${timeoutStack}`);
+
+    if (suite.totalTimeout) suiteTimeout.push(suite.totalTimeout);
+    output.log(`Timeouts: ${suiteTimeout}`);
   });
 
   event.dispatcher.on(event.test.before, (test) => {
     currentTest = test;
-    timeout = test.totalTimeout || timeoutStack[timeoutStack.length - 1];
+    let testTimeout = null;
+
+    let timeoutConfig = Config.get('timeout');
+
+    if (typeof timeoutConfig === 'object' || Array.isArray(timeoutConfig)) {
+      if (!Array.isArray(timeoutConfig)) {
+        timeoutConfig = [timeoutConfig];
+      }
+
+      for (const config of timeoutConfig.filter(c => !!c.Scenario)) {
+        console.log('Test Timeout', config, test.title.includes(config.grep));
+        if (config.grep) {
+          if (!test.title.includes(config.grep)) continue;
+        }
+        testTimeout = config.Scenario;
+      }
+    }
+
+    timeout = test.totalTimeout || testTimeout || suiteTimeout[suiteTimeout.length - 1];
     if (!timeout) return;
     currentTimeout = timeout;
     output.debug(`Test Timeout: ${timeout}s`);
@@ -70,3 +101,9 @@ module.exports = function () {
     }
   });
 };
+
+function checkForSeconds(timeout) {
+  if (timeout >= 1000) {
+    console.log(`Warning: Timeout was set to ${timeout}secs.\nGlobal timeout should be specified in seconds.`);
+  }
+}

package/lib/mochaFactory.js

@@ -47,13 +47,13 @@ class MochaFactory {
         let missingFeatureInFile = [];
         const seenTests = [];
         mocha.suite.eachTest(test => {
-          test.id = genTestId(test);
+          test.uid = genTestId(test);
 
           const name = test.fullTitle();
-          if (seenTests.includes(test.id)) {
+          if (seenTests.includes(test.uid)) {
             dupes.push(name);
           }
-          seenTests.push(test.id);
+          seenTests.push(test.uid);
 
           if (name.slice(0, name.indexOf(':')) === '') {
             missingFeatureInFile.push(test.file);