Fix errors for old junit format (#34)

* fix errors for old junit format * 1.0.17
MishaKav · Oct 15, 2022 · ebc4ddf · ebc4ddf
1 parent 3c924f4
commit ebc4ddf
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 6 deletions.
diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "jest-coverage-comment",
-  "version": "1.0.16",
+  "version": "1.0.17",
   "description": "Comments a pull request or commit with the jest code coverage badge, full report and tests summary",
   "author": "Misha Kav <[email protected]>",
   "license": "MIT",

diff --git a/src/junit.ts b/src/junit.ts
@@ -22,14 +22,19 @@ async function parseJunit(xmlContent: string): Promise<Junit | null> {
 
     const main = parsedJunit.testsuites['$']
     const testsuites = parsedJunit.testsuites.testsuite
+    const errors =
+      testsuites
+        ?.map((t: any) => Number(t['$'].errors))
+        .reduce((sum: number, a: number) => sum + a, 0) || 0
+
     const skipped =
       testsuites
         ?.map((t: any) => Number(t['$'].skipped))
         .reduce((sum: number, a: number) => sum + a, 0) || 0
 
     return {
       skipped,
-      errors: Number(main.errors),
+      errors: Number(main.errors || errors),
       failures: Number(main.failures),
       tests: Number(main.tests),
       time: Number(main.time),