@arghajit/dummy 0.3.40 → 0.3.42

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

@@ -601,7 +601,8 @@ class PlaywrightPulseReporter {
         else {
             // Logic for appending/merging reports
             const pulseResultsDir = path.join(this.outputDir, this.individualReportsSubDir);
-            const individualReportPath = path.join(pulseResultsDir, `playwright-pulse-report-${Date.now()}.json`);
+            const shardPrefix = this.baseOutputFile.replace(".json", "-");
+            const individualReportPath = path.join(pulseResultsDir, `${shardPrefix}${Date.now()}.json`);
             try {
                 await this._ensureDirExists(pulseResultsDir);
                 await fs.writeFile(individualReportPath, JSON.stringify(finalReport, jsonReplacer, 2));

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@arghajit/dummy",
   "author": "Arghajit Singha",
-  "version": "0.3.40",
+  "version": "0.3.42",
   "description": "A Playwright reporter and dashboard for visualizing test results.",
   "homepage": "https://arghajit47.github.io/playwright-pulse/",
   "repository": {
@@ -35,7 +35,7 @@
   ],
   "license": "MIT",
   "bin": {
-    "logo": "node scripts/terminal-logo.mjs",
+    "logo": "scripts/terminal-logo.mjs",
     "generate-pulse-report": "scripts/generate-static-report.mjs",
     "generate-report": "scripts/generate-report.mjs",
     "merge-pulse-report": "scripts/merge-pulse-report.mjs",

package/scripts/config-reader.mjs

@@ -34,17 +34,34 @@ async function extractReporterOptionsFromConfig(configPath) {
 
   // 1. Strategy: Text Parsing (Safe & Fast)
   try {
-    const outputDirMatch = fileContent.match(/outputDir:\s*["']([^"']+)["']/);
-    if (outputDirMatch && outputDirMatch[1]) options.outputDir = outputDirMatch[1];
-
-    const outputFileMatch = fileContent.match(/outputFile:\s*["']([^"']+)["']/);
-    if (outputFileMatch && outputFileMatch[1]) options.outputFile = outputFileMatch[1];
-
-    const resetOnEachRunMatch = fileContent.match(/resetOnEachRun:\s*(true|false)/);
-    if (resetOnEachRunMatch) options.resetOnEachRun = resetOnEachRunMatch[1] === "true";
+    // Find the Pulse/Dummy reporter block
+    // It usually looks like ["@arghajit/playwright-pulse-report", { ... }] or ["playwright-pulse-report", { ... }]
+    const reporterBlockRegex = /\[\s*["'](?:@arghajit\/)?(?:playwright-pulse-report|dummy)["']\s*,\s*(\{[\s\S]*?\})\s*,?\s*\]/g;
+    let match;
+    while ((match = reporterBlockRegex.exec(fileContent)) !== null) {
+      const block = match[1];
+      
+      const outputDirMatch = block.match(/outputDir:\s*["']([^"']+)["']/);
+      if (outputDirMatch && outputDirMatch[1]) options.outputDir = outputDirMatch[1];
+
+      const outputFileMatch = block.match(/outputFile:\s*["']([^"']+)["']/);
+      if (outputFileMatch && outputFileMatch[1]) options.outputFile = outputFileMatch[1];
+
+      const resetOnEachRunMatch = block.match(/resetOnEachRun:\s*(true|false)/);
+      if (resetOnEachRunMatch) options.resetOnEachRun = resetOnEachRunMatch[1] === "true";
+
+      const individualReportsSubDirMatch = block.match(/individualReportsSubDir:\s*["']([^"']+)["']/);
+      if (individualReportsSubDirMatch && individualReportsSubDirMatch[1]) options.individualReportsSubDir = individualReportsSubDirMatch[1];
+      
+      // If we found any Pulse-specific options, we're likely in the right block
+      if (Object.keys(options).length > 0) break;
+    }
 
-    const individualReportsSubDirMatch = fileContent.match(/individualReportsSubDir:\s*["']([^"']+)["']/);
-    if (individualReportsSubDirMatch && individualReportsSubDirMatch[1]) options.individualReportsSubDir = individualReportsSubDirMatch[1];
+    // Fallback for global outputDir if not found in reporter block
+    if (!options.outputDir) {
+      const globalOutputDirMatch = fileContent.match(/outputDir:\s*["']([^"']+)["']/);
+      if (globalOutputDirMatch && globalOutputDirMatch[1]) options.outputDir = globalOutputDirMatch[1];
+    }
   } catch (e) { }
 
   // 2. Safety Check and Dynamic Import
@@ -104,7 +121,8 @@ async function extractReporterOptionsFromConfig(configPath) {
 
                 if (typeof reporterName === "string" && 
                    (reporterName.includes("playwright-pulse-report") || 
-                    reporterName.includes("@arghajit/playwright-pulse-report"))) {
+                    reporterName.includes("@arghajit/playwright-pulse-report") ||
+                    reporterName.includes("@arghajit/dummy"))) {
                   if (reporterOptions) {
                     if (reporterOptions.outputDir) options.outputDir = reporterOptions.outputDir;
                     if (reporterOptions.outputFile) options.outputFile = reporterOptions.outputFile;

package/scripts/merge-sequential-reports.mjs

@@ -4,8 +4,8 @@ import * as path from "path";
 import { getReporterConfig } from "./config-reader.mjs";
 
 /**
- * Reads all `playwright-pulse-report-*.json` files in the `pulse-results` directory
- * and merges them into a single `playwright-pulse-report.json`.
+ * Reads all `<outputFile>-*.json` files in the `pulse-results` directory
+ * and merges them into a single `<outputFile>.json`.
  * It resolves duplicate tests using exactly the same logic as the reporter.
  * 
  * @param {string} customOutputDir The base report directory override (from CLI).
@@ -25,12 +25,15 @@ export async function mergeSequentialReportsIfNeeded(customOutputDir) {
   const pulseResultsDir = path.join(outputDir, individualReportsSubDir);
   const finalOutputPath = path.join(outputDir, baseOutputFile);
 
+  // Use the actual outputFile name as seed for shard files (e.g. "results.json" -> "results-")
+  const shardPrefix = baseOutputFile.replace(".json", "-");
+
   let reportFiles;
   try {
     const allFiles = await fs.readdir(pulseResultsDir);
     reportFiles = allFiles.filter(
       (file) =>
-        file.startsWith("playwright-pulse-report-") && file.endsWith(".json"),
+        file.startsWith(shardPrefix) && file.endsWith(".json"),
     );
   } catch (error) {
     if (error.code === "ENOENT") {
@@ -64,15 +67,24 @@ export async function mergeSequentialReportsIfNeeded(customOutputDir) {
       const content = await fs.readFile(filePath, "utf-8");
       const json = JSON.parse(content);
 
+      let currentRunId = `run-${Date.now()}`;
       if (json.run) {
+        if (json.run.id) currentRunId = json.run.id;
+        
         const runTimestamp = new Date(json.run.timestamp);
         if (runTimestamp > latestTimestamp) {
           latestTimestamp = runTimestamp;
           lastRunEnvironment = json.run.environment || undefined;
         }
       }
+
       if (json.results) {
-        allResultsFromAllFiles.push(...json.results);
+        // Tag each result with its runId to ensure we can sum them up if they have same IDs but different runs
+        const resultsWithRunId = json.results.map((r) => ({
+          ...r,
+          runId: currentRunId,
+        }));
+        allResultsFromAllFiles.push(...resultsWithRunId);
       }
     } catch (err) {
       console.warn(
@@ -90,7 +102,7 @@ export async function mergeSequentialReportsIfNeeded(customOutputDir) {
   );
 
   const combinedRun = {
-    id: `run-${Date.now()}-581d5ad8-ce75-4ca5-94a6-ed29c466c815`,
+    id: `run-${Date.now()}`,
     timestamp: latestTimestamp.toISOString(),
     environment: lastRunEnvironment,
     totalTests: finalMergedResults.length,
@@ -155,10 +167,12 @@ function getFinalizedResults(allResults) {
   const resultsMap = new Map();
 
   for (const result of allResults) {
-    if (!resultsMap.has(result.id)) {
-      resultsMap.set(result.id, []);
+    // Unique key is runId + testId to support summation of tests across different sequential runs
+    const compositeId = `${result.runId || "unknown"}-${result.id}`;
+    if (!resultsMap.has(compositeId)) {
+      resultsMap.set(compositeId, []);
     }
-    resultsMap.get(result.id).push(result);
+    resultsMap.get(compositeId).push(result);
   }
 
   const finalResults = [];