@brillout/test-e2e 0.1.12 → 0.1.14

package/dist/Logs.d.ts

@@ -3,17 +3,17 @@ export declare const Logs: {
     add: typeof add;
     flush: typeof flush;
     clear: typeof clear;
-    getBrowserErrors: typeof getBrowserErrors;
+    hasError: typeof hasError;
     flushEagerly: boolean;
 };
-declare type LogSource = 'stdout' | 'stderr' | 'Browser Error' | 'Browser Log' | 'Playwright' | 'run()' | 'test()' | 'Connection Error';
+declare type LogSource = 'stdout' | 'stderr' | 'Browser Error' | 'Browser Warning' | 'Browser Log' | 'Playwright' | 'run()' | 'test()' | 'Connection Error';
 declare type LogEntry = {
     logSource: LogSource;
     logText: string;
     logTimestamp: string;
     isExpectedError: boolean;
 };
-declare function getBrowserErrors(): LogEntry[];
+declare function hasError(): boolean;
 declare function clear(): void;
 declare function flush(): void;
 declare function add({ logSource, logText }: {

package/dist/Logs.js

@@ -8,7 +8,7 @@ exports.Logs = {
     add,
     flush,
     clear,
-    getBrowserErrors,
+    hasError,
     flushEagerly: false,
 };
 const utils_1 = require("./utils");
@@ -16,8 +16,9 @@ const picocolors_1 = __importDefault(require("picocolors"));
 const getCurrentTest_1 = require("./getCurrentTest");
 let logEntries = [];
 let logEntriesNotPrinted = [];
-function getBrowserErrors() {
-    return logEntries.filter(({ logSource, isExpectedError }) => logSource === 'Browser Error' && !isExpectedError);
+function hasError() {
+    const logErrors = logEntries.filter(({ logSource, isExpectedError }) => (logSource === 'Browser Error' || logSource === 'Browser Warning' || logSource === 'stderr') && !isExpectedError);
+    return logErrors.length > 0;
 }
 function clear() {
     logEntries = [];
@@ -85,11 +86,13 @@ function colorize(logSource) {
     const { bold } = picocolors_1.default;
     if (logSource === 'stderr' || logSource === 'Browser Error' || logSource === 'Connection Error')
         return bold(picocolors_1.default.red(logSource));
+    if (logSource === 'Browser Warning')
+        return bold(picocolors_1.default.yellow(logSource));
     if (logSource === 'stdout' || logSource === 'Browser Log')
         return bold(picocolors_1.default.blue(logSource));
     if (logSource === 'Playwright')
         return bold(picocolors_1.default.magenta(logSource));
     if (logSource === 'run()' || logSource === 'test()')
-        return bold(picocolors_1.default.yellow(logSource));
+        return bold(picocolors_1.default.cyan(logSource));
     (0, utils_1.assert)(false);
 }

package/dist/runTests.js

@@ -15,9 +15,9 @@ async function runTests(browser) {
     testInfo.page = page;
     // Set when user calls `skip()`
     if (testInfo.skipped) {
+        logTestsResult(false);
         (0, utils_1.assertUsage)(!testInfo.runInfo, 'You cannot call `run()` after calling `skip()`');
         (0, utils_1.assertUsage)(testInfo.tests === undefined, 'You cannot call `test()` after calling `skip()`');
-        logTestsResult(false);
         return;
     }
     // Set when user calls `run()`
@@ -32,9 +32,9 @@ async function runTests(browser) {
         await testInfo.startServer();
     }
     catch (err) {
+        logTestsResult(false);
         console.log(err);
         Logs_1.Logs.flush();
-        logTestsResult(false);
         process.exit(1);
     }
     for (const { testDesc, testFn } of testInfo.tests) {
@@ -52,18 +52,17 @@ async function runTests(browser) {
         }
         done(!!err);
         testInfo.afterEach(!!err);
-        const browserErrors = Logs_1.Logs.getBrowserErrors();
-        const browserThrewError = browserErrors.length > 0;
-        const isFailure = err || browserThrewError;
+        const hasErrorLog = Logs_1.Logs.hasError();
+        const isFailure = err || hasErrorLog;
         if (isFailure) {
+            logTestsResult(false);
             if (err) {
                 console.error(err);
             }
-            if (browserThrewError) {
-                console.log(new Error('The browser threw one or more error'));
+            if (hasErrorLog) {
+                console.log(new Error('The browser/server threw/logged one or more error, see logs below.'));
             }
             Logs_1.Logs.flush();
-            logTestsResult(false);
             process.exit(1);
         }
         else {

package/dist/setup.js

@@ -90,7 +90,15 @@ additionalTimeout = 0, serverIsReadyMessage, serverIsReadyDelay = 1000, inspect
     function onConsole(msg) {
         const type = msg.type();
         Logs_1.Logs.add({
-            logSource: type === 'error' ? 'Browser Error' : 'Browser Log',
+            logSource: (() => {
+                if (type === 'error') {
+                    return 'Browser Error';
+                }
+                if (type === 'warning') {
+                    return 'Browser Warning';
+                }
+                return 'Browser Log';
+            })(),
             logText: JSON.stringify({
                 type,
                 text: msg.text(),

package/package.json

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