Merge branch '3.x' into refactor/mocha-hooks

Author: DavertMik

bin/codecept.js

@@ -32,7 +32,7 @@ const commandFlags = {
 }
 
 const errorHandler =
-  (fn) =>
+  fn =>
   async (...args) => {
     try {
       await fn(...args)
@@ -286,7 +286,7 @@ program
 
   .action(require('../lib/command/run-rerun'))
 
-program.on('command:*', (cmd) => {
+program.on('command:*', cmd => {
   console.log(`\nUnknown command ${cmd}\n`)
   program.outputHelp()
 })

docs/helpers/ApiDataFactory.md

@@ -146,7 +146,7 @@ delete: (id) => ({ method: 'delete', url: '/posts', data: { id } })
 Requests can be updated on the fly by using `onRequest` function. For instance, you can pass in current session from a cookie.
 
 ```js
-onRequest: async (request) => {
+onRequest: async request => {
   // using global codeceptjs instance
   let cookie = await codeceptjs.container.helpers('WebDriver').grabCookie('session')
   request.headers = { Cookie: `session=${cookie.value}` }

docs/helpers/Appium.md

@@ -183,7 +183,7 @@ Also capabilities can be checked by a function.
 
 ```js
 I.runOnAndroid(
-  (caps) => {
+  caps => {
     // caps is current config of desiredCapabiliites
     return caps.platformVersion >= 6
   },
@@ -222,7 +222,7 @@ In this case, code will be executed only on Android >= 6.
 
 ```js
 I.runOnAndroid(
-  (caps) => {
+  caps => {
     // caps is current config of desiredCapabiliites
     return caps.platformVersion >= 6
   },

docs/helpers/GraphQLDataFactory.md

@@ -58,7 +58,7 @@ 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.
-module.exports = new Factory((buildObj) => ({
+module.exports = new Factory(buildObj => ({
   input: { ...buildObj },
 }))
   // 'attr'-id can be left out depending on the GraphQl resolvers
@@ -137,7 +137,7 @@ Each operation must have the following:
 Requests can be updated on the fly by using `onRequest` function. For instance, you can pass in current session from a cookie.
 
 ```js
-onRequest: async (request) => {
+onRequest: async request => {
   // using global codeceptjs instance
   let cookie = await codeceptjs.container.helpers('WebDriver').grabCookie('session')
   request.headers = { Cookie: `session=${cookie.value}` }

docs/helpers/JSONResponse.md

@@ -227,13 +227,14 @@ I.seeResponseMatchesJsonSchema(joi.object({
 
 ### seeResponseValidByCallback
 
-Executes a callback function passing in `response` object and chai assertions with `expect`
+Executes a callback function passing in `response` object and assert
 Use it to perform custom checks of response data
 
 ```js
-I.seeResponseValidByCallback(({ data, status, expect }) => {
-  expect(status).to.eql(200)
-  expect(data).keys.to.include(['user', 'company'])
+I.seeResponseValidByCallback(({ data, status }) => {
+  assert.strictEqual(status, 200)
+  assert('user' in data)
+  assert('company' in data)
 })
 ```
 

docs/helpers/MockRequest.md

@@ -184,7 +184,7 @@ Use `I.mockServer()` to customize which requests should be recorded and under wh
 
 ```js
 I.startMocking()
-I.mockServer((server) => {
+I.mockServer(server => {
   // mock request only from ap1.com and api2.com and
   // store recording into two different files
   server.any('https://api1.com/*').passthrough(false).recordingName('api1')
@@ -243,7 +243,7 @@ I.mockRequest('GET', ['/secrets', '/v2/secrets'], 403)
 Use PollyJS [Server Routes API][12] to declare mocks via callback function:
 
 ```js
-I.mockServer((server) => {
+I.mockServer(server => {
   // basic usage
   server.get('/api/v2/users').intercept((req, res) => {
     res.sendStatus(200).json({ users })
@@ -258,7 +258,7 @@ In record replay mode you can define which routes should be recorded and where t
 
 ```js
 I.startMocking('mock')
-I.mockServer((server) => {
+I.mockServer(server => {
   // record requests from cdn1.com and save them to data/recording/xml
   server.any('https://cdn1.com/*').passthrough(false).recordingName('xml')
 

docs/helpers/Nightmare.md

@@ -148,7 +148,7 @@ if none provided clears all cookies.
 
 ```js
 I.clearCookie()
-I.clearCookie('test') // Playwright currently doesn't support clear a particular cookie name
+I.clearCookie('test')
 ```
 
 #### Parameters
@@ -1197,7 +1197,7 @@ I.waitForFunction(fn[, [args[, timeout]])
 ```js
 I.waitForFunction(() => window.requests == 0)
 I.waitForFunction(() => window.requests == 0, 5) // waits for 5 sec
-I.waitForFunction((count) => window.requests == count, [3], 5) // pass args and wait for 5 sec
+I.waitForFunction(count => window.requests == count, [3], 5) // pass args and wait for 5 sec
 ```
 
 #### Parameters

docs/helpers/Playwright.md

@@ -542,11 +542,12 @@ if none provided clears all cookies.
 
 ```js
 I.clearCookie()
-I.clearCookie('test') // Playwright currently doesn't support clear a particular cookie name
+I.clearCookie('test')
 ```
 
 #### Parameters
 
+- `cookieName`  
 - `cookie` **[string][9]?** (optional, `null` by default) cookie name
 
 ### clearField
@@ -1021,7 +1022,7 @@ Get JS log from browser.
 
 ```js
 const logs = await I.grabBrowserLogs()
-const errors = logs.map((l) => ({ type: l.type(), text: l.text() })).filter((l) => l.type === 'error')
+const errors = logs.map(l => ({ type: l.type(), text: l.text() })).filter(l => l.type === 'error')
 console.log(JSON.stringify(errors))
 ```
 
@@ -1499,7 +1500,7 @@ Returns **[Promise][22]<[object][6]>** response
 Mocks network request using [`browserContext.route`][30] of Playwright
 
 ```js
-I.mockRoute(/(.png$)|(.jpg$)/, (route) => route.abort())
+I.mockRoute(/(.png$)|(.jpg$)/, route => route.abort())
 ```
 
 This method allows intercepting and mocking requests & responses. [Learn more about it][31]
@@ -2495,7 +2496,7 @@ I.waitForFunction(fn[, [args[, timeout]])
 ```js
 I.waitForFunction(() => window.requests == 0)
 I.waitForFunction(() => window.requests == 0, 5) // waits for 5 sec
-I.waitForFunction((count) => window.requests == count, [3], 5) // pass args and wait for 5 sec
+I.waitForFunction(count => window.requests == count, [3], 5) // pass args and wait for 5 sec
 ```
 
 #### Parameters
@@ -2553,7 +2554,7 @@ Waits for a network request.
 
 ```js
 I.waitForRequest('http://example.com/resource')
-I.waitForRequest((request) => request.url() === 'http://example.com' && request.method() === 'GET')
+I.waitForRequest(request => request.url() === 'http://example.com' && request.method() === 'GET')
 ```
 
 #### Parameters
@@ -2567,7 +2568,7 @@ Waits for a network response.
 
 ```js
 I.waitForResponse('http://example.com/resource')
-I.waitForResponse((response) => response.url() === 'https://example.com' && response.status() === 200)
+I.waitForResponse(response => response.url() === 'https://example.com' && response.status() === 200)
 ```
 
 #### Parameters

docs/helpers/Protractor.md

@@ -267,7 +267,7 @@ if none provided clears all cookies.
 
 ```js
 I.clearCookie()
-I.clearCookie('test') // Playwright currently doesn't support clear a particular cookie name
+I.clearCookie('test')
 ```
 
 #### Parameters
@@ -1582,7 +1582,7 @@ I.waitForFunction(fn[, [args[, timeout]])
 ```js
 I.waitForFunction(() => window.requests == 0)
 I.waitForFunction(() => window.requests == 0, 5) // waits for 5 sec
-I.waitForFunction((count) => window.requests == count, [3], 5) // pass args and wait for 5 sec
+I.waitForFunction(count => window.requests == count, [3], 5) // pass args and wait for 5 sec
 ```
 
 #### Parameters

docs/helpers/Puppeteer.md

@@ -388,7 +388,7 @@ if none provided clears all cookies.
 
 ```js
 I.clearCookie()
-I.clearCookie('test') // Playwright currently doesn't support clear a particular cookie name
+I.clearCookie('test')
 ```
 
 #### Parameters
@@ -1270,7 +1270,7 @@ I.seeFile('avatar.jpg')
 Mocks network request using [`Request Interception`][19]
 
 ```js
-I.mockRoute(/(.png$)|(.jpg$)/, (route) => route.abort())
+I.mockRoute(/(.png$)|(.jpg$)/, route => route.abort())
 ```
 
 This method allows intercepting and mocking requests & responses. [Learn more about it][19]
@@ -2181,7 +2181,7 @@ I.waitForFunction(fn[, [args[, timeout]])
 ```js
 I.waitForFunction(() => window.requests == 0)
 I.waitForFunction(() => window.requests == 0, 5) // waits for 5 sec
-I.waitForFunction((count) => window.requests == count, [3], 5) // pass args and wait for 5 sec
+I.waitForFunction(count => window.requests == count, [3], 5) // pass args and wait for 5 sec
 ```
 
 #### Parameters
@@ -2239,7 +2239,7 @@ Waits for a network request.
 
 ```js
 I.waitForRequest('http://example.com/resource')
-I.waitForRequest((request) => request.url() === 'http://example.com' && request.method() === 'GET')
+I.waitForRequest(request => request.url() === 'http://example.com' && request.method() === 'GET')
 ```
 
 #### Parameters
@@ -2253,7 +2253,7 @@ Waits for a network response.
 
 ```js
 I.waitForResponse('http://example.com/resource')
-I.waitForResponse((response) => response.url() === 'http://example.com' && response.request().method() === 'GET')
+I.waitForResponse(response => response.url() === 'http://example.com' && response.request().method() === 'GET')
 ```
 
 #### Parameters