@jsenv/test 3.5.1 → 3.5.3

package/dist/jsenv_test.js

@@ -5464,7 +5464,7 @@ const pingServer = async (url) => {
 
 const startServerUsingCommand = async (
   webServer,
-  { signal, allocatedMs, logger, teardownCallbackSet },
+  { signal, allocatedMs, logger, teardownCallbackSet, shell = true },
 ) => {
   const spawnedProcess = spawn(webServer.command, [], {
     // On non-windows platforms, `detached: true` makes child process a leader of a new
@@ -5472,7 +5472,7 @@ const startServerUsingCommand = async (
     // @see https://nodejs.org/api/child_process.html#child_process_options_detached
     detached: process.platform !== "win32",
     stdio: ["pipe", "pipe", "pipe"],
-    shell: true,
+    shell,
     cwd: webServer.cwd,
   });
   if (!spawnedProcess.pid) {
@@ -5499,13 +5499,28 @@ const startServerUsingCommand = async (
   // stderr.on("line", (data) => {
   //   logger.debug(`[pid=${spawnedProcess.pid}][err] ${data}`);
   // });
+  let isAbort = false;
+  let isTeardown = false;
   let processClosed = false;
+  let stderr = "";
+  spawnedProcess.stderr.on("data", (data) => {
+    stderr += String(data);
+  });
   const closedPromise = new Promise((resolve) => {
     spawnedProcess.once("exit", (exitCode, signal) => {
-      logger.debug(
-        `[pid=${spawnedProcess.pid}] <process did exit: exitCode=${exitCode}, signal=${signal}>`,
-      );
       processClosed = true;
+      if (isAbort || isTeardown) {
+        logger.debug(
+          `web server process exit exitCode=${exitCode}, exitSignal=${signal}, pid=${spawnedProcess.pid}`,
+        );
+      } else {
+        logger.error(
+          `web server process premature exit exitCode=${exitCode}, exitSignal=${signal}, pid=${spawnedProcess.pid}
+--- stderr ---
+${stderr}
+--------------`,
+        );
+      }
       resolve();
     });
   });
@@ -5550,8 +5565,14 @@ const startServerUsingCommand = async (
   const startOperation = Abort.startOperation();
   startOperation.addAbortSignal(signal);
   const timeoutAbortSource = startOperation.timeout(allocatedMs);
-  startOperation.addAbortCallback(killProcess);
-  teardownCallbackSet.add(killProcess);
+  startOperation.addAbortCallback(async () => {
+    isAbort = true;
+    await killProcess();
+  });
+  teardownCallbackSet.add(async () => {
+    isTeardown = true;
+    await killProcess();
+  });
 
   const startedPromise = (async () => {
     const logScale = [100, 250, 500];
@@ -5598,7 +5619,8 @@ const startServerUsingModuleUrl = async (webServer, params) => {
   return startServerUsingCommand(
     {
       ...webServer,
-      command: `node ${fileURLToPath(webServer.moduleUrl)}`,
+      command: `node ${fileURLToPath(webServer.moduleUrl)} --jsenv-test`,
+      shell: false,
     },
     params,
   );

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/test",
-  "version": "3.5.1",
+  "version": "3.5.3",
   "license": "MIT",
   "repository": {
     "type": "git",

package/src/execution/web_server/start_using_command.js

@@ -4,7 +4,7 @@ import { pingServer } from "../../helpers/ping_server.js";
 
 export const startServerUsingCommand = async (
   webServer,
-  { signal, allocatedMs, logger, teardownCallbackSet },
+  { signal, allocatedMs, logger, teardownCallbackSet, shell = true },
 ) => {
   const spawnedProcess = spawn(webServer.command, [], {
     // On non-windows platforms, `detached: true` makes child process a leader of a new
@@ -12,7 +12,7 @@ export const startServerUsingCommand = async (
     // @see https://nodejs.org/api/child_process.html#child_process_options_detached
     detached: process.platform !== "win32",
     stdio: ["pipe", "pipe", "pipe"],
-    shell: true,
+    shell,
     cwd: webServer.cwd,
   });
   if (!spawnedProcess.pid) {
@@ -39,13 +39,28 @@ export const startServerUsingCommand = async (
   // stderr.on("line", (data) => {
   //   logger.debug(`[pid=${spawnedProcess.pid}][err] ${data}`);
   // });
+  let isAbort = false;
+  let isTeardown = false;
   let processClosed = false;
+  let stderr = "";
+  spawnedProcess.stderr.on("data", (data) => {
+    stderr += String(data);
+  });
   const closedPromise = new Promise((resolve) => {
     spawnedProcess.once("exit", (exitCode, signal) => {
-      logger.debug(
-        `[pid=${spawnedProcess.pid}] <process did exit: exitCode=${exitCode}, signal=${signal}>`,
-      );
       processClosed = true;
+      if (isAbort || isTeardown) {
+        logger.debug(
+          `web server process exit exitCode=${exitCode}, exitSignal=${signal}, pid=${spawnedProcess.pid}`,
+        );
+      } else {
+        logger.error(
+          `web server process premature exit exitCode=${exitCode}, exitSignal=${signal}, pid=${spawnedProcess.pid}
+--- stderr ---
+${stderr}
+--------------`,
+        );
+      }
       resolve();
     });
   });
@@ -90,8 +105,14 @@ export const startServerUsingCommand = async (
   const startOperation = Abort.startOperation();
   startOperation.addAbortSignal(signal);
   const timeoutAbortSource = startOperation.timeout(allocatedMs);
-  startOperation.addAbortCallback(killProcess);
-  teardownCallbackSet.add(killProcess);
+  startOperation.addAbortCallback(async () => {
+    isAbort = true;
+    await killProcess();
+  });
+  teardownCallbackSet.add(async () => {
+    isTeardown = true;
+    await killProcess();
+  });
 
   const startedPromise = (async () => {
     const logScale = [100, 250, 500];

package/src/execution/web_server/start_using_module_url.js

@@ -10,7 +10,8 @@ export const startServerUsingModuleUrl = async (webServer, params) => {
   return startServerUsingCommand(
     {
       ...webServer,
-      command: `node ${fileURLToPath(webServer.moduleUrl)}`,
+      command: `node ${fileURLToPath(webServer.moduleUrl)} --jsenv-test`,
+      shell: false,
     },
     params,
   );

package/src/execution/web_server_param.js

@@ -1,5 +1,4 @@
 import { assertAndNormalizeDirectoryUrl } from "@jsenv/filesystem";
-
 import { basicFetch } from "../helpers/basic_fetch.js";
 import { pingServer } from "../helpers/ping_server.js";
 import { startServerUsingCommand } from "./web_server/start_using_command.js";