playwright-slack-report-burak 3.0.22 → 3.0.24

package/dist/src/LayoutGenerator.js

@@ -8,24 +8,6 @@ Object.defineProperty(exports, "__esModule", { value: true });
  exports.generateAllRunSuites = 
  void 0;
 
-// Helper function to get build URL and CI platform name
-const getBuildInfo = () => {
-    if (process.env.GITHUB_ACTIONS || process.env.SAFETYWINGTEST_GITHUB_TOKEN) {
-        const repo = process.env.GITHUB_REPOSITORY;
-        const runId = process.env.GITHUB_RUN_ID;
-        const serverUrl = process.env.GITHUB_SERVER_URL || 'https://github.com';
-        const buildUrl = `${serverUrl}/${repo}/actions/runs/${runId}`;
-        return {
-            url: buildUrl,
-            platform: 'GitHub Actions'
-        };
-    }
-    return {
-        url: '#',
-        platform: 'CI'
-    };
-};
-
 const generateAllRunSuites = async (summaryResults) => {
     const suitesResults = [];
     const allSuites = [];
@@ -273,13 +255,12 @@ const generateProblemCaseList = async (summaryResults) => {
         let text = `${title}${list.map((value, index) => `*${index + 1}.* ${value}`).join('\n')}`;
         if (text.length > 2700) {
             text = text.substring(0, 2700) + '...';
-            const buildInfo = getBuildInfo();
             array.push({
                 type: 'section',
                 text: {
                     type: 'mrkdwn',
-                    text: text + '\n\n\n' + `⚠️ *There are too many items to display here. ` +
-                          `You can view more on ${buildInfo.platform}* ⚠️`,
+                    text: text + '\n\n\n' + '⚠️ *There are too many items to display here. ' +
+                          'You can view more on Github Actions* ⚠️',
                 },
             },
             {
@@ -290,7 +271,7 @@ const generateProblemCaseList = async (summaryResults) => {
                         type: 'plain_text',
                         text: 'Go to Build Details'
                     },
-                    url: buildInfo.url
+                    url: 'https://github.com/safetywing/test-automation/actions'
                 }]
             });
         } else {
@@ -375,13 +356,12 @@ const generateFailuresReasons = async (summaryResults) => {
                 },
             });
             if (i === summaryResults.failures.length-1) {
-                const buildInfo = getBuildInfo();
                 failsList.push({
                     type: 'section',
                     text: {
                         type: 'mrkdwn',
                         text: '⚠️ *There are too many failures to display in full detail. '+
-                            `You can view more on ${buildInfo.platform}* ⚠️`,
+                            'You can view more on Github Actions* ⚠️',
                     },
                 },
                 {
@@ -393,7 +373,7 @@ const generateFailuresReasons = async (summaryResults) => {
                             type: 'plain_text',
                             text: 'Go to Build Details'
                         },
-                        url: buildInfo.url
+                        url: 'https://github.com/safetywing/test-automation/actions'
                     }
                 ]
                 });
@@ -411,13 +391,12 @@ const generateFailuresReasons = async (summaryResults) => {
                 },
             });
             if (i > 23) {
-                const buildInfo = getBuildInfo();
                 failsList.push({
                     type: 'section',
                     text: {
                         type: 'mrkdwn',
                         text: '⚠️ *There are too many failures to display here '+
-                            `You can view more on ${buildInfo.platform}* ⚠️`,
+                            'You can view more on Github Actions* ⚠️',
                     },
                 },
                 {
@@ -428,7 +407,7 @@ const generateFailuresReasons = async (summaryResults) => {
                             type: 'plain_text',
                             text: 'Go to Build Details'
                         },
-                        url: buildInfo.url
+                        url: 'https://github.com/safetywing/test-automation/actions'
                     }]
                 });
                 break;

package/dist/src/SlackReporter.js

@@ -123,12 +123,17 @@ class SlackReporter {
         // This ensures ResultsParser always has correct values, even if we block posting
         this.resultsParser.updateShardInfo(shardIndexNum, totalShards);
         
+        // CRITICAL: ALL shards must create their node_summary files, even if they don't post to Slack
+        // This must happen BEFORE the early return for non-shard-1 shards
+        const resultSummary = await this.resultsParser.getParsedResults();
+        
         // Only allow shard 1 to post when there are multiple shards
         // This ensures only one shard posts the aggregated report
         if (totalShards > 1 && shardIndexNum !== 1) {
             this.log(`❌ [Shard Detection] BLOCKING: Non-shard-1 detected (shard ${shardIndexNum} of ${totalShards})`);
             this.log(`❌ Stopping reporter for shard ${shardIndexNum} of ${totalShards}`);
             this.log(`ℹ️ Only shard 1 will post the aggregated report.`);
+            this.log(`✅ Node summary file created for shard ${shardIndexNum}`);
             return;
         }
         
@@ -138,8 +143,6 @@ class SlackReporter {
         } else {
             this.log(`✅ [Shard Detection] ALLOWING: Multiple shards detected (${totalShards}). Shard ${shardIndexNum} will post aggregated report.`);
         }
-        
-        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

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