codeceptjs 3.7.5-beta.7 → 3.7.5-beta.9

package/lib/mocha/test.js

@@ -121,6 +121,7 @@ function serializeTest(test, error = null) {
   }
 
   return {
+    file: test.file ? relativeDir(test.file) : undefined,
     opts: test.opts || {},
     tags: test.tags || [],
     uid: test.uid,

package/lib/plugin/failedTestsTracker.js

@@ -41,72 +41,77 @@ module.exports = function (config) {
 
   // Track test failures - only when not using workers
   event.dispatcher.on(event.test.failed, test => {
-    // Skip if main process is using workers (will collect from consolidated results)
-    if (store.hasWorkers) {
-      return
-    }
-    
-    // Skip if we're in a worker thread (CodeceptJS uses worker_threads)
+    // Skip collection in worker threads to avoid duplicates
     try {
       const { isMainThread } = require('worker_threads')
-      if (!isMainThread) {
-        return
-      }
+      if (!isMainThread) return
     } catch (e) {
-      // worker_threads not available, assume main thread
+      // worker_threads not available, continue
     }
 
-    allTestsPassed = false
+    if (store.hasWorkers) return // Skip if running with workers
 
-    const failedTest = {
-      title: test.title,
-      fullTitle: test.fullTitle(),
-      file: test.file || (test.parent && test.parent.file),
-      uid: test.uid,
-      timestamp: new Date().toISOString(),
-    }
+    // Only collect on final failure (when retries are exhausted or no retries configured)
+    const currentRetry = test._currentRetry || 0
+    const maxRetries = typeof test.retries === 'function' ? test.retries() : (test.retries || 0)
+    
+    // Only add to failed tests if this is the final attempt
+    if (currentRetry >= maxRetries) {
+      allTestsPassed = false
+
+      const failedTest = {
+        title: test.title,
+        fullTitle: test.fullTitle(),
+        file: test.file || (test.parent && test.parent.file),
+        uid: test.uid,
+        timestamp: new Date().toISOString(),
+      }
 
-    // Add parent suite information
-    if (test.parent) {
-      failedTest.suite = test.parent.title
-      failedTest.suiteFile = test.parent.file
-    }
+      // Add parent suite information
+      if (test.parent) {
+        failedTest.suite = test.parent.title
+        failedTest.suiteFile = test.parent.file
+      }
 
-    // Add error information if available
-    if (test.err && options.includeStackTrace) {
-      failedTest.error = {
-        message: test.err.message || 'Test failed',
-        stack: test.err.stack || '',
-        name: test.err.name || 'Error',
+      // Add error information if available
+      if (test.err && options.includeStackTrace) {
+        failedTest.error = {
+          message: test.err.message || 'Test failed',
+          stack: test.err.stack || '',
+          name: test.err.name || 'Error',
+        }
       }
-    }
 
-    // Add metadata if available
-    if (options.includeMetadata) {
-      failedTest.metadata = {
-        tags: test.tags || [],
-        meta: test.meta || {},
-        opts: test.opts || {},
-        duration: test.duration || 0,
-        retries: test.retries || 0,
+      // Add metadata if available
+      if (options.includeMetadata) {
+        failedTest.metadata = {
+          tags: test.tags || [],
+          meta: test.meta || {},
+          opts: test.opts || {},
+          duration: test.duration || 0,
+          // Only include retries if it represents actual retry attempts, not the config value
+          ...(test._currentRetry > 0 && { actualRetries: test._currentRetry }),
+          ...(maxRetries > 0 && maxRetries !== -1 && { maxRetries: maxRetries }),
+        }
       }
-    }
 
-    // Add BDD/Gherkin information if available
-    if (test.parent && test.parent.feature) {
-      failedTest.bdd = {
-        feature: test.parent.feature.name || test.parent.title,
-        scenario: test.title,
-        featureFile: test.parent.file,
+      // Add BDD/Gherkin information if available
+      if (test.parent && test.parent.feature) {
+        failedTest.bdd = {
+          feature: test.parent.feature.name || test.parent.title,
+          scenario: test.title,
+          featureFile: test.parent.file,
+        }
       }
-    }
 
-    failedTests.push(failedTest)
-    output.print(`Failed Tests Tracker: Recorded failed test - ${test.title}`)
+      failedTests.push(failedTest)
+      output.print(`Failed Tests Tracker: Recorded failed test - ${test.title}`)
+    }
   })
 
   // Handle test completion and save failed tests
   event.dispatcher.on(event.all.result, (result) => {
+    
     // Respect CodeceptJS output directory like other plugins
     const outputDir = global.output_dir || './output'
     const outputPath = path.isAbsolute(options.outputFile)
@@ -114,14 +119,27 @@ module.exports = function (config) {
       : path.resolve(outputDir, options.outputFile)
     let allFailedTests = [...failedTests]
 
-    // In worker mode, collect failed tests from consolidated result
-    if (store.hasWorkers && result && result.tests) {
-      const workerFailedTests = result.tests.filter(test => test.state === 'failed' || test.err)
+    // Collect failed tests from result (both worker and single-process modes)
+    if (result) {
+      let resultFailedTests = []
+      
+      // Worker mode: result.tests
+      if (store.hasWorkers && result.tests) {
+        resultFailedTests = result.tests.filter(test => test.state === 'failed' || test.err)
+      }
+      // Single-process mode: result._failures or result._tests
+      else if (!store.hasWorkers && (result._failures || result._tests)) {
+        if (result._failures && result._failures.length > 0) {
+          resultFailedTests = result._failures.map(failure => failure.test || failure)
+        } else if (result._tests) {
+          resultFailedTests = result._tests.filter(test => test.state === 'failed' || test.err)
+        }
+      }
 
       // Use a Set to track unique test identifiers to prevent duplicates
       const existingTestIds = new Set(allFailedTests.map(test => test.uid || `${test.file}:${test.title}`))
 
-      workerFailedTests.forEach(test => {
+      resultFailedTests.forEach(test => {
         // Create unique identifier for deduplication
         const testId = test.uid || `${test.file || 'unknown'}:${test.title}`
 
@@ -177,7 +195,7 @@ module.exports = function (config) {
         existingTestIds.add(testId)
       })
 
-      output.print(`Failed Tests Tracker: Collected ${workerFailedTests.length} failed tests from workers`)
+      output.print(`Failed Tests Tracker: Collected ${resultFailedTests.length} failed tests from result`)
     }
 
     if (allFailedTests.length === 0) {
@@ -242,12 +260,22 @@ module.exports = function (config) {
         
         // If still unknown, try to extract from error stack trace
         if (filePath === 'unknown' && test.err && test.err.stack) {
-          const stackMatch = test.err.stack.match(/at.*\(([^)]+\.js):\d+:\d+\)/)
-          if (stackMatch && stackMatch[1]) {
-            // Convert absolute path to relative path
-            const absolutePath = stackMatch[1]
-            const relativePath = absolutePath.replace(process.cwd() + '/', '')
-            filePath = relativePath
+          // Try multiple regex patterns for different stack trace formats
+          const patterns = [
+            /at.*\(([^)]+\.js):\d+:\d+\)/,  // Standard format
+            /at.*\(.*[\/\\]([^\/\\]+\.js):\d+:\d+\)/,  // With path separators
+            /\(([^)]*\.js):\d+:\d+\)/,  // Simpler format
+            /([^\/\\]+\.js):\d+:\d+/,   // Just filename with line numbers
+          ]
+          
+          for (const pattern of patterns) {
+            const stackMatch = test.err.stack.match(pattern)
+            if (stackMatch && stackMatch[1]) {
+              const absolutePath = stackMatch[1]
+              const relativePath = absolutePath.replace(process.cwd() + '/', '').replace(/^.*[\/\\]/, '')
+              filePath = relativePath
+              break
+            }
           }
         }
         
@@ -308,7 +336,7 @@ module.exports = function (config) {
         allFailedTests.push(failedTest)
       })
 
-      output.print(`Failed Tests Tracker: Collected ${workerFailedTests.length} failed tests from workers`)
+      output.print(`Failed Tests Tracker: Collected ${allFailedTests.length - failedTests.length} failed tests from workers`)
     }
 
     if (allFailedTests.length === 0) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codeceptjs",
-  "version": "3.7.5-beta.7",
+  "version": "3.7.5-beta.9",
   "description": "Supercharged End 2 End Testing Framework for NodeJS",
   "keywords": [
     "acceptance",