@arghajit/dummy 0.3.19 → 0.3.27

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

@@ -116,33 +116,22 @@ class PlaywrightPulseReporter {
         const severityAnnotation = annotations.find((a) => a.type === "pulse_severity");
         return (severityAnnotation === null || severityAnnotation === void 0 ? void 0 : severityAnnotation.description) || "Medium";
     }
-    async extractCodeSnippet(filePath, targetLine, targetColumn) {
+    extractCodeSnippet(filePath, targetLine, targetColumn) {
+        var _a;
         try {
-            const fileContent = await fs.readFile(filePath, "utf-8");
-            const lines = fileContent.split("\n");
-            if (targetLine < 1 || targetLine > lines.length) {
-                return undefined;
+            const fsSync = require('fs');
+            if (!fsSync.existsSync(filePath)) {
+                return '';
             }
-            const contextLines = 2;
-            const startLine = Math.max(0, targetLine - contextLines - 1);
-            const endLine = Math.min(lines.length, targetLine + contextLines);
-            const snippetLines = [];
-            for (let i = startLine; i < endLine; i++) {
-                const lineNum = i + 1;
-                const isTargetLine = lineNum === targetLine;
-                const lineContent = lines[i] || "";
-                const prefix = isTargetLine ? ">" : " ";
-                snippetLines.push(`${prefix} ${lineNum.toString().padStart(4)} | ${lineContent}`);
-                if (isTargetLine && targetColumn > 0) {
-                    const pointer = "  " + " ".repeat(4) + " | " + " ".repeat(Math.max(0, targetColumn)) + "^";
-                    snippetLines.push(pointer);
-                }
+            const content = fsSync.readFileSync(filePath, 'utf8');
+            const lines = content.split('\n');
+            if (targetLine < 1 || targetLine > lines.length) {
+                return '';
             }
-            return snippetLines.length > 0 ? snippetLines.join("\n") : undefined;
+            return ((_a = lines[targetLine - 1]) === null || _a === void 0 ? void 0 : _a.trim()) || '';
         }
-        catch (error) {
-            console.error(`Failed to extract code snippet from ${filePath}:${targetLine}:${targetColumn}`, error);
-            return undefined;
+        catch (e) {
+            return '';
         }
     }
     getBrowserDetails(test) {
@@ -195,7 +184,7 @@ class PlaywrightPulseReporter {
         }
         return finalString.trim();
     }
-    async processStep(step, testId, browserDetails, testCase, isFailedStep = false) {
+    async processStep(step, testId, browserDetails, testCase) {
         var _a, _b, _c, _d;
         let stepStatus = "passed";
         let errorMessage = ((_a = step.error) === null || _a === void 0 ? void 0 : _a.message) || undefined;
@@ -209,18 +198,10 @@ class PlaywrightPulseReporter {
         const startTime = new Date(step.startTime);
         const endTime = new Date(startTime.getTime() + Math.max(0, duration));
         let codeLocation = "";
-        let codeSnippet = undefined;
+        let codeSnippet = '';
         if (step.location) {
             codeLocation = `${path.relative(this.config.rootDir, step.location.file)}:${step.location.line}:${step.location.column}`;
-            try {
-                codeSnippet = await this.extractCodeSnippet(step.location.file, step.location.line, step.location.column);
-                if (!codeSnippet) {
-                    console.warn(`Pulse Reporter: extractCodeSnippet returned undefined for step "${step.title}" at ${step.location.file}:${step.location.line}:${step.location.column}`);
-                }
-            }
-            catch (error) {
-                console.error(`Pulse Reporter: Failed to extract code snippet for step "${step.title}":`, error);
-            }
+            codeSnippet = this.extractCodeSnippet(step.location.file, step.location.line, step.location.column);
         }
         return {
             id: `${testId}_step_${startTime.toISOString()}-${duration}-${(0, crypto_1.randomUUID)()}`,
@@ -240,7 +221,6 @@ class PlaywrightPulseReporter {
                     ? "before"
                     : "after"
                 : undefined,
-            isFailedStep: isFailedStep,
             steps: [],
         };
     }
@@ -248,33 +228,35 @@ class PlaywrightPulseReporter {
         var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l, _m, _o, _p;
         const project = (_a = test.parent) === null || _a === void 0 ? void 0 : _a.project();
         const browserDetails = this.getBrowserDetails(test);
-        const testStatus = convertStatus(result.status, test);
+        // Captured outcome from Playwright
+        const outcome = test.outcome();
+        // Calculate final status based on the last result (Last-Run-Wins)
+        // result.status in onTestEnd is typically the status of the test run (passed if flaky passed)
+        // But we double check the last result in test.results just to be sure/consistent
+        const lastResult = test.results[test.results.length - 1];
+        const finalStatus = convertStatus(lastResult ? lastResult.status : result.status, test);
+        // Existing behavior: fail if flaky (implied by user request "existing status field should remain failed")
+        // If outcome is flaky, status should be 'failed' to indicate initial failure, but final_status is 'passed'
+        let testStatus = finalStatus;
+        if (outcome === 'flaky') {
+            testStatus = 'flaky';
+        }
         const startTime = new Date(result.startTime);
         const endTime = new Date(startTime.getTime() + result.duration);
-        const processAllSteps = async (steps, parentFailed = false) => {
+        const processAllSteps = async (steps) => {
             let processed = [];
-            let foundFailedStep = false;
             for (const step of steps) {
-                const isThisStepFailed = !foundFailedStep && (step.error !== undefined || (testStatus === "failed" && !foundFailedStep));
-                if (isThisStepFailed) {
-                    foundFailedStep = true;
-                }
-                const processedStep = await this.processStep(step, test.id, browserDetails, test, isThisStepFailed);
+                const processedStep = await this.processStep(step, test.id, browserDetails, test);
                 processed.push(processedStep);
                 if (step.steps && step.steps.length > 0) {
-                    processedStep.steps = await processAllSteps(step.steps, isThisStepFailed);
+                    processedStep.steps = await processAllSteps(step.steps);
                 }
             }
             return processed;
         };
-        let codeSnippet = undefined;
-        try {
-            if (((_b = test.location) === null || _b === void 0 ? void 0 : _b.file) && ((_c = test.location) === null || _c === void 0 ? void 0 : _c.line) && ((_d = test.location) === null || _d === void 0 ? void 0 : _d.column)) {
-                codeSnippet = await this.extractCodeSnippet(test.location.file, test.location.line, test.location.column);
-            }
-        }
-        catch (e) {
-            console.warn(`Pulse Reporter: Could not extract code snippet for ${test.title}`, e);
+        let codeSnippet = '';
+        if (((_b = test.location) === null || _b === void 0 ? void 0 : _b.file) && ((_c = test.location) === null || _c === void 0 ? void 0 : _c.line) && ((_d = test.location) === null || _d === void 0 ? void 0 : _d.column)) {
+            codeSnippet = this.extractCodeSnippet(test.location.file, test.location.line, test.location.column);
         }
         // 1. Get Spec File Name
         const specFileName = ((_e = test.location) === null || _e === void 0 ? void 0 : _e.file)
@@ -308,6 +290,8 @@ class PlaywrightPulseReporter {
             name: test.titlePath().join(" > "),
             suiteName: (project === null || project === void 0 ? void 0 : project.name) || ((_g = this.config.projects[0]) === null || _g === void 0 ? void 0 : _g.name) || "Default Suite",
             status: testStatus,
+            outcome: outcome === 'flaky' ? outcome : undefined, // Only Include if flaky
+            final_status: finalStatus, // New Field
             duration: result.duration,
             startTime: startTime,
             endTime: endTime,
@@ -366,15 +350,35 @@ class PlaywrightPulseReporter {
         this.results.push(pulseResult);
     }
     _getFinalizedResults(allResults) {
-        const finalResultsMap = new Map();
+        const resultsMap = 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);
+            if (!resultsMap.has(result.id)) {
+                resultsMap.set(result.id, []);
+            }
+            resultsMap.get(result.id).push(result);
+        }
+        const finalResults = [];
+        for (const [testId, attempts] of resultsMap.entries()) {
+            attempts.sort((a, b) => a.retries - b.retries);
+            const firstAttempt = attempts[0];
+            const retryAttempts = attempts.slice(1);
+            if (retryAttempts.length > 0) {
+                firstAttempt.retryHistory = retryAttempts;
+                // Calculate final status and outcome from the last attempt if retries exist
+                const lastAttempt = attempts[attempts.length - 1];
+                firstAttempt.final_status = lastAttempt.status;
+                // If the last attempt was flaky, ensure outcome is set on the main result
+                if (lastAttempt.outcome === 'flaky') {
+                    firstAttempt.outcome = 'flaky';
+                }
+            }
+            else {
+                // If no retries, ensure final_status is undefined (as requested)
+                delete firstAttempt.final_status;
             }
+            finalResults.push(firstAttempt);
         }
-        return Array.from(finalResultsMap.values());
+        return finalResults;
     }
     onError(error) {
         var _a;

package/dist/types/index.d.ts

@@ -1,5 +1,5 @@
 import type { LucideIcon } from "lucide-react";
-export type TestStatus = "passed" | "failed" | "skipped" | "expected-failure" | "unexpected-success" | "explicitly-skipped";
+export type TestStatus = "passed" | "failed" | "skipped" | "expected-failure" | "unexpected-success" | "explicitly-skipped" | "flaky";
 export interface TestStep {
     id: string;
     title: string;
@@ -14,7 +14,6 @@ export interface TestStep {
     codeSnippet?: string;
     isHook?: boolean;
     hookType?: "before" | "after";
-    isFailedStep?: boolean;
     steps?: TestStep[];
 }
 export interface TestResult {
@@ -37,6 +36,8 @@ export interface TestResult {
     suiteName?: string;
     runId: string;
     browser: string;
+    outcome?: string;
+    final_status?: TestStatus;
     screenshots?: string[];
     videoPath?: string[];
     tracePath?: string;
@@ -60,6 +61,7 @@ export interface TestResult {
             column: number;
         };
     }[];
+    retryHistory?: TestResult[];
 }
 export interface TestRun {
     id: string;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@arghajit/dummy",
   "author": "Arghajit Singha",
-  "version": "0.3.19",
+  "version": "0.3.27",
   "description": "A Playwright reporter and dashboard for visualizing test results.",
   "homepage": "https://arghajit47.github.io/playwright-pulse/",
   "repository": {

package/scripts/generate-email-report.mjs

@@ -242,6 +242,17 @@ function generateMinifiedHTML(reportData) {
           severity
         )}; font-size: 0.8em; font-weight: 600; padding: 3px 8px; border-radius: 4px; color: #fff; margin-left: 10px; white-space: nowrap;">${severity}</span>`;
 
+        // --- NEW: Retry Count Badge ---
+        const retryCountBadge = (test.retryHistory && test.retryHistory.length > 0)
+          ? `<span style="background-color: #f59e0b; border: 1px solid #d97706; font-size: 0.8em; font-weight: 700; padding: 4px 10px; border-radius: 50px; color: #fff; margin-left: 10px; white-space: nowrap; display: inline-flex; align-items: center; gap: 4px;">
+              <svg width="12" height="12" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" style="vertical-align: middle;">
+                <path d="M1 4v6h6"/>
+                <path d="M3.51 15a9 9 0 1 0 2.13-9.36L1 10"/>
+              </svg>
+              Retry Count: ${test.retryHistory.length}
+            </span>`
+          : '';
+
         // --- NEW: Tags Logic ---
         const tagsBadges = (test.tags || [])
           .map(
@@ -266,6 +277,7 @@ function generateMinifiedHTML(reportData) {
                 test.name
               )}">${sanitizeHTML(testTitle)}</span>
               
+              ${retryCountBadge}
               ${severityBadge}
               ${tagsBadges}
             </li>
@@ -532,10 +544,10 @@ function generateMinifiedHTML(reportData) {
             </div>
             <div class="run-info">
                 <strong>Run Date:</strong> ${formatDate(
-                  runSummary.timestamp
+                  runSummary.timestamp,
                 )}<br>
                 <strong>Total Duration:</strong> ${formatDuration(
-                  runSummary.duration
+                  runSummary.duration,
                 )}
             </div>
         </header>
@@ -583,11 +595,11 @@ function generateMinifiedHTML(reportData) {
             <div class="filters-section">
                 <input type="text" id="filter-min-name" placeholder="Search by test name...">
                 <select id="filter-min-status">
-                    <option value="">All Statuses</option>
-                    <option value="passed">Passed</option>
-                    <option value="failed">Failed</option>
-                    <option value="skipped">Skipped</option>
-                    <option value="unknown">Unknown</option>
+                    <option value="">All Status</option>
+                    <option value="passed">✅ Passed</option>
+                    <option value="failed">❌ Failed</option>
+                    <option value="skipped">⏭️ Skipped</option>
+                    <option value="unknown">❓ Unknown</option>
                 </select>
                 <select id="filter-min-browser">
                     <option value="">All Browsers</option>
@@ -595,8 +607,8 @@ function generateMinifiedHTML(reportData) {
                       .map(
                         (browser) =>
                           `<option value="${sanitizeHTML(
-                            browser.toLowerCase()
-                          )}">${sanitizeHTML(capitalize(browser))}</option>`
+                            browser.toLowerCase(),
+                          )}">${sanitizeHTML(capitalize(browser))}</option>`,
                       )
                       .join("")}
                 </select>