@brillout/test-e2e 0.6.9 → 0.6.11

package/dist/Logs.d.ts

@@ -12,6 +12,7 @@ type LogSource = 'stdout' | 'stderr' | 'Browser Error' | 'Browser Warning' | 'Br
 type LogEntry = {
     logSource: LogSource;
     logText: string;
+    logInfo: string;
     logTimestamp: string;
     isNotFailure: boolean;
     loggedAfterExit: boolean;
@@ -20,9 +21,10 @@ declare function hasFailLogs(failOnWarning: boolean): boolean;
 declare function clearLogs(): void;
 declare function logErrorsAndWarnings(): void;
 declare function flushLogs(): void;
-declare function add({ logSource, logText, loggedAfterExit, }: {
+declare function add({ logSource, logText, logInfo, loggedAfterExit, }: {
     logSource: LogSource;
     logText: string;
+    logInfo?: string;
     loggedAfterExit?: boolean;
 }): void;
 /**

package/dist/Logs.js

@@ -7,9 +7,9 @@ export const Logs = {
     hasFailLogs,
     logEagerly: false,
 };
-import { assert, ensureNewTerminalLine, isWindows } from './utils.js';
+import { assert, cliOptions, ensureNewTerminalLine, isWindows } from './utils.js';
 import pc from '@brillout/picocolors';
-import { getCurrentTestOptional } from './getCurrentTest.js';
+import { getCurrentTest, getCurrentTestOptional } from './getCurrentTest.js';
 import { logSection } from './logSection.js';
 import { isTolerateError } from './Logs/isTolerateError.js';
 import stripAnsi from 'strip-ansi';
@@ -57,11 +57,12 @@ function flushLogs() {
     logEntries.filter((logEntry) => !logEntry.loggedAfterExit).forEach((logEntry) => printLog(logEntry));
     logEntries = [];
 }
-function add({ logSource, logText, loggedAfterExit = false, }) {
+function add({ logSource, logText, logInfo = '', loggedAfterExit = false, }) {
     const logTimestamp = getTimestamp();
     const logEntry = {
         logSource,
         logText,
+        logInfo,
         logTimestamp,
         isNotFailure: isTolerateError({ logSource, logText }),
         loggedAfterExit,
@@ -77,6 +78,25 @@ function add({ logSource, logText, loggedAfterExit = false, }) {
         if (shouldLog)
             printLog(logEntry);
     }
+    if (logSource === 'Browser Error' || logSource === 'stderr') {
+        terminateUponErrorLog();
+    }
+}
+async function terminateUponErrorLog() {
+    if (!cliOptions.bail)
+        return;
+    const testInfo = getCurrentTest();
+    testInfo.aborted = true;
+    // Trick to abort the test: page.close() triggers the following error.
+    // ```console
+    // proxy.waitForFunction: Target closed
+    //     at clientSideNavigation (.testRun.ts:16:16)
+    //     at .testRun.ts:11:5
+    //     at file:///home/rom/code/test-e2e/src/runAll.ts:278:7 {
+    //   name: 'Error'
+    // }
+    // ```
+    await testInfo.page.close();
 }
 /**
  * Expect a log to be printed.

package/dist/run.js

@@ -69,12 +69,11 @@ additionalTimeout = 0, serverIsReadyMessage, serverIsReadyDelay = 1000, inspect
     };
     return;
     // Also called when the page throws an error or a warning
-    async function onConsole(msg) {
+    function onConsole(msg) {
         const type = msg.type();
-        const isError = type === 'error';
         Logs.add({
             logSource: (() => {
-                if (isError) {
+                if (type === 'error') {
                     return 'Browser Error';
                 }
                 if (type === 'warning') {
@@ -83,24 +82,15 @@ additionalTimeout = 0, serverIsReadyMessage, serverIsReadyDelay = 1000, inspect
                 return 'Browser Log';
             })(),
             logText: msg.text(),
-            /* Seems like the information below isn't that useful.
-            JSON.stringify(
-              {
+            logInfo: JSON.stringify({
                 type,
-                text: msg.text(),
                 location: msg.location(),
                 args: msg.args(),
-              },
-              null,
-              2,
-            ),
-            */
+            }, null, 2),
         });
-        if (isError)
-            await terminateUponBrowserError();
     }
     // For uncaught exceptions
-    async function onPageError(err) {
+    function onPageError(err) {
         Logs.add({
             logSource: 'Browser Error',
             logText: JSON.stringify({
@@ -108,22 +98,6 @@ additionalTimeout = 0, serverIsReadyMessage, serverIsReadyDelay = 1000, inspect
                 location: err.stack,
             }, null, 2),
         });
-        await terminateUponBrowserError();
-    }
-    async function terminateUponBrowserError() {
-        if (!cliOptions.bail)
-            return;
-        testInfo.aborted = true;
-        // Trick to abort the test: page.close() triggers the following error.
-        // ```console
-        // proxy.waitForFunction: Target closed
-        //     at clientSideNavigation (.testRun.ts:16:16)
-        //     at .testRun.ts:11:5
-        //     at file:///home/rom/code/test-e2e/src/runAll.ts:278:7 {
-        //   name: 'Error'
-        // }
-        // ```
-        await testInfo.page.close();
     }
 }
 async function startProcess({ serverIsReadyMessage, serverIsReadyDelay, additionalTimeout, cmd, }) {

package/dist/utils/cliOptions.js

@@ -1,10 +1,11 @@
 export { cliOptions };
 import { check, prepare, validate } from './parseCli.js';
+import { isCI } from './platform.js';
 prepare();
 const verbose = check('--verbose');
 const inspect = check('--inspect');
 const debugEsbuild = check('--debug-esbuild');
-const bail = check('--bail');
+const bail = check('--bail') || !isCI();
 // Defined & used in ./findFiles.ts
 const exclude = check('--exclude');
 validate();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@brillout/test-e2e",
-  "version": "0.6.9",
+  "version": "0.6.11",
   "bin": "./bin.js",
   "main": "./dist/index.js",
   "type": "module",