@brillout/test-e2e 0.1.19 → 0.1.20

package/dist/setup.js

@@ -83,15 +83,7 @@ additionalTimeout = 0, serverIsReadyMessage, serverIsReadyDelay = 1000, inspect
         page_1.page.off('pageerror', onPageError);
         // `runProcess` is `undefined` if `start()` failed.
         if (runProcess) {
-            try {
-                await runProcess.terminate('SIGINT');
-            }
-            catch (err) {
-                Logs_1.Logs.add({
-                    logSource: 'run()',
-                    logText: String(err),
-                });
-            }
+            await runProcess.terminate('SIGINT');
         }
     };
     return;
@@ -141,6 +133,8 @@ async function start() {
     let rejectServerStart;
     const promise = new Promise((_resolve, _reject) => {
         resolveServerStart = () => {
+            (0, utils_2.assert)(!processHasExited());
+            (0, utils_2.assert)(getRunInfo().cmd === cmd);
             Logs_1.Logs.add({
                 logSource: 'run()',
                 logText: 'server is ready',
@@ -151,45 +145,40 @@ async function start() {
             _resolve(runProcess);
         };
         rejectServerStart = async (err) => {
+            (0, utils_2.assert)(processHasExited());
             (0, utils_2.assert)(getRunInfo().cmd === cmd);
             done(true);
-            Logs_1.Logs.add({
-                logSource: 'run() failure',
-                logText: String(err),
-            });
             clearTimeout(serverStartTimeout);
-            try {
-                await terminate('SIGKILL');
-            }
-            catch (err) {
-                Logs_1.Logs.add({
-                    logSource: 'run()',
-                    logText: String(err),
-                });
-            }
             _reject(err);
         };
     });
     const timeoutTotal = TIMEOUT_NPM_SCRIPT + additionalTimeout;
     let hasTimedout = false;
-    const serverStartTimeout = setTimeout(() => {
+    const serverStartTimeout = setTimeout(async () => {
+        hasTimedout = true;
         let errMsg = '';
         errMsg += `Server still didn't start after ${(0, utils_1.humanizeTime)(timeoutTotal)} of running the npm script \`${cmd}\`.`;
         if (serverIsReadyMessage) {
             errMsg += ` (The stdout of the npm script did not include: "${serverIsReadyMessage}".)`;
         }
-        hasTimedout = true;
+        Logs_1.Logs.add({
+            logSource: 'run() failure',
+            logText: errMsg,
+        });
+        await terminate();
         rejectServerStart(new Error(errMsg));
     }, timeoutTotal);
     // Kill any process that listens to port `3000`
     if (!process.env.CI && isLinux()) {
         await (0, utils_1.runCommand)('fuser -k 3000/tcp', { swallowError: true, timeout: 10 * 1000 });
     }
-    const { terminate } = execRunScript({
-        onError(err) {
+    const { terminate, processHasExited } = execRunScript({
+        async onFailure(err) {
+            (0, utils_2.assert)(processHasExited());
             rejectServerStart(err);
         },
         onExit() {
+            (0, utils_2.assert)(processHasExited());
             const exitIsPossible = hasSuccessfullyStarted === true || hasTimedout;
             return exitIsPossible;
         },
@@ -211,11 +200,6 @@ async function start() {
                 done();
             }
         },
-        onStderr(data) {
-            if (data.includes('EADDRINUSE')) {
-                rejectServerStart(new Error('Port conflict? Port already in use EADDRINUSE.'));
-            }
-        },
     });
     return promise;
 }
@@ -276,7 +260,7 @@ function stopProcess({ proc, cwd, cmd, signal, }) {
     }
     return promise;
 }
-function execRunScript({ onStdout, onStderr, onError, onExit, }) {
+function execRunScript({ onStdout, onFailure, onExit, }) {
     const { cwd, cmd } = getRunInfo();
     let [command, ...args] = cmd.split(' ');
     let detached = true;
@@ -291,10 +275,20 @@ function execRunScript({ onStdout, onStderr, onError, onExit, }) {
         logText: `Spawn command \`${cmd}\``,
     });
     const proc = (0, child_process_1.spawn)(command, args, { cwd, detached });
+    let procExited = false;
+    const exitAndFail = async (err) => {
+        Logs_1.Logs.add({
+            logText: err.message,
+            logSource: 'run() failure',
+        });
+        await terminate();
+        onFailure(err);
+    };
     proc.stdin.on('data', async (data) => {
-        onError(new Error(`Command is \`${cmd}\` (${cwd}) is invoking \`stdin\`: ${data}.`));
+        await exitAndFail(new Error(`Command is \`${cmd}\` (${cwd}) is invoking \`stdin\`: ${data}.`));
     });
     proc.stdout.on('data', (data) => {
+        (0, utils_2.assert)(!procExited);
         (0, utils_2.assert)(getRunInfo().cmd === cmd);
         data = data.toString();
         Logs_1.Logs.add({
@@ -303,16 +297,20 @@ function execRunScript({ onStdout, onStderr, onError, onExit, }) {
         });
         onStdout?.(data);
     });
-    proc.stderr.on('data', (data) => {
+    proc.stderr.on('data', async (data) => {
+        (0, utils_2.assert)(!procExited);
         (0, utils_2.assert)(getRunInfo().cmd === cmd);
         data = data.toString();
         Logs_1.Logs.add({
             logSource: 'stderr',
             logText: data,
         });
-        onStderr?.(data);
+        if (data.includes('EADDRINUSE')) {
+            await exitAndFail(new Error('Port conflict? Port already in use EADDRINUSE.'));
+        }
     });
     proc.on('exit', (code) => {
+        procExited = true;
         (0, utils_2.assert)(getRunInfo().cmd === cmd);
         const exitIsPossible = onExit();
         const isSuccessCode = [0, null].includes(code) || (isWindows() && code === 1);
@@ -323,16 +321,7 @@ function execRunScript({ onStdout, onStderr, onError, onExit, }) {
                 logText: errMsg,
                 logSource: 'run() failure',
             });
-            onError(new Error(errMsg));
-            /*
-            try {
-              await terminate('SIGKILL')
-            } catch (err: unknown) {
-              if( !err.code === 'ESRCH' ) {
-                onError(err as Error)
-              }
-            }
-            */
+            onFailure(new Error(errMsg));
         }
         else {
             Logs_1.Logs.add({
@@ -341,8 +330,11 @@ function execRunScript({ onStdout, onStderr, onError, onExit, }) {
             });
         }
     });
-    return { terminate };
-    async function terminate(signal) {
+    return { terminate, processHasExited };
+    function processHasExited() {
+        return procExited;
+    }
+    async function terminate(signal = 'SIGKILL') {
         let resolve;
         let reject;
         const promise = new Promise((_resolve, _reject) => {
@@ -350,15 +342,28 @@ function execRunScript({ onStdout, onStderr, onError, onExit, }) {
             reject = _reject;
         });
         const timeout = setTimeout(() => {
-            reject(new Error('Process termination timeout. Cmd: ' + cmd));
+            const errMsg = 'Process termination timeout. Cmd: ' + cmd;
+            Logs_1.Logs.add({
+                logSource: 'run() failure',
+                logText: errMsg,
+            });
+            reject(new Error(errMsg));
         }, TIMEOUT_PROCESS_TERMINATION);
         (0, utils_2.assert)(proc);
-        await stopProcess({
-            proc,
-            cwd,
-            cmd,
-            signal,
-        });
+        try {
+            await stopProcess({
+                proc,
+                cwd,
+                cmd,
+                signal,
+            });
+        }
+        catch (err) {
+            Logs_1.Logs.add({
+                logSource: 'run() failure',
+                logText: String(err),
+            });
+        }
         clearTimeout(timeout);
         resolve();
         return promise;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@brillout/test-e2e",
-  "version": "0.1.19",
+  "version": "0.1.20",
   "bin": "./bin.js",
   "main": "./dist/index.js",
   "scripts": {