codeceptjs 3.7.6-beta.3 → 3.7.6

package/lib/codecept.js

@@ -1,6 +1,5 @@
 const { existsSync, readFileSync } = require('fs')
 const { globSync } = require('glob')
-const shuffle = require('lodash.shuffle')
 const fsPath = require('path')
 const { resolve } = require('path')
 
@@ -185,8 +184,10 @@ class Codecept {
       }
     }
 
+    this.testFiles.sort()
+
     if (this.opts.shuffle) {
-      this.testFiles = shuffle(this.testFiles)
+      this.testFiles = this.shuffle(this.testFiles)
     }
 
     if (this.opts.shard) {
@@ -194,6 +195,18 @@ class Codecept {
     }
   }
 
+  /**
+   * Fisher–Yates shuffle (non-mutating)
+   */
+  shuffle(array) {
+    const arr = [...array] // clone to avoid mutating input
+    for (let i = arr.length - 1; i > 0; i--) {
+      const j = Math.floor(Math.random() * (i + 1))
+      ;[arr[i], arr[j]] = [arr[j], arr[i]] // swap
+    }
+    return arr
+  }
+
   /**
    * Apply sharding to test files based on shard configuration
    *

package/lib/helper/JSONResponse.js

@@ -72,11 +72,11 @@ class JSONResponse extends Helper {
     if (!this.helpers[this.options.requestHelper]) {
       throw new Error(`Error setting JSONResponse, helper ${this.options.requestHelper} is not enabled in config, helpers: ${Object.keys(this.helpers)}`)
     }
-    const origOnResponse = this.helpers[this.options.requestHelper].config.onResponse;
+    const origOnResponse = this.helpers[this.options.requestHelper].config.onResponse
     this.helpers[this.options.requestHelper].config.onResponse = response => {
-      this.response = response;
-      if (typeof origOnResponse === 'function') origOnResponse(response);
-    };
+      this.response = response
+      if (typeof origOnResponse === 'function') origOnResponse(response)
+    }
   }
 
   _before() {

package/lib/helper/Playwright.js

@@ -1876,11 +1876,15 @@ class Playwright extends Helper {
    * {{> type }}
    */
   async type(keys, delay = null) {
+    // Always use page.keyboard.type for any string (including single character and national characters).
     if (!Array.isArray(keys)) {
       keys = keys.toString()
-      keys = keys.split('')
+      const typeDelay = typeof delay === 'number' ? delay : this.options.pressKeyDelay
+      await this.page.keyboard.type(keys, { delay: typeDelay })
+      return
     }
 
+    // For array input, treat each as a key press to keep working combinations such as ['Control', 'A'] or ['T', 'e', 's', 't'].
     for (const key of keys) {
       await this.page.keyboard.press(key)
       if (delay) await this.wait(delay / 1000)

package/lib/helper/WebDriver.js

@@ -998,7 +998,7 @@ class WebDriver extends Helper {
    * {{ react }}
    */
   async click(locator, context = null) {
-  const clickMethod = this.browser.isMobile && this.browser.capabilities.platformName !== 'android' ? 'touchClick' : 'elementClick'
+    const clickMethod = this.browser.isMobile && this.browser.capabilities.platformName !== 'android' ? 'touchClick' : 'elementClick'
     const locateFn = prepareLocateFn.call(this, context)
 
     const res = await findClickable.call(this, locator, locateFn)
@@ -1217,7 +1217,7 @@ class WebDriver extends Helper {
    * {{> checkOption }}
    */
   async checkOption(field, context = null) {
-  const clickMethod = this.browser.isMobile && this.browser.capabilities.platformName !== 'android' ? 'touchClick' : 'elementClick'
+    const clickMethod = this.browser.isMobile && this.browser.capabilities.platformName !== 'android' ? 'touchClick' : 'elementClick'
     const locateFn = prepareLocateFn.call(this, context)
 
     const res = await findCheckable.call(this, field, locateFn)
@@ -1237,7 +1237,7 @@ class WebDriver extends Helper {
    * {{> uncheckOption }}
    */
   async uncheckOption(field, context = null) {
-  const clickMethod = this.browser.isMobile && this.browser.capabilities.platformName !== 'android' ? 'touchClick' : 'elementClick'
+    const clickMethod = this.browser.isMobile && this.browser.capabilities.platformName !== 'android' ? 'touchClick' : 'elementClick'
     const locateFn = prepareLocateFn.call(this, context)
 
     const res = await findCheckable.call(this, field, locateFn)

package/lib/listener/globalTimeout.js

@@ -20,14 +20,29 @@ module.exports = function () {
 
   // disable timeout for BeforeSuite/AfterSuite hooks
   // add separate configs to them?
+  // When a BeforeSuite/AfterSuite hook starts we want to disable the
+  // per-test timeout during that hook execution only. Previously the
+  // code cleared `suiteTimeout` permanently which caused the suite
+  // level timeout to be lost for subsequent tests. Save previous
+  // values and restore them when the hook finishes.
+  let __prevTimeout = undefined
+  let __prevSuiteTimeout = undefined
+
   event.dispatcher.on(event.hook.started, hook => {
-    if (hook instanceof BeforeSuiteHook) {
+    if (hook instanceof BeforeSuiteHook || hook instanceof AfterSuiteHook) {
+      __prevTimeout = timeout
+      // copy array to preserve original values
+      __prevSuiteTimeout = suiteTimeout.slice()
       timeout = null
       suiteTimeout = []
     }
-    if (hook instanceof AfterSuiteHook) {
-      timeout = null
-      suiteTimeout = []
+  })
+
+  event.dispatcher.on(event.hook.finished, hook => {
+    if (hook instanceof BeforeSuiteHook || hook instanceof AfterSuiteHook) {
+      // restore previously stored values
+      timeout = __prevTimeout
+      suiteTimeout = __prevSuiteTimeout.slice()
     }
   })
 

package/lib/listener/steps.js

@@ -79,14 +79,14 @@ module.exports = function () {
       return currentHook.steps.push(step)
     }
     if (!currentTest || !currentTest.steps) return
-    
+
     // Check if we're in a session that should be excluded from main test steps
     const currentSessionId = recorder.getCurrentSessionId()
     if (currentSessionId && EXCLUDED_SESSIONS.includes(currentSessionId)) {
       // Skip adding this step to the main test steps
       return
     }
-    
+
     currentTest.steps.push(step)
   })
 

package/lib/mocha/test.js

@@ -153,14 +153,16 @@ function cloneTest(test) {
 function testToFileName(test, { suffix = '', unique = false } = {}) {
   let fileName = test.title
 
-  if (unique) fileName = `${fileName}_${test?.uid || Math.floor(new Date().getTime() / 1000)}`
-  if (suffix) fileName = `${fileName}_${suffix}`
   // remove tags with empty string (disable for now)
   // fileName = fileName.replace(/\@\w+/g, '')
   fileName = fileName.slice(0, 100)
   if (fileName.indexOf('{') !== -1) {
     fileName = fileName.substr(0, fileName.indexOf('{') - 3).trim()
   }
+
+  // Apply unique suffix AFTER removing data part to ensure uniqueness
+  if (unique) fileName = `${fileName}_${test?.uid || Math.floor(new Date().getTime())}`
+  if (suffix) fileName = `${fileName}_${suffix}`
   if (test.ctx && test.ctx.test && test.ctx.test.type === 'hook') fileName = clearString(`${test.title}_${test.ctx.test.title}`)
   // TODO: add suite title to file name
   // if (test.parent && test.parent.title) {

package/lib/output.js

@@ -222,12 +222,10 @@ module.exports = {
     /**
      * @param {Mocha.Test} test
      */
-
     started(test) {
       if (outputLevel < 1) return
       print(`  ${colors.dim.bold('Scenario()')}`)
     },
-
     /**
      * @param {Mocha.Test} test
      */
@@ -273,10 +271,12 @@ module.exports = {
   },
 
   /**
+   * Prints the stats of a test run to the console.
    * @param {number} passed
    * @param {number} failed
    * @param {number} skipped
    * @param {number|string} duration
+   * @param {number} [failedHooks]
    */
   result(passed, failed, skipped, duration, failedHooks = 0) {
     let style = colors.bgGreen