playwright-slack-report-burak 3.2.3 → 3.3.0

package/dist/src/ResultsParser.js

@@ -53,8 +53,8 @@ const getGcsObjectPath = (relativePath) => `${getRunFolder()}/${relativePath}`;
 class ResultsParser {
     result;
     separateFlakyTests;
-    totalShardCount = process.env.TOTAL_SHARDS 
-        ? parseInt(process.env.TOTAL_SHARDS, 10) 
+    totalShardCount = process.env.CI && process.env.TOTAL_SHARDS
+        ? parseInt(process.env.TOTAL_SHARDS, 10)
         : 1;
     shardIndex = process.env.SHARD_INDEX !== undefined
         ? parseInt(process.env.SHARD_INDEX, 10)
@@ -129,21 +129,34 @@ class ResultsParser {
           }
         }
 
-        if (this.shardIndex === 1 && this.allNodeSummaryFilesExist() && this.allBlobZipsExist()) {
+        // Single shard: copy playwright-report to SUMMARIES_DIR so merge logic finds files
+        if (this.shardIndex === 1 && this.totalShardCount === 1) {
+          if (!fs.existsSync(SUMMARIES_DIR)) {
+            fs.mkdirSync(SUMMARIES_DIR, { recursive: true });
+          }
+          if (fs.existsSync(PLAYWRIGHT_REPORT_DIR)) {
+            this.copyDirectoryRecursive(PLAYWRIGHT_REPORT_DIR, SUMMARIES_DIR);
+          }
+        }
+
+        const canMerge = this.shardIndex === 1 && this.allNodeSummaryFilesExist() && this.allBlobZipsExist();
+        const singleShardNoBlob = this.shardIndex === 1 && this.totalShardCount === 1 && this.allNodeSummaryFilesExist() && !this.allBlobZipsExist();
+
+        if (canMerge) {
             // Merge all node summaries into the final summary
             for (let i = 1; i <= this.totalShardCount; i++) {
                 const nodeSummaryFile = path.join(SUMMARIES_DIR, `node_summary_${i}.json`);
                 const fileToRead = nodeSummaryFile;
 
                 if (fs.existsSync(fileToRead)) {
-                    const nodeSummary = JSON.parse(fs.readFileSync(fileToRead, 'utf-8'));
+                    const nodeSummaryData = JSON.parse(fs.readFileSync(fileToRead, 'utf-8'));
                     // Don't add the inflated totals - calculate from outcomes instead
-                    summary.passed += nodeSummary.passed;
-                    summary.failed += nodeSummary.failed;
-                    summary.flaky += nodeSummary.flaky;
-                    summary.skipped += nodeSummary.skipped;
-                    summary.failures = summary.failures.concat(nodeSummary.failures);
-                    summary.tests = summary.tests.concat(nodeSummary.tests);
+                    summary.passed += nodeSummaryData.passed;
+                    summary.failed += nodeSummaryData.failed;
+                    summary.flaky += nodeSummaryData.flaky;
+                    summary.skipped += nodeSummaryData.skipped;
+                    summary.failures = summary.failures.concat(nodeSummaryData.failures);
+                    summary.tests = summary.tests.concat(nodeSummaryData.tests);
                 }
             }
             
@@ -169,6 +182,17 @@ class ResultsParser {
             
             await this.mergeReports();
             await this.pushReportsToGCS();
+        } else if (singleShardNoBlob) {
+            // Single shard without blob (e.g. different reporter config): use node summary for Slack report
+            const nodeSummaryFile = path.join(SUMMARIES_DIR, 'node_summary_1.json');
+            const nodeSummaryData = JSON.parse(fs.readFileSync(nodeSummaryFile, 'utf-8'));
+            summary.passed = nodeSummaryData.passed;
+            summary.failed = nodeSummaryData.failed;
+            summary.flaky = nodeSummaryData.flaky;
+            summary.skipped = nodeSummaryData.skipped;
+            summary.failures = nodeSummaryData.failures || [];
+            summary.tests = nodeSummaryData.tests || [];
+            summary.total = summary.passed + summary.failed + summary.flaky + summary.skipped;
         }
         return summary;
     }
@@ -420,8 +444,12 @@ class ResultsParser {
                 },
                 params: { per_page: 100 }
             });
+            const runPrefix = `${GITHUB_RUN_ID}-${GITHUB_RUN_ATTEMPT}`;
             const artifact = (listResponse.data.artifacts || []).find(
-                (a) => (a.name || '').endsWith(`-artifacts-shard-${shard}`)
+                (a) => {
+                    const name = a.name || '';
+                    return name.startsWith(runPrefix) && name.endsWith(`-artifacts-shard-${shard}`);
+                }
             );
             if (!artifact?.archive_download_url) return false;
             const downloadResponse = await axios.get(artifact.archive_download_url, {
@@ -437,7 +465,7 @@ class ResultsParser {
                 const baseName = path.basename(entry.entryName);
                 if (targetFiles.includes(baseName)) {
                     fs.writeFileSync(path.join(SUMMARIES_DIR, baseName), entry.getData());
-                    console.log(`Successfully fetched ${baseName} from shard ${shard}`);
+                    console.log(`Successfully fetched ${baseName} from artifact ${artifact.name}`);
                 }
             }
             return true;

package/package.json

@@ -33,7 +33,7 @@
     "lint-fix": "npx eslint . --ext .ts --fix"
   },
   "name": "playwright-slack-report-burak",
-  "version": "3.2.3",
+  "version": "3.3.0",
   "main": "index.js",
   "types": "dist/index.d.ts",
   "author": "Burak B. <burak.boluk@hotmail.com>",