Merge branch 'fix-empty-test-suites'

src/test_parser.ts

@@ -201,9 +201,9 @@ export async function parseTap(data: string): Promise<TestResult> {
 async function parseJunitXml(xml: any): Promise<TestResult> {
     let testsuites
 
-    if (xml.testsuites) {
-        testsuites = xml.testsuites.testsuite
-    } else if (xml.testsuite) {
+    if ('testsuites' in xml) {
+        testsuites = xml.testsuites.testsuite || [ ]
+    } else if ('testsuite' in xml) {
         testsuites = [ xml.testsuite ]
     } else {
         throw new Error("expected top-level testsuites or testsuite node")

test/junit.ts

@@ -105,5 +105,23 @@ describe("junit", async () => {
         expect(result.suites[0].cases[0].status).to.eql(TestStatus.Fail)
         expect(result.suites[1].cases[0].name).to.eql("dummy/path/to/project/and/passing_test_target")
         expect(result.suites[1].cases[0].status).to.eql(TestStatus.Pass)
-  })
+    })
+
+    it("parses empty testsuites", async () => {
+        const result = await parseJunitFile(`${resourcePath}/05-empty.xml`)
+
+        expect(result.counts.passed).to.eql(0)
+        expect(result.counts.failed).to.eql(0)
+        expect(result.counts.skipped).to.eql(0)
+        expect(result.suites.length).to.eql(0)
+    })
+
+    it("parses empty testsuite", async () => {
+        const result = await parseJunitFile(`${resourcePath}/06-empty.xml`)
+
+        expect(result.counts.passed).to.eql(0)
+        expect(result.counts.failed).to.eql(0)
+        expect(result.counts.skipped).to.eql(0)
+        expect(result.suites.length).to.eql(0)
+    })
 })

test/resources/junit/05-empty.xml

@@ -0,0 +1 @@
+<testsuites />

test/resources/junit/06-empty.xml

@@ -0,0 +1 @@
+<testsuite />