playwright-slack-report-burak 1.3.5 → 1.3.7

package/dist/src/LayoutGenerator.js

@@ -1,6 +1,13 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.generateFailures = exports.generateFailuresReasons = exports.generateProblemCaseList = exports.generateBlocks = exports.generateAllRunSuites = void 0;
+exports.generateFailures =
+ exports.generateFailuresReasons =
+ exports.generateProblemSuiteList =
+ exports.generateProblemCaseList =
+ exports.generateBlocks =
+ exports.generateAllRunSuites = 
+ void 0;
+
 const generateBlocks = async (summaryResults, maxNumberOfFailures) => {
     const meta = [];
     const header = {
@@ -137,47 +144,39 @@ 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 generateProblemCaseList = async (summaryResults) => {
+const generateProblemSuiteList = async (summaryResults) => {
     const suitesResults = [];
-    const allSuites = [];
+    const failedSuites = [];
     const flakySuites = [];
     const skippedSuites = [];
-    const failsList = [];
-    const flakyList = [];
-    const skipsList = [];
-    const allSuitesTitle = [];
+    const failedSuitesTitle = [];
     const flakySuitesTitle = [];
     const skippedSuitesTitle = [];
-    const failedNamesTitle = [];
-    const flakyNamesTitle = [];
-    const skippedNamesTitle = [];
-    let previousTestName = undefined;
     let previousSuiteName = undefined;
   
     //  Create a list for names of test suites with failed test cases
@@ -191,7 +190,7 @@ const generateProblemCaseList = async (summaryResults) => {
         ? `${feature.split('-').map(word => word.charAt(0).toUpperCase() + word.slice(1)).join(' ')} (${category})`
         : backSlashedParts;
       if (formattedSuiteName !== previousSuiteName) {
-        allSuites.push(formattedSuiteName);
+        failedSuites.push(formattedSuiteName);
         previousSuiteName = formattedSuiteName;
       }
     }
@@ -236,6 +235,65 @@ const generateProblemCaseList = async (summaryResults) => {
     }
     previousSuiteName = undefined;
   
+    //  Decide on Titles For Listing Failures, Skips & Flakies
+     if (failedSuites.length === 1) {
+      failedSuitesTitle[0] = '*Suite With Failed Test Case(s):*\n';
+    }
+    if (failedSuites.length > 1) {
+      failedSuitesTitle[0] = '*Suites With Failed Test Case(s):*\n';
+    }
+     if (flakySuites.length === 1) {
+      flakySuitesTitle[0] = '\n*Suite With Flaky Test Case(s):*\n';
+    }
+    if (flakySuites.length > 1) {
+      flakySuitesTitle[0] = '\n*Suites With Flaky Test Case(s):*\n';
+    }
+    if (skippedSuites.length === 1) {
+      skippedSuitesTitle[0] = '\n*Suite With Skipped Test Case(s):*\n';
+    }
+    if (skippedSuites.length > 1) {
+      skippedSuitesTitle[0] = '\n*Suites With Skipped Test Case(s):*\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:
+          `${failedSuitesTitle}${[...new Set(failedSuites)].map((value, index) => `*${index + 1}.* ${value}`).join('\n')}` +
+          `\n${flakySuitesTitle}${[...new Set(flakySuites)].map((value, index) => `*${index + 1}.* ${value}`).join('\n')}` +
+          `\n${skippedSuitesTitle}${[...new Set(skippedSuites)].map((value, index) => `*${index + 1}.* ${value}`).join('\n')} `,
+        },
+      },
+      {
+        type: 'section',
+        text: {
+          type: 'mrkdwn',
+          text: ` `,
+        },
+      },
+      {
+        type: 'divider',
+      },
+    )}
+    return [
+        ...suitesResults,
+    ];
+};
+
+const generateProblemCaseList = async (summaryResults) => {
+    const casesResults = [];
+    const failsList = [];
+    const flakyList = [];
+    const skipsList = [];
+    const failedNamesTitle = [];
+    const flakyNamesTitle = [];
+    const skippedNamesTitle = [];
+    let previousTestName = undefined;
+    let previousSuiteName = undefined;
+  
     //  Create a list for names of failed test cases
     for (let i = 0; i < summaryResults.failures.length; i += 1) {
       const { test, suiteName } = summaryResults.failures[i];
@@ -279,24 +337,6 @@ const generateProblemCaseList = async (summaryResults) => {
     previousSuiteName = undefined;
   
     //  Decide on Titles For Listing Failures, Skips & Flakies
-     if (allSuites.length === 1) {
-      allSuitesTitle[0] = '*Failed Suite:*\n';
-    }
-    if (allSuites.length > 1) {
-      allSuitesTitle[0] = '*Failed Suites:*\n';
-    }
-     if (flakySuites.length === 1) {
-      flakySuitesTitle[0] = '\n*Flaky Suite:*\n';
-    }
-    if (flakySuites.length > 1) {
-      flakySuitesTitle[0] = '\n*Flaky Suites:*\n';
-    }
-    if (skippedSuites.length === 1) {
-      skippedSuitesTitle[0] = '\n*Skipped Suite:*\n';
-    }
-    if (skippedSuites.length > 1) {
-      skippedSuitesTitle[0] = '\n*Skipped Suites:*\n';
-    }
     if (summaryResults.failed === 1) {
       failedNamesTitle[0] = '\n*Failed Test Case:*\n';
     }
@@ -318,16 +358,7 @@ const generateProblemCaseList = async (summaryResults) => {
   
     //  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')}` +
-          `\n${flakySuitesTitle}${[...new Set(flakySuites)].map((value, index) => `*${index + 1}.* ${value}`).join('\n')}` +
-          `\n${skippedSuitesTitle}${[...new Set(skippedSuites)].map((value, index) => `*${index + 1}.* ${value}`).join('\n')} `,
-        },
-      },
+      casesResults.push(
       {
           type: 'section',
           text: {
@@ -350,7 +381,7 @@ const generateProblemCaseList = async (summaryResults) => {
       },
     )}
     return [
-        ...suitesResults,
+        ...casesResults,
     ];
 };
 
@@ -475,5 +506,6 @@ const generateFailuresReasons = async (summaryResults) => {
 };
 exports.generateFailures = generateFailures;
 exports.generateFailuresReasons = generateFailuresReasons;
+exports.generateProblemSuiteList = generateProblemSuiteList;
 exports.generateProblemCaseList = generateProblemCaseList;
-exports.generateAllRunSuites = generateAllRunSuites;
+exports.generateAllRunSuites = generateAllRunSuites;

package/dist/src/SlackClient.js

@@ -102,7 +102,30 @@ class SlackClient {
       }
       return result;
     }
-
+    async attachDetailsToThreadSuites({ channelIds, ts, summaryResults, maxNumberOfFailures, disableUnfurl, fakeRequest, }) {
+      const result = [];
+      const blocks = await (0, LayoutGenerator_1.generateProblemSuiteList)(summaryResults, maxNumberOfFailures);
+      for (const channel of channelIds) {
+        // under test
+        let chatResponse;
+        if (fakeRequest) {
+            chatResponse = await fakeRequest();
+        }
+        else {
+            chatResponse = await SlackClient.doPostRequest(this.slackWebClient, channel, blocks, disableUnfurl, ts);
+        }
+        if (chatResponse.ok) {
+            // eslint-disable-next-line no-console
+            console.log(`✅ Message sent to ${channel} within thread ${ts}`);
+            result.push({
+                channel,
+                outcome: `✅ Message sent to ${channel} within thread ${ts}`,
+                ts: chatResponse.ts,
+            });
+        }
+      }
+      return result;
+    }
     async attachDetailsToThreadCases({ channelIds, ts, summaryResults, maxNumberOfFailures, disableUnfurl, fakeRequest, }) {
       const result = [];
       const blocks = await (0, LayoutGenerator_1.generateProblemCaseList)(summaryResults, maxNumberOfFailures);

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,25 +131,37 @@ class SlackReporter {
                     });
                 }
 
-                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,
-                    });
-                }
+                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,
+                      });
+                  }
+              }
             }
         }
     }

package/package.json

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