chore: suite metadata hostname and device (#244)

test-results-reporter · Sep 1, 2024 · e58016e · e58016e
1 parent 0f471ae
commit e58016e
Show file tree

Hide file tree

Showing 8 changed files with 37 additions and 20 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "testbeats",
-  "version": "2.1.1",
+  "version": "2.1.2",
   "description": "Publish test results to Microsoft Teams, Google Chat, Slack and InfluxDB",
   "main": "src/index.js",
   "types": "./src/index.d.ts",
@@ -55,7 +55,7 @@
     "pretty-ms": "^7.0.1",
     "rosters": "0.0.1",
     "sade": "^1.8.1",
-    "test-results-parser": "0.2.4"
+    "test-results-parser": "0.2.5"
   },
   "devDependencies": {
     "c8": "^7.12.0",

diff --git a/src/extensions/error-clusters.extension.js b/src/extensions/error-clusters.extension.js
@@ -1,5 +1,6 @@
 const { BaseExtension } = require('./base.extension');
 const { STATUS, HOOK } = require("../helpers/constants");
+const { truncate } = require('../helpers/helper');
 
 class ErrorClustersExtension extends BaseExtension {
 
@@ -35,7 +36,7 @@ class ErrorClustersExtension extends BaseExtension {
 
     const texts = [];
     for (const cluster of clusters) {
-      texts.push(`${cluster.failure} - ${this.bold(`(x${cluster.count})`)}`);
+      texts.push(`${truncate(cluster.failure, 150)} - ${this.bold(`(x${cluster.count})`)}`);
     }
     this.text = this.mergeTexts(texts);
   }

diff --git a/src/helpers/helper.js b/src/helpers/helper.js
@@ -46,9 +46,15 @@ function processData(data) {
   return data;
 }
 
+/**
+ *
+ * @param {string} text
+ * @param {number} length
+ * @returns
+ */
 function truncate(text, length) {
   if (text && text.length > length) {
-    return text.slice(0, length) + "...";
+    return text.slice(0, length).trim() + "...";
   } else {
     return text;
   }

diff --git a/src/platforms/base.platform.js b/src/platforms/base.platform.js
@@ -103,12 +103,22 @@ class BasePlatform {
 
     const texts = [];
 
+    // webdriver io
+    if (suite.metadata.device && typeof suite.metadata.device === 'string') {
+      texts.push(`${suite.metadata.device}`);
+    }
+
     if (suite.metadata.platform && suite.metadata.platform.name && suite.metadata.platform.version) {
-      texts.push(`${suite.metadata.platform.name} ${suite.metadata.platform.version}`)
+      texts.push(`${suite.metadata.platform.name} ${suite.metadata.platform.version}`);
     }
 
     if (suite.metadata.browser && suite.metadata.browser.name && suite.metadata.browser.version) {
-      texts.push(`${suite.metadata.browser.name} ${suite.metadata.browser.version}`)
+      texts.push(`${suite.metadata.browser.name} ${suite.metadata.browser.version}`);
+    }
+
+    // playwright
+    if (suite.metadata.hostname && typeof suite.metadata.hostname === 'string') {
+      texts.push(`${suite.metadata.hostname}`);
     }
 
     return texts.join(' • ');

diff --git a/test/mocks/chat.mock.js b/test/mocks/chat.mock.js
@@ -605,7 +605,7 @@ addInteractionHandler('post test-summary with suite metadata to chat', () => {
                 "widgets": [
                   {
                     "textParagraph": {
-                      "text": "<b>✅ Addition</b><br><br><b>Results</b>: 1 / 1 Passed (100%)<br><b>Duration</b>: 1ms<br><br>Windows 11 • firefox 129.0<br><br><b>✅ Addition</b><br><br><b>Results</b>: 1 / 1 Passed (100%)<br><b>Duration</b>: 1ms<br><br>Windows 11 • chrome 129.0"
+                      "text": "<b>✅ Addition</b><br><br><b>Results</b>: 1 / 1 Passed (100%)<br><b>Duration</b>: 1ms<br><br>Desktop • Windows 11 • firefox 129.0<br><br><b>✅ Addition</b><br><br><b>Results</b>: 1 / 1 Passed (100%)<br><b>Duration</b>: 1ms<br><br>Desktop • Windows 11 • chrome 129.0"
                     }
                   }
                 ]

diff --git a/test/mocks/slack.mock.js b/test/mocks/slack.mock.js
@@ -853,14 +853,14 @@ addInteractionHandler('post test-summary to slack with suite metadata', () => {
                 "type": "section",
                 "text": {
                   "type": "mrkdwn",
-                  "text": "*✅ Addition*\n\n*Results*: 1 / 1 Passed (100%)\n*Duration*: 1ms\n\nWindows 11 • firefox 129.0"
+                  "text": "*✅ Addition*\n\n*Results*: 1 / 1 Passed (100%)\n*Duration*: 1ms\n\nDesktop • Windows 11 • firefox 129.0"
                 }
               },
               {
                 "type": "section",
                 "text": {
                   "type": "mrkdwn",
-                  "text": "*✅ Addition*\n\n*Results*: 1 / 1 Passed (100%)\n*Duration*: 1ms\n\nWindows 11 • chrome 129.0"
+                  "text": "*✅ Addition*\n\n*Results*: 1 / 1 Passed (100%)\n*Duration*: 1ms\n\nDesktop • Windows 11 • chrome 129.0"
                 }
               }
             ],

diff --git a/test/mocks/teams.mock.js b/test/mocks/teams.mock.js
@@ -1849,7 +1849,7 @@ addInteractionHandler('post test-summary with suite metadata to teams', () => {
                 },
                 {
                   "type": "TextBlock",
-                  "text": "Windows 11 • firefox 129.0",
+                  "text": "Desktop • Windows 11 • firefox 129.0",
                   "wrap": true
                 },
                 {
@@ -1874,7 +1874,7 @@ addInteractionHandler('post test-summary with suite metadata to teams', () => {
                 },
                 {
                   "type": "TextBlock",
-                  "text": "Windows 11 • chrome 129.0",
+                  "text": "Desktop • Windows 11 • chrome 129.0",
                   "wrap": true
                 }
               ],