@brillout/test-e2e 0.4.7 → 0.4.9

package/dist/setup.js

@@ -314,22 +314,32 @@ function execRunScript({ onStdout, onFailure, onExit, }) {
     });
     proc.stdout.on('data', (chunk) => {
         const data = String(chunk);
-        assertNotExited(data);
-        (0, utils_2.assert)(getRunInfo().cmd === cmd);
         Logs_1.Logs.add({
             logSource: 'stdout',
             logText: data,
         });
+        /* These assertions sometimes fails, see comments in assertNotExited()
+        assertNotExited(data)
+        assert(getRunInfo().cmd === cmd)
+        /*/
+        if (procExited)
+            return undefined;
+        //*/
         onStdout?.(data);
     });
     proc.stderr.on('data', async (chunk) => {
         const data = String(chunk);
-        assertNotExited(data);
-        (0, utils_2.assert)(getRunInfo().cmd === cmd);
         Logs_1.Logs.add({
             logSource: 'stderr',
             logText: data,
         });
+        /* These assertions sometimes fails, see comments in assertNotExited()
+        assertNotExited(data)
+        assert(getRunInfo().cmd === cmd)
+        /*/
+        if (procExited)
+            return undefined;
+        //*/
         if (data.includes('EADDRINUSE')) {
             await exitAndFail(new Error('Port conflict? Port already in use EADDRINUSE.'));
         }
@@ -359,14 +369,15 @@ function execRunScript({ onStdout, onFailure, onExit, }) {
         exitPromiseResolve();
     });
     return { terminate, processHasExited };
-    function assertNotExited(data) {
-        if (!procExited)
-            return;
-        // Sometimes, a single white space is emitted after 'exit' has been emitted.
-        if (data.trim().length === 0)
-            return;
-        (0, utils_2.assert)(false, { data: data, dataLength: data.length });
+    /* Node.js seems to occasionally emit 'data' events after having the emitted 'exit' event, for example:
+     *  - `{"data":"[0] npm run serve:cdn  exited with code SIGTERM\n","dataLength":48}`.
+    function assertNotExited(data: string) {
+      if (!procExited) return
+      // Sometimes, a single white space is emitted after 'exit' has been emitted.
+      if (data.trim().length === 0) return
+      assert(false, { data, dataLength: data.length })
     }
+    */
     function processHasExited() {
         return procExited;
     }

package/dist/utils/assert.js

@@ -2,7 +2,6 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.projectErrorPrefix = exports.getProjectError = exports.assertInfo = exports.assertWarning = exports.assertUsage = exports.assert = void 0;
 const runTests_1 = require("../runTests");
-const createErrorWithCleanStackTrace_1 = require("./createErrorWithCleanStackTrace");
 const projectInfo_1 = require("./projectInfo");
 const Logs_1 = require("../Logs");
 const getCurrentTest_1 = require("../getCurrentTest");
@@ -26,13 +25,13 @@ function assert(condition, debugInfo) {
         const debugInfoSerialized = typeof debugInfo === 'string' ? debugInfo : '`' + JSON.stringify(debugInfo) + '`';
         return `Debug info (this is for the ${projectInfo_1.projectInfo.projectName} maintainers; you can ignore this): ${debugInfoSerialized}.`;
     })();
-    const internalError = (0, createErrorWithCleanStackTrace_1.createErrorWithCleanStackTrace)([
+    const internalError = new Error([
         `${internalErrorPrefix} You stumbled upon a bug in ${projectInfo_1.projectInfo.projectName}'s source code.`,
         `Reach out at ${projectInfo_1.projectInfo.githubRepository}/issues/new and include this error stack (the error stack is usually enough to fix the problem).`,
         'A maintainer will fix the bug (usually under 24 hours).',
         `Do not hesitate to reach out as it makes ${projectInfo_1.projectInfo.projectName} more robust.`,
         debugStr,
-    ].join(' '), numberOfStackTraceLinesToRemove);
+    ].join(' '));
     (0, runTests_1.logFailure)('a bug occurred in @brillout/test-e2e');
     (0, logError_1.logError)(internalError, 'BUG');
     logDebugInfo();
@@ -46,13 +45,12 @@ function assertUsage(condition, errorMessage) {
         return;
     }
     const whiteSpace = errorMessage.startsWith('[') ? '' : ' ';
-    const usageError = (0, createErrorWithCleanStackTrace_1.createErrorWithCleanStackTrace)(`${usageErrorPrefix}${whiteSpace}${errorMessage}`, numberOfStackTraceLinesToRemove);
-    throw usageError;
+    throw new Error(`${usageErrorPrefix}${whiteSpace}${errorMessage}`);
 }
 exports.assertUsage = assertUsage;
 function getProjectError(errorMessage) {
-    const pluginError = (0, createErrorWithCleanStackTrace_1.createErrorWithCleanStackTrace)(`${errorPrefix} ${errorMessage}`, numberOfStackTraceLinesToRemove);
-    return pluginError;
+    const projectError = new Error(`${errorPrefix} ${errorMessage}`);
+    return projectError;
 }
 exports.getProjectError = getProjectError;
 let loggedWarnings = new Set();

package/package.json

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

package/dist/utils/createErrorWithCleanStackTrace.d.ts

@@ -1,2 +0,0 @@
-export { createErrorWithCleanStackTrace };
-declare function createErrorWithCleanStackTrace(errorMessage: string, numberOfStackTraceLinesToRemove: number): Error;

package/dist/utils/createErrorWithCleanStackTrace.js

@@ -1,43 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.createErrorWithCleanStackTrace = void 0;
-function createErrorWithCleanStackTrace(errorMessage, numberOfStackTraceLinesToRemove) {
-    let err;
-    {
-        var stackTraceLimit__original = Error.stackTraceLimit;
-        Error.stackTraceLimit = Infinity;
-        err = new Error(errorMessage);
-        Error.stackTraceLimit = stackTraceLimit__original;
-    }
-    err.stack = clean(err.stack, numberOfStackTraceLinesToRemove);
-    return err;
-}
-exports.createErrorWithCleanStackTrace = createErrorWithCleanStackTrace;
-function clean(errStack, numberOfStackTraceLinesToRemove) {
-    if (!errStack) {
-        return errStack;
-    }
-    const stackLines = splitByLine(errStack);
-    let linesRemoved = 0;
-    const stackLine__cleaned = stackLines
-        .filter((line) => {
-        // Remove internal stack traces
-        if (line.includes(' (internal/') || line.includes(' (node:internal')) {
-            return false;
-        }
-        if (linesRemoved < numberOfStackTraceLinesToRemove && isStackTraceLine(line)) {
-            linesRemoved++;
-            return false;
-        }
-        return true;
-    })
-        .join('\n');
-    return stackLine__cleaned;
-}
-function isStackTraceLine(line) {
-    return line.startsWith('    at ');
-}
-function splitByLine(str) {
-    // https://stackoverflow.com/questions/21895233/how-in-node-to-split-string-by-newline-n
-    return str.split(/\r?\n/);
-}