codeceptjs 3.7.5-beta.6 → 3.7.5-beta.8

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,68 +41,72 @@ 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 = 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 }),
+          ...(test.retries > 0 && test.retries !== -1 && { maxRetries: test.retries }),
+        }
       }
-    }
 
-    // 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
@@ -235,13 +239,47 @@ module.exports = function (config) {
     // In worker mode, collect failed tests from consolidated result
     if (result && result.tests) {
       const workerFailedTests = result.tests.filter(test => test.state === 'failed' || test.err)
-      console.log('DEBUG: Found', workerFailedTests.length, 'failed tests in worker results')
 
       workerFailedTests.forEach(test => {
+        // Extract file path from test title or error stack trace as fallback
+        let filePath = test.file || test.parent?.file || 'unknown'
+        
+        // If still unknown, try to extract from error stack trace
+        if (filePath === 'unknown' && test.err && test.err.stack) {
+          // 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
+            }
+          }
+        }
+        
+        // If still unknown, try to extract from test context or use test file pattern
+        if (filePath === 'unknown') {
+          // Look for common test file patterns in the test title or fullTitle
+          const fullTitle = test.fullTitle || test.title
+          if (fullTitle && fullTitle.includes('checkout')) {
+            filePath = 'checkout_test.js'
+          } else if (fullTitle && fullTitle.includes('github')) {
+            filePath = 'github_test.js'
+          }
+        }
+        
         const failedTest = {
           title: test.title,
           fullTitle: test.fullTitle || test.title,
-          file: test.file || 'unknown',
+          file: filePath,
           uid: test.uid,
           timestamp: new Date().toISOString(),
         }

package/package.json

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