codeceptjs 3.7.6-beta.3 → 3.7.6-beta.4

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/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/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/plugin/htmlReporter.js

@@ -1,3 +1,7 @@
+// @ts-nocheck
+// TypeScript: Import Node.js types for process, fs, path, etc.
+/// <reference types="node" />
+
 const fs = require('fs')
 const path = require('path')
 const mkdirp = require('mkdirp')
@@ -11,7 +15,7 @@ const output = require('../output')
 const Codecept = require('../codecept')
 
 const defaultConfig = {
-  output: global.output_dir || './output',
+  output: typeof global !== 'undefined' && global.output_dir ? global.output_dir : './output',
   reportFileName: 'report.html',
   includeArtifacts: true,
   showSteps: true,
@@ -60,6 +64,9 @@ const defaultConfig = {
  */
 module.exports = function (config) {
   const options = { ...defaultConfig, ...config }
+  /**
+   * TypeScript: Explicitly type reportData arrays as any[] to avoid 'never' errors
+   */
   let reportData = {
     stats: {},
     tests: [],
@@ -82,8 +89,8 @@ module.exports = function (config) {
 
   // Track overall test execution
   event.dispatcher.on(event.all.before, () => {
-    reportData.startTime = new Date()
-    output.plugin('htmlReporter', 'Starting HTML report generation...')
+    reportData.startTime = new Date().toISOString()
+    output.print('HTML Reporter: Starting HTML report generation...')
   })
 
   // Track test start to initialize steps and hooks collection
@@ -138,10 +145,30 @@ module.exports = function (config) {
     if (step.htmlReporterStartTime) {
       step.duration = Date.now() - step.htmlReporterStartTime
     }
+
+    // Serialize args immediately to preserve them through worker serialization
+    let serializedArgs = []
+    if (step.args && Array.isArray(step.args)) {
+      serializedArgs = step.args.map(arg => {
+        try {
+          // Try to convert to JSON-friendly format
+          if (typeof arg === 'string') return arg
+          if (typeof arg === 'number') return arg
+          if (typeof arg === 'boolean') return arg
+          if (arg === null || arg === undefined) return arg
+          // For objects, try to serialize them
+          return JSON.parse(JSON.stringify(arg))
+        } catch (e) {
+          // If serialization fails, convert to string
+          return String(arg)
+        }
+      })
+    }
+
     currentTestSteps.push({
       name: step.name,
       actor: step.actor,
-      args: step.args || [],
+      args: serializedArgs,
       status: step.failed ? 'failed' : 'success',
       duration: step.duration || 0,
     })
@@ -211,19 +238,25 @@ module.exports = function (config) {
     // Debug logging
     output.debug(`HTML Reporter: Test finished - ${test.title}, State: ${test.state}, Retries: ${retryAttempts}`)
 
-    // Detect if this is a BDD/Gherkin test
-    const isBddTest = isBddGherkinTest(test, currentSuite)
+    // Detect if this is a BDD/Gherkin test - use test.parent directly instead of currentSuite
+    const suite = test.parent || test.suite || currentSuite
+    const isBddTest = isBddGherkinTest(test, suite)
     const steps = isBddTest ? currentBddSteps : currentTestSteps
-    const featureInfo = isBddTest ? getBddFeatureInfo(test, currentSuite) : null
+    const featureInfo = isBddTest ? getBddFeatureInfo(test, suite) : null
 
     // Check if this test already exists in reportData.tests (from a previous retry)
     const existingTestIndex = reportData.tests.findIndex(t => t.id === testId)
-    const hasFailedBefore = existingTestIndex >= 0 && reportData.tests[existingTestIndex].state === 'failed'
+    const hasFailedBefore = existingTestIndex >= 0 && reportData.tests[existingTestIndex] && reportData.tests[existingTestIndex].state === 'failed'
     const currentlyFailed = test.state === 'failed'
 
     // Debug artifacts collection (but don't process them yet - screenshots may not be ready)
     output.debug(`HTML Reporter: Test ${test.title} artifacts at test.finished: ${JSON.stringify(test.artifacts)}`)
 
+    // Extract parent/suite title before serialization (for worker mode)
+    // This ensures the feature name is preserved when test data is JSON stringified
+    const parentTitle = test.parent?.title || test.suite?.title || (suite && suite.title) || null
+    const suiteTitle = test.suite?.title || (suite && suite.title) || null
+
     const testData = {
       ...test,
       id: testId,
@@ -239,11 +272,14 @@ module.exports = function (config) {
       uid: test.uid,
       isBdd: isBddTest,
       feature: featureInfo,
+      // Store parent/suite titles as simple strings for worker mode serialization
+      parentTitle: parentTitle,
+      suiteTitle: suiteTitle,
     }
 
     if (existingTestIndex >= 0) {
       // Update existing test with final result (including failed state)
-      reportData.tests[existingTestIndex] = testData
+      if (existingTestIndex >= 0) reportData.tests[existingTestIndex] = testData
       output.debug(`HTML Reporter: Updated existing test - ${test.title}, Final state: ${test.state}`)
     } else {
       // Add new test
@@ -288,14 +324,14 @@ module.exports = function (config) {
         finalState: test.state,
         duration: test.duration || 0,
       })
-    output.debug(`HTML Reporter: Fallback retry detection for failed test ${test.title}, attempts: ${fallbackAttempts}`)
+      output.debug(`HTML Reporter: Fallback retry detection for failed test ${test.title}, attempts: ${fallbackAttempts}`)
     }
   })
 
   // Generate final report
-  event.dispatcher.on(event.all.result, result => {
-    reportData.endTime = new Date()
-    reportData.duration = reportData.endTime - reportData.startTime
+  event.dispatcher.on(event.all.result, async result => {
+    reportData.endTime = new Date().toISOString()
+    reportData.duration = new Date(reportData.endTime).getTime() - new Date(reportData.startTime).getTime()
 
     // Process artifacts now that all async tasks (including screenshots) are complete
     output.debug(`HTML Reporter: Processing artifacts for ${reportData.tests.length} tests after all async tasks complete`)
@@ -345,7 +381,11 @@ module.exports = function (config) {
       .filter(t => t.state === 'failed')
       .map(t => {
         const testName = t.title || 'Unknown Test'
-        const featureName = t.parent?.title || 'Unknown Feature'
+        // Try to get feature name from BDD, preserved titles (worker mode), or direct access
+        let featureName = t.feature?.name || t.parentTitle || t.suiteTitle || t.parent?.title || t.suite?.title || 'Unknown Feature'
+        if (featureName === 'Unknown Feature' && t.suite && t.suite.feature && t.suite.feature.name) {
+          featureName = t.suite.feature.name
+        }
 
         if (t.err) {
           const errorMessage = t.err.message || t.err.toString() || 'Test failed'
@@ -410,15 +450,20 @@ module.exports = function (config) {
         // Always overwrite the file with the latest complete data from this worker
         // This prevents double-counting when the event is triggered multiple times
         fs.writeFileSync(jsonPath, safeJsonStringify(reportData))
-        output.print(`HTML Reporter: Generated worker JSON results: ${jsonFileName}`)
+        output.debug(`HTML Reporter: Generated worker JSON results: ${jsonFileName}`)
       } catch (error) {
-        output.print(`HTML Reporter: Failed to write worker JSON: ${error.message}`)
+        output.debug(`HTML Reporter: Failed to write worker JSON: ${error.message}`)
       }
       return
     }
 
     // Single process mode - generate report normally
-    generateHtmlReport(reportData, options)
+    try {
+      await generateHtmlReport(reportData, options)
+    } catch (error) {
+      output.print(`Failed to generate HTML report: ${error.message}`)
+      output.debug(`HTML Reporter error stack: ${error.stack}`)
+    }
 
     // Export stats if configured
     if (options.exportStats) {
@@ -542,8 +587,8 @@ module.exports = function (config) {
       const testName = replicateTestToFileName(originalTestName)
       const featureName = replicateTestToFileName(originalFeatureName)
 
-      output.print(`HTML Reporter: Original test title: "${originalTestName}"`)
-      output.print(`HTML Reporter: CodeceptJS filename: "${testName}"`)
+      output.debug(`HTML Reporter: Original test title: "${originalTestName}"`)
+      output.debug(`HTML Reporter: CodeceptJS filename: "${testName}"`)
 
       // Generate possible screenshot names based on CodeceptJS patterns
       const possibleNames = [
@@ -567,19 +612,19 @@ module.exports = function (config) {
         'failure.jpg',
       ]
 
-      output.print(`HTML Reporter: Checking ${possibleNames.length} possible screenshot names for "${testName}"`)
+      output.debug(`HTML Reporter: Checking ${possibleNames.length} possible screenshot names for "${testName}"`)
 
       // Search for screenshots in possible directories
       for (const dir of possibleDirs) {
-        output.print(`HTML Reporter: Checking directory: ${dir}`)
+        output.debug(`HTML Reporter: Checking directory: ${dir}`)
         if (!fs.existsSync(dir)) {
-          output.print(`HTML Reporter: Directory does not exist: ${dir}`)
+          output.debug(`HTML Reporter: Directory does not exist: ${dir}`)
           continue
         }
 
         try {
           const files = fs.readdirSync(dir)
-          output.print(`HTML Reporter: Found ${files.length} files in ${dir}`)
+          output.debug(`HTML Reporter: Found ${files.length} files in ${dir}`)
 
           // Look for exact matches first
           for (const name of possibleNames) {
@@ -587,7 +632,7 @@ module.exports = function (config) {
               const fullPath = path.join(dir, name)
               if (!screenshots.includes(fullPath)) {
                 screenshots.push(fullPath)
-                output.print(`HTML Reporter: Found screenshot: ${fullPath}`)
+                output.debug(`HTML Reporter: Found screenshot: ${fullPath}`)
               }
             }
           }
@@ -618,16 +663,16 @@ module.exports = function (config) {
             const fullPath = path.join(dir, file)
             if (!screenshots.includes(fullPath)) {
               screenshots.push(fullPath)
-              output.print(`HTML Reporter: Found related screenshot: ${fullPath}`)
+              output.debug(`HTML Reporter: Found related screenshot: ${fullPath}`)
             }
           }
         } catch (error) {
           // Ignore directory read errors
-          output.print(`HTML Reporter: Could not read directory ${dir}: ${error.message}`)
+          output.debug(`HTML Reporter: Could not read directory ${dir}: ${error.message}`)
         }
       }
     } catch (error) {
-      output.print(`HTML Reporter: Error collecting screenshots: ${error.message}`)
+      output.debug(`HTML Reporter: Error collecting screenshots: ${error.message}`)
     }
 
     return screenshots
@@ -654,7 +699,7 @@ module.exports = function (config) {
     const statsPath = path.resolve(reportDir, config.exportStatsPath)
 
     const exportData = {
-      timestamp: data.endTime.toISOString(),
+      timestamp: data.endTime, // Already an ISO string
       duration: data.duration,
       stats: data.stats,
       retries: data.retries,
@@ -698,7 +743,7 @@ module.exports = function (config) {
 
     // Add current run to history
     history.unshift({
-      timestamp: data.endTime.toISOString(),
+      timestamp: data.endTime, // Already an ISO string
       duration: data.duration,
       stats: data.stats,
       retries: data.retries.length,
@@ -725,11 +770,11 @@ module.exports = function (config) {
     const jsonFiles = fs.readdirSync(reportDir).filter(file => file.startsWith('worker-') && file.endsWith('-results.json'))
 
     if (jsonFiles.length === 0) {
-      output.print('HTML Reporter: No worker JSON results found to consolidate')
+      output.debug('HTML Reporter: No worker JSON results found to consolidate')
       return
     }
 
-    output.print(`HTML Reporter: Found ${jsonFiles.length} worker JSON files to consolidate`)
+    output.debug(`HTML Reporter: Found ${jsonFiles.length} worker JSON files to consolidate`)
 
     // Initialize consolidated data structure
     const consolidatedData = {
@@ -821,9 +866,9 @@ module.exports = function (config) {
         saveTestHistory(consolidatedData, config)
       }
 
-      output.print(`HTML Reporter: Successfully consolidated ${jsonFiles.length} worker reports`)
+      output.debug(`HTML Reporter: Successfully consolidated ${jsonFiles.length} worker reports`)
     } catch (error) {
-      output.print(`HTML Reporter: Failed to consolidate worker reports: ${error.message}`)
+      output.debug(`HTML Reporter: Failed to consolidate worker reports: ${error.message}`)
     }
   }
 
@@ -844,7 +889,7 @@ module.exports = function (config) {
 
       // Add current run to history for chart display (before saving to file)
       const currentRun = {
-        timestamp: data.endTime.toISOString(),
+        timestamp: data.endTime, // Already an ISO string
         duration: data.duration,
         stats: data.stats,
         retries: data.retries.length,
@@ -863,7 +908,7 @@ module.exports = function (config) {
 
     const html = template(getHtmlTemplate(), {
       title: `CodeceptJS Test Report v${Codecept.version()}`,
-      timestamp: data.endTime.toISOString(),
+      timestamp: data.endTime, // Already an ISO string
       duration: formatDuration(data.duration),
       stats: JSON.stringify(data.stats),
       history: JSON.stringify(history),
@@ -929,7 +974,9 @@ module.exports = function (config) {
     return tests
       .map(test => {
         const statusClass = test.state || 'unknown'
-        const feature = test.isBdd && test.feature ? test.feature.name : test.parent?.title || 'Unknown Feature'
+        // Use preserved parent/suite titles (for worker mode) or fallback to direct access
+        const feature = test.isBdd && test.feature ? test.feature.name : test.parentTitle || test.suiteTitle || test.parent?.title || test.suite?.title || 'Unknown Feature'
+        // Always try to show steps if available, even for unknown feature
         const steps = config.showSteps && test.steps ? (test.isBdd ? generateBddStepsHtml(test.steps) : generateStepsHtml(test.steps)) : ''
         const featureDetails = test.isBdd && test.feature ? generateBddFeatureHtml(test.feature) : ''
         const hooks = test.hooks && test.hooks.length > 0 ? generateHooksHtml(test.hooks) : ''
@@ -1135,19 +1182,19 @@ module.exports = function (config) {
 
   function generateArtifactsHtml(artifacts, isFailedTest = false) {
     if (!artifacts || artifacts.length === 0) {
-      output.print(`HTML Reporter: No artifacts found for test`)
+      output.debug(`HTML Reporter: No artifacts found for test`)
       return ''
     }
 
-    output.print(`HTML Reporter: Processing ${artifacts.length} artifacts, isFailedTest: ${isFailedTest}`)
-    output.print(`HTML Reporter: Artifacts: ${JSON.stringify(artifacts)}`)
+    output.debug(`HTML Reporter: Processing ${artifacts.length} artifacts, isFailedTest: ${isFailedTest}`)
+    output.debug(`HTML Reporter: Artifacts: ${JSON.stringify(artifacts)}`)
 
     // Separate screenshots from other artifacts
     const screenshots = []
     const otherArtifacts = []
 
     artifacts.forEach(artifact => {
-      output.print(`HTML Reporter: Processing artifact: ${artifact} (type: ${typeof artifact})`)
+      output.debug(`HTML Reporter: Processing artifact: ${artifact} (type: ${typeof artifact})`)
 
       // Handle different artifact formats
       let artifactPath = artifact
@@ -1162,14 +1209,14 @@ module.exports = function (config) {
       // Check if it's a screenshot file
       if (typeof artifactPath === 'string' && artifactPath.match(/\.(png|jpg|jpeg|gif|webp|bmp|svg)$/i)) {
         screenshots.push(artifactPath)
-        output.print(`HTML Reporter: Found screenshot: ${artifactPath}`)
+        output.debug(`HTML Reporter: Found screenshot: ${artifactPath}`)
       } else {
         otherArtifacts.push(artifact)
-        output.print(`HTML Reporter: Found other artifact: ${artifact}`)
+        output.debug(`HTML Reporter: Found other artifact: ${artifact}`)
       }
     })
 
-    output.print(`HTML Reporter: Found ${screenshots.length} screenshots and ${otherArtifacts.length} other artifacts`)
+    output.debug(`HTML Reporter: Found ${screenshots.length} screenshots and ${otherArtifacts.length} other artifacts`)
 
     let artifactsHtml = ''
 
@@ -1198,7 +1245,7 @@ module.exports = function (config) {
             }
           }
 
-          output.print(`HTML Reporter: Screenshot ${filename} -> ${relativePath}`)
+          output.debug(`HTML Reporter: Screenshot ${filename} -> ${relativePath}`)
 
           return `
               <div class="screenshot-container">
@@ -1242,7 +1289,7 @@ module.exports = function (config) {
             }
           }
 
-          output.print(`HTML Reporter: Screenshot ${filename} -> ${relativePath}`)
+          output.debug(`HTML Reporter: Screenshot ${filename} -> ${relativePath}`)
           return `<img src="${relativePath}" alt="Screenshot" class="artifact-image" onclick="openImageModal(this.src)"/>`
         })
         .join('')
@@ -1556,7 +1603,7 @@ module.exports = function (config) {
         <section class="history-section" style="display: {{showHistory}};">
             <h2>Test History</h2>
             <div class="history-chart-container">
-                <canvas id="historyChart" width="800" height="300"></canvas>
+                <canvas id="historyChart" width="1600" height="600"></canvas>
             </div>
         </section>
 
@@ -2457,6 +2504,10 @@ body {
 
   function getJsScripts() {
     return `
+// Go to Top button
+function scrollToTop() {
+  window.scrollTo({ top: 0, behavior: 'smooth' });
+}
 function toggleTestDetails(testId) {
     const details = document.getElementById('details-' + testId);
     if (details.style.display === 'none' || details.style.display === '') {
@@ -2939,9 +2990,26 @@ function drawHistoryChart() {
 // Initialize charts and filters
 document.addEventListener('DOMContentLoaded', function() {
     
-    // Draw charts
-    drawPieChart();
-    drawHistoryChart();
+  // Draw charts
+  drawPieChart();
+  drawHistoryChart();
+  // Add Go to Top button
+  const goTopBtn = document.createElement('button');
+  goTopBtn.innerText = '↑ Top';
+  goTopBtn.id = 'goTopBtn';
+  goTopBtn.style.position = 'fixed';
+  goTopBtn.style.bottom = '30px';
+  goTopBtn.style.right = '30px';
+  goTopBtn.style.zIndex = '9999';
+  goTopBtn.style.padding = '12px 18px';
+  goTopBtn.style.borderRadius = '50%';
+  goTopBtn.style.background = '#27ae60';
+  goTopBtn.style.color = '#fff';
+  goTopBtn.style.fontSize = '20px';
+  goTopBtn.style.boxShadow = '0 2px 8px rgba(0,0,0,0.2)';
+  goTopBtn.style.cursor = 'pointer';
+  goTopBtn.onclick = scrollToTop;
+  document.body.appendChild(goTopBtn);
     
     // Set up filter event listeners
     document.getElementById('statusFilter').addEventListener('change', applyFilters);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codeceptjs",
-  "version": "3.7.6-beta.3",
+  "version": "3.7.6-beta.4",
   "description": "Supercharged End 2 End Testing Framework for NodeJS",
   "keywords": [
     "acceptance",
@@ -48,6 +48,9 @@
   "repository": "Codeception/codeceptjs",
   "scripts": {
     "test-server": "node bin/test-server.js test/data/rest/db.json --host 0.0.0.0 -p 8010",
+    "mock-server:start": "node test/mock-server/start-mock-server.js",
+    "mock-server:stop": "kill -9 $(lsof -t -i:3001)",
+    "test:with-mock-server": "npm run mock-server:start && npm test",
     "json-server:graphql": "node test/data/graphql/index.js",
     "lint": "eslint bin/ examples/ lib/ test/ translations/ runok.js",
     "lint-fix": "eslint bin/ examples/ lib/ test/ translations/ runok.js --fix",
@@ -138,7 +141,7 @@
     "@pollyjs/core": "6.0.6",
     "@types/chai": "5.2.2",
     "@types/inquirer": "9.0.9",
-    "@types/node": "^24.6.0",
+    "@types/node": "^24.6.2",
     "@wdio/sauce-service": "9.12.5",
     "@wdio/selenium-standalone-service": "8.15.0",
     "@wdio/utils": "9.19.2",