playwright-slack-report-burak 2.3.0 → 2.4.0

package/dist/src/ResultsParser.js

@@ -97,7 +97,7 @@ class ResultsParser {
 
                 if (fs.existsSync(fileToRead)) {
                     const nodeSummary = JSON.parse(fs.readFileSync(fileToRead, 'utf-8'));
-                    summary.total += nodeSummary.total;
+                    // Don't add the inflated totals - calculate from outcomes instead
                     summary.passed += nodeSummary.passed;
                     summary.failed += nodeSummary.failed;
                     summary.flaky += nodeSummary.flaky;
@@ -106,6 +106,27 @@ class ResultsParser {
                     summary.tests = summary.tests.concat(nodeSummary.tests);
                 }
             }
+            
+            // Calculate correct total as sum of outcomes (not inflated shard totals)
+            summary.total = summary.passed + summary.failed + summary.flaky + summary.skipped;
+            
+            // Deduplicate tests array in case there are duplicates from retries
+            if (summary.tests && summary.tests.length > 0) {
+                const uniqueTests = new Map();
+                for (const test of summary.tests) {
+                    const key = `${test.suiteName}_${test.name}`;
+                    // Keep the test with the highest retry number (most recent attempt)
+                    if (!uniqueTests.has(key) || test.retry > uniqueTests.get(key).retry) {
+                        uniqueTests.set(key, test);
+                    }
+                }
+                const deduplicatedTests = Array.from(uniqueTests.values());
+                if (deduplicatedTests.length < summary.tests.length) {
+                    console.log(`[Fix] Deduplicating merged tests: ${summary.tests.length} → ${deduplicatedTests.length}`);
+                    summary.tests = deduplicatedTests;
+                }
+            }
+            
             this.mergeReports();
         }
         return summary;
@@ -176,27 +197,58 @@ class ResultsParser {
     }
     updateResults(data) {
         if (data.testSuite.tests.length > 0) {
-            this.result.push(data);
+            // Check if we already have this test suite
+            const existingSuiteIndex = this.result.findIndex(r => r.testSuite.title === data.testSuite.title);
+            
+            if (existingSuiteIndex >= 0) {
+                // Suite exists, need to check for duplicate tests
+                const existingSuite = this.result[existingSuiteIndex];
+                
+                for (const newTest of data.testSuite.tests) {
+                    // Find if this test already exists (by name)
+                    const existingTestIndex = existingSuite.testSuite.tests.findIndex(
+                        t => t.name === newTest.name
+                    );
+                    
+                    if (existingTestIndex >= 0) {
+                        // Test exists - replace with newer result (higher retry number)
+                        if (newTest.retry > existingSuite.testSuite.tests[existingTestIndex].retry) {
+                            console.log(`[Fix] Updating test "${newTest.name}" from retry ${existingSuite.testSuite.tests[existingTestIndex].retry} to ${newTest.retry}`);
+                            existingSuite.testSuite.tests[existingTestIndex] = newTest;
+                        }
+                    } else {
+                        // New test, add it
+                        existingSuite.testSuite.tests.push(newTest);
+                    }
+                }
+            } else {
+                // New suite, add it
+                this.result.push(data);
+            }
         }
     }
     addTestResult(suiteName, testCase, projectBrowserMapping) {
         const testResults = [];
         const projectSettings = this.determineBrowser(testCase._projectId, projectBrowserMapping);
-        for (const result of testCase.results) {
-            testResults.push({
-                suiteName,
-                name: testCase.title,
-                status: result.status,
-                browser: projectSettings.browser,
-                projectName: projectSettings.projectName,
-                retry: result.retry,
-                retries: testCase.retries,
-                startedAt: new Date(result.startTime).toISOString(),
-                endedAt: new Date(new Date(result.startTime).getTime() + result.duration).toISOString(),
-                reason: this.safelyDetermineFailure(result),
-                attachments: result.attachments,
-            });
-        }
+        
+        // Only add the FINAL result (last retry attempt), not all attempts
+        // This prevents counting retry attempts as separate tests
+        const finalResult = testCase.results[testCase.results.length - 1];
+        
+        testResults.push({
+            suiteName,
+            name: testCase.title,
+            status: finalResult.status,
+            browser: projectSettings.browser,
+            projectName: projectSettings.projectName,
+            retry: finalResult.retry,
+            retries: testCase.retries,
+            startedAt: new Date(finalResult.startTime).toISOString(),
+            endedAt: new Date(new Date(finalResult.startTime).getTime() + finalResult.duration).toISOString(),
+            reason: this.safelyDetermineFailure(finalResult),
+            attachments: finalResult.attachments,
+        });
+        
         this.updateResults({
             testSuite: {
                 title: suiteName,
@@ -320,13 +372,20 @@ class ResultsParser {
         }
     }
     mergeReports() {
+      let breakMergeWaiter = false;
       try {
         // Execute the command to merge reports
         exec(`npx playwright merge-reports --reporter html -c ./playwright-report playwright-report`);
         // Wait until index.html exists
         while (!fs.existsSync(path.join('./playwright-report', 'index.html'))) {
-          console.log('Waiting for merged html report to be generated...');
-          setTimeout(() => {}, 3000);
+          console.log('Waiting 2 seconds for merged html report to be generated...');
+          const currentTime = new Date().getTime();
+          breakMergeWaiter = false;
+          while (!breakMergeWaiter) {
+            if(new Date().getTime() - currentTime > 2000) {
+              breakMergeWaiter = true;
+            }
+          }
         }
         console.log('Reports merged successfully.');
         console.log('Contents of playwright-report:', fs.readdirSync('./playwright-report'));

package/package.json

@@ -30,7 +30,7 @@
     "lint-fix": "npx eslint . --ext .ts --fix"
   },
   "name": "playwright-slack-report-burak",
-  "version": "2.3.0",
+  "version": "2.4.0",
   "main": "index.js",
   "types": "dist/index.d.ts",
   "repository": "git@github.com:ryanrosello-og/playwright-slack-report.git",