@arghajit/dummy 0.1.0-beta-23 → 0.1.0-beta-25

package/dist/reporter/playwright-pulse-reporter.d.ts

@@ -19,6 +19,7 @@ export declare class PlaywrightPulseReporter implements Reporter {
     private getBrowserDetails;
     private processStep;
     onTestEnd(test: TestCase, result: PwTestResult): Promise<void>;
+    private _getFinalizedResults;
     onError(error: any): void;
     private _getEnvDetails;
     private _writeShardResults;

package/dist/reporter/playwright-pulse-reporter.js

@@ -296,15 +296,18 @@ class PlaywrightPulseReporter {
                 console.error(`Pulse Reporter: Failed to process attachment "${attachment.name}" for test ${pulseResult.name}. Error: ${err.message}`);
             }
         }
-        const existingTestIndex = this.results.findIndex((r) => r.id === test.id);
-        if (existingTestIndex !== -1) {
-            if (pulseResult.retries >= this.results[existingTestIndex].retries) {
-                this.results[existingTestIndex] = pulseResult;
+        this.results.push(pulseResult);
+    }
+    _getFinalizedResults(allResults) {
+        const finalResultsMap = new Map();
+        for (const result of allResults) {
+            const existing = finalResultsMap.get(result.id);
+            // Keep the result with the highest retry attempt for each test ID
+            if (!existing || result.retries >= existing.retries) {
+                finalResultsMap.set(result.id, result);
             }
         }
-        else {
-            this.results.push(pulseResult);
-        }
+        return Array.from(finalResultsMap.values());
     }
     onError(error) {
         var _a;
@@ -359,14 +362,7 @@ class PlaywrightPulseReporter {
                 }
             }
         }
-        let finalUniqueResultsMap = new Map();
-        for (const result of allShardProcessedResults) {
-            const existing = finalUniqueResultsMap.get(result.id);
-            if (!existing || result.retries >= existing.retries) {
-                finalUniqueResultsMap.set(result.id, result);
-            }
-        }
-        const finalResultsList = Array.from(finalUniqueResultsMap.values());
+        const finalResultsList = this._getFinalizedResults(allShardProcessedResults);
         finalResultsList.forEach((r) => (r.runId = finalRunData.id));
         finalRunData.passed = finalResultsList.filter((r) => r.status === "passed").length;
         finalRunData.failed = finalResultsList.filter((r) => r.status === "failed").length;
@@ -417,6 +413,8 @@ class PlaywrightPulseReporter {
             await this._writeShardResults();
             return;
         }
+        // De-duplicate and handle retries here, in a safe, single-threaded context.
+        const finalResults = this._getFinalizedResults(this.results);
         const runEndTime = Date.now();
         const duration = runEndTime - this.runStartTime;
         const runId = `run-${this.runStartTime}-581d5ad8-ce75-4ca5-94a6-ed29c466c815`;
@@ -424,26 +422,25 @@ class PlaywrightPulseReporter {
         const runData = {
             id: runId,
             timestamp: new Date(this.runStartTime),
-            totalTests: 0,
-            passed: 0,
-            failed: 0,
-            skipped: 0,
+            // Use the length of the de-duplicated array for all counts
+            totalTests: finalResults.length,
+            passed: finalResults.filter((r) => r.status === "passed").length,
+            failed: finalResults.filter((r) => r.status === "failed").length,
+            skipped: finalResults.filter((r) => r.status === "skipped").length,
             duration,
             environment: environmentDetails,
         };
+        finalResults.forEach((r) => (r.runId = runId));
         let finalReport = undefined;
         if (this.isSharded) {
+            // The _mergeShardResults method will handle its own de-duplication
             finalReport = await this._mergeShardResults(runData);
         }
         else {
-            this.results.forEach((r) => (r.runId = runId));
-            runData.passed = this.results.filter((r) => r.status === "passed").length;
-            runData.failed = this.results.filter((r) => r.status === "failed").length;
-            runData.skipped = this.results.filter((r) => r.status === "skipped").length;
-            runData.totalTests = this.results.length;
             finalReport = {
                 run: runData,
-                results: this.results,
+                // Use the de-duplicated results
+                results: finalResults,
                 metadata: { generatedAt: new Date().toISOString() },
             };
         }
@@ -515,47 +512,48 @@ class PlaywrightPulseReporter {
             }
             return;
         }
-        const combinedRun = {
-            id: `merged-${Date.now()}`,
-            timestamp: new Date(0),
-            totalTests: 0,
-            passed: 0,
-            failed: 0,
-            skipped: 0,
-            duration: 0,
-            environment: undefined,
-        };
-        const allResults = [];
+        const allResultsFromAllFiles = [];
         let latestTimestamp = new Date(0);
+        let lastRunEnvironment = undefined;
+        let totalDuration = 0;
         for (const file of reportFiles) {
             const filePath = path.join(pulseResultsDir, file);
             try {
                 const content = await fs.readFile(filePath, "utf-8");
                 const json = JSON.parse(content);
                 if (json.run) {
-                    combinedRun.totalTests += json.run.totalTests || 0;
-                    combinedRun.passed += json.run.passed || 0;
-                    combinedRun.failed += json.run.failed || 0;
-                    combinedRun.skipped += json.run.skipped || 0;
-                    combinedRun.duration += json.run.duration || 0;
                     const runTimestamp = new Date(json.run.timestamp);
                     if (runTimestamp > latestTimestamp) {
                         latestTimestamp = runTimestamp;
-                        combinedRun.environment = json.run.environment || undefined;
+                        lastRunEnvironment = json.run.environment || undefined;
                     }
                 }
                 if (json.results) {
-                    allResults.push(...json.results);
+                    allResultsFromAllFiles.push(...json.results);
                 }
             }
             catch (err) {
                 console.warn(`Pulse Reporter: Could not parse report file ${filePath}. Skipping. Error: ${err.message}`);
             }
         }
-        combinedRun.timestamp = latestTimestamp;
+        // De-duplicate the results from ALL merged files using the helper function
+        const finalMergedResults = this._getFinalizedResults(allResultsFromAllFiles);
+        // Sum the duration from the final, de-duplicated list of tests
+        totalDuration = finalMergedResults.reduce((acc, r) => acc + (r.duration || 0), 0);
+        const combinedRun = {
+            id: `merged-${Date.now()}`,
+            timestamp: latestTimestamp,
+            environment: lastRunEnvironment,
+            // Recalculate counts based on the truly final, de-duplicated list
+            totalTests: finalMergedResults.length,
+            passed: finalMergedResults.filter((r) => r.status === "passed").length,
+            failed: finalMergedResults.filter((r) => r.status === "failed").length,
+            skipped: finalMergedResults.filter((r) => r.status === "skipped").length,
+            duration: totalDuration,
+        };
         const finalReport = {
             run: combinedRun,
-            results: allResults,
+            results: finalMergedResults, // Use the de-duplicated list
             metadata: {
                 generatedAt: new Date().toISOString(),
             },
@@ -567,7 +565,7 @@ class PlaywrightPulseReporter {
                 return value;
             }, 2));
             if (this.printsToStdio()) {
-                console.log(`PlaywrightPulseReporter: ✅ Merged report with ${allResults.length} total results saved to ${finalOutputPath}`);
+                console.log(`PlaywrightPulseReporter: ✅ Merged report with ${finalMergedResults.length} total results saved to ${finalOutputPath}`);
             }
         }
         catch (err) {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@arghajit/dummy",
   "author": "Arghajit Singha",
-  "version": "0.1.0-beta-23",
+  "version": "0.1.0-beta-25",
   "description": "A Playwright reporter and dashboard for visualizing test results.",
   "homepage": "https://playwright-pulse-report.netlify.app/",
   "keywords": [