Fix bug with empty testsuites

Author: thlcodes

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "live-spec",
-  "version": "0.0.5",
+  "version": "0.0.6",
   "description": "[WIP] Generate live spec from feature files & JUnit test results",
   "bin": "cli.js",
   "scripts": {
@@ -50,4 +50,4 @@
     "ts-jest": "^27.1.3",
     "typescript": "4.5.x"
   }
-}
+}

package.json

@@ -10,6 +10,8 @@ describe('junit', () => {
 
 const VALID_JUNIT_XML = `<?xml version="1.0" encoding="UTF-8"?>
 <testsuites name="my testsuites">
+  <testsuite name="empty testsuire" errors="0" failures="0" skipped="0" tests="0" timestamp="2022-03-01T08:00:00">
+  </testsuite>
   <testsuite name="my first testsuite" errors="1" failures="1" skipped="1" tests="4" timestamp="2022-03-01T08:00:00">
     <testcase name="my ok testcase">
     </testcase>

src/__tests__/junit.test.ts

@@ -54,7 +54,7 @@ function testcaseDetails(from: ErrorOrFailure): string | undefined {
 export async function parseJunitXml(xml: string): Promise<Testsuite[]> {
   const { testsuites } = await parseXml(xml);
   return (testsuites?.testsuite || []).map(
-    ({ $: { name, timestamp, failures, errors }, testcase: testcases }: JUnitTestsuite) => ({
+    ({ $: { name, timestamp, failures, errors }, testcase: testcases }: JUnitTestsuite) => ((testcases == null || testcases.length == 0) ? null : {
       name,
       timestamp,
       status: Number.parseInt(failures) + Number.parseInt(errors) > 0 ? 'failure' : 'success',
@@ -65,7 +65,7 @@ export async function parseJunitXml(xml: string): Promise<Testsuite[]> {
         details: testcaseDetails((error || [])[0] || (failure || [])[0])
       }))
     })
-  );
+  ).filter((ts: unknown) => ts != null);
 }
 
 // loads a XML file and parses its contents into a list of testsuites