playwright-slack-report-burak 2.0.12 → 2.0.14

package/dist/src/ResultsParser.js

@@ -93,14 +93,12 @@ class ResultsParser {
         fs.writeFileSync(nodeSummaryFile, JSON.stringify(nodeSummary, null, 2));
 
         if (this.shardIndex === 0 && this.totalShardCount > 1) {
-      /*       console.log('Fetching all artifacts...');
-            await this.fetchAllArtifacts(); */
           if (process.env.CI) {
             console.log('Fetching all artifacts...');
             await this.fetchAllArtifacts();
           } else {
             while (!this.allNodeSummaryFilesExist() || !this.allBlobZipsExist()) {
-              console.log('Waiting for all blob zips to exist...');
+              console.log('Waiting for all both to exist...');
               await new Promise(resolve => setTimeout(resolve, 1000)); // Wait for 1 second
             }
           }
@@ -108,7 +106,7 @@ class ResultsParser {
 
         console.log('Contents of playwright-report for node', this.shardIndex, ':', fs.readdirSync(summariesDir));
 
-        if (this.allNodeSummaryFilesExist() && this.allBlobZipsExist()) {
+        if (this.allNodeSummaryFilesExist() && this.allBlobZipsExist() && this.shardIndex === 0) {
             // Merge all node summaries into the final summary
             for (let i = 0; i < this.totalShardCount; i++) {
                 const nodeSummaryFile = path.join(summariesDir, `node_summary_${i}.json`);
@@ -343,9 +341,10 @@ class ResultsParser {
     mergeReports() {
       try {
         // Execute the command to merge reports
-        exec('npx playwright merge-reports --reporter html ./playwright-report', { stdio: 'inherit' });
+        exec('npx playwright merge-reports --reporter html playwright-report');
+        //new Promise(resolve => setTimeout(resolve, 5000));
         console.log('Reports merged successfully.');
-        console.log('Contents of playwright-report for node after merge:', fs.readdirSync('./playwright-report'));
+        console.log('Contents of playwright-report:', fs.readdirSync('./playwright-report'));
       } catch (error) {
           // Log a warning instead of throwing an error
           console.warn('Warning: Failed to merge reports. This may not affect the overall process.', error.message);

package/dist/src/SlackReporter.js

@@ -70,12 +70,12 @@ class SlackReporter {
             this.log(message);
             return;
         }
-        const { stopReporter, currentShardIndex, totalShardCount } = await this.resultsParser.getParsedResults();
-        if (stopReporter) {
-            this.log(`❌ Stopping reporter for non-zero index node ${currentShardIndex} of ${totalShardCount}`);
+        const resultSummary = await this.resultsParser.getParsedResults();
+        // SHARDING SUPPORT - Stop slack messages for non-zero index node
+        if (process.env.CIRCLE_NODE_INDEX && process.env.CIRCLE_NODE_INDEX !== '0') {
+            this.log(`❌ Stopping reporter for non-zero index node ${process.env.CIRCLE_NODE_INDEX} of ${process.env.CIRCLE_NODE_TOTAL}`);
             return;
         }
-        const resultSummary = await this.resultsParser.getParsedResults();
         resultSummary.meta = this.meta;
         const maxRetry = Math.max(...resultSummary.tests.map((o) => o.retry));
         if (this.sendResults === 'on-failure'

package/package.json

@@ -30,7 +30,7 @@
     "lint-fix": "npx eslint . --ext .ts --fix"
   },
   "name": "playwright-slack-report-burak",
-  "version": "2.0.12",
+  "version": "2.0.14",
   "main": "index.js",
   "types": "dist/index.d.ts",
   "repository": "git@github.com:ryanrosello-og/playwright-slack-report.git",