@brillout/test-e2e 0.1.18 → 0.1.20

package/dist/Logs.d.ts

@@ -6,7 +6,7 @@ export declare const Logs: {
     hasError: typeof hasError;
     flushEagerly: boolean;
 };
-declare type LogSource = 'stdout' | 'stderr' | 'Browser Error' | 'Browser Warning' | 'Browser Log' | 'Playwright' | 'run()' | 'test()' | 'Connection Error';
+declare type LogSource = 'stdout' | 'stderr' | 'Browser Error' | 'Browser Warning' | 'Browser Log' | 'Playwright' | 'run()' | 'run() failure' | 'test()' | 'Connection Error';
 declare type LogEntry = {
     logSource: LogSource;
     logText: string;

package/dist/Logs.js

@@ -22,6 +22,9 @@ function hasError(onlyFailOnBrowserError) {
         if (isNotFailure) {
             return;
         }
+        if (logSource === 'run() failure') {
+            return true;
+        }
         if (!onlyFailOnBrowserError && (logSource === 'Browser Warning' || logSource === 'stderr')) {
             return true;
         }
@@ -106,15 +109,23 @@ function printLog(logEntry) {
 }
 function colorize(logSource) {
     const { bold } = picocolors_1.default;
-    if (logSource === 'stderr' || logSource === 'Browser Error' || logSource === 'Connection Error')
+    if (logSource === 'stderr' ||
+        logSource === 'Browser Error' ||
+        logSource === 'Connection Error' ||
+        logSource === 'run() failure') {
         return bold(picocolors_1.default.red(logSource));
-    if (logSource === 'Browser Warning')
+    }
+    if (logSource === 'Browser Warning') {
         return bold(picocolors_1.default.yellow(logSource));
-    if (logSource === 'stdout' || logSource === 'Browser Log')
+    }
+    if (logSource === 'stdout' || logSource === 'Browser Log') {
         return bold(picocolors_1.default.blue(logSource));
-    if (logSource === 'Playwright')
+    }
+    if (logSource === 'Playwright') {
         return bold(picocolors_1.default.magenta(logSource));
-    if (logSource === 'run()' || logSource === 'test()')
+    }
+    if (logSource === 'run()' || logSource === 'test()') {
         return bold(picocolors_1.default.cyan(logSource));
+    }
     (0, utils_1.assert)(false);
 }

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,44 +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: 'stderr',
-                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;
         },
@@ -210,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;
 }
@@ -275,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;
@@ -290,10 +275,21 @@ 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', async (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({
             logSource: 'stdout',
@@ -302,14 +298,20 @@ function execRunScript({ onStdout, onStderr, onError, onExit, }) {
         onStdout?.(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', async (code) => {
+    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);
         const isExpected = isSuccessCode && exitIsPossible;
@@ -317,18 +319,9 @@ function execRunScript({ onStdout, onStderr, onError, onExit, }) {
             const errMsg = `Unexpected premature process termination, exit code: ${code}`;
             Logs_1.Logs.add({
                 logText: errMsg,
-                logSource: 'stderr',
+                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({
@@ -337,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) => {
@@ -346,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.18",
+  "version": "0.1.20",
   "bin": "./bin.js",
   "main": "./dist/index.js",
   "scripts": {