@brillout/test-e2e 0.4.12 → 0.4.13

package/dist/Logs.d.ts

@@ -2,9 +2,9 @@ export { expectError };
 export declare const Logs: {
     add: typeof add;
     flushLogs: typeof flushLogs;
-    logErrors: typeof logErrors;
+    logErrorsAndWarnings: typeof logErrorsAndWarnings;
     clearLogs: typeof clearLogs;
-    hasErrorLogs: typeof hasErrorLogs;
+    hasFailLogs: typeof hasFailLogs;
     logEagerly: boolean;
 };
 declare type LogSource = 'stdout' | 'stderr' | 'Browser Error' | 'Browser Warning' | 'Browser Log' | 'Playwright' | 'run()' | 'run() failure' | 'test()' | 'Connection Error';
@@ -14,9 +14,9 @@ declare type LogEntry = {
     logTimestamp: string;
     isNotFailure: boolean;
 };
-declare function hasErrorLogs(failOnWarning: boolean): boolean;
+declare function hasFailLogs(failOnWarning: boolean): boolean;
 declare function clearLogs(): void;
-declare function logErrors(failOnWarning: boolean): void;
+declare function logErrorsAndWarnings(): void;
 declare function flushLogs(): void;
 declare function add({ logSource, logText }: {
     logSource: LogSource;

package/dist/Logs.js

@@ -7,9 +7,9 @@ exports.Logs = exports.expectError = void 0;
 exports.Logs = {
     add,
     flushLogs,
-    logErrors,
+    logErrorsAndWarnings,
     clearLogs,
-    hasErrorLogs,
+    hasFailLogs,
     logEagerly: false,
 };
 const utils_1 = require("./utils");
@@ -18,11 +18,11 @@ const getCurrentTest_1 = require("./getCurrentTest");
 const getConfig_1 = require("./getConfig");
 const logSection_1 = require("./logSection");
 let logEntries = [];
-function hasErrorLogs(failOnWarning) {
-    const errorLogs = getErrorLogs(failOnWarning);
-    return errorLogs.length > 0;
+function hasFailLogs(failOnWarning) {
+    const failLogs = getErrorLogs(failOnWarning);
+    return failLogs.length > 0;
 }
-function getErrorLogs(failOnWarning) {
+function getErrorLogs(returnWarnings) {
     const errorLogs = logEntries.filter(({ logSource, isNotFailure }) => {
         if (isNotFailure) {
             return;
@@ -30,7 +30,7 @@ function getErrorLogs(failOnWarning) {
         if (logSource === 'run() failure') {
             return true;
         }
-        if (failOnWarning && (logSource === 'Browser Warning' || logSource === 'stderr')) {
+        if (returnWarnings && (logSource === 'Browser Warning' || logSource === 'stderr')) {
             return true;
         }
         if (logSource === 'Browser Error') {
@@ -43,12 +43,12 @@ function getErrorLogs(failOnWarning) {
 function clearLogs() {
     logEntries = [];
 }
-function logErrors(failOnWarning) {
-    if (!exports.Logs.hasErrorLogs(failOnWarning))
+function logErrorsAndWarnings() {
+    const errorAndWarningLogs = getErrorLogs(true);
+    if (errorAndWarningLogs.length === 0)
         return;
-    (0, logSection_1.logSection)(`ERROR${!failOnWarning ? '' : '/WARNING'} LOGS`);
-    const errorLogs = getErrorLogs(failOnWarning);
-    errorLogs.forEach((logEntry) => printLog(logEntry));
+    (0, logSection_1.logSection)('ERROR & WARNING LOGS');
+    errorAndWarningLogs.forEach((logEntry) => printLog(logEntry));
 }
 function flushLogs() {
     (0, logSection_1.logSection)('ALL LOGS');

package/dist/runAll.js

@@ -116,7 +116,7 @@ async function runTests(browser, isSecondAttempt) {
         testInfo.afterEach(!!err);
         {
             const failOnWarning = !testInfo.runInfo.doNotFailOnWarning;
-            const hasErrorLog = Logs_1.Logs.hasErrorLogs(failOnWarning);
+            const hasErrorLog = Logs_1.Logs.hasFailLogs(failOnWarning);
             const isFailure = err || hasErrorLog;
             if (isFailure) {
                 if (err) {
@@ -129,7 +129,7 @@ async function runTests(browser, isSecondAttempt) {
                 else {
                     (0, utils_1.assert)(false);
                 }
-                Logs_1.Logs.logErrors(failOnWarning);
+                Logs_1.Logs.logErrorsAndWarnings();
                 Logs_1.Logs.flushLogs();
                 return failure();
             }
@@ -139,11 +139,11 @@ async function runTests(browser, isSecondAttempt) {
     // Check whether stderr emitted during testInfo.terminateServer()
     {
         const failOnWarning = true;
-        if (Logs_1.Logs.hasErrorLogs(failOnWarning) &&
+        if (Logs_1.Logs.hasFailLogs(failOnWarning) &&
             // See comments about taskkill in src/setup.ts
             !(0, utils_1.isWindows)()) {
             (0, logTestStatus_1.logFail)(`${getErrorType(failOnWarning)} occurred during server termination`, isFinalAttempt);
-            Logs_1.Logs.logErrors(failOnWarning);
+            Logs_1.Logs.logErrorsAndWarnings();
             Logs_1.Logs.flushLogs();
             return failure();
         }

package/dist/utils/assert.js

@@ -34,7 +34,7 @@ function assert(condition, debugInfo) {
     (0, logTestStatus_1.logFail)('a bug occurred in @brillout/test-e2e', true);
     (0, logError_1.logError)(internalError, 'BUG');
     logDebugInfo();
-    Logs_1.Logs.logErrors(true);
+    Logs_1.Logs.logErrorsAndWarnings();
     Logs_1.Logs.flushLogs();
     throw new Error('Bug. See messages above.');
 }

package/package.json

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