@brillout/test-e2e 0.6.1 → 0.6.3

package/dist/Logs.js

@@ -87,7 +87,7 @@ function add({ logSource, logText, loggedAfterExit = false, }) {
  */
 function expectLog(logText, { filter: logFilter, allLogs } = {}) {
     const logList = allLogs ? logEntriesAll : logEntries;
-    let logsFound = logList.filter((logEntry) => {
+    const logsFound = logList.filter((logEntry) => {
         if (removeAnsi(logEntry).logText.includes(logText)) {
             logEntry.isNotFailure = true;
             return true;
@@ -95,13 +95,14 @@ function expectLog(logText, { filter: logFilter, allLogs } = {}) {
         return false;
     });
     if (logsFound.length === 0) {
-        throw new Error(`The following log was expected but it wasn't logged: "${logText}"`);
+        throw new Error(`The following log is expected but it wasn't logged: "${logText}"`);
     }
     if (!logFilter)
         return;
-    logsFound = logsFound.filter((logEntry) => logFilter(removeAnsi(logEntry)));
-    if (logsFound.length === 0) {
-        throw new Error(`The following log was logged as expected, but it didn't match the logFilter() you provided: "${logText}"`);
+    const logsFoundAfterFilter = logsFound.filter((logEntry) => logFilter(removeAnsi(logEntry)));
+    if (logsFoundAfterFilter.length === 0) {
+        console.log(JSON.stringify(logsFound, null, 2));
+        throw new Error(`Logs above are matching "${logText}" but don't match the filter you provided.`);
     }
 }
 function removeAnsi(logEntry) {

package/dist/runAll.js

@@ -20,9 +20,6 @@ async function runAll(filter) {
     const hasFailedTestFile = failedTestFiles.length > 0;
     const hasFailLog = hasFail();
     if (hasFailedTestFile || hasFailLog) {
-        // hasFailedTestFile and hasFailLog are redundant
-        //  - When assert.ts calls logFail() this code block isn't run
-        assert(hasFailedTestFile && hasFailLog);
         assert(failedTestFiles.length > 0);
         throw new Error(pc.red(pc.bold([
             `Following test files failed, see logs above for more information (search for [${TEST_FAIL}]):`,

package/dist/utils/assert.js

@@ -5,11 +5,6 @@ export { assertInfo };
 export { getProjectError };
 export { errorPrefix as projectErrorPrefix };
 import { projectInfo } from './projectInfo.js';
-import { Logs } from '../Logs.js';
-import { getCurrentTest } from '../getCurrentTest.js';
-import { logSection } from '../logSection.js';
-import { logError } from '../logError.js';
-import { logFail } from '../logTestStatus.js';
 const errorPrefix = `[${projectInfo.npmPackageName}]`;
 const internalErrorPrefix = `${errorPrefix}[Bug]`;
 const usageErrorPrefix = `${errorPrefix}[Wrong Usage]`;
@@ -33,12 +28,7 @@ function assert(condition, debugInfo) {
         `Do not hesitate to reach out as it makes ${projectInfo.projectName} more robust.`,
         debugStr,
     ].join(' '));
-    logFail('a bug occurred in @brillout/test-e2e', true);
-    logError(internalError, 'BUG');
-    logDebugInfo();
-    Logs.logErrorsAndWarnings();
-    Logs.flushLogs();
-    throw new Error('Bug. See messages above.');
+    throw internalError;
 }
 function assertUsage(condition, errorMessage) {
     if (condition) {
@@ -74,13 +64,3 @@ function assertInfo(condition, errorMessage) {
     }
     console.warn(`${infoPrefix} ${errorMessage}`);
 }
-function logDebugInfo() {
-    logSection('DEBUG INFO');
-    const testInfo = getCurrentTest();
-    console.log('testInfo.hasStartedRunning', testInfo.hasStartedRunning);
-    console.log('testInfo.skipped?.reason', testInfo.skipped?.reason);
-    console.log('testInfo.runInfo', testInfo.runInfo);
-    console.log('testInfo.testFile', testInfo.testFile);
-    console.log('testInfo.testName', testInfo.testName);
-    console.log('process.platform', process.platform);
-}

package/package.json

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