@brillout/test-e2e 0.5.23 → 0.5.25

package/dist/Logs.js

@@ -92,7 +92,7 @@ function expectLog(logText, logFilter) {
     const logsFoundWithFilter = logsFound.filter((logEntry) => logFilter?.(removeAnsi(logEntry)));
     if (logsFoundWithFilter.length === 0) {
         if (logsFound.length === 0) {
-            throw new Error(`The following log was expected but wasn't logged: "${logText}"`);
+            throw new Error(`The following log was expected but it wasn't logged: "${logText}"`);
         }
         else {
             throw new Error(`The following log was logged as expected, but it didn't match the logFilter() you provided`);

package/dist/cli.js

@@ -30,7 +30,9 @@ function initUserExitHandler() {
         Logs_1.Logs.flushLogs();
     });
 }
-// Much simpler alternative: https://github.com/brillout/vite-plugin-ssr/blob/77b1b0e8b6c1ed4efe3e3039f7c0b23a00651663/vite-plugin-ssr/node/plugin/plugins/importUserCode/v1-design/transpileAndLoadFile.ts#L200-L208
+// This mechanism seems necessary for the source map to apply for errors that happen a long time after the build file, which contains the inline source map, is removed.
+//  - The much simpler alternative used by Vike doesn't seem to work: https://github.com/brillout/vite-plugin-ssr/blob/77b1b0e8b6c1ed4efe3e3039f7c0b23a00651663/vite-plugin-ssr/node/plugin/plugins/importUserCode/v1-design/transpileAndLoadFile.ts#L200-L208
+//  - Failed attempt to make the simpler alternative work: https://github.com/brillout/test-e2e/commit/681abd19459b8c7d3ade9363ba32be86e3fdcb53
 function initSourceMap() {
     source_map_support_1.default.install({
         retrieveSourceMap: function (source) {

package/dist/logTestStatus.d.ts

@@ -3,6 +3,7 @@ export { logWarn };
 export { logFail };
 export { logBoot };
 export { hasFail };
+export declare const TEST_FAIL = "TEST_FAIL";
 declare function logPass(): void;
 declare function logWarn(warning: string): void;
 declare function logFail(reason: string, isFinalAttempt: boolean): void;

package/dist/logTestStatus.js

@@ -3,7 +3,8 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.hasFail = exports.logBoot = exports.logFail = exports.logWarn = exports.logPass = void 0;
+exports.TEST_FAIL = exports.hasFail = exports.logBoot = exports.logFail = exports.logWarn = exports.logPass = void 0;
+exports.TEST_FAIL = 'TEST_FAIL';
 const getCurrentTest_1 = require("./getCurrentTest");
 const utils_1 = require("./utils");
 const picocolors_1 = __importDefault(require("@brillout/picocolors"));
@@ -24,7 +25,7 @@ function logFail(reason, isFinalAttempt) {
     }
     const { FAIL } = getStatusTags();
     const color = (s) => picocolors_1.default.red(picocolors_1.default.bold(s));
-    const msg = `Test ${isFinalAttempt ? FAIL : 'failed'} because ${reason}, see below. [TEST_FAILURE]`;
+    const msg = `Test ${isFinalAttempt ? FAIL : 'failed'} because ${reason}, see below. [${exports.TEST_FAIL}]`;
     console.log(color(msg));
 }
 exports.logFail = logFail;

package/dist/runAll.js

@@ -30,8 +30,8 @@ async function runAll(filter) {
         (0, utils_1.assert)(hasFailedTestFile && hasFailLog);
         (0, utils_1.assert)(failedTestFiles.length > 0);
         throw new Error(picocolors_1.default.red(picocolors_1.default.bold([
-            'Following test files failed, see logs above for more information (search for [TEST_FAILURE]):',
-            ...failedTestFiles.map((testFile) => `  ${testFile}`),
+            `Following test files failed, see logs above for more information (search for [${logTestStatus_1.TEST_FAIL}]):`,
+            ...failedTestFiles.map((testFile) => `❌ ${testFile}`),
         ].join('\n'))));
     }
 }
@@ -74,14 +74,15 @@ async function runTestFiles(testFiles, browser) {
 }
 async function buildAndTest(testFile, browser, isThirdAttempt) {
     (0, utils_1.assert)(testFile.endsWith('.ts'));
-    const testFileJs = testFile.replace('.ts', '.mjs');
+    const cacheBuster = Date.now();
+    const testFileJs = testFile.replace('.ts', `.${cacheBuster}.mjs`);
     (0, utils_1.assert)(testFileJs.endsWith('.mjs'));
     const cleanBuild = await (0, buildTs_1.buildTs)(testFile, testFileJs);
     (0, getCurrentTest_2.setCurrentTest)(testFile);
     (0, logTestStatus_1.logBoot)();
     let execErr;
     try {
-        await import((0, utils_2.fsWindowsBugWorkaround)(testFileJs) + `?cacheBuster=${Date.now()}`);
+        await import((0, utils_2.fsWindowsBugWorkaround)(testFileJs));
     }
     catch (err) {
         (0, utils_1.assert)(err);

package/dist/testScreenshotFixture.js

@@ -17,7 +17,7 @@ async function testScreenshotFixture({ screenshotFixturePath, doNotTestLocally,
     const pngPaths = getPngPaths();
     const pngFixturPath = screenshotFixturePath || pngPaths.pngFixturPath;
     if (doNotTestLocally && !(0, utils_1.isCI)()) {
-        console.log(`${picocolors_1.default.blue(picocolors_1.default.bold('INFO'))} screenshot test ${pngFixturPath} skipped because running test locally (i.e. not in a CI environment)`);
+        console.log(`\n${picocolors_1.default.blue(picocolors_1.default.bold('INFO'))} screenshot test ${pngFixturPath} skipped because running test locally (i.e. not in a CI environment)`);
         return;
     }
     if (!fs_1.default.existsSync(pngFixturPath)) {

package/package.json

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