@brillout/test-e2e 0.5.3 → 0.5.5

package/dist/Logs/isTolerateError.js

@@ -7,14 +7,14 @@ function isTolerateError({ logSource, logText }) {
         return true;
     }
     const config = (0, getConfig_1.getConfig)();
-    if (!config.tolerateError) {
-        return false;
+    if (config.tolerateError?.({ logSource, logText })) {
+        return true;
     }
-    return config.tolerateError({ logSource, logText });
+    return false;
 }
 exports.isTolerateError = isTolerateError;
 function tolerateError({ logSource, logText }) {
-    return isFetchExperimentalWarning() && isTerminationEPIPE();
+    return isFetchExperimentalWarning() || isTerminationEPIPE();
     function isFetchExperimentalWarning() {
         return (logSource === 'stderr' &&
             logText.includes('ExperimentalWarning: The Fetch API is an experimental feature. This feature could change at any time'));

package/dist/Logs.js

@@ -23,12 +23,13 @@ function hasFailLogs(failOnWarning) {
     return failLogs.length > 0;
 }
 function getErrorLogs(includeWarnings) {
-    const errorLogs = logEntries.filter(({ logSource, isNotFailure, loggedAfterExit }) => {
-        if (isNotFailure) {
+    const errorLogs = logEntries.filter((logEntry) => {
+        if (logEntry.isNotFailure) {
             return false;
         }
+        const { logSource } = logEntry;
         // taskkill makes process exit with exit code `1` which makes npm emit logs on stderr
-        if (loggedAfterExit && (0, utils_1.isWindows)() && logSource === 'stderr') {
+        if (logEntry.loggedAfterExit && (0, utils_1.isWindows)() && logSource === 'stderr') {
             return false;
         }
         if (logSource === 'run() failure') {
@@ -61,12 +62,11 @@ function flushLogs() {
 }
 function add({ logSource, logText, loggedAfterExit = false, }) {
     const logTimestamp = getTimestamp();
-    const isNotFailure = (0, isTolerateError_1.isTolerateError)({ logSource, logText });
     const logEntry = {
         logSource,
         logText,
         logTimestamp,
-        isNotFailure,
+        isNotFailure: (0, isTolerateError_1.isTolerateError)({ logSource, logText }),
         loggedAfterExit,
     };
     logEntries.push(logEntry);

package/dist/getConfig.d.ts

@@ -2,7 +2,6 @@ export { getConfig };
 export { loadConfig };
 declare type Config = {
     tolerateError?: Function;
-    tolerateSkip?: boolean;
     ci?: never;
 };
 declare function getConfig(): Config;

package/dist/getConfig.js

@@ -30,9 +30,6 @@ function assertConfig(config) {
         if (key === 'tolerateError') {
             (0, utils_1.assertUsage)((0, utils_1.isCallable)(val), `${wrongType} function`);
         }
-        else if (key === 'tolerateSkip') {
-            (0, utils_1.assertUsage)(typeof val === 'boolean', `${wrongType} boolean`);
-        }
         else if (key === 'ci') {
             // Only used by .github/workflows/ci/getTestJobs.mjs
             // https://github.com/brillout/vite-plugin-ssr/blob/4fa329e4081655c39c4c70436ae73e563489439e/.github/workflows/ci/getTestJobs.mjs#L72

package/dist/getCurrentTest.d.ts

@@ -23,7 +23,9 @@ declare type TestInfo = {
         isFlaky: boolean;
     };
     hasStartedRunning?: boolean;
-    skipped?: string;
+    skipped?: {
+        reason: string;
+    };
     startServer?: () => Promise<void>;
     terminateServer?: () => Promise<void>;
     afterEach?: (hasFailed: boolean) => void;

package/dist/runAll.js

@@ -63,7 +63,7 @@ async function runServerAndTests(browser, isSecondAttempt) {
     // Set when user calls `skip()`
     if (testInfo.skipped) {
         assertSkipUsage(testInfo);
-        (0, logTestStatus_1.logWarn)(testInfo.skipped);
+        (0, logTestStatus_1.logWarn)(testInfo.skipped.reason);
         return true;
     }
     // Set when user calls `run()`
@@ -191,6 +191,7 @@ function runTest(testFn, testFunctionTimeout) {
     return promise;
 }
 function assertSkipUsage(testInfo) {
+    (0, utils_1.assert)(testInfo.skipped);
     {
         const err = 'You cannot call run() after calling skip()';
         (0, utils_1.assertUsage)(testInfo.runInfo === undefined, err);
@@ -199,13 +200,4 @@ function assertSkipUsage(testInfo) {
         const err = 'You cannot call test() after calling skip()';
         (0, utils_1.assertUsage)(testInfo.tests === undefined, err);
     }
-    const config = (0, getConfig_1.getConfig)();
-    {
-        const err = 'Using skip() while config tolerateSkip is set to false';
-        (0, utils_1.assertUsage)(config.tolerateSkip !== false, err);
-    }
-    {
-        const err = 'Using skip() while config tolerateSkip is set to undefined and tests are running in a parallel CI';
-        (0, utils_1.assertUsage)(!(config.tolerateSkip === undefined && (0, parallel_ci_1.isParallelCI)()), err);
-    }
 }

package/dist/setup.js

@@ -31,7 +31,7 @@ const TIMEOUT_AUTORETRY = TIMEOUT_TEST_FUNCTION / 2;
 const TIMEOUT_PLAYWRIGHT = TIMEOUT_TEST_FUNCTION / 2;
 function skip(reason) {
     const testInfo = (0, getCurrentTest_1.getCurrentTest)();
-    testInfo.skipped = reason;
+    testInfo.skipped = { reason };
 }
 exports.skip = skip;
 function run(cmd, { 

package/dist/utils/assert.js

@@ -81,7 +81,7 @@ function logDebugInfo() {
     (0, logSection_1.logSection)('DEBUG INFO');
     const testInfo = (0, getCurrentTest_1.getCurrentTest)();
     console.log('testInfo.hasStartedRunning', testInfo.hasStartedRunning);
-    console.log('testInfo.skipped', testInfo.skipped);
+    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);

package/package.json

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