codeceptjs 4.0.0-rc.23 → 4.0.0-rc.24

package/lib/command/run-multiple.js

@@ -138,9 +138,8 @@ function executeRun(runName, runConfig) {
 
   outputDir = clearString(outputDir)
 
-  // tweaking default output directories and for mochawesome
+  // tweaking default output directories
   overriddenConfig = replaceValueDeep(overriddenConfig, 'output', path.join(config.output, outputDir))
-  overriddenConfig = replaceValueDeep(overriddenConfig, 'reportDir', path.join(config.output, outputDir))
   overriddenConfig = replaceValueDeep(overriddenConfig, 'mochaFile', path.join(config.output, outputDir, `${browserName}_report.xml`))
 
   // override tests configuration

package/lib/helper/Playwright.js

@@ -36,7 +36,7 @@ import MultipleElementsFound from './errors/MultipleElementsFound.js'
 import RemoteBrowserConnectionRefused from './errors/RemoteBrowserConnectionRefused.js'
 import Popup from './extras/Popup.js'
 import Console from './extras/Console.js'
-import { findReact, findByPlaywrightLocator } from './extras/PlaywrightReactVueLocator.js'
+import { findByPlaywrightLocator } from './extras/PlaywrightLocator.js'
 import { dropFile } from './scripts/dropFile.js'
 import WebElement from '../element/WebElement.js'
 import { selectElement } from './extras/elementSelection.js'
@@ -2081,15 +2081,6 @@ class Playwright extends Helper {
     return proceedClick.call(this, locator, context, options)
   }
 
-  /**
-   * Clicks link and waits for navigation (deprecated)
-   */
-  async clickLink(locator, context = null) {
-    console.log('clickLink deprecated: Playwright automatically waits for navigation to happen.')
-    console.log('Replace I.clickLink with I.click')
-    return this.click(locator, context)
-  }
-
   /**
    * {{> forceClick }}
    */
@@ -3306,8 +3297,6 @@ class Playwright extends Helper {
   }
 
   /**
-   * This method accepts [React selectors](https://codecept.io/react).
-   *
    * {{> waitForVisible }}
    */
   async waitForVisible(locator, sec) {
@@ -4231,10 +4220,8 @@ async function findByRole(context, locator) {
 }
 
 async function findElements(matcher, locator) {
-  const isReactLocator = locator.type === 'react' || (locator.locator && locator.locator.react) || locator.react
   const isPwLocator = locator.type === 'pw' || (locator.locator && locator.locator.pw) || locator.pw
 
-  if (isReactLocator) return findReact(matcher, locator)
   if (isPwLocator) return findByPlaywrightLocator.call(this, matcher, locator)
 
   // Handle role locators with text/exact options (e.g., {role: 'button', text: 'Submit', exact: true})
@@ -4249,7 +4236,6 @@ async function findElements(matcher, locator) {
 }
 
 async function findElement(matcher, locator) {
-  if (locator.react) return findReact(matcher, locator)
   if (locator.pw) return findByPlaywrightLocator.call(this, matcher, locator)
 
   locator = new Locator(locator, 'css')

package/lib/helper/Puppeteer.js

@@ -821,7 +821,6 @@ class Puppeteer extends Helper {
 
   /**
    * {{> moveCursorTo }}
-   * {{ react }}
    */
   async moveCursorTo(locator, offsetX = 0, offsetY = 0) {
     let context = null
@@ -992,7 +991,6 @@ class Puppeteer extends Helper {
    * const elements = await this.helpers['Puppeteer']._locate({name: 'password'});
    * ```
    *
-   * {{ react }}
    */
   async _locate(locator) {
     const context = await this.context
@@ -1007,7 +1005,6 @@ class Puppeteer extends Helper {
    * const element = await this.helpers['Puppeteer']._locateElement({name: 'password'});
    * ```
    *
-   * {{ react }}
    */
   async _locateElement(locator) {
     const context = await this.context
@@ -1191,7 +1188,6 @@ class Puppeteer extends Helper {
 
   /**
    * {{> seeElement }}
-   * {{ react }}
    */
   async seeElement(locator, context = null) {
     let els
@@ -1215,7 +1211,6 @@ class Puppeteer extends Helper {
 
   /**
    * {{> dontSeeElement }}
-   * {{ react }}
    */
   async dontSeeElement(locator, context = null) {
     let els
@@ -1264,7 +1259,6 @@ class Puppeteer extends Helper {
   /**
    * {{> click }}
    *
-   * {{ react }}
    */
   async click(locator = '//body', context = null) {
     return proceedClick.call(this, locator, context)
@@ -1273,7 +1267,6 @@ class Puppeteer extends Helper {
   /**
    * {{> forceClick }}
    *
-   * {{ react }}
    */
   async forceClick(locator, context = null) {
     let matcher = await this.context
@@ -1303,7 +1296,6 @@ class Puppeteer extends Helper {
   /**
    * {{> clickLink }}
    *
-   * {{ react }}
    */
   async clickLink(locator, context = null) {
     return proceedClick.call(this, locator, context, { waitForNavigation: true })
@@ -1411,7 +1403,6 @@ class Puppeteer extends Helper {
   /**
    * {{> doubleClick }}
    *
-   * {{ react }}
    */
   async doubleClick(locator, context = null) {
     return proceedClick.call(this, locator, context, { clickCount: 2 })
@@ -1420,7 +1411,6 @@ class Puppeteer extends Helper {
   /**
    * {{> rightClick }}
    *
-   * {{ react }}
    */
   async rightClick(locator, context = null) {
     return proceedClick.call(this, locator, context, { button: 'right' })
@@ -1593,7 +1583,6 @@ class Puppeteer extends Helper {
 
   /**
    * {{> fillField }}
-   * {{ react }}
    */
   async fillField(field, value, context = null) {
     let els = await findVisibleFields.call(this, field, context)
@@ -1632,7 +1621,6 @@ class Puppeteer extends Helper {
   /**
    * {{> appendField }}
    *
-   * {{ react }}
    */
   async appendField(field, value, context = null) {
     const els = await findVisibleFields.call(this, field, context)
@@ -1734,7 +1722,6 @@ class Puppeteer extends Helper {
 
   /**
    * {{> grabNumberOfVisibleElements }}
-   * {{ react }}
    */
   async grabNumberOfVisibleElements(locator) {
     let els = await this._locate(locator)
@@ -1796,7 +1783,6 @@ class Puppeteer extends Helper {
   /**
    * {{> see }}
    *
-   * {{ react }}
    */
   async see(text, context = null) {
     return proceedSee.call(this, 'assert', text, context)
@@ -1812,7 +1798,6 @@ class Puppeteer extends Helper {
   /**
    * {{> dontSee }}
    *
-   * {{ react }}
    */
   async dontSee(text, context = null) {
     return proceedSee.call(this, 'negate', text, context)
@@ -1866,7 +1851,6 @@ class Puppeteer extends Helper {
   /**
    * {{> seeNumberOfElements }}
    *
-   * {{ react }}
    */
   async seeNumberOfElements(locator, num) {
     const elements = await this._locate(locator)
@@ -1876,7 +1860,6 @@ class Puppeteer extends Helper {
   /**
    * {{> seeNumberOfVisibleElements }}
    *
-   * {{ react }}
    */
   async seeNumberOfVisibleElements(locator, num) {
     const res = await this.grabNumberOfVisibleElements(locator)
@@ -2003,7 +1986,6 @@ class Puppeteer extends Helper {
 
   /**
    * {{> grabTextFromAll }}
-   * {{ react }}
    */
   async grabTextFromAll(locator) {
     const els = await this._locate(locator)
@@ -2016,7 +1998,6 @@ class Puppeteer extends Helper {
 
   /**
    * {{> grabTextFrom }}
-   * {{ react }}
    */
   async grabTextFrom(locator) {
     const texts = await this.grabTextFromAll(locator)
@@ -2077,7 +2058,6 @@ class Puppeteer extends Helper {
 
   /**
    * {{> grabCssPropertyFromAll }}
-   * {{ react }}
    */
   async grabCssPropertyFromAll(locator, cssProperty) {
     const els = await this._locate(locator)
@@ -2089,7 +2069,6 @@ class Puppeteer extends Helper {
 
   /**
    * {{> grabCssPropertyFrom }}
-   * {{ react }}
    */
   async grabCssPropertyFrom(locator, cssProperty) {
     const cssValues = await this.grabCssPropertyFromAll(locator, cssProperty)
@@ -2104,7 +2083,6 @@ class Puppeteer extends Helper {
 
   /**
    * {{> seeCssPropertiesOnElements }}
-   * {{ react }}
    */
   async seeCssPropertiesOnElements(locator, cssProperties) {
     const res = await this._locate(locator)
@@ -2139,7 +2117,6 @@ class Puppeteer extends Helper {
 
   /**
    * {{> seeAttributesOnElements }}
-   * {{ react }}
    */
   async seeAttributesOnElements(locator, attributes) {
     const elements = await this._locate(locator)
@@ -2177,7 +2154,6 @@ class Puppeteer extends Helper {
 
   /**
    * {{> dragSlider }}
-   * {{ react }}
    */
   async dragSlider(locator, offsetX = 0) {
     const src = await this._locate(locator)
@@ -2199,7 +2175,6 @@ class Puppeteer extends Helper {
 
   /**
    * {{> grabAttributeFromAll }}
-   * {{ react }}
    */
   async grabAttributeFromAll(locator, attr) {
     const els = await this._locate(locator)
@@ -2213,7 +2188,6 @@ class Puppeteer extends Helper {
 
   /**
    * {{> grabAttributeFrom }}
-   * {{ react }}
    */
   async grabAttributeFrom(locator, attr) {
     const attrs = await this.grabAttributeFromAll(locator, attr)
@@ -2376,7 +2350,6 @@ class Puppeteer extends Helper {
 
   /**
    * {{> waitNumberOfVisibleElements }}
-   * {{ react }}
    */
   async waitNumberOfVisibleElements(locator, num, sec) {
     const waitTimeout = sec ? sec * 1000 : this.options.waitForTimeout
@@ -2424,7 +2397,6 @@ class Puppeteer extends Helper {
 
   /**
    * {{> waitForElement }}
-   * {{ react }}
    */
   async waitForElement(locator, sec) {
     const waitTimeout = sec ? sec * 1000 : this.options.waitForTimeout
@@ -2445,7 +2417,6 @@ class Puppeteer extends Helper {
   /**
    * {{> waitForVisible }}
    *
-   * {{ react }}
    */
   async waitForVisible(locator, sec) {
     const waitTimeout = sec ? sec * 1000 : this.options.waitForTimeout
@@ -3022,10 +2993,6 @@ export default Puppeteer
  * @returns {Promise<Array>} Array of ElementHandle objects
  */
 async function findElements(matcher, locator) {
-  // Check if locator is a Locator object with react type, or a raw object with react property
-  const isReactLocator = locator.type === 'react' || (locator.locator && locator.locator.react) || locator.react
-  if (isReactLocator) return findReactElements.call(this, locator)
-  
   locator = new Locator(locator, 'css')
 
   // Check if locator is a role locator and call findByRole
@@ -3092,7 +3059,6 @@ async function findElements(matcher, locator) {
  * @returns {Promise<Object>} Single ElementHandle object
  */
 async function findElement(matcher, locator) {
-  if (locator.react) return findReactElements.call(this, locator)
   locator = new Locator(locator, 'css')
 
   // Check if locator is a role locator and call findByRole
@@ -3591,75 +3557,6 @@ function _waitForElement(locator, options) {
   }
 }
 
-async function findReactElements(locator) {
-  // Handle both Locator objects and raw locator objects
-  const resolved = locator.locator ? locator.locator : toLocatorConfig(locator, 'react')
-  this.debug(`Finding React elements: ${JSON.stringify(resolved)}`)
-
-  // Use createRequire to access require.resolve in ESM
-  const { createRequire } = await import('module')
-  const require = createRequire(import.meta.url)
-  const resqScript = await fs.promises.readFile(require.resolve('resq'), 'utf-8')
-  await this.page.evaluate(resqScript.toString())
-
-  await this.page.evaluate(() => window.resq.waitToLoadReact())
-  const arrayHandle = await this.page.evaluateHandle(
-    obj => {
-      const { selector, props, state } = obj
-      let elements = window.resq.resq$$(selector)
-      if (Object.keys(props).length) {
-        elements = elements.byProps(props)
-      }
-      if (Object.keys(state).length) {
-        elements = elements.byState(state)
-      }
-
-      if (!elements.length) {
-        return []
-      }
-
-      // 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 = []
-
-      elements.forEach(element => {
-        let { node, isFragment } = element
-
-        if (!node) {
-          isFragment = true
-          node = element.children
-        }
-
-        if (isFragment) {
-          nodes = nodes.concat(node)
-        } else {
-          nodes.push(node)
-        }
-      })
-
-      return [...nodes]
-    },
-    {
-      selector: resolved.react,
-      props: resolved.props || {},
-      state: resolved.state || {},
-    },
-  )
-
-  const properties = await arrayHandle.getProperties()
-  const result = []
-  for (const property of properties.values()) {
-    const elementHandle = property.asElement()
-    if (elementHandle) {
-      result.push(elementHandle)
-    }
-  }
-
-  await arrayHandle.dispose()
-  return result
-}
-
 async function findByRole(matcher, locator) {
   const resolved = toLocatorConfig(locator, 'role')
   const roleSelector = buildRoleSelector(resolved)

package/lib/helper/WebDriver.js

@@ -97,11 +97,7 @@ const config = {}
  * WebDriver helper which wraps [webdriverio](http://webdriver.io/) library to
  * manipulate browser using Selenium WebDriver or PhantomJS.
  *
- * WebDriver requires Selenium Server and ChromeDriver/GeckoDriver to be installed. Those tools can be easily installed via NPM. Please check [Testing with WebDriver](https://codecept.io/webdriver/#testing-with-webdriver) for more details.
- *
- * With the release of WebdriverIO version v8.14.0, and onwards, all driver management hassles are now a thing of the past 🙌. Read more [here](https://webdriver.io/blog/2023/07/31/driver-management/).
- * One of the significant advantages of this update is that you can now get rid of any driver services you previously had to manage, such as
- * `wdio-chromedriver-service`, `wdio-geckodriver-service`, `wdio-edgedriver-service`, `wdio-safaridriver-service`, and even `@wdio/selenium-standalone-service`.
+ * No Selenium Server, ChromeDriver, or GeckoDriver to install or start. Since WebdriverIO 9, driver management is fully automatic — WebdriverIO downloads and starts the matching driver for you. Read more [here](https://webdriver.io/blog/2023/07/31/driver-management/). Please check [Testing with WebDriver](https://codecept.io/webdriver/#testing-with-webdriver) for more details.
  *
  * For those who require custom driver options, fear not; WebDriver Helper allows you to pass in driver options through custom WebDriver configuration.
  * If you have a custom grid, use a cloud service, or prefer to run your own driver, there's no need to worry since WebDriver Helper will only start a driver when there are no other connection information settings like hostname or port specified.
@@ -908,13 +904,6 @@ class WebDriver extends Helper {
       return els
     }
 
-    // special locator type for React
-    if (locator.react) {
-      const els = await this.browser.react$$(locator.react, locator.props || undefined, locator.state || undefined)
-      this.debugSection('Elements', `Found ${els.length} react components`)
-      return els
-    }
-
     // special locator type for ARIA roles
     if (locator.role) {
       return this._locateByRole(locator)
@@ -1084,7 +1073,6 @@ class WebDriver extends Helper {
   /**
    * {{> click }}
    *
-   * {{ react }}
    */
   async click(locator, context = null) {
     const clickMethod = this.browser.isMobile && this.browser.capabilities.platformName !== 'android' ? 'touchClick' : 'elementClick'
@@ -1104,7 +1092,6 @@ class WebDriver extends Helper {
   /**
    * {{> forceClick }}
    *
-   * {{ react }}
    */
   async forceClick(locator, context = null) {
     const locateFn = prepareLocateFn.call(this, context)
@@ -1131,7 +1118,6 @@ class WebDriver extends Helper {
   /**
    * {{> doubleClick }}
    *
-   * {{ react }}
    */
   async doubleClick(locator, context = null) {
     const locateFn = prepareLocateFn.call(this, context)
@@ -1151,7 +1137,6 @@ class WebDriver extends Helper {
   /**
    * {{> rightClick }}
    *
-   * {{ react }}
    */
   async rightClick(locator, context) {
     const locateFn = prepareLocateFn.call(this, context)
@@ -1246,7 +1231,6 @@ class WebDriver extends Helper {
   /**
    * {{> forceRightClick }}
    *
-   * {{ react }}
    */
   async forceRightClick(locator, context = null) {
     const locateFn = prepareLocateFn.call(this, context)
@@ -1271,7 +1255,6 @@ class WebDriver extends Helper {
 
   /**
    * {{> fillField }}
-   * {{ react }}
    * {{ custom }}
    *
    */
@@ -1301,7 +1284,6 @@ class WebDriver extends Helper {
 
   /**
    * {{> appendField }}
-   * {{ react }}
    */
   async appendField(field, value, context = null) {
     const res = await findFields.call(this, field, context)
@@ -1599,7 +1581,6 @@ class WebDriver extends Helper {
   /**
    * {{> see }}
    *
-   * {{ react }}
    */
   async see(text, context = null) {
     return proceedSee.call(this, 'assert', text, context)
@@ -1615,7 +1596,6 @@ class WebDriver extends Helper {
   /**
    * {{> dontSee }}
    *
-   * {{ react }}
    */
   async dontSee(text, context = null) {
     return proceedSee.call(this, 'negate', text, context)
@@ -1657,7 +1637,6 @@ class WebDriver extends Helper {
 
   /**
    * {{> seeElement }}
-   * {{ react }}
    *
    */
   async seeElement(locator, context = null) {
@@ -1674,7 +1653,6 @@ class WebDriver extends Helper {
 
   /**
    * {{> dontSeeElement }}
-   * {{ react }}
    */
   async dontSeeElement(locator, context = null) {
     const locateFn = prepareLocateFn.call(this, context)
@@ -1759,7 +1737,6 @@ class WebDriver extends Helper {
 
   /**
    * {{> seeNumberOfElements }}
-   * {{ react }}
    */
   async seeNumberOfElements(locator, num) {
     const res = await this._locate(locator)
@@ -1768,7 +1745,6 @@ class WebDriver extends Helper {
 
   /**
    * {{> seeNumberOfVisibleElements }}
-   * {{ react }}
    */
   async seeNumberOfVisibleElements(locator, num) {
     const res = await this.grabNumberOfVisibleElements(locator)
@@ -3501,9 +3477,6 @@ function prepareLocateFn(context) {
     l = new Locator(l, 'css')
     return this._locate(context, true).then(async res => {
       assertElementExists(res, context, 'Context element')
-      if (l.react) {
-        return res[0].react$$(l.react, l.props || undefined)
-      }
       return res[0].$$(l.simplify())
     })
   }

package/lib/helper/extras/PlaywrightLocator.js

@@ -0,0 +1,10 @@
+async function findByPlaywrightLocator(matcher, locator) {
+  const pwLocator = locator.locator || locator
+  if (pwLocator && pwLocator.toString && pwLocator.toString().includes(process.env.testIdAttribute)) {
+    return matcher.getByTestId(pwLocator.pw.value.split('=')[1])
+  }
+  const pwValue = typeof pwLocator.pw === 'string' ? pwLocator.pw : pwLocator.pw
+  return matcher.locator(pwValue).all()
+}
+
+export { findByPlaywrightLocator }

package/lib/locator.js

@@ -58,9 +58,6 @@ class Locator {
     if (isShadow(locator)) {
       this.type = 'shadow'
     }
-    if (isPlaywrightLocator(locator)) {
-      this.type = 'pw'
-    }
 
     Locator.filters.forEach(f => f(locator, this))
   }
@@ -753,16 +750,6 @@ function removePrefix(xpath) {
   return xpath.replace(/^(\.|\/)+/, '')
 }
 
-/**
- * @private
- * check if the locator is a Playwright locator
- * @param {string} locator
- * @returns {boolean}
- */
-function isPlaywrightLocator(locator) {
-  return locator.includes('_react') || locator.includes('_vue')
-}
-
 /**
  * @private
  * check if the locator is a role locator

package/lib/plugin/analyze.js

@@ -155,10 +155,9 @@ const defaultConfig = {
       if (config.vision && test.artifacts.screenshot) {
         debug('Adding screenshot to prompt')
         messages[0].content.push({
-          type: 'image_url',
-          image_url: {
-            url: 'data:image/png;base64,' + base64EncodeFile(test.artifacts.screenshot),
-          },
+          type: 'image',
+          image: base64EncodeFile(test.artifacts.screenshot),
+          mediaType: 'image/png',
         })
       }
 

package/lib/plugin/pauseOnFail.js

@@ -4,7 +4,9 @@ import pause from './pause.js'
 let warned = false
 
 /**
- * @deprecated Use the `pause` plugin with `on: 'fail'` (the default).
+ * Starts an interactive pause when a test fails.
+ *
+ * **Deprecated:** use the `pause` plugin with `on: 'fail'`, which is the default behavior.
  */
 export default function (config = {}) {
   if (!warned) {

package/lib/plugin/retryFailedStep.js

@@ -8,7 +8,10 @@ const debug = debugModule('codeceptjs:retryFailedStep')
 const defaultConfig = {
   retries: 3,
   defaultIgnoredSteps: ['amOnPage', 'wait*', 'send*', 'execute*', 'run*', 'have*'],
+  minTimeout: 150,
+  maxTimeout: 10000,
   factor: 1.5,
+  randomize: false,
   ignoredSteps: [],
   deferToScenarioRetries: true,
 }
@@ -44,10 +47,9 @@ const RETRY_PRIORITIES = {
  * #### Configuration:
  *
  * * `retries` - number of retries (by default 3),
- * * `when` - function, when to perform a retry (accepts error as parameter)
  * * `factor` - The exponential factor to use. Default is 1.5.
- * * `minTimeout` - The number of milliseconds before starting the first retry. Default is 1000.
- * * `maxTimeout` - The maximum number of milliseconds between two retries. Default is Infinity.
+ * * `minTimeout` - The number of milliseconds before starting the first retry. Default is 150.
+ * * `maxTimeout` - The maximum number of milliseconds between two retries. Default is 10000.
  * * `randomize` - Randomizes the timeouts by multiplying with a factor from 1 to 2. Default is false.
  * * `defaultIgnoredSteps` - an array of steps to be ignored for retry. Includes:
  *     * `amOnPage`
@@ -77,7 +79,7 @@ const RETRY_PRIORITIES = {
  *
  * #### Disable Per Test
  *
- * This plugin can be disabled per test. In this case you will need to stet `I.retry()` to all flaky steps:
+ * This plugin can be disabled per test. In this case you will need to add `step.retry()` to all flaky steps:
  *
  * Use scenario configuration to disable plugin for a test
  *
@@ -89,9 +91,8 @@ const RETRY_PRIORITIES = {
  *
  */
 export default function (config) {
-  config = Object.assign(defaultConfig, config)
+  config = Object.assign({}, defaultConfig, config)
   config.ignoredSteps = config.ignoredSteps.concat(config.defaultIgnoredSteps)
-  const customWhen = config.when
 
   let enableRetry = false
 
@@ -101,7 +102,6 @@ export default function (config) {
     if (!store.autoRetries) return false
     if (err && err.isTerminal) return false
     if (err && err.message && (err.message.includes('ERR_ABORTED') || err.message.includes('frame was detached') || err.message.includes('Target page, context or browser has been closed'))) return false
-    if (customWhen) return customWhen(err)
     return true
   }
   config.when = when

package/lib/plugin/screenshot.js

@@ -86,14 +86,9 @@ export default function (config = {}) {
   const trigger = resolveTrigger(cliArgs, config, { on: defaultConfig.on }, { name: 'screenshot' })
   if (!trigger) return
 
-  const helpers = Container.helpers()
   const options = Object.assign({}, defaultConfig, helper.options, config)
   options.slides = cliArgs.slides ?? config.slides ?? defaultConfig.slides
 
-  if (helpers.Mochawesome?.config) {
-    options.uniqueScreenshotNames = helpers.Mochawesome.config.uniqueScreenshotNames
-  }
-
   if (Codeceptjs.container.mocha()) {
     options.reportDir = Codeceptjs.container.mocha()?.options?.reporterOptions
       && Codeceptjs.container.mocha()?.options?.reporterOptions?.reportDir

package/lib/plugin/screenshotOnFail.js

@@ -4,7 +4,9 @@ import screenshot from './screenshot.js'
 let warned = false
 
 /**
- * @deprecated Use the `screenshot` plugin with `on: 'fail'` (the default).
+ * Saves a screenshot when a test fails.
+ *
+ * **Deprecated:** use the `screenshot` plugin with `on: 'fail'`, which is the default behavior.
  */
 export default function (config = {}) {
   if (!warned) {

package/lib/plugin/stepTimeout.js

@@ -32,7 +32,7 @@ const defaultConfig = {
  * #### Configuration:
  *
  * * `timeout` - global step timeout, default 150 seconds
- * * `overrideStepLimits` - whether to use timeouts set in plugin config to override step timeouts set in code with I.limitTime(x).action(...), default false
+ * * `overrideStepLimits` - whether to use timeouts set in plugin config to override step timeouts set in code with `I.action(..., step.timeout(x))`, default false
  * * `noTimeoutSteps` - an array of steps with no timeout. Default:
  *     * `amOnPage`
  *     * `wait*`

package/lib/recorder.js

@@ -217,7 +217,7 @@ export default {
       }
 
       const retryRules = this.retries.slice().reverse()
-      return promiseRetry(Object.assign(defaultRetryOptions, retryOpts), (retry, number) => {
+      return promiseRetry(Object.assign({}, defaultRetryOptions, retryOpts), (retry, number) => {
         if (number > 1) output.log(`${currentQueue()}Retrying... Attempt #${number}`)
         const [promise, timer] = getTimeoutPromise(timeout, taskName)
         return Promise.race([promise, Promise.resolve(res).then(fn)])