@reporters/junit 1.0.3 → 1.1.0

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+## [1.1.0](https://github.com/MoLow/reporters/compare/junit-v1.0.4...junit-v1.1.0) (2023-05-30)
+
+
+### Features
+
+* report node 20 features ([c99a76c](https://github.com/MoLow/reporters/commit/c99a76c0f6bef75abb2c053c82c88448b0c82690))
+
+## [1.0.4](https://github.com/MoLow/reporters/compare/junit-v1.0.3...junit-v1.0.4) (2023-04-03)
+
+
+### Bug Fixes
+
+* allow multiple roots ([590df94](https://github.com/MoLow/reporters/commit/590df948f8a4626fc29e8ce185e08d2226a307ba))
+
 ## [1.0.3](https://github.com/MoLow/reporters/compare/junit-v1.0.2...junit-v1.0.3) (2023-03-12)
 
 

package/README.md

@@ -1,4 +1,5 @@
-[![npm version](https://img.shields.io/npm/v/@reporters/junit)](https://www.npmjs.com/package/@reporters/junit) ![tests](https://github.com/MoLow/reporters/actions/workflows/test.yaml/badge.svg?branch=main)
+[![npm version](https://img.shields.io/npm/v/@reporters/junit)](https://www.npmjs.com/package/@reporters/junit) ![tests](https://github.com/MoLow/reporters/actions/workflows/test.yaml/badge.svg?branch=main) [![codecov](https://codecov.io/gh/MoLow/reporters/branch/main/graph/badge.svg?token=0LFVC8SCQV)](https://codecov.io/gh/MoLow/reporters)
+
 # Junit Reporter
 A Junit reporter for `node:test`.
 intendend for use with major CI tools like Jenkins, CircleCI, etc that consume Junit reports.

package/index.js

@@ -41,6 +41,7 @@ module.exports = async function* junitReporter(source) {
   yield '<?xml version="1.0" encoding="utf-8"?>\n';
   yield '<testsuites>\n';
   let currentSuite = null;
+  const roots = [];
 
   function startTest(event) {
     const originalSuite = currentSuite;
@@ -51,6 +52,9 @@ module.exports = async function* junitReporter(source) {
       children: [],
     };
     originalSuite?.children.push(currentSuite);
+    if (!currentSuite.parent) {
+      roots.push(currentSuite);
+    }
   }
 
   for await (const event of source) {
@@ -69,7 +73,7 @@ module.exports = async function* junitReporter(source) {
           startTest(event);
         }
         const currentTest = currentSuite;
-        if (currentSuite?.nesting === event.data.nesting && currentSuite?.parent) {
+        if (currentSuite?.nesting === event.data.nesting) {
           currentSuite = currentSuite.parent;
         }
         currentTest.props.time = (event.data.details.duration_ms / 1000).toFixed(6);
@@ -113,6 +117,8 @@ module.exports = async function* junitReporter(source) {
         break;
     }
   }
-  yield treeToXML(currentSuite);
+  for (const suite of roots) {
+    yield treeToXML(suite);
+  }
   yield '</testsuites>\n';
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reporters/junit",
-  "version": "1.0.3",
+  "version": "1.1.0",
   "description": "A jUnit reporter for `node:test`",
   "keywords": [
     "junit",

package/tests/index.test.js

@@ -1,9 +1,10 @@
 const { spawnSync } = require('child_process');
 const assert = require('assert');
 const { compareLines } = require('../../../tests/utils');
-const output = require('./output');
+// eslint-disable-next-line import/no-dynamic-require
+const output = require(`./output.${process.version.split('.')[0]}`);
 
-const child = spawnSync(process.execPath, ['--test-reporter', './index.js', '../../tests/example']);
+const child = spawnSync(process.execPath, ['--test-reporter', './index.js', '../../tests/example'], { env: {} });
 
 assert.strictEqual(child.stderr?.toString(), '');
 compareLines(child.stdout?.toString(), output.stdout);

package/tests/{output.js → output.v18.js}

@@ -1,14 +1,14 @@
 module.exports = {
   stdout: `<\\?xml version="1.0" encoding="utf-8"\\?>
 <testsuites>
-\t<testsuite name="tests" time=".*" disabled="0" errors="0" tests="2" failures="1" skipped="0" hostname=".*">
+\t<testsuite name="tests" time=".*" disabled="0" errors="0" tests="3" failures="1" skipped="0" hostname=".*">
 \t\t<testcase name="is ok" time=".*" classname="test"/>
 \t\t<testcase name="fails" time=".*" classname="test" failure="this is an error">
 \t\t\t<failure type="testCodeFailure" message="this is an error">
 \\[Error \\[ERR_TEST_FAILURE\\]: this is an error\\] {
   failureType: 'testCodeFailure',
   cause: Error: this is an error
-      at Object.&lt;anonymous&gt; (.*)
+      .*
       .*
       .*
       .*
@@ -21,6 +21,10 @@ module.exports = {
 }
 \t\t\t</failure>
 \t\t</testcase>
+\t\t<testcase name="is a diagnostic" time=".*" classname="test"/>
+\t</testsuite>
+\t<testsuite name="more tests" time=".*" disabled="0" errors="0" tests="1" failures="0" skipped="0" hostname=".*">
+\t\t<testcase name="is ok" time=".*" classname="test"/>
 \t</testsuite>
 </testsuites>
 `,

package/tests/output.v19.js

@@ -0,0 +1,31 @@
+module.exports = {
+  stdout: `<\\?xml version="1.0" encoding="utf-8"\\?>
+<testsuites>
+\t<testsuite name="tests" time=".*" disabled="0" errors="0" tests="3" failures="1" skipped="0" hostname=".*">
+\t\t<testcase name="is ok" time=".*" classname="test"/>
+\t\t<testcase name="fails" time=".*" classname="test" failure="this is an error">
+\t\t\t<failure type="testCodeFailure" message="this is an error">
+\\[Error \\[ERR_TEST_FAILURE\\]: this is an error\\] {
+  failureType: 'testCodeFailure',
+  cause: Error: this is an error
+      .*
+      .*
+      .*
+      .*
+      .*
+      .*
+      .*
+      .*
+      .*
+  code: 'ERR_TEST_FAILURE'
+}
+\t\t\t</failure>
+\t\t</testcase>
+\t\t<testcase name="is a diagnostic" time=".*" classname="test"/>
+\t</testsuite>
+\t<testsuite name="more tests" time=".*" disabled="0" errors="0" tests="1" failures="0" skipped="0" hostname=".*">
+\t\t<testcase name="is ok" time=".*" classname="test"/>
+\t</testsuite>
+</testsuites>
+`,
+};

package/tests/output.v20.js

@@ -0,0 +1,31 @@
+module.exports = {
+  stdout: `<\\?xml version="1.0" encoding="utf-8"\\?>
+<testsuites>
+\t<testsuite name="tests" time=".*" disabled="0" errors="0" tests="3" failures="1" skipped="0" hostname=".*">
+\t\t<testcase name="is ok" time=".*" classname="test"/>
+\t\t<testcase name="fails" time=".*" classname="test" failure="this is an error">
+\t\t\t<failure type="testCodeFailure" message="this is an error">
+\\[Error \\[ERR_TEST_FAILURE\\]: this is an error\\] {
+  failureType: 'testCodeFailure',
+  cause: Error: this is an error
+      .*
+      .*
+      .*
+      .*
+      .*
+      .*
+      .*
+      .*
+      .*
+  code: 'ERR_TEST_FAILURE'
+}
+\t\t\t</failure>
+\t\t</testcase>
+\t\t<testcase name="is a diagnostic" time=".*" classname="test"/>
+\t</testsuite>
+\t<testsuite name="more tests" time=".*" disabled="0" errors="0" tests="1" failures="0" skipped="0" hostname=".*">
+\t\t<testcase name="is ok" time=".*" classname="test"/>
+\t</testsuite>
+</testsuites>
+`,
+};