@brillout/test-e2e 0.3.8 → 0.3.10

package/dist/Logs.d.ts

@@ -14,9 +14,9 @@ declare type LogEntry = {
     logTimestamp: string;
     isNotFailure: boolean;
 };
-declare function hasErrorLogs(treatWarningAsError: boolean): boolean;
+declare function hasErrorLogs(failOnWarning: boolean): boolean;
 declare function clearLogs(): void;
-declare function logErrors(treatWarningAsError: boolean): void;
+declare function logErrors(failOnWarning: boolean): void;
 declare function flushLogs(): void;
 declare function add({ logSource, logText }: {
     logSource: LogSource;

package/dist/Logs.js

@@ -17,11 +17,11 @@ const picocolors_1 = __importDefault(require("picocolors"));
 const getCurrentTest_1 = require("./getCurrentTest");
 const getConfig_1 = require("./getConfig");
 let logEntries = [];
-function hasErrorLogs(treatWarningAsError) {
-    const errorLogs = getErrorLogs(treatWarningAsError);
+function hasErrorLogs(failOnWarning) {
+    const errorLogs = getErrorLogs(failOnWarning);
     return errorLogs.length > 0;
 }
-function getErrorLogs(treatWarningAsError) {
+function getErrorLogs(failOnWarning) {
     const errorLogs = logEntries.filter(({ logSource, isNotFailure }) => {
         if (isNotFailure) {
             return;
@@ -29,7 +29,7 @@ function getErrorLogs(treatWarningAsError) {
         if (logSource === 'run() failure') {
             return true;
         }
-        if (treatWarningAsError && (logSource === 'Browser Warning' || logSource === 'stderr')) {
+        if (failOnWarning && (logSource === 'Browser Warning' || logSource === 'stderr')) {
             return true;
         }
         if (logSource === 'Browser Error') {
@@ -42,11 +42,12 @@ function getErrorLogs(treatWarningAsError) {
 function clearLogs() {
     logEntries = [];
 }
-function logErrors(treatWarningAsError) {
-    const errorLogs = getErrorLogs(treatWarningAsError);
+function logErrors(failOnWarning) {
+    const errorLogs = getErrorLogs(failOnWarning);
     errorLogs.forEach((logEntry) => printLog(logEntry));
 }
 function flushLogs() {
+    console.log(picocolors_1.default.bold('vvvv ALL LOGS vvvv'));
     logEntries.forEach((logEntry) => printLog(logEntry));
     logEntries = [];
 }
@@ -98,7 +99,7 @@ function printLog(logEntry) {
     const testInfo = (0, getCurrentTest_1.getCurrentTestOptional)();
     // See https://github.com/nodejs/node/issues/8033#issuecomment-388323687
     if (!(0, utils_1.ensureNewTerminalLine)()) {
-        process.stderr.write(`\n`);
+        console.log('');
     }
     let msg = logText;
     // I don't know why but sometimes `logText` is `undefined`
@@ -111,7 +112,7 @@ function printLog(logEntry) {
         (0, utils_1.assert)(testInfo.runInfo);
         testInfoLabels = `[${testInfo.testName}][${testInfo.runInfo.cmd}]`;
     }
-    process.stderr.write(`[${logTimestamp}]${testInfoLabels}[${logSourceLabel}] ${msg}`);
+    console.log(`[${logTimestamp}]${testInfoLabels}[${logSourceLabel}] ${msg}`);
 }
 function colorize(logSource) {
     const { bold } = picocolors_1.default;

package/dist/runTests.js

@@ -61,13 +61,10 @@ async function runTests(browser) {
         if (isFailure) {
             logTestsResult(false);
             if (err) {
-                console.error(err);
+                console.log(err);
             }
-            if (hasErrorLog) {
+            else if (hasErrorLog) {
                 logIsFailure(false, !doNotFailOnWarning);
-                console.log(picocolors_1.default.bold('vvvv FAIL LOGS vvvv'));
-                Logs_1.Logs.logErrors(!doNotFailOnWarning);
-                console.log(picocolors_1.default.bold('vvvv ALL LOGS vvvv'));
             }
             Logs_1.Logs.flushLogs();
             process.exit(1);
@@ -135,11 +132,13 @@ function logTestsResult(success) {
         console.log(`${failStyle(' FAIL ')} ${testInfo.testFile}`);
     }
 }
-function logIsFailure(isTermination, canBeAWarning) {
+function logIsFailure(isTermination, failOnWarning) {
     console.log(picocolors_1.default.red(picocolors_1.default.bold([
         'Test FAIL because encountered an error',
-        !canBeAWarning ? '' : ' or warning',
+        !failOnWarning ? '' : ' or warning',
         !isTermination ? '' : ' during termination',
         ', see logs below.',
     ].join(''))));
+    console.log(picocolors_1.default.bold('vvvv FAIL LOG vvvv'));
+    Logs_1.Logs.logErrors(failOnWarning);
 }

package/package.json

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