@applitools/core 1.0.7 → 1.0.8

package/dist/close-manager.js

@@ -2,6 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.makeCloseManager = void 0;
 const test_error_1 = require("./errors/test-error");
+const internal_error_1 = require("./errors/internal-error");
 function makeCloseManager({ core, storage, logger: defaultLogger, }) {
     return async function closeManager({ settings, logger = defaultLogger, } = {}) {
         const containers = await Promise.all(storage.map(async ({ eyes, promise }) => {
@@ -17,7 +18,7 @@ function makeCloseManager({ core, storage, logger: defaultLogger, }) {
                 });
             }
             catch (error) {
-                return [{ error, userTestId: error.userTestId, renderer: error.renderer }];
+                return [{ error: new internal_error_1.InternalError(error), ...error.info }];
             }
         }));
         const batches = storage.reduce((batches, { eyes }) => {

package/dist/errors/abort-error.js

@@ -0,0 +1,6 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.AbortError = void 0;
+class AbortError extends Error {
+}
+exports.AbortError = AbortError;

package/dist/errors/internal-error.js

@@ -0,0 +1,11 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.InternalError = void 0;
+const core_base_1 = require("@applitools/core-base");
+class InternalError extends core_base_1.CoreError {
+    constructor(error) {
+        super(error.message, { reason: 'internal', ...error.info });
+        this.stack = error.stack;
+    }
+}
+exports.InternalError = InternalError;

package/dist/errors/test-error.js

@@ -24,6 +24,9 @@ class TestError extends core_base_1.CoreError {
                 });
             }
         }
+        else {
+            super('');
+        }
     }
 }
 exports.TestError = TestError;

package/dist/open-eyes.js

@@ -34,24 +34,24 @@ const close_1 = require("./close");
 const utils = __importStar(require("@applitools/utils"));
 function makeOpenEyes({ spec, core, concurrency, logger: defaultLogger, }) {
     return async function openEyes({ type, target, settings, config, logger = defaultLogger, }) {
-        var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l, _m, _o, _p, _q, _r, _s, _t;
-        var _u, _v, _w, _x, _y;
+        var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l, _m, _o, _p, _q, _r, _s, _t, _u;
+        var _v, _w, _x, _y, _z;
         settings = { ...config === null || config === void 0 ? void 0 : config.open, ...settings };
-        settings.userTestId = `${settings.testName}--${utils.general.guid()}`;
-        (_a = settings.serverUrl) !== null && _a !== void 0 ? _a : (settings.serverUrl = (_b = utils.general.getEnvValue('SERVER_URL')) !== null && _b !== void 0 ? _b : 'https://eyesapi.applitools.com');
-        (_c = settings.apiKey) !== null && _c !== void 0 ? _c : (settings.apiKey = utils.general.getEnvValue('API_KEY'));
-        (_d = settings.batch) !== null && _d !== void 0 ? _d : (settings.batch = {});
-        (_e = (_u = settings.batch).id) !== null && _e !== void 0 ? _e : (_u.id = (_f = utils.general.getEnvValue('BATCH_ID')) !== null && _f !== void 0 ? _f : utils.general.guid());
-        (_g = (_v = settings.batch).name) !== null && _g !== void 0 ? _g : (_v.name = utils.general.getEnvValue('BATCH_NAME'));
-        (_h = (_w = settings.batch).sequenceName) !== null && _h !== void 0 ? _h : (_w.sequenceName = utils.general.getEnvValue('BATCH_SEQUENCE'));
-        (_j = (_x = settings.batch).notifyOnCompletion) !== null && _j !== void 0 ? _j : (_x.notifyOnCompletion = utils.general.getEnvValue('BATCH_NOTIFY', 'boolean'));
-        (_k = settings.keepBatchOpen) !== null && _k !== void 0 ? _k : (settings.keepBatchOpen = utils.general.getEnvValue('DONT_CLOSE_BATCHES', 'boolean'));
-        (_l = settings.branchName) !== null && _l !== void 0 ? _l : (settings.branchName = utils.general.getEnvValue('BRANCH'));
-        (_m = settings.parentBranchName) !== null && _m !== void 0 ? _m : (settings.parentBranchName = utils.general.getEnvValue('PARENT_BRANCH'));
-        (_o = settings.baselineBranchName) !== null && _o !== void 0 ? _o : (settings.baselineBranchName = utils.general.getEnvValue('BASELINE_BRANCH'));
-        (_p = settings.ignoreBaseline) !== null && _p !== void 0 ? _p : (settings.ignoreBaseline = false);
-        (_q = settings.compareWithParentBranch) !== null && _q !== void 0 ? _q : (settings.compareWithParentBranch = false);
-        (_r = (_y = settings).renderConcurrency) !== null && _r !== void 0 ? _r : (_y.renderConcurrency = (_t = (_s = config === null || config === void 0 ? void 0 : config.check) === null || _s === void 0 ? void 0 : _s.renderers) === null || _t === void 0 ? void 0 : _t.length);
+        (_a = settings.userTestId) !== null && _a !== void 0 ? _a : (settings.userTestId = `${settings.testName}--${utils.general.guid()}`);
+        (_b = settings.serverUrl) !== null && _b !== void 0 ? _b : (settings.serverUrl = (_c = utils.general.getEnvValue('SERVER_URL')) !== null && _c !== void 0 ? _c : 'https://eyesapi.applitools.com');
+        (_d = settings.apiKey) !== null && _d !== void 0 ? _d : (settings.apiKey = utils.general.getEnvValue('API_KEY'));
+        (_e = settings.batch) !== null && _e !== void 0 ? _e : (settings.batch = {});
+        (_f = (_v = settings.batch).id) !== null && _f !== void 0 ? _f : (_v.id = (_g = utils.general.getEnvValue('BATCH_ID')) !== null && _g !== void 0 ? _g : utils.general.guid());
+        (_h = (_w = settings.batch).name) !== null && _h !== void 0 ? _h : (_w.name = utils.general.getEnvValue('BATCH_NAME'));
+        (_j = (_x = settings.batch).sequenceName) !== null && _j !== void 0 ? _j : (_x.sequenceName = utils.general.getEnvValue('BATCH_SEQUENCE'));
+        (_k = (_y = settings.batch).notifyOnCompletion) !== null && _k !== void 0 ? _k : (_y.notifyOnCompletion = utils.general.getEnvValue('BATCH_NOTIFY', 'boolean'));
+        (_l = settings.keepBatchOpen) !== null && _l !== void 0 ? _l : (settings.keepBatchOpen = utils.general.getEnvValue('DONT_CLOSE_BATCHES', 'boolean'));
+        (_m = settings.branchName) !== null && _m !== void 0 ? _m : (settings.branchName = utils.general.getEnvValue('BRANCH'));
+        (_o = settings.parentBranchName) !== null && _o !== void 0 ? _o : (settings.parentBranchName = utils.general.getEnvValue('PARENT_BRANCH'));
+        (_p = settings.baselineBranchName) !== null && _p !== void 0 ? _p : (settings.baselineBranchName = utils.general.getEnvValue('BASELINE_BRANCH'));
+        (_q = settings.ignoreBaseline) !== null && _q !== void 0 ? _q : (settings.ignoreBaseline = false);
+        (_r = settings.compareWithParentBranch) !== null && _r !== void 0 ? _r : (settings.compareWithParentBranch = false);
+        (_s = (_z = settings).renderConcurrency) !== null && _s !== void 0 ? _s : (_z.renderConcurrency = (_u = (_t = config === null || config === void 0 ? void 0 : config.check) === null || _t === void 0 ? void 0 : _t.renderers) === null || _u === void 0 ? void 0 : _u.length);
         if (!utils.types.has(core, 'type')) {
             core = type === 'ufg' ? (0, core_2.makeCore)({ spec, core, concurrency, logger }) : (0, core_1.makeCore)({ spec, core, logger });
         }

package/dist/ufg/abort.js

@@ -1,15 +1,27 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.makeAbort = void 0;
+const abort_error_1 = require("../errors/abort-error");
 function makeAbort({ storage, controller, logger: defaultLogger }) {
     return async function ({ logger = defaultLogger, } = {}) {
         controller.abort();
-        const results = await Promise.allSettled(storage);
-        const eyes = results.reduce((eyes, result) => {
-            const value = result.status === 'fulfilled' ? result.value : result.reason;
-            return eyes.set(value.eyes, value.renderer);
-        }, new Map());
-        return Promise.all(Array.from(eyes.entries(), async ([eyes, renderer]) => {
+        return Promise.all(storage.map(async (promise) => {
+            let eyes, renderer;
+            try {
+                const value = await promise;
+                eyes = value.eyes;
+                renderer = value.renderer;
+            }
+            catch (error) {
+                eyes = error.info.eyes;
+                renderer = error.info.renderer;
+                if (!eyes) {
+                    if (error instanceof abort_error_1.AbortError)
+                        return error.info;
+                    else
+                        throw error;
+                }
+            }
             const [result] = await eyes.abort({ logger });
             return { ...result, renderer };
         }));

package/dist/ufg/check.js

@@ -32,6 +32,7 @@ const take_snapshots_1 = require("./utils/take-snapshots");
 const wait_for_lazy_load_1 = require("../utils/wait-for-lazy-load");
 const to_base_check_settings_1 = require("../utils/to-base-check-settings");
 const generate_safe_selectors_1 = require("./utils/generate-safe-selectors");
+const abort_error_1 = require("../errors/abort-error");
 const utils = __importStar(require("@applitools/utils"));
 const chalk_1 = __importDefault(require("chalk"));
 function makeCheck({ spec, getEyes, client, test, signal, target: defaultTarget, logger: defaultLogger, }) {
@@ -40,7 +41,7 @@ function makeCheck({ spec, getEyes, client, test, signal, target: defaultTarget,
         logger.log('Command "check" is called with settings', settings);
         if (signal.aborted) {
             logger.warn('Command "check" was called after test was already aborted');
-            throw new Error('Command "check" was called after test was already aborted');
+            throw new abort_error_1.AbortError('Command "check" was called after test was already aborted');
         }
         const { elementReferencesToCalculate, elementReferenceToTarget, getBaseCheckSettings } = (0, to_base_check_settings_1.toBaseCheckSettings)({ settings });
         let snapshots, snapshotUrl, snapshotTitle, regionToTarget, selectorsToCalculate;
@@ -112,7 +113,7 @@ function makeCheck({ spec, getEyes, client, test, signal, target: defaultTarget,
             try {
                 if (signal.aborted) {
                     logger.warn('Command "check" was aborted before rendering');
-                    throw new Error('Command "check" was aborted before rendering');
+                    throw new abort_error_1.AbortError('Command "check" was aborted before rendering');
                 }
                 const { cookies, ...snapshot } = snapshots[index];
                 const renderTargetPromise = client.createRenderTarget({
@@ -135,21 +136,21 @@ function makeCheck({ spec, getEyes, client, test, signal, target: defaultTarget,
                 try {
                     if (signal.aborted) {
                         logger.warn('Command "check" was aborted before rendering');
-                        throw new Error('Command "check" was aborted before rendering');
+                        throw new abort_error_1.AbortError('Command "check" was aborted before rendering');
                     }
                     else if (eyes.aborted) {
                         logger.warn(`Renderer with id ${rendererId} was aborted during one of the previous steps`);
-                        throw new Error(`Renderer with id "${rendererId}" was aborted during one of the previous steps`);
+                        throw new abort_error_1.AbortError(`Renderer with id "${rendererId}" was aborted during one of the previous steps`);
                     }
                     request.settings.rendererId = rendererId;
                     request.target = await renderTargetPromise;
                     if (signal.aborted) {
                         logger.warn('Command "check" was aborted before rendering');
-                        throw new Error('Command "check" was aborted before rendering');
+                        throw new abort_error_1.AbortError('Command "check" was aborted before rendering');
                     }
                     else if (eyes.aborted) {
                         logger.warn(`Renderer with id ${rendererId} was aborted during one of the previous steps`);
-                        throw new Error(`Renderer with id "${rendererId}" was aborted during one of the previous steps`);
+                        throw new abort_error_1.AbortError(`Renderer with id "${rendererId}" was aborted during one of the previous steps`);
                     }
                     const { renderId, selectorRegions, ...baseTarget } = await client.render({ request, signal });
                     const baseSettings = getBaseCheckSettings({
@@ -163,28 +164,27 @@ function makeCheck({ spec, getEyes, client, test, signal, target: defaultTarget,
                     baseTarget.name = snapshotTitle;
                     if (signal.aborted) {
                         logger.warn('Command "check" was aborted after rendering');
-                        throw new Error('Command "check" was aborted after rendering');
+                        throw new abort_error_1.AbortError('Command "check" was aborted after rendering');
                     }
                     else if (eyes.aborted) {
                         logger.warn(`Renderer with id ${rendererId} was aborted during one of the previous steps`);
-                        throw new Error(`Renderer with id "${rendererId}" was aborted during one of the previous steps`);
+                        throw new abort_error_1.AbortError(`Renderer with id "${rendererId}" was aborted during one of the previous steps`);
                     }
                     const [result] = await eyes.check({ target: baseTarget, settings: baseSettings, logger });
                     if (eyes.aborted) {
                         logger.warn(`Renderer with id ${rendererId} was aborted during one of the previous steps`);
-                        throw new Error(`Renderer with id "${rendererId}" was aborted during one of the previous steps`);
+                        throw new abort_error_1.AbortError(`Renderer with id "${rendererId}" was aborted during one of the previous steps`);
                     }
                     return { ...result, eyes, renderer };
                 }
                 catch (error) {
-                    error.eyes = eyes;
                     await eyes.abort();
+                    error.info = { eyes };
                     throw error;
                 }
             }
             catch (error) {
-                error.userTestId = test.userTestId;
-                error.renderer = renderer;
+                error.info = { ...error.info, userTestId: test.userTestId, renderer };
                 throw error;
             }
         });

package/dist/ufg/close.js

@@ -11,13 +11,13 @@ function makeClose({ storage, logger: defaultLogger }) {
                 value = result.value;
             }
             else {
-                value = result.reason;
+                value = result.reason.info;
                 error !== null && error !== void 0 ? error : (error = result.reason);
             }
             return eyes.set(value.eyes, value.renderer);
         }, new Map());
         if (error) {
-            await Promise.all(Array.from(eyes.entries(), async ([eyes]) => eyes.abort({ logger })));
+            await Promise.all(Array.from(eyes.entries(), async ([eyes]) => eyes === null || eyes === void 0 ? void 0 : eyes.abort({ logger })));
             throw error;
         }
         else {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@applitools/core",
-  "version": "1.0.7",
+  "version": "1.0.8",
   "homepage": "https://applitools.com",
   "bugs": {
     "url": "https://github.com/applitools/eyes.sdk.javascript1/issues"
@@ -86,7 +86,7 @@
     "@typescript-eslint/parser": "^5.38.0",
     "eslint": "^8.23.1",
     "eslint-config-prettier": "^8.5.0",
-    "eslint-plugin-mocha-no-only": "^1.1.1",
+    "eslint-plugin-no-only-tests": "^3.0.0",
     "eslint-plugin-node": "^11.1.0",
     "eslint-plugin-prettier": "^4.0.0",
     "husky": "^4.3.7",

package/types/errors/abort-error.d.ts

@@ -0,0 +1,3 @@
+export declare class AbortError extends Error {
+    info: Record<string, any>;
+}

package/types/errors/internal-error.d.ts

@@ -0,0 +1,6 @@
+import { CoreError } from '@applitools/core-base';
+export declare class InternalError extends CoreError {
+    constructor(error: Error & {
+        info?: Record<string, any>;
+    });
+}