@applitools/core 4.37.1 → 4.38.1-debug-20250507.0

package/CHANGELOG.md

@@ -1,5 +1,95 @@
 # Changelog
 
+## [4.38.1](https://github.com/Applitools-Dev/sdk/compare/js/core@4.38.0...js/core@4.38.1) (2025-05-07)
+
+
+### Bug Fixes
+
+* improve retry timeout logic in makeReqUFG function | AD-9895 ([#2949](https://github.com/Applitools-Dev/sdk/issues/2949)) ([f6c7048](https://github.com/Applitools-Dev/sdk/commit/f6c704846faa3d16bf3df01d289e5961363d88ae))
+* offline test dist readings | AD-9894 ([#2945](https://github.com/Applitools-Dev/sdk/issues/2945)) ([92293f4](https://github.com/Applitools-Dev/sdk/commit/92293f40ddbbb015fba224a70158bbd0cdbcf799))
+* silent tunnel warning | AD-9842 ([#2932](https://github.com/Applitools-Dev/sdk/issues/2932)) ([e83a2ff](https://github.com/Applitools-Dev/sdk/commit/e83a2fffc5c73edda8c52c101356f94d0c600959))
+
+
+### Code Refactoring
+
+* simplify upload resources logic ([#2952](https://github.com/Applitools-Dev/sdk/issues/2952)) ([9918341](https://github.com/Applitools-Dev/sdk/commit/9918341567b6c0148c7ea957932bd4f57896c1fb))
+
+
+### Dependencies
+
+* @applitools/utils bumped to 1.8.1
+  #### Bug Fixes
+
+  * offline test dist readings | AD-9894 ([#2945](https://github.com/Applitools-Dev/sdk/issues/2945)) ([92293f4](https://github.com/Applitools-Dev/sdk/commit/92293f40ddbbb015fba224a70158bbd0cdbcf799))
+* @applitools/logger bumped to 2.1.3
+  #### Bug Fixes
+
+  * silent tunnel warning | AD-9842 ([#2932](https://github.com/Applitools-Dev/sdk/issues/2932)) ([e83a2ff](https://github.com/Applitools-Dev/sdk/commit/e83a2fffc5c73edda8c52c101356f94d0c600959))
+
+
+
+* @applitools/socket bumped to 1.2.4
+
+* @applitools/tunnel-client bumped to 1.6.6
+  #### Bug Fixes
+
+  * silent tunnel warning | AD-9842 ([#2932](https://github.com/Applitools-Dev/sdk/issues/2932)) ([e83a2ff](https://github.com/Applitools-Dev/sdk/commit/e83a2fffc5c73edda8c52c101356f94d0c600959))
+
+
+
+* @applitools/ufg-client bumped to 1.16.10
+  #### Bug Fixes
+
+  * improve retry timeout logic in makeReqUFG function | AD-9895 ([#2949](https://github.com/Applitools-Dev/sdk/issues/2949)) ([f6c7048](https://github.com/Applitools-Dev/sdk/commit/f6c704846faa3d16bf3df01d289e5961363d88ae))
+  * offline test dist readings | AD-9894 ([#2945](https://github.com/Applitools-Dev/sdk/issues/2945)) ([92293f4](https://github.com/Applitools-Dev/sdk/commit/92293f40ddbbb015fba224a70158bbd0cdbcf799))
+
+
+  #### Code Refactoring
+
+  * simplify upload resources logic ([#2952](https://github.com/Applitools-Dev/sdk/issues/2952)) ([9918341](https://github.com/Applitools-Dev/sdk/commit/9918341567b6c0148c7ea957932bd4f57896c1fb))
+
+
+
+* @applitools/dom-snapshot bumped to 4.11.20
+
+* @applitools/req bumped to 1.7.11
+
+* @applitools/image bumped to 1.1.19
+
+* @applitools/driver bumped to 1.21.3
+
+* @applitools/spec-driver-webdriver bumped to 1.2.4
+
+* @applitools/spec-driver-selenium bumped to 1.5.100
+
+* @applitools/spec-driver-puppeteer bumped to 1.4.29
+
+* @applitools/screenshoter bumped to 3.11.3
+
+* @applitools/nml-client bumped to 1.9.3
+
+* @applitools/core-base bumped to 1.25.1
+
+* @applitools/ec-client bumped to 1.10.12
+
+
+## [4.38.0](https://github.com/Applitools-Dev/sdk/compare/js/core@4.37.1...js/core@4.38.0) (2025-04-30)
+
+
+### Features
+
+* openCheckAndClose for offline executions ([#2941](https://github.com/Applitools-Dev/sdk/issues/2941)) ([d44a274](https://github.com/Applitools-Dev/sdk/commit/d44a27423520e17bfc06117b28c17e5f0ea78876))
+
+
+### Dependencies
+
+* @applitools/core-base bumped to 1.25.0
+  #### Features
+
+  * openCheckAndClose for offline executions ([#2941](https://github.com/Applitools-Dev/sdk/issues/2941)) ([d44a274](https://github.com/Applitools-Dev/sdk/commit/d44a27423520e17bfc06117b28c17e5f0ea78876))
+* @applitools/ec-client bumped to 1.10.11
+
+
 ## [4.37.1](https://github.com/Applitools-Dev/sdk/compare/js/core@4.37.0...js/core@4.37.1) (2025-04-29)
 
 

package/dist/errors/test-error.js

@@ -0,0 +1,32 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.TestError = void 0;
+const core_base_1 = require("@applitools/core-base");
+class TestError extends core_base_1.CoreError {
+    constructor(result) {
+        if (result.status === 'Failed') {
+            super(`Test '${result.name}' of '${result.appName}' is failed! See details at ${result.url}`, {
+                reason: 'test failed',
+                result,
+            });
+        }
+        else if (result.status === 'Unresolved') {
+            if (result.isNew) {
+                super(`Test '${result.name}' of '${result.appName}' is new! Please approve the new baseline at ${result.url}`, {
+                    reason: 'test new',
+                    result,
+                });
+            }
+            else {
+                super(`Test '${result.name}' of '${result.appName}' detected differences! See details at: ${result.url}`, {
+                    reason: 'test different',
+                    result,
+                });
+            }
+        }
+        else {
+            super('');
+        }
+    }
+}
+exports.TestError = TestError;

package/dist/offline/run-offline-snapshots.js

@@ -31,13 +31,14 @@ const fs_1 = __importDefault(require("fs"));
 const path_1 = __importDefault(require("path"));
 const core_1 = require("../core");
 const get_ufg_client_1 = require("../ufg/get-ufg-client");
-const logger_1 = require("@applitools/logger");
 const to_base_check_settings_1 = require("../automation/utils/to-base-check-settings");
+const logger_1 = require("@applitools/logger");
 const chalk_1 = __importDefault(require("chalk"));
 const utils = __importStar(require("@applitools/utils"));
 const merge_configs_1 = require("./merge-configs");
 const format_results_1 = require("../utils/format-results");
 const throat_1 = __importDefault(require("throat"));
+const throttledReadFile = (0, throat_1.default)(Number(process.env.APPLITOOLS_FILE_READING_CONCURRENCY) || 1000, fs_1.default.promises.readFile);
 function getConsoleLogLimit() {
     var _a;
     return (_a = utils.general.getEnvValue('OFFLINE_LOG_LIMIT', 'number')) !== null && _a !== void 0 ? _a : 20;
@@ -45,6 +46,7 @@ function getConsoleLogLimit() {
 async function runOfflineSnapshots(options) {
     var _a;
     const limitConsoleLogs = getConsoleLogLimit();
+    const throttledRender = (0, throat_1.default)(Number(process.env.APPLITOOLS_RENDER_CONCURRENCY) || 1000, render);
     if (!options.offlineLocationPath)
         throw new Error('offlineLocationPath is required');
     const offlineLocationPath = path_1.default.resolve(options.offlineLocationPath);
@@ -127,71 +129,120 @@ async function runOfflineSnapshots(options) {
         const environment = await ufgClient.getActualEnvironment({
             settings: { environment: openSettings.environment.requested },
         });
+        const closeSettings = await getCloseSettings(testPath);
+        let resultsGetter;
+        if (closeSettings) {
+            const targets = await getCheckTargets(testPath, logger);
+            const uploadResourcePromise = uploadResources(targets, logger).then(() => logger.log('uploaded resources for test', testPath));
+            if (targets.length === 1) {
+                logger.log('only one check found for test', testPath);
+                await uploadResourcePromise;
+                const target = targets[0];
+                resultsGetter = await runOpenCheckAndClose({
+                    ...target,
+                    settings: {
+                        ...(0, merge_configs_1.mergeConfigs)((0, merge_configs_1.mergeConfigs)(closeSettings, openSettings), target.settings),
+                        ...account.eyesServer,
+                        environment: {
+                            ...openSettings.environment,
+                            ...target.settings.environment,
+                        },
+                    },
+                }, logger, environment);
+                logger.log('finished running checks for test', testPath);
+            }
+            else {
+                const eyes = await openEyes(openSettings, environment, logger);
+                logger.log('opened eyes for test', testPath);
+                await uploadResourcePromise;
+                const lastTarget = targets.pop();
+                await Promise.all(targets.map((target, index) => runCheck(eyes, target, index, logger)));
+                if (lastTarget) {
+                    lastTarget.settings = (0, merge_configs_1.mergeConfigs)(lastTarget.settings, closeSettings);
+                    await runCheckAndClose(eyes, lastTarget, targets.length - 1, logger);
+                    logger.log('finished running checks for test', testPath);
+                }
+                else {
+                    logger.log('no checks found for test', testPath);
+                    // on empty test, `checkAndClose` will not be called so we need to call `close` instead
+                    await eyes.close({ settings: closeSettings });
+                }
+                resultsGetter = eyes;
+            }
+        }
+        else {
+            logger.log('no close settings found for test - aborting test', testPath);
+            resultsGetter = await makeAbortedTest(openSettings, environment, logger);
+        }
+        logger.log('done running test', testPath);
+        return { batchId: openSettings.batch.id, results: (await resultsGetter.getResults({ logger }))[0] };
+    }
+    async function getCloseSettings(testPath) {
+        const closeSettingsPath = path_1.default.resolve(testPath, 'close.json');
+        if (fs_1.default.existsSync(closeSettingsPath)) {
+            const fileCloseSettings = await throttledReadFile(closeSettingsPath, 'utf-8').then(JSON.parse);
+            const closeSettings = (0, merge_configs_1.mergeConfigs)(fileCloseSettings, options.config.close);
+            return closeSettings;
+        }
+        return undefined;
+    }
+    async function getCheckTargets(testPath, logger) {
+        const checkFolders = (await fs_1.default.promises.readdir(testPath)).filter(folderpath => folderpath.startsWith('check-'));
+        logger.log('check folders', checkFolders);
+        const targets = await Promise.all(checkFolders.map(async (checkFolder) => {
+            const snapshot = await fs_1.default.promises
+                .readFile(path_1.default.resolve(testPath, checkFolder, 'snapshot.json'), 'utf-8')
+                .then(JSON.parse);
+            snapshot.settings.environment.environmentId = utils.general.guid();
+            return snapshot;
+        }));
+        return targets;
+    }
+    async function makeAbortedTest(openSettings, environment, logger) {
+        const eyes = await openEyes(openSettings, environment, logger);
+        await eyes.abort({
+            settings: {
+                reason: 'internal',
+            },
+        });
+        return eyes;
+    }
+    async function openEyes(settings, environment, logger) {
         const eyes = await core.base.openEyes({
             settings: {
-                ...openSettings,
+                ...settings,
                 environment: {
-                    ...openSettings.environment,
+                    ...settings.environment,
                     ...environment,
                 },
                 ...account.eyesServer,
             },
             logger,
         });
-        const closeSettingsPath = path_1.default.resolve(testPath, 'close.json');
-        if (fs_1.default.existsSync(closeSettingsPath)) {
-            const checkFolders = (await fs_1.default.promises.readdir(testPath)).filter(folderpath => folderpath.startsWith('check-'));
-            logger.log('running checks for test', testPath, ':', checkFolders);
-            const targets = await Promise.all(checkFolders.map(async (checkFolder) => {
-                const snapshot = await fs_1.default.promises
-                    .readFile(path_1.default.resolve(testPath, checkFolder, 'snapshot.json'), 'utf-8')
-                    .then(JSON.parse);
-                snapshot.settings.environment.environmentId = utils.general.guid();
-                return snapshot;
-            }));
-            await uploadResources(targets, logger);
-            // logger.log('resource hashes for test', testFolder, ':', resourceHashes)
-            logger.log('uploaded resources for test', testPath);
-            await Promise.all(targets.map((target, index) => runCheck(eyes, target, index, logger)));
-            const fileCloseSettings = await fs_1.default.promises.readFile(closeSettingsPath, 'utf-8').then(JSON.parse);
-            const closeSettings = (0, merge_configs_1.mergeConfigs)(fileCloseSettings, options.config.close);
-            await eyes.close({ settings: closeSettings });
-        }
-        else {
-            await eyes.abort({
-                settings: {
-                    reason: 'internal',
-                },
-            });
-        }
-        logger.log('done running test', testPath);
-        return { batchId: openSettings.batch.id, results: (await eyes.getResults({ logger }))[0] };
+        return eyes;
     }
     async function uploadResources(targets, logger) {
         const uploadLogger = logger.extend({ tags: ['upload-resources'] });
         const promises = targets.map(async ({ target }) => {
-            let resourcePromises = Object.values(target.resources)
+            const resourcePromises = Object.values(target.resources)
                 .filter(isHashedResource)
                 .map(async (resource) => {
-                const contentfuleResource = {
-                    id: '',
-                    url: '',
-                    value: await fs_1.default.promises.readFile(path_1.default.join(offlineLocationPath, 'resources', resource.hash)),
-                    contentType: resource.contentType,
-                    hash: resource,
-                };
-                return ufgClient.uploadResource({ resource: contentfuleResource, logger: uploadLogger });
+                return ufgClient.uploadResource({
+                    resource: {
+                        hash: resource,
+                        async getter() {
+                            return {
+                                id: '',
+                                url: '',
+                                value: await throttledReadFile(path_1.default.join(offlineLocationPath, 'resources', resource.hash)),
+                                contentType: resource.contentType,
+                                hash: resource,
+                            };
+                        },
+                    },
+                    logger: uploadLogger,
+                });
             });
-            resourcePromises = resourcePromises.concat(ufgClient.uploadResource({
-                resource: {
-                    id: '',
-                    url: '',
-                    value: await fs_1.default.promises.readFile(path_1.default.join(offlineLocationPath, 'resources', target.snapshot.hash)),
-                    contentType: target.snapshot.contentType,
-                    hash: target.snapshot,
-                },
-                logger: uploadLogger,
-            }));
             return Promise.all(resourcePromises);
         });
         await Promise.all(promises);
@@ -202,6 +253,53 @@ async function runOfflineSnapshots(options) {
     async function runCheck(eyes, target, index, logger) {
         var _a;
         const checkLogger = logger.extend({ tags: [`check-${index}`] });
+        const { mergedCheckSettings, baseTarget } = await throttledRender(target, checkLogger);
+        (_a = mergedCheckSettings.stepIndex) !== null && _a !== void 0 ? _a : (mergedCheckSettings.stepIndex = index);
+        await eyes.check({
+            target: { ...baseTarget, isTransformed: true },
+            settings: mergedCheckSettings,
+            logger: checkLogger,
+        });
+        logger.log('check finished', mergedCheckSettings);
+        return eyes;
+    }
+    async function runCheckAndClose(eyes, target, index, logger) {
+        var _a;
+        const checkLogger = logger.extend({ tags: [`check-${index}`] });
+        const { mergedCheckSettings, baseTarget } = await throttledRender(target, checkLogger);
+        (_a = mergedCheckSettings.stepIndex) !== null && _a !== void 0 ? _a : (mergedCheckSettings.stepIndex = index);
+        await eyes.checkAndClose({
+            target: { ...baseTarget, isTransformed: true },
+            settings: mergedCheckSettings,
+            logger: checkLogger,
+        });
+        logger.log('checkAndClose finished', mergedCheckSettings);
+        return eyes;
+    }
+    async function runOpenCheckAndClose(target, logger, actualEnvironment) {
+        const checkLogger = logger.extend({ tags: ['open-check-and-close'] });
+        const { mergedCheckSettings, baseTarget } = await throttledRender(target, checkLogger);
+        const settings = (0, merge_configs_1.mergeConfigs)(target.settings, mergedCheckSettings);
+        return core.base.openCheckAndCloseEyes({
+            target: { ...baseTarget, isTransformed: true },
+            settings: {
+                ...settings,
+                environment: {
+                    ...settings.environment,
+                    ...actualEnvironment,
+                },
+            },
+            logger: checkLogger,
+            heartbeat: {
+                processId: '',
+                startPeriodicHeartbeatMessaging(settings) {
+                    logger.log('heartbeat is not used (offline)', settings);
+                },
+            },
+        });
+    }
+    async function render(target, logger) {
+        var _a;
         const { elementReferences: selectors, getBaseCheckSettings } = (0, to_base_check_settings_1.toBaseCheckSettings)({
             settings: target.settings,
         });
@@ -217,7 +315,7 @@ async function runOfflineSnapshots(options) {
                 uploadUrl: account.uploadUrl,
                 stitchingServiceUrl: account.stitchingServiceUrl,
             },
-            logger: checkLogger,
+            logger,
         });
         const baseSettings = getBaseCheckSettings({
             calculatedRegions: selectors.calculate.map((_, index) => {
@@ -228,15 +326,11 @@ async function runOfflineSnapshots(options) {
             }),
         });
         baseSettings.renderId = renderId;
-        baseSettings.stepIndex = index;
+        // baseSettings.stepIndex = index
         baseTarget.source = target.target.source; // TODO verify
         // baseTarget.name = snapshot.title // TODO figure out
         const mergedCheckSettings = (0, merge_configs_1.mergeConfigs)(baseSettings, options.config.check);
-        await eyes.check({
-            target: { ...baseTarget, isTransformed: true },
-            settings: mergedCheckSettings,
-            logger: checkLogger,
-        });
+        return { mergedCheckSettings, baseTarget };
     }
 }
 exports.runOfflineSnapshots = runOfflineSnapshots;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@applitools/core",
-  "version": "4.37.1",
+  "version": "4.38.1-debug-20250507.0",
   "homepage": "https://applitools.com",
   "bugs": {
     "url": "https://github.com/applitools/eyes.sdk.javascript1/issues"
@@ -75,20 +75,20 @@
     }
   },
   "dependencies": {
-    "@applitools/core-base": "1.24.0",
+    "@applitools/core-base": "1.25.1",
     "@applitools/dom-capture": "11.5.5",
-    "@applitools/dom-snapshot": "4.11.19",
-    "@applitools/driver": "1.21.2",
-    "@applitools/ec-client": "1.10.10",
-    "@applitools/logger": "2.1.2",
-    "@applitools/nml-client": "1.9.2",
-    "@applitools/req": "1.7.10",
-    "@applitools/screenshoter": "3.11.2",
+    "@applitools/dom-snapshot": "4.11.20",
+    "@applitools/driver": "1.21.3",
+    "@applitools/ec-client": "1.10.12",
+    "@applitools/logger": "2.1.3",
+    "@applitools/nml-client": "1.9.3",
+    "@applitools/req": "1.7.11",
+    "@applitools/screenshoter": "3.11.3",
     "@applitools/snippets": "2.6.5",
-    "@applitools/socket": "1.2.3",
-    "@applitools/spec-driver-webdriver": "1.2.3",
-    "@applitools/ufg-client": "1.16.9",
-    "@applitools/utils": "1.8.0",
+    "@applitools/socket": "1.2.4",
+    "@applitools/spec-driver-webdriver": "1.2.4",
+    "@applitools/ufg-client": "1.16.10-debug-20250507.0",
+    "@applitools/utils": "1.8.1-debug-20250507.0",
     "@types/ws": "8.5.5",
     "abort-controller": "3.0.0",
     "chalk": "4.1.2",
@@ -102,11 +102,11 @@
   "devDependencies": {
     "@applitools/bongo": "^5.10.0",
     "@applitools/sea": "^1.0.0",
-    "@applitools/spec-driver-puppeteer": "^1.4.28",
-    "@applitools/spec-driver-selenium": "^1.5.99",
+    "@applitools/spec-driver-puppeteer": "^1.4.29",
+    "@applitools/spec-driver-selenium": "^1.5.100",
     "@applitools/test-server": "^1.2.2",
     "@applitools/test-utils": "^1.5.17",
-    "@applitools/tunnel-client": "^1.6.5",
+    "@applitools/tunnel-client": "^1.6.6",
     "@types/mocha": "^10.0.7",
     "@types/node": "^12.20.55",
     "@types/selenium-webdriver": "^4.1.2",

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

@@ -0,0 +1,5 @@
+import type { TestResult } from '@applitools/core-base';
+import { CoreError } from '@applitools/core-base';
+export declare class TestError extends CoreError {
+    constructor(result: TestResult);
+}

package/types/offline/merge-configs.d.ts

@@ -1 +1 @@
-export declare function mergeConfigs<TBase extends Record<string, any>>(base: TBase, other: Record<string, any>): TBase;
+export declare function mergeConfigs<TBase extends Record<string, any>, TBase2 extends Record<string, any>>(base: TBase, other: TBase2): TBase & TBase2;

package/types/offline/run-offline-snapshots.d.ts

@@ -1,5 +1,5 @@
-import { Config, type EyesServerSettings } from '../types';
 import { type Logger } from '@applitools/logger';
+import type { Config, EyesServerSettings } from '../types';
 import type { TestResult } from '@applitools/core-base';
 import type { SpecType } from '@applitools/driver';
 export type OfflineSnapshotsSettings = EyesServerSettings & {