Merge pull request #18 from test-results-reporter/17-nan-in-my-report…

…-using-newman

fix - junit report from newman

package.json

@@ -1,7 +1,7 @@
 {
   "name": "test-results-parser",
-  "version": "0.1.0",
-  "description": "Parse test results from JUnit, TestNG, xUnit and many more",
+  "version": "0.1.1",
+  "description": "Parse test results from JUnit, TestNG, xUnit, cucumber and many more",
   "main": "src/index.js",
   "types": "./src/index.d.ts",
   "scripts": {
@@ -20,7 +20,9 @@
     "testing",
     "results",
     "result",
-    "automation"
+    "automation",
+    "mocha",
+    "cucumber"
   ],
   "author": "Anudeep <asa.anudeep4@gmail.com>",
   "license": "MIT",

src/parsers/junit.js

@@ -34,6 +34,28 @@ function getTestSuite(rawSuite) {
   return suite;
 }
 
+/**
+ * @param {TestResult} result 
+ */
+function setAggregateResults(result) {
+  if (Number.isNaN(result.passed) || Number.isNaN(result.failed)) {
+    let passed = 0;
+    let failed = 0;
+    let errors = 0;
+    let skipped = 0;
+    result.suites.forEach(_suite => {
+      passed = _suite.passed + passed;
+      failed = _suite.failed + failed;
+      errors = _suite.errors + errors;
+      skipped = _suite.skipped + skipped;
+    });
+    result.passed = passed;
+    result.failed = failed;
+    result.errors = errors;
+    result.skipped = skipped;
+  }
+}
+
 function getTestResult(json) {
   const result = new TestResult();
   const rawResult = json["testsuites"][0];
@@ -56,6 +78,7 @@ function getTestResult(json) {
   for (let i = 0; i < filteredSuites.length; i++) {
     result.suites.push(getTestSuite(filteredSuites[i]));
   }
+  setAggregateResults(result);
   result.status = result.total === result.passed ? 'PASS' : 'FAIL';
   return result;
 }

tests/data/junit/newman.xml

@@ -0,0 +1,6 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<testsuites name="MyCollection" tests="1" time="32.937">
+  <testsuite name="1 - Authenticate " id="u68c4966-0877-4766-1e5b-uu96efb7f84" timestamp="2022-09-20T16:49:56.147Z" tests="1" failures="0" errors="0" time="0.807">
+    <testcase name="Teste" time="0.807" classname="TestePostamn"/>
+  </testsuite>
+ </testsuites> 

tests/parser.junit.spec.js

@@ -282,4 +282,49 @@ describe('Parser - JUnit', () => {
     });
   });
 
+  it('parse newman reporter', () => {
+    const result = parse({ type: 'junit', files: ['tests/data/junit/newman.xml'] });
+    assert.deepEqual(result, {
+      id: '',
+      name: 'MyCollection',
+      total: 1,
+      passed: 1,
+      failed: 0,
+      errors: 0,
+      skipped: 0,
+      retried: 0,
+      duration: 32937,
+      status: 'PASS',
+      suites: [
+        {
+          id: '',
+          name: '1 - Authenticate',
+          total: 1,
+          passed: 1,
+          failed: 0,
+          errors: 0,
+          skipped: 0,
+          duration: 807,
+          status: 'PASS',
+          cases: [
+            {
+              duration: 807,
+              errors: 0,
+              failed: 0,
+              failure: "",
+              id: "",
+              name: "Teste",
+              passed: 0,
+              skipped: 0,
+              stack_trace: "",
+              status: "PASS",
+              steps: [],
+              total: 0
+            }
+          ]
+        }
+      ]
+    });
+  });
+
 });