@arghajit/dummy 0.1.0-beta-22 → 0.1.0-beta-24

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;
@@ -26,6 +27,6 @@ export declare class PlaywrightPulseReporter implements Reporter {
     private _cleanupTemporaryFiles;
     private _ensureDirExists;
     onEnd(result: FullResult): Promise<void>;
-    private falseResetOnEachRun;
+    private _mergeAllRunReports;
 }
 export default PlaywrightPulseReporter;

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

@@ -36,7 +36,6 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.PlaywrightPulseReporter = void 0;
 const fs = __importStar(require("fs/promises"));
 const path = __importStar(require("path"));
-const fsSync = __importStar(require("fs"));
 const crypto_1 = require("crypto");
 const ua_parser_js_1 = require("ua-parser-js");
 const os = __importStar(require("os"));
@@ -61,6 +60,7 @@ const convertStatus = (status, testCase) => {
 };
 const TEMP_SHARD_FILE_PREFIX = ".pulse-shard-results-";
 const ATTACHMENTS_SUBDIR = "attachments";
+const INDIVIDUAL_REPORTS_SUBDIR = "pulse-results";
 class PlaywrightPulseReporter {
     constructor(options = {}) {
         var _a, _b, _c;
@@ -72,7 +72,7 @@ class PlaywrightPulseReporter {
         this.baseOutputFile = (_a = options.outputFile) !== null && _a !== void 0 ? _a : this.baseOutputFile;
         this.outputDir = (_b = options.outputDir) !== null && _b !== void 0 ? _b : "pulse-report";
         this.attachmentsDir = path.join(this.outputDir, ATTACHMENTS_SUBDIR);
-        this.resetOnEachRun = (_c = options.resetOnEachRun) !== null && _c !== void 0 ? _c : false;
+        this.resetOnEachRun = (_c = options.resetOnEachRun) !== null && _c !== void 0 ? _c : true;
     }
     printsToStdio() {
         return this.shardIndex === undefined || this.shardIndex === 0;
@@ -96,7 +96,7 @@ class PlaywrightPulseReporter {
             : undefined;
         this._ensureDirExists(this.outputDir)
             .then(() => {
-            if (this.shardIndex === undefined || this.shardIndex === 0) {
+            if (this.printsToStdio()) {
                 console.log(`PlaywrightPulseReporter: Starting test run with ${suite.allTests().length} tests${this.isSharded ? ` across ${totalShards} shards` : ""}. Pulse outputting to ${this.outputDir}`);
                 if (this.shardIndex === undefined ||
                     (this.isSharded && this.shardIndex === 0)) {
@@ -262,27 +262,19 @@ class PlaywrightPulseReporter {
             stderr: stderrMessages.length > 0 ? stderrMessages : undefined,
             ...testSpecificData,
         };
-        // --- CORRECTED ATTACHMENT PROCESSING LOGIC ---
         for (const [index, attachment] of result.attachments.entries()) {
             if (!attachment.path)
                 continue;
             try {
-                // Create a sanitized, unique folder name for this specific test
                 const testSubfolder = test.id.replace(/[^a-zA-Z0-9_-]/g, "_");
-                // Sanitize the original attachment name to create a safe filename
                 const safeAttachmentName = path
                     .basename(attachment.path)
                     .replace(/[^a-zA-Z0-9_.-]/g, "_");
-                // Create a unique filename to prevent collisions, especially in retries
                 const uniqueFileName = `${index}-${Date.now()}-${safeAttachmentName}`;
-                // This is the relative path that will be stored in the JSON report
                 const relativeDestPath = path.join(ATTACHMENTS_SUBDIR, testSubfolder, uniqueFileName);
-                // This is the absolute path used for the actual file system operation
                 const absoluteDestPath = path.join(this.outputDir, relativeDestPath);
-                // Ensure the unique, test-specific attachment directory exists
                 await this._ensureDirExists(path.dirname(absoluteDestPath));
                 await fs.copyFile(attachment.path, absoluteDestPath);
-                // Categorize the attachment based on its content type
                 if (attachment.contentType.startsWith("image/")) {
                     (_j = pulseResult.screenshots) === null || _j === void 0 ? void 0 : _j.push(relativeDestPath);
                 }
@@ -294,8 +286,8 @@ class PlaywrightPulseReporter {
                 }
                 else {
                     (_l = pulseResult.attachments) === null || _l === void 0 ? void 0 : _l.push({
-                        name: attachment.name, // The original, human-readable name
-                        path: relativeDestPath, // The safe, relative path for linking
+                        name: attachment.name,
+                        path: relativeDestPath,
                         contentType: attachment.contentType,
                     });
                 }
@@ -304,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;
@@ -367,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;
@@ -425,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`;
@@ -432,38 +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);
-            if (finalReport && finalReport.run && !finalReport.run.environment) {
-                finalReport.run.environment = environmentDetails;
-            }
         }
         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;
-            const reviveDates = (key, value) => {
-                const isoDateRegex = /^\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}(\.\d+)?Z$/;
-                if (typeof value === "string" && isoDateRegex.test(value)) {
-                    const date = new Date(value);
-                    return !isNaN(date.getTime()) ? date : value;
-                }
-                return value;
-            };
-            const properlyTypedResults = JSON.parse(JSON.stringify(this.results), reviveDates);
             finalReport = {
                 run: runData,
-                results: properlyTypedResults,
+                // Use the de-duplicated results
+                results: finalResults,
                 metadata: { generatedAt: new Date().toISOString() },
             };
         }
@@ -471,17 +448,18 @@ class PlaywrightPulseReporter {
             console.error("PlaywrightPulseReporter: CRITICAL - finalReport object was not generated. Cannot create summary.");
             return;
         }
-        if (this.resetOnEachRun == true) {
+        const jsonReplacer = (key, value) => {
+            if (value instanceof Date)
+                return value.toISOString();
+            if (typeof value === "bigint")
+                return value.toString();
+            return value;
+        };
+        if (this.resetOnEachRun) {
             const finalOutputPath = path.join(this.outputDir, this.baseOutputFile);
             try {
                 await this._ensureDirExists(this.outputDir);
-                await fs.writeFile(finalOutputPath, JSON.stringify(finalReport, (key, value) => {
-                    if (value instanceof Date)
-                        return value.toISOString();
-                    if (typeof value === "bigint")
-                        return value.toString();
-                    return value;
-                }, 2));
+                await fs.writeFile(finalOutputPath, JSON.stringify(finalReport, jsonReplacer, 2));
                 if (this.printsToStdio()) {
                     console.log(`PlaywrightPulseReporter: JSON report written to ${finalOutputPath}`);
                 }
@@ -489,101 +467,109 @@ class PlaywrightPulseReporter {
             catch (error) {
                 console.error(`Pulse Reporter: Failed to write final JSON report to ${finalOutputPath}. Error: ${error.message}`);
             }
-            finally {
-                if (this.isSharded) {
-                    await this._cleanupTemporaryFiles();
-                }
-            }
         }
         else {
-            console.warn("PlaywrightPulseReporter: resetOnEachRun is set to false. The finalReport will display all the results present in '/pulse-results'.");
-            const finalOutputPath = path.join(`${this.outputDir}/pulse-results`, `playwright-pulse-report-${Date.now()}.json`);
+            // Logic for appending/merging reports
+            const pulseResultsDir = path.join(this.outputDir, INDIVIDUAL_REPORTS_SUBDIR);
+            const individualReportPath = path.join(pulseResultsDir, `playwright-pulse-report-${Date.now()}.json`);
             try {
-                await this._ensureDirExists(this.outputDir);
-                await fs.writeFile(finalOutputPath, JSON.stringify(finalReport, (key, value) => {
-                    if (value instanceof Date)
-                        return value.toISOString();
-                    if (typeof value === "bigint")
-                        return value.toString();
-                    return value;
-                }, 2));
+                await this._ensureDirExists(pulseResultsDir);
+                await fs.writeFile(individualReportPath, JSON.stringify(finalReport, jsonReplacer, 2));
                 if (this.printsToStdio()) {
-                    console.log(`PlaywrightPulseReporter: JSON report written to ${finalOutputPath}`);
+                    console.log(`PlaywrightPulseReporter: Individual run report for merging written to ${individualReportPath}`);
                 }
+                await this._mergeAllRunReports();
             }
             catch (error) {
-                console.error(`Pulse Reporter: Failed to write final JSON report to ${finalOutputPath}. Error: ${error.message}`);
+                console.error(`Pulse Reporter: Failed to write or merge report. Error: ${error.message}`);
             }
-            finally {
-                if (this.isSharded) {
-                    await this._cleanupTemporaryFiles();
+        }
+        if (this.isSharded) {
+            await this._cleanupTemporaryFiles();
+        }
+    }
+    async _mergeAllRunReports() {
+        const pulseResultsDir = path.join(this.outputDir, INDIVIDUAL_REPORTS_SUBDIR);
+        const finalOutputPath = path.join(this.outputDir, this.baseOutputFile);
+        let reportFiles;
+        try {
+            const allFiles = await fs.readdir(pulseResultsDir);
+            reportFiles = allFiles.filter((file) => file.startsWith("playwright-pulse-report-") && file.endsWith(".json"));
+        }
+        catch (error) {
+            if (error.code === "ENOENT") {
+                if (this.printsToStdio()) {
+                    console.log(`Pulse Reporter: No individual reports directory found at ${pulseResultsDir}. Skipping merge.`);
                 }
-                await this.falseResetOnEachRun();
+                return;
             }
+            console.error(`Pulse Reporter: Error reading report directory ${pulseResultsDir}:`, error);
+            return;
         }
-    }
-    async falseResetOnEachRun() {
-        const REPORT_DIR = "./pulse-report"; // Or change this to your reports directory
-        const OUTPUT_FILE = "playwright-pulse-report.json";
-        function getReportFiles(dir) {
-            return fsSync
-                .readdirSync(dir)
-                .filter((file) => file.startsWith("playwright-pulse-report-") &&
-                file.endsWith(".json"));
+        if (reportFiles.length === 0) {
+            if (this.printsToStdio()) {
+                console.log("Pulse Reporter: No matching JSON report files found to merge.");
+            }
+            return;
         }
-        function mergeReports(files) {
-            var _a;
-            let combinedRun = {
-                totalTests: 0,
-                passed: 0,
-                failed: 0,
-                skipped: 0,
-                duration: 0,
-                environment: {},
-            };
-            let combinedResults = [];
-            let latestTimestamp = "";
-            let latestGeneratedAt = "";
-            for (const file of files) {
-                const filePath = path.join(`${REPORT_DIR}/pulse-results`, file);
-                const json = JSON.parse(fsSync.readFileSync(filePath, "utf-8"));
-                const run = json.run || {};
-                combinedRun.totalTests += run.totalTests || 0;
-                combinedRun.passed += run.passed || 0;
-                combinedRun.failed += run.failed || 0;
-                combinedRun.skipped += run.skipped || 0;
-                combinedRun.duration += run.duration || 0;
-                combinedRun.environment = run.environment || {};
+        const combinedRun = {
+            id: `merged-${Date.now()}`,
+            timestamp: new Date(0),
+            totalTests: 0,
+            passed: 0,
+            failed: 0,
+            skipped: 0,
+            duration: 0,
+            environment: undefined,
+        };
+        const allResults = [];
+        let latestTimestamp = new Date(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;
+                    }
+                }
                 if (json.results) {
-                    combinedResults.push(...json.results);
+                    allResults.push(...json.results);
                 }
-                if (run.timestamp > latestTimestamp)
-                    latestTimestamp = run.timestamp;
-                if (((_a = json.metadata) === null || _a === void 0 ? void 0 : _a.generatedAt) > latestGeneratedAt)
-                    latestGeneratedAt = json.metadata.generatedAt;
             }
-            const finalJson = {
-                run: {
-                    id: `merged-${Date.now()}`,
-                    timestamp: latestTimestamp,
-                    ...combinedRun,
-                },
-                results: combinedResults,
-                metadata: {
-                    generatedAt: latestGeneratedAt,
-                },
-            };
-            return finalJson;
+            catch (err) {
+                console.warn(`Pulse Reporter: Could not parse report file ${filePath}. Skipping. Error: ${err.message}`);
+            }
         }
-        // Main execution
-        const reportFiles = getReportFiles(REPORT_DIR);
-        if (reportFiles.length === 0) {
-            console.log("No matching JSON report files found.");
-            process.exit(1);
+        combinedRun.timestamp = latestTimestamp;
+        const finalReport = {
+            run: combinedRun,
+            results: allResults,
+            metadata: {
+                generatedAt: new Date().toISOString(),
+            },
+        };
+        try {
+            await fs.writeFile(finalOutputPath, JSON.stringify(finalReport, (key, value) => {
+                if (value instanceof Date)
+                    return value.toISOString();
+                return value;
+            }, 2));
+            if (this.printsToStdio()) {
+                console.log(`PlaywrightPulseReporter: ✅ Merged report with ${allResults.length} total results saved to ${finalOutputPath}`);
+            }
+        }
+        catch (err) {
+            console.error(`Pulse Reporter: Failed to write final merged report to ${finalOutputPath}. Error: ${err.message}`);
         }
-        const merged = mergeReports(reportFiles);
-        fsSync.writeFileSync(path.join(REPORT_DIR, OUTPUT_FILE), JSON.stringify(merged, null, 2));
-        console.log(`✅ Merged report saved as ${OUTPUT_FILE}`);
     }
 }
 exports.PlaywrightPulseReporter = PlaywrightPulseReporter;

package/package.json

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