tewip-ai 0.18.0 → 0.18.2

package/dist/engine/import/junit.js

@@ -116,6 +116,9 @@ function junitCases(root, workspace) {
         }
         out.push({ testId, file: file || (testcase.attrs.classname ?? '').trim(), duration: Number.isFinite(seconds) ? Math.round(seconds * 1000) : 0, attempts });
     };
+    // <testrun> is the root Eclipse's JUnit runner and Ant write; everything inside it is ordinary
+    // JUnit XML. A real report in that shape was rejected outright as "no test report Tewip can
+    // read", which is a poor answer to a file holding 36 perfectly readable results.
     walk(root, root.name === 'testsuite' ? root : undefined);
     // Attempts whose error the runner did not record borrow the final one's, when there is one.
     for (const c of out) {
@@ -193,6 +196,26 @@ export function junitFiles(path) {
  * Playwright; every test, passing or failing, is counted in the run summary so its history
  * shows the passes too.
  */
+/**
+ * A runner's own timestamp, when it is one a machine can read.
+ *
+ * Maven Surefire writes `timestamp="2025-03-05T11:53:11 IST"` - a timezone abbreviation with no
+ * offset, which is not a date any parser accepts. `new Date(...).toISOString()` then threw, and
+ * Tewip died on one of the most widely used test runners there is. This was found by reading real
+ * Surefire reports from public repositories; every fixture collected for this project had a clean
+ * timestamp. A report Tewip cannot date is still a report worth reading, so an unreadable one falls
+ * back to the time of import rather than stopping everything.
+ */
+function readTimestamp(raw) {
+    if (!raw)
+        return null;
+    const asIs = new Date(raw);
+    if (!Number.isNaN(asIs.getTime()))
+        return asIs.toISOString();
+    // Drop a trailing zone abbreviation (IST, CEST, PDT...) and read the rest as local time.
+    const withoutZone = new Date(raw.replace(/\s+[A-Z]{2,5}$/, ''));
+    return Number.isNaN(withoutZone.getTime()) ? null : withoutZone.toISOString();
+}
 export function importJunit(path, meta) {
     const records = [];
     const attempts = {};
@@ -229,7 +252,7 @@ export function importJunit(path, meta) {
             if (start && (!startedAt || start < startedAt))
                 startedAt = start;
         }
-        else if (root.name === 'testsuites' || root.name === 'testsuite') {
+        else if (root.name === 'testsuites' || root.name === 'testsuite' || root.name === 'testrun') {
             cases = junitCases(root, meta.workspace);
             const stamp = root.attrs.timestamp ?? root.children.find((c) => c.name === 'testsuite')?.attrs.timestamp;
             if (stamp && (!startedAt || stamp < startedAt))
@@ -288,7 +311,7 @@ export function importJunit(path, meta) {
         runId: meta.runId,
         gitSha: meta.gitSha,
         ...(meta.branch ? { branch: meta.branch } : {}),
-        startedAt: startedAt ? new Date(startedAt).toISOString() : now,
+        startedAt: readTimestamp(startedAt) ?? now,
         finishedAt: now,
         status: anyFailed ? 'failed' : 'passed',
         attempts,
@@ -297,7 +320,7 @@ export function importJunit(path, meta) {
     if (unread.length > 0) {
         const list = unread.slice(0, 5).join(', ');
         if (parsedFiles === 0) {
-            throw new Error(`no test report Tewip can read in ${path}: ${list}. It reads JUnit XML (<testsuites>/<testsuite>), NUnit 3 (<test-run>), .NET TRX (<TestRun>), ` +
+            throw new Error(`no test report Tewip can read in ${path}: ${list}. It reads JUnit XML (<testsuites>/<testsuite>/<testrun>), NUnit 3 (<test-run>), .NET TRX (<TestRun>), ` +
                 'Robot Framework output.xml (<robot>), JMeter .jtl (<testResults>) and a k6 summary (--summary-export). ' +
                 'Most runners can write JUnit XML; for TestNG use the Surefire report, and for NUnit use --result=x.xml.');
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tewip-ai",
-  "version": "0.18.0",
+  "version": "0.18.2",
   "description": "Tewip finds why each failing test failed - real bug, flaky test, environment, test data or changed selector - for any stack, from the command line, CI, or your editor.",
   "keywords": [
     "test-failures",