Merge pull request #2 from cristidumitru/main

Avoid NaN in totalTime

__tests__/__outputs__/go-junit-report-test-results.md

@@ -1,6 +1,6 @@
 ![Tests failed](https://img.shields.io/badge/tests-2%20passed%2C%202%20failed%2C%201%20skipped-critical)
 ## ❌ <a id="user-content-r0" href="#r0">fixtures/external/go/go-junit-report.xml</a>
-**5** tests were completed in **NaNms** with **2** passed, **2** failed and **1** skipped.
+**5** tests were completed in **1s** with **2** passed, **2** failed and **1** skipped.
 |Test suite|Passed|Failed|Skipped|Time|
 |:---|---:|---:|---:|---:|
 |[github.com/company/package1](#r0s0)||||0ms|

__tests__/__snapshots__/go-junit.test.ts.snap

@@ -123,6 +123,6 @@ TestRunResult {
       "totalTime": 835,
     },
   ],
-  "totalTime": NaN,
+  "totalTime": undefined,
 }
 `;

src/parsers/go-junit/go-junit-parser.ts

@@ -43,7 +43,10 @@ export class GoJunitParser implements TestParser {
             return sr
           })
 
-    const time = parseFloat(junit.testsuites.$.time) * 1000
+    const time =
+      junit.testsuites.$?.time === undefined
+        ? undefined
+        : parseFloat(junit.testsuites.$.time) * 1000
     return new TestRunResult(path, suites, time)
   }