@testomatio/reporter 0.8.0-beta.31 → 0.8.0-beta.32

package/lib/util.js

@@ -7,7 +7,7 @@ const isValid = require('is-valid-path');
 /**
  * @param {String} testTitle - Test title
  *
- * @returns {String} testId
+ * @returns {String|null} testId
  */
 const parseTest = testTitle => {
   const captures = testTitle.match(/@T([\w\d]+)/);
@@ -21,7 +21,7 @@ const parseTest = testTitle => {
 /**
  * @param {String} suiteTitle - suite title
  *
- * @returns {String} suiteId
+ * @returns {String|null} suiteId
  */
 const parseSuite = suiteTitle => {
   const captures = suiteTitle.match(/@S([\w\d]+)/);
@@ -63,12 +63,12 @@ const fetchSourceCodeFromStackTrace = (stack = '') => {
     // .map(l => l.match(/\[(.*?)\]/)?.[1] || l) // minitest format
     // .map(l => l.split(':')[0])
     .map(l => l.trim())
-    .map(l => l.split(' ').find(p => p.includes(':')))
-    .filter(l => isValid(l.split(':')[0]))
+    .map(l => l.split(' ').find(p => p.includes(':')) || '')
+    .filter(l => isValid(l?.split(':')[0]))
 
     // // filter out 3rd party libs
-    .filter(l => !l.includes(`vendor${  sep}`))
-    .filter(l => !l.includes(`node_modules${  sep}`))
+    .filter(l => !l?.includes(`vendor${  sep}`))
+    .filter(l => !l?.includes(`node_modules${  sep}`))
     .filter(l => fs.existsSync(l.split(':')[0]))
     .filter(l => fs.lstatSync(l.split(':')[0]).isFile())
 
@@ -154,31 +154,31 @@ const getCurrentDateTime = () => {
 }
 
 /**
- * @param {String} test - Test adapter object
+ * @param {Object} test - Test adapter object
  *
- * @returns {String} testInfo as one string 
+ * @returns {String|null} testInfo as one string 
  */
 const specificTestInfo = test => {
-  //TODO: afterEach has another context.... need to add specific handler, maybe...
+  // TODO: afterEach has another context.... need to add specific handler, maybe...
   if (test?.title && test?.file) {
 
-    return basename(test.file).split(".").join("#")
-             + "#" 
-             + test.title.split(" ").join("#");
+    return `${basename(test.file).split(".").join("#")
+              }#${ 
+              test.title.split(" ").join("#")}`;
   }
 
   return null;
 };
 
 module.exports = {
-  parseTest,
-  parseSuite,
-  ansiRegExp,
   isSameTest,
-  isValidUrl,
   fetchSourceCode,
   fetchSourceCodeFromStackTrace,
   fetchFilesFromStackTrace,
   getCurrentDateTime,
-  specificTestInfo
-};
+  specificTestInfo,
+  isValidUrl,
+  ansiRegExp,
+  parseTest,
+  parseSuite 
+}

package/lib/xmlReader.js

@@ -2,15 +2,14 @@ const debug = require('debug')('@testomatio/reporter:xml');
 const path = require("path");
 const chalk = require('chalk');
 const fs = require("fs");
-
-// const util = require("util"); // you can see a result
 const { XMLParser } = require("fast-xml-parser");
-const { APP_PREFIX, PASSED, FAILED, SKIPPED } = require('./constants');
+const { APP_PREFIX, STATUS } = require('./constants');
 const { fetchFilesFromStackTrace, fetchSourceCode, fetchSourceCodeFromStackTrace } = require('./util');
 const upload = require('./fileUploader');
 const pipesFactory = require('./pipe');
 const adapterFactory = require('./junit-adapter');
 
+
 const TESTOMATIO_URL = process.env.TESTOMATIO_URL || "https://app.testomat.io";
 const TESTOMATIO = process.env.TESTOMATIO; // key?
 const { TESTOMATIO_RUNGROUP_TITLE, TESTOMATIO_TITLE, TESTOMATIO_ENV, TESTOMATIO_RUN } = process.env;
@@ -37,7 +36,9 @@ class XmlReader {
     };
     this.runId = opts.runId || TESTOMATIO_RUN;
     this.adapter = adapterFactory(null, opts)
-    this.opts = opts;
+    if (!this.adapter) throw new Error('XML adapter for this format not found');
+
+    this.opts = opts || {};
     const store = {}
     this.pipes = pipesFactory(opts, store);
 
@@ -93,7 +94,7 @@ class XmlReader {
       create_tests: true,
       name,
       tests_count: parseInt(tests, 10),
-      passed_count: parseInt(tests - failures, 10),
+      passed_count: parseInt(tests, 10) - parseInt(failures, 10),
       failed_count: parseInt(failures, 10),
       skipped_count: 0,
       tests: resultTests,
@@ -160,9 +161,9 @@ class XmlReader {
       if (test.example) r.example = test.example;
       if (test.file) r.file = test.file;
       r.create = true;
-      if (r.status === 'Passed') r.status = PASSED;
-      if (r.status === 'Failed') r.status = FAILED;
-      if (r.status === 'Skipped') r.status = SKIPPED;
+      if (r.status === 'Passed') r.status = STATUS.PASSED;
+      if (r.status === 'Failed') r.status = STATUS.FAILED;
+      if (r.status === 'Skipped') r.status = STATUS.SKIPPED;
       delete r.id;
     });
 
@@ -172,8 +173,8 @@ class XmlReader {
 
     const failed_count = parseInt(counters.failed, 10) + parseInt(counters.error, 10);
 
-    let status = PASSED;
-    if (failed_count > 0) status = FAILED;
+    let status = STATUS.PASSED.toString();
+    if (failed_count > 0) status = STATUS.FAILED;
 
     this.tests = results.filter(t => !!t.title);
 
@@ -313,7 +314,7 @@ class XmlReader {
     const dataString = {
       ...this.stats,
       api_key: this.requestParams.apiKey,
-      status_event: 'finish',
+      status: 'finished',
       tests: this.tests,
     };
 
@@ -348,9 +349,9 @@ function reduceTestCases(prev, item) {
     // prepend system output
     stack = `${testCaseItem['system-out'] || testCaseItem.output || testCaseItem.log || ''}\n\n${stack}`.trim()
 
-    let status = PASSED;
-    if ('failure' in testCaseItem || 'error' in testCaseItem) status = FAILED;
-    if ('skipped' in testCaseItem) status = SKIPPED;
+    let status = STATUS.PASSED.toString();
+    if ('failure' in testCaseItem || 'error' in testCaseItem) status = STATUS.FAILED;
+    if ('skipped' in testCaseItem) status = STATUS.SKIPPED;
 
     prev.push({
       create: true,

package/package.json

@@ -1,8 +1,9 @@
 {
   "name": "@testomatio/reporter",
-  "version": "0.8.0-beta.31",
+  "version": "0.8.0-beta.32",
   "description": "Testomatio Reporter Client",
   "main": "./lib/reporter.js",
+  "typings": "typings/index.d.ts",
   "repository": "git@github.com:testomatio/reporter.git",
   "author": "Michael Bodnarchuk <davert@testomat.io>,Koushik Mohan <koushikmohan1996@gmail.com>",
   "license": "MIT",
@@ -15,6 +16,8 @@
     "callsite-record": "^4.1.4",
     "chalk": "^4.1.0",
     "commander": "^4.1.1",
+    "csv-writer": "^1.6.0",
+    "debug": "^4.3.4",
     "dotenv": "^16.0.1",
     "fast-xml-parser": "^4.0.8",
     "glob": "^8.0.3",
@@ -24,7 +27,6 @@
     "json-cycle": "^1.3.0",
     "lodash.memoize": "^4.1.2",
     "lodash.merge": "^4.6.2",
-    "csv-writer": "^1.6.0",
     "uuid": "^9.0.0"
   },
   "files": [
@@ -50,6 +52,7 @@
   },
   "devDependencies": {
     "@cucumber/cucumber": "^8.6.0",
+    "@redocly/cli": "^1.0.0-beta.125",
     "@wdio/reporter": "^7.16.13",
     "chai": "^4.3.6",
     "codeceptjs": "^3.2.3",