@testomatio/reporter 1.4.2-beta-specflow-compat.2 → 1.4.3

package/lib/bin/reportXml.js

@@ -44,15 +44,10 @@ program
 
     let timeoutTimer;
     if (opts.timelimit) {
-      timeoutTimer = setTimeout(
-        () => {
-          console.log(
-            `⚠️  Reached timeout of ${opts.timelimit}s. Exiting... (Exit code is 0 to not fail the pipeline)`,
-          );
-          process.exit(0);
-        },
-        parseInt(opts.timelimit, 10) * 1000,
-      );
+      timeoutTimer = setTimeout(() => {
+        console.log(`⚠️  Reached timeout of ${opts.timelimit}s. Exiting... (Exit code is 0 to not fail the pipeline)`);
+        process.exit(0);
+      }, parseInt(opts.timelimit, 10) * 1000);
     }
 
     try {

package/lib/fileUploader.js

@@ -103,9 +103,9 @@ const uploadUsingS3 = async (filePath, runId) => {
   let Key;
 
   if (typeof filePath === 'object') {
-    ContentType = filePath.type;
-    filePath = filePath.path;
-    Key = filePath.name;
+    ContentType = filePath?.type;
+    filePath = filePath?.path;
+    Key = filePath?.name;
   }
 
   const { TESTOMATIO_PRIVATE_ARTIFACTS, S3_BUCKET } = getConfig();

package/lib/junit-adapter/csharp.js

@@ -11,7 +11,6 @@ class CSharpAdapter extends Adapter {
     t.title = title.trim();
     return t;
   }
-
 }
 
 module.exports = CSharpAdapter;

package/lib/pipe/gitlab.js

@@ -79,13 +79,13 @@ class GitLabPipe {
     let summary = `${this.hiddenCommentData}
     
   | [![Testomat.io Report](${testomatLogoURL})](https://testomat.io)  | ${statusEmoji(
-    runParams.status,
-  )} ${runParams.status.toUpperCase()} ${statusEmoji(runParams.status)} |
+      runParams.status,
+    )} ${runParams.status.toUpperCase()} ${statusEmoji(runParams.status)} |
   | --- | --- |
   | Tests | ✔️  **${this.tests.length}** tests run  |
   | Summary | ${statusEmoji('failed')} **${failedCount}** failed; ${statusEmoji(
-    'passed',
-  )} **${passedCount}** passed; **${statusEmoji('skipped')}** ${skippedCount} skipped |
+      'passed',
+    )} **${passedCount}** passed; **${statusEmoji('skipped')}** ${skippedCount} skipped |
   | Duration | 🕐  **${humanizeDuration(
     parseInt(
       this.tests.reduce((a, t) => a + (t.run_time || 0), 0),

package/lib/utils/utils.js

@@ -246,7 +246,9 @@ const foundedTestLog = (app, tests) => {
 };
 
 const humanize = text => {
-  text = decamelize(text);
+  // if there are no spaces, decamelize
+  if (!text.trim().includes(' ')) text = decamelize(text);
+  
   return text
     .replace(/_./g, match => ` ${match.charAt(1).toUpperCase()}`)
     .trim()

package/lib/xmlReader.js

@@ -69,7 +69,19 @@ class XmlReader {
   }
 
   parse(fileName) {
-    const xmlData = fs.readFileSync(path.resolve(fileName));
+    let xmlData = fs.readFileSync(path.resolve(fileName)).toString();
+
+    // we remove too long stack traces
+    const cutRegexes = [
+      /(<output><!\[CDATA\[)([\s\S]*?)(\]\]><\/output>)/g,
+      /(<system-err><!\[CDATA\[)([\s\S]*?)(\]\]><\/system-err>)/g,
+      /(<system-out><!\[CDATA\[)([\s\S]*?)(\]\]><\/system-out>)/g,
+    ];
+
+    for (const regex of cutRegexes) {
+      xmlData = xmlData.replace(regex, (_, p1, p2, p3) => `${p1}${p2.substring(0, 5000)}${p3}`);
+    }
+
     const jsonResult = this.parser.parse(xmlData);
     let jsonSuite;
 
@@ -422,6 +434,10 @@ function reduceTestCases(prev, item) {
   if (!Array.isArray(testCases)) {
     testCases = [testCases];
   }
+
+  // suite inside test case
+  if (item['test-suite'] && item['test-suite']['test-case']) testCases.push(...item['test-suite']['test-case']);
+
   const suiteOutput = item['system-out'] || item.output || item.log || '';
   const suiteErr = item['system-err'] || item.output || item.log || '';
   testCases
@@ -429,7 +445,6 @@ function reduceTestCases(prev, item) {
     .forEach(testCaseItem => {
       const file = testCaseItem.file || item.filepath || '';
 
-
       let stack = '';
       let message = '';
       if (testCaseItem.error) stack = testCaseItem.error;
@@ -444,12 +459,16 @@ function reduceTestCases(prev, item) {
 
       // SpecFlow config
       let { title, tags } = fetchProperties(item.type === 'ParameterizedMethod' ? item : testCaseItem);
+      let example = null;
 
       title ||= testCaseItem.name || testCaseItem.methodname || testCaseItem.classname;
       tags ||= [];
 
-      let example = testCaseItem.name.match(/\((.*?)\)/);
-      if (example) example = { ...example[1].split(',') };
+      const exampleMatches = title.match(/\((.*?)\)/);
+      if (exampleMatches) {
+        example = { ...exampleMatches[1].split(',').map(v => v.replace(/^['"]|['"]$/g, '')) };
+        title = title.replace(/\(.*?\)/, '').trim();
+      }
 
       // eslint-disable-next-line
       stack = `${
@@ -484,26 +503,28 @@ function processTestSuite(testsuite) {
   if (!testsuite) return [];
   if (testsuite.testsuite) return processTestSuite(testsuite.testsuite);
   if (testsuite['test-suite'] && !testsuite['test-case']) return processTestSuite(testsuite['test-suite']);
-  
+
   let suites = testsuite;
   if (!Array.isArray(testsuite)) {
     suites = [testsuite];
   }
-  if (testsuite['test-suite']) suites.push(testsuite['test-suite']);
 
   const res = suites.flat().reduce(reduceTestCases, []);
-  
+
   return res;
 }
 
 function fetchProperties(item) {
-  let tags = [];
+  const tags = [];
   let title = '';
 
   if (!item.properties) return {};
 
   const prop = [item.properties?.property].flat().find(p => p.name === 'Description');
   if (prop) title = prop.value;
-  [item.properties?.property].flat().filter(p => p.name === 'Category').forEach(p => tags.push(p.value));
-  return { title, tags }
-}
+  [item.properties?.property]
+    .flat()
+    .filter(p => p.name === 'Category')
+    .forEach(p => tags.push(p.value));
+  return { title, tags };
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@testomatio/reporter",
-  "version": "1.4.2-beta-specflow-compat.2",
+  "version": "1.4.3",
   "description": "Testomatio Reporter Client",
   "main": "./lib/reporter.js",
   "typings": "typings/index.d.ts",
@@ -41,7 +41,7 @@
   ],
   "scripts": {
     "clear-exportdir": "rm -rf export/",
-    "pretty": "npx prettier --debug-check --check .",
+    "pretty": "npx prettier --check .",
     "pretty:fix": "prettier --write .",
     "lint": "eslint lib",
     "lint:fix": "eslint lib --fix",