playwright-slack-report 1.1.15 → 1.1.17

package/README.md

@@ -129,7 +129,7 @@ An example advanced configuration is shown below:
         ],
         slackOAuthToken: 'YOUR_SLACK_OAUTH_TOKEN',
         slackLogLevel: LogLevel.DEBUG,
-        enableUnfurl: false,
+        disableUnfurl: true,
         showInThread: true,
       },
 
@@ -154,8 +154,14 @@ Limits the number of failures shown in the Slack message, defaults to 10.
 ### **slackOAuthToken**
 Instead of providing an environment variable `SLACK_BOT_USER_OAUTH_TOKEN` you can specify the token in the config in the `slackOAuthToken` field.
 ### **slackLogLevel** (default LogLevel.DEBUG)
-This option allows you to control slack client severity levels for log entries. It accepts a value from @slack/web-api `LogLevel` enum
-### **enableUnfurl** (default: true)
+This option allows you to control slack client severity levels for log entries. It accepts a value from @slack/web-api `LogLevel` enum:
+- ERROR
+- WARN
+- INFO
+- DEBUG
+
+Example: `slackLogLevel: "ERROR",` will only log errors to the console.
+### **disableUnfurl** (default: true)
 Enable or disable unfurling of links in Slack messages.
 ### **showInThread** (default: false)
 Instructs the reporter to show the failure details in a thread instead of the main channel.
@@ -327,7 +333,7 @@ Generates the following message in Slack:
 ![Final](https://github.com/ryanrosello-og/playwright-slack-report/blob/main/assets/2022-08-13_8-17-46.png?raw=true)
 
 
-**Example 3: - with screenshots and/or recorded videos**
+**Example 3: - With screenshots and/or recorded videos (using AWS S3)**
 
 In your, `playwright.config.ts` file, add these params (Make sure you use **layoutAsync** rather than **layout**):
 
@@ -461,7 +467,19 @@ export async function generateCustomLayoutAsync (summaryResults: SummaryResults)
 
 ```
 
-**Also you can upload the attachments to slack.** But it might be more expensive for you and also you'll have to extend the scope.
+**Example 4: - Upload the attachments to directly to Slack**
+
+To enable this functionality, make sure the slackbot user has the following additional scopes:
+- `files:write`
+- `files:read`
+
+You will need to re-install the app and re-invite the bot into the channel.
+
+The value of the channel_id should be the channel id of the channel you want to upload the file to. This channel id can be found in the url when you are in the channel. e.g.
+
+**https://app.slack.com/client/T02RVEEFPDH/C05H7TKVDUK**
+
+^ the bit starting with 'C...' is your channel id. In this case, the channel id is `C05H7TKVDUK`
 
 ```typescript
 ...
@@ -471,7 +489,7 @@ const slackClient = new web_api_1.WebClient(process.env.SLACK_BOT_USER_OAUTH_TOK
 async function uploadFile(filePath) {
   try {
     const result = await slackClient.files.uploadV2({
-      channels: 'you_cannel_name',
+      channel_id: 'C05H7TKVDUK',  << this is the channel id not channel name! ☠️
       file: fs.createReadStream(filePath),
       filename: filePath.split('/').at(-1),
     });

package/dist/src/SlackClient.d.ts

@@ -16,7 +16,7 @@ export default class SlackClient {
             maxNumberOfFailures: number;
             slackOAuthToken?: string;
             slackLogLevel?: LogLevel;
-            unfurlEnable?: boolean;
+            disableUnfurl?: boolean;
             summaryResults: SummaryResults;
             showInThread: boolean;
         };
@@ -25,12 +25,12 @@ export default class SlackClient {
         outcome: string;
         ts: string;
     }>>;
-    attachDetailsToThread({ channelIds, ts, summaryResults, maxNumberOfFailures, unfurlEnable, fakeRequest, }: {
+    attachDetailsToThread({ channelIds, ts, summaryResults, maxNumberOfFailures, disableUnfurl, fakeRequest, }: {
         channelIds: Array<string>;
         ts: string;
         summaryResults: SummaryResults;
         maxNumberOfFailures: number;
-        unfurlEnable?: boolean;
+        disableUnfurl?: boolean;
         fakeRequest?: Function;
     }): Promise<any[]>;
     static doPostRequest(slackWebClient: WebClient, channel: string, blocks: Array<KnownBlock | Block>, unfurl: boolean, threadTimestamp?: string): Promise<ChatPostMessageResponse>;

package/dist/src/SlackClient.js

@@ -26,7 +26,7 @@ class SlackClient {
             throw new Error(`Channel ids [${options.channelIds}] is not valid`);
         }
         const result = [];
-        const unfurl = options.unfurlEnable;
+        const unfurl = !options.disableUnfurl;
         for (const channel of options.channelIds) {
             let chatResponse;
             try {
@@ -63,7 +63,7 @@ class SlackClient {
         }
         return result;
     }
-    async attachDetailsToThread({ channelIds, ts, summaryResults, maxNumberOfFailures, unfurlEnable, fakeRequest, }) {
+    async attachDetailsToThread({ channelIds, ts, summaryResults, maxNumberOfFailures, disableUnfurl, fakeRequest, }) {
         const result = [];
         const blocks = await (0, LayoutGenerator_1.generateFailures)(summaryResults, maxNumberOfFailures);
         for (const channel of channelIds) {
@@ -73,7 +73,7 @@ class SlackClient {
                 chatResponse = await fakeRequest();
             }
             else {
-                chatResponse = await SlackClient.doPostRequest(this.slackWebClient, channel, blocks, unfurlEnable, ts);
+                chatResponse = await SlackClient.doPostRequest(this.slackWebClient, channel, blocks, disableUnfurl, ts);
             }
             if (chatResponse.ok) {
                 // eslint-disable-next-line no-console

package/dist/src/SlackReporter.d.ts

@@ -10,7 +10,7 @@ declare class SlackReporter implements Reporter {
     private slackChannels;
     private slackLogLevel;
     private slackOAuthToken;
-    private enableUnfurl;
+    private disableUnfurl;
     private suite;
     logs: string[];
     onBegin(fullConfig: FullConfig, suite: Suite): void;
@@ -21,5 +21,6 @@ declare class SlackReporter implements Reporter {
         message?: string;
     };
     log(message: string | undefined): void;
+    printsToStdio(): boolean;
 }
 export default SlackReporter;

package/dist/src/SlackReporter.js

@@ -14,7 +14,7 @@ class SlackReporter {
     slackChannels = [];
     slackLogLevel;
     slackOAuthToken;
-    enableUnfurl;
+    disableUnfurl;
     suite;
     logs = [];
     onBegin(fullConfig, suite) {
@@ -29,8 +29,9 @@ class SlackReporter {
             this.slackChannels = slackReporterConfig.channels;
             this.maxNumberOfFailuresToShow = slackReporterConfig.maxNumberOfFailuresToShow || 10;
             this.slackOAuthToken = slackReporterConfig.slackOAuthToken || undefined;
-            this.enableUnfurl = slackReporterConfig.enableUnfurl || true;
+            this.disableUnfurl = slackReporterConfig.disableUnfurl || false;
             this.showInThread = slackReporterConfig.showInThread || false;
+            this.slackLogLevel = slackReporterConfig.slackLogLevel || web_api_1.LogLevel.DEBUG;
         }
         this.resultsParser = new ResultsParser_1.default();
     }
@@ -62,7 +63,7 @@ class SlackReporter {
                 customLayout: this.customLayout,
                 customLayoutAsync: this.customLayoutAsync,
                 maxNumberOfFailures: this.maxNumberOfFailuresToShow,
-                unfurlEnable: this.enableUnfurl,
+                disableUnfurl: this.disableUnfurl,
                 summaryResults: resultSummary,
                 showInThread: this.showInThread,
             },
@@ -126,5 +127,9 @@ class SlackReporter {
             this.logs.push(message);
         }
     }
+    // eslint-disable-next-line class-methods-use-this
+    printsToStdio() {
+        return false;
+    }
 }
 exports.default = SlackReporter;

package/package.json

@@ -1,15 +1,13 @@
 {
   "dependencies": {
-    "@slack/web-api": "^6.7.2"
+    "@slack/web-api": "^6.8.1"
   },
   "devDependencies": {
     "@playwright/test": "^1.23.3",
-    "dotenv": "^16.0.1",
-    "playwright": "^1.23.3",
-    "typescript": "^4.7.4",
     "@slack/types": "^2.7.0",
     "@typescript-eslint/eslint-plugin": "^5.30.6",
     "@typescript-eslint/parser": "^5.30.6",
+    "dotenv": "^16.0.1",
     "eslint": "^7.32.0 || ^8.2.0",
     "eslint-config-airbnb-base": "^15.0.0",
     "eslint-config-node": "^4.1.0",
@@ -19,21 +17,29 @@
     "eslint-plugin-prettier": "^4.2.1",
     "nyc": "^15.1.0",
     "prettier": "^2.7.1",
-    "ts-mockito": "^2.6.1"
+    "ts-mockito": "^2.6.1",
+    "typescript": "^4.7.4"
   },
   "scripts": {
     "prettier": "prettier --write --loglevel warn \"**/**/*.ts\"",
     "pw": "nyc playwright test && nyc report --reporter=lcov",
     "build": "tsc -p ./tsconfig.json",
-    "lint":"npx eslint . --ext .ts"
+    "lint": "npx eslint . --ext .ts"
   },
   "name": "playwright-slack-report",
-  "version": "1.1.15",
+  "version": "1.1.17",
   "main": "index.js",
   "types": "dist/index.d.ts",
   "repository": "git@github.com:ryanrosello-og/playwright-slack-report.git",
   "author": "Ryan Rosello <ryanrosello@hotmail.com>",
   "license": "MIT",
-  "files": ["/dist/src"],
-  "keywords": ["slack", "report", "playwright", "typescript"]
+  "files": [
+    "/dist/src"
+  ],
+  "keywords": [
+    "slack",
+    "report",
+    "playwright",
+    "typescript"
+  ]
 }