playwright-slack-report-burak 1.3.6 → 1.4.0

package/dist/src/LayoutGenerator.js

@@ -1,13 +1,14 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.generateFailures =
+//exports.generateBlocks =
+ //exports.generateFailures =
  exports.generateFailuresReasons =
  exports.generateProblemSuiteList =
  exports.generateProblemCaseList =
- exports.generateBlocks =
  exports.generateAllRunSuites = 
  void 0;
 
+/*
 const generateBlocks = async (summaryResults, maxNumberOfFailures) => {
     const meta = [];
     const header = {
@@ -113,6 +114,7 @@ const generateFailures = async (summaryResults) => {
         ...thread,
     ];
 };
+*/
 
 const generateAllRunSuites = async (summaryResults) => {
     const suitesResults = [];
@@ -144,30 +146,29 @@ const generateAllRunSuites = async (summaryResults) => {
       allSuitesTitle[0] = '*Suites Run:*\n';
     }
   
-    //  Push data into array to create the "List of Problematic Test Suites"
-    if (summaryResults.tests.length !== summaryResults.passed){
-      suitesResults.push({
-        type: 'section',
-        text: {
-          type: 'mrkdwn',
-          text:
-          `${allSuitesTitle}${[...new Set(allSuites)].map((value, index) => `*${index + 1}.* ${value}`).join('\n')}`
-        },
-      },
-      {
-        type: 'section',
-        text: {
-          type: 'mrkdwn',
-          text: ` `,
-        },
+    //  Push data into array to create the "List of All Run Test Suites"
+    suitesResults.push({
+      type: 'section',
+      text: {
+        type: 'mrkdwn',
+        text:
+        `${allSuitesTitle}${[...new Set(allSuites)].map((value, index) => `*${index + 1}.* ${value}`).join('\n')}`
       },
-      {
-        type: 'divider',
+    },
+    {
+      type: 'section',
+      text: {
+        type: 'mrkdwn',
+        text: ` `,
       },
-    )}
-    return [
-        ...suitesResults,
-    ];
+    },
+    {
+      type: 'divider',
+    },
+  )
+  return [
+      ...suitesResults,
+  ];
 };
 
 const generateProblemSuiteList = async (summaryResults) => {
@@ -339,48 +340,80 @@ const generateProblemCaseList = async (summaryResults) => {
   
     //  Decide on Titles For Listing Failures, Skips & Flakies
     if (summaryResults.failed === 1) {
-      failedNamesTitle[0] = '\n*Failed Test Case:*\n';
+      failedNamesTitle[0] = '*Failed Test Case:*\n';
     }
     if (summaryResults.failed > 1) {
-      failedNamesTitle[0] = '\n*Failed Test Cases:*\n';
+      failedNamesTitle[0] = '*Failed Test Cases:*\n';
     }
     if (summaryResults.flaky === 1) {
-      flakyNamesTitle[0] = '\n*Flaky Test Case:*\n';
+      flakyNamesTitle[0] = '*Flaky Test Case:*\n';
     }
     if (summaryResults.flaky > 1) {
-      flakyNamesTitle[0] = '\n*Flaky Test Cases:*\n';
+      flakyNamesTitle[0] = '*Flaky Test Cases:*\n';
     }
     if (summaryResults.skipped === 1) {
-      skippedNamesTitle[0] = '\n*Skipped Test Case:*\n';
+      skippedNamesTitle[0] = '*Skipped Test Case:*\n';
     }
     if (summaryResults.skipped > 1) {
-      skippedNamesTitle[0] = '\n*Skipped Test Cases:*\n';
+      skippedNamesTitle[0] = '*Skipped Test Cases:*\n';
     }
   
-    //  Push data into array to create the "List of Problematic Test Suites"
-    if (summaryResults.tests.length !== summaryResults.passed){
-      casesResults.push(
-      {
-          type: 'section',
-          text: {
-              type: 'mrkdwn',
-              text:
-              `\n${failedNamesTitle}${failsList.map((value, index) => `*${index + 1}.* ${value}`).join('\n')}` +  // List of Failed Test Cases
-              `\n${flakyNamesTitle}${flakyList.map((value, index) => `*${index + 1}.* ${value}`).join('\n')}` +  // List of Flaky Test Cases
-              `\n${skippedNamesTitle}${skipsList.map((value, index) => `*${index + 1}.* ${value}`).join('\n')} `, // List of Skipped Test Cases
-          },
-      },
-      {
-        type: 'section',
-        text: {
-          type: 'mrkdwn',
-          text: ` `,
+    // Function to safely push text to casesResults
+    const safePush = (title, list, array) => {
+        let text = `${title}${list.map((value, index) => `*${index + 1}.* ${value}`).join('\n')}\n`;
+        if (text.length > 2700) {
+            text = text.substring(0, 2700) + '...';
+            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 CircleCI* ⚠️',
+                },
+            },
+            {
+                type: 'actions',
+                elements: [{
+                    type: 'button',
+                    text: {
+                        type: 'plain_text',
+                        text: 'Go to Build Details'
+                    },
+                    url: process.env.CIRCLE_BUILD_URL
+                }]
+            });
+        } else {
+            casesResults.push({
+                type: 'section',
+                text: {
+                    type: 'mrkdwn',
+                    text: text,
+                },
+            });
+        }
+    };
+
+    // Push each section with character limit check
+    if (summaryResults.tests.length !== summaryResults.passed) {
+        safePush(failedNamesTitle, failsList, casesResults);
+        safePush(flakyNamesTitle, flakyList, casesResults);
+        safePush(skippedNamesTitle, skipsList, casesResults);
+    }
+
+    // Add additional sections if needed
+    casesResults.push(
+        {
+            type: 'section',
+            text: {
+                type: 'mrkdwn',
+                text: ` `,
+            },
         },
-      },
-      {
-        type: 'divider',
-      },
-    )}
+        {
+            type: 'divider',
+        },
+    );
+
     return [
         ...casesResults,
     ];
@@ -505,7 +538,7 @@ const generateFailuresReasons = async (summaryResults) => {
         ...failsList,
     ];
 };
-exports.generateFailures = generateFailures;
+//exports.generateFailures = generateFailures;
 exports.generateFailuresReasons = generateFailuresReasons;
 exports.generateProblemSuiteList = generateProblemSuiteList;
 exports.generateProblemCaseList = generateProblemCaseList;

package/dist/src/SlackReporter.js

@@ -111,8 +111,8 @@ class SlackReporter {
             });
             // eslint-disable-next-line no-console
             console.log(JSON.stringify(result, null, 2));
-            if (this.showInThread && (resultSummary.failures.length > 0 || resultSummary.skipped > 0 || resultSummary.flaky > 0)) {
-                /*  for (let i = 0; i < result.length; i += 1) {
+            if (this.showInThread) {
+                  /*for (let i = 0; i < result.length; i += 1) {
                     // eslint-disable-next-line no-await-in-loop
                     await slackClient.attachDetailsToThread({
                         channelIds: [result[i].channel],
@@ -120,7 +120,7 @@ class SlackReporter {
                         summaryResults: resultSummary,
                         maxNumberOfFailures: this.maxNumberOfFailuresToShow,
                     });
-}*/
+                  }*/
                 for (let i = 0; i < result.length; i += 1) {
                     // eslint-disable-next-line no-await-in-loop
                     await slackClient.attachDetailsToThreadRuns({
@@ -131,35 +131,37 @@ class SlackReporter {
                     });
                 }
 
-                for (let i = 0; i < result.length; i += 1) {
-                    // eslint-disable-next-line no-await-in-loop
-                    await slackClient.attachDetailsToThreadSuites({
-                        channelIds: [result[i].channel],
-                        ts: result[i].ts,
-                        summaryResults: resultSummary,
-                        maxNumberOfFailures: this.maxNumberOfFailuresToShow,
-                    });
-                }
+                if ((resultSummary.failures.length > 0 || resultSummary.skipped > 0 || resultSummary.flaky > 0)) {
+                    for (let i = 0; i < result.length; i += 1) {
+                      // eslint-disable-next-line no-await-in-loop
+                      await slackClient.attachDetailsToThreadSuites({
+                          channelIds: [result[i].channel],
+                          ts: result[i].ts,
+                          summaryResults: resultSummary,
+                          maxNumberOfFailures: this.maxNumberOfFailuresToShow,
+                      });
+                  }
 
-                for (let i = 0; i < result.length; i += 1) {
-                    // eslint-disable-next-line no-await-in-loop
-                    await slackClient.attachDetailsToThreadCases({
-                        channelIds: [result[i].channel],
-                        ts: result[i].ts,
-                        summaryResults: resultSummary,
-                        maxNumberOfFailures: this.maxNumberOfFailuresToShow,
-                    });
-                }
-                
-                for (let i = 0; i < result.length; i += 1) {
-                    // eslint-disable-next-line no-await-in-loop
-                    await slackClient.attachDetailsToThreadReasons({
-                        channelIds: [result[i].channel],
-                        ts: result[i].ts,
-                        summaryResults: resultSummary,
-                        maxNumberOfFailures: this.maxNumberOfFailuresToShow,
-                    });
-                }
+                  for (let i = 0; i < result.length; i += 1) {
+                      // eslint-disable-next-line no-await-in-loop
+                      await slackClient.attachDetailsToThreadCases({
+                          channelIds: [result[i].channel],
+                          ts: result[i].ts,
+                          summaryResults: resultSummary,
+                          maxNumberOfFailures: this.maxNumberOfFailuresToShow,
+                      });
+                  }
+                  
+                  for (let i = 0; i < result.length; i += 1) {
+                      // eslint-disable-next-line no-await-in-loop
+                      await slackClient.attachDetailsToThreadReasons({
+                          channelIds: [result[i].channel],
+                          ts: result[i].ts,
+                          summaryResults: resultSummary,
+                          maxNumberOfFailures: this.maxNumberOfFailuresToShow,
+                      });
+                  }
+              }
             }
         }
     }

package/package.json

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