@jsenv/test 3.4.1 → 3.4.3

package/dist/jsenv_test.js

@@ -4143,26 +4143,28 @@ const formatErrorForTerminal = (
       error.site.url.startsWith("file:") &&
       typeof error.site.line === "number"
     ) {
-      const content = readFileSync(new URL(error.site.url), "utf8");
-      text += generateContentFrame({
-        content,
-        line: error.site.line,
-        column: error.site.column,
-        linesAbove: 2,
-        linesBelow: 0,
-        lineMaxWidth: process.stdout.columns,
-        format: (string, type) => {
-          return {
-            line_number_aside: () => ANSI.color(string, ANSI.GREY),
-            char: () => string,
-            marker_overflow_left: () => ANSI.color(string, ANSI.GREY),
-            marker_overflow_right: () => ANSI.color(string, ANSI.GREY),
-            marker_line: () => ANSI.color(string, ANSI.RED),
-            marker_column: () => ANSI.color(string, ANSI.RED),
-          }[type]();
-        },
-      });
-      text += `\n`;
+      try {
+        const content = readFileSync(new URL(error.site.url), "utf8");
+        text += generateContentFrame({
+          content,
+          line: error.site.line,
+          column: error.site.column,
+          linesAbove: 2,
+          linesBelow: 0,
+          lineMaxWidth: process.stdout.columns,
+          format: (string, type) => {
+            return {
+              line_number_aside: () => ANSI.color(string, ANSI.GREY),
+              char: () => string,
+              marker_overflow_left: () => ANSI.color(string, ANSI.GREY),
+              marker_overflow_right: () => ANSI.color(string, ANSI.GREY),
+              marker_line: () => ANSI.color(string, ANSI.RED),
+              marker_column: () => ANSI.color(string, ANSI.RED),
+            }[type]();
+          },
+        });
+        text += `\n`;
+      } catch {}
     }
   }
   text += `${error.name}: ${error.message}`;
@@ -5576,7 +5578,7 @@ const startServerUsingCommand = async (
       if (timeoutAbortSource.signal.aborted) {
         // aborted by timeout
         throw new Error(
-          `"${webServer.command}" command did not start a server in less than ${allocatedMs}ms`,
+          `"${webServer.command}" command did not start a server at ${webServer.origin} in less than ${allocatedMs}ms`,
         );
       }
       if (signal.aborted) {
@@ -9208,11 +9210,24 @@ const reportAsJunitXml = async (
       const testCase = testSuite.createNode("testcase", {
         file: fileRelativeUrl,
         name: group,
-        time: mockFluctuatingValues ? "[mock]" : executionResult.timings.end, // duration of this execution
-        timestamp: mockFluctuatingValues
+        // duration of this execution
+        time: mockFluctuatingValues
           ? "[mock]"
-          : new Date(executionResult.timings.origin).toISOString().slice(0, -5), //  Date and time of when the test run was executed
-        tests: 1, // Total number of tests for this execution
+          : executionResult.status === "skipped"
+            ? 0
+            : executionResult.timings.end,
+        // Date and time of when the test run was executed
+        ...(executionResult.status === "skipped"
+          ? {}
+          : {
+              timestamp: mockFluctuatingValues
+                ? "[mock]"
+                : new Date(executionResult.timings.origin)
+                    .toISOString()
+                    .slice(0, -5),
+            }),
+        // Total number of tests for this execution
+        tests: 1,
       });
       testSuite.appendChild(testCase);
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/test",
-  "version": "3.4.1",
+  "version": "3.4.3",
   "license": "MIT",
   "repository": {
     "type": "git",
@@ -41,7 +41,7 @@
     "@jsenv/ast": "6.2.16",
     "@jsenv/exception": "1.1.2",
     "@jsenv/filesystem": "4.10.2",
-    "@jsenv/github-check-run": "0.1.32",
+    "@jsenv/github-check-run": "0.2.0",
     "@jsenv/humanize": "1.2.8",
     "@jsenv/plugin-supervisor": "1.5.18",
     "@jsenv/sourcemap": "1.2.23",

package/src/execution/execute_test_plan.js

@@ -25,7 +25,6 @@ import {
 import { memoryUsage } from "node:process";
 import { takeCoverage } from "node:v8";
 import stripAnsi from "strip-ansi";
-
 import { generateCoverage } from "../coverage/generate_coverage.js";
 import { createCallOrderer } from "../helpers/call_orderer.js";
 import { startMeasuringTotalCpuUsage } from "../helpers/cpu_usage.js";

package/src/execution/format_error_for_terminal.js

@@ -25,26 +25,28 @@ export const formatErrorForTerminal = (
       error.site.url.startsWith("file:") &&
       typeof error.site.line === "number"
     ) {
-      const content = readFileSync(new URL(error.site.url), "utf8");
-      text += generateContentFrame({
-        content,
-        line: error.site.line,
-        column: error.site.column,
-        linesAbove: 2,
-        linesBelow: 0,
-        lineMaxWidth: process.stdout.columns,
-        format: (string, type) => {
-          return {
-            line_number_aside: () => ANSI.color(string, ANSI.GREY),
-            char: () => string,
-            marker_overflow_left: () => ANSI.color(string, ANSI.GREY),
-            marker_overflow_right: () => ANSI.color(string, ANSI.GREY),
-            marker_line: () => ANSI.color(string, ANSI.RED),
-            marker_column: () => ANSI.color(string, ANSI.RED),
-          }[type]();
-        },
-      });
-      text += `\n`;
+      try {
+        const content = readFileSync(new URL(error.site.url), "utf8");
+        text += generateContentFrame({
+          content,
+          line: error.site.line,
+          column: error.site.column,
+          linesAbove: 2,
+          linesBelow: 0,
+          lineMaxWidth: process.stdout.columns,
+          format: (string, type) => {
+            return {
+              line_number_aside: () => ANSI.color(string, ANSI.GREY),
+              char: () => string,
+              marker_overflow_left: () => ANSI.color(string, ANSI.GREY),
+              marker_overflow_right: () => ANSI.color(string, ANSI.GREY),
+              marker_line: () => ANSI.color(string, ANSI.RED),
+              marker_column: () => ANSI.color(string, ANSI.RED),
+            }[type]();
+          },
+        });
+        text += `\n`;
+      } catch {}
     }
   }
   text += `${error.name}: ${error.message}`;

package/src/execution/junit_xml_file/report_as_junit_xml.js

@@ -38,11 +38,24 @@ export const reportAsJunitXml = async (
       const testCase = testSuite.createNode("testcase", {
         file: fileRelativeUrl,
         name: group,
-        time: mockFluctuatingValues ? "[mock]" : executionResult.timings.end, // duration of this execution
-        timestamp: mockFluctuatingValues
+        // duration of this execution
+        time: mockFluctuatingValues
           ? "[mock]"
-          : new Date(executionResult.timings.origin).toISOString().slice(0, -5), //  Date and time of when the test run was executed
-        tests: 1, // Total number of tests for this execution
+          : executionResult.status === "skipped"
+            ? 0
+            : executionResult.timings.end,
+        // Date and time of when the test run was executed
+        ...(executionResult.status === "skipped"
+          ? {}
+          : {
+              timestamp: mockFluctuatingValues
+                ? "[mock]"
+                : new Date(executionResult.timings.origin)
+                    .toISOString()
+                    .slice(0, -5),
+            }),
+        // Total number of tests for this execution
+        tests: 1,
       });
       testSuite.appendChild(testCase);
 

package/src/execution/web_server/start_using_command.js

@@ -117,7 +117,7 @@ export const startServerUsingCommand = async (
       if (timeoutAbortSource.signal.aborted) {
         // aborted by timeout
         throw new Error(
-          `"${webServer.command}" command did not start a server in less than ${allocatedMs}ms`,
+          `"${webServer.command}" command did not start a server at ${webServer.origin} in less than ${allocatedMs}ms`,
         );
       }
       if (signal.aborted) {