@applitools/core 4.37.0-debug-20250429 → 4.37.0

package/dist/automation/utils/to-base-check-settings.js

@@ -38,7 +38,7 @@ async function extractMobileElementsIntoCheckSettings({ settings, driver, logger
     logger = logger.extend({ tags: [`extract-mobile-elements-${utils.general.shortid()}`] });
     const environment = await driver.getEnvironment();
     if (settings.region) {
-        logger.log('extract element form region', settings.region);
+        logger.log('extract element from region', settings.region);
         const region = await driver.element(settings.region).catch(error => logger.log(error));
         if (!region) {
             throw new Error('region not found');
@@ -63,7 +63,7 @@ async function extractMobileElementsIntoCheckSettings({ settings, driver, logger
         if (!settings[`${regionType}Regions`])
             return obj;
         obj[`${regionType}Regions`] = (await Promise.all((_b = (_a = settings[`${regionType}Regions`]) === null || _a === void 0 ? void 0 : _a.flatMap(async (reference) => {
-            logger.log('extract element form region', reference);
+            logger.log('extract element from region', reference);
             const { region } = utils.types.has(reference, 'region') ? reference : { region: reference };
             const handleRegion = new Set();
             const elements = await driver.elements(region).catch(error => logger.log(error));

package/dist/classic/check.js

@@ -355,7 +355,7 @@ async function takeNMLScreenshot({ driver, settings, environments, logger, eyes,
                 aggregateCalculatedRegions.push({ regions, selector: reference });
             }
             else {
-                logger.log(`Regions "${reference}" was not found in the screenshot`);
+                logger.log(`Regions "${JSON.stringify(reference)}" was not found in the screenshot`);
             }
         });
         if ((_a = baseTarget.dom) === null || _a === void 0 ? void 0 : _a.error) {

package/dist/ufg/check.js

@@ -174,6 +174,12 @@ function makeCheck({ eyes, target: defaultTarget, environments: defaultEnvironme
                             },
                         });
                     }
+                    try {
+                        logInvalidSettings(logger, settings);
+                    }
+                    catch (err) {
+                        logger.log('Failed to log invalid check settings', err);
+                    }
                     await baseEyes.check({
                         target: { ...baseTarget, isTransformed: true },
                         settings: baseSettings,
@@ -204,3 +210,14 @@ function makeCheck({ eyes, target: defaultTarget, environments: defaultEnvironme
     };
 }
 exports.makeCheck = makeCheck;
+function logInvalidSettings(logger, settings) {
+    const invalidKeys = ['overlap', 'scrollRootElement', 'stitchMode'];
+    if (!settings)
+        return;
+    const invalidEntries = Object.entries(settings)
+        .filter(([key]) => invalidKeys.includes(key))
+        .filter(([_, value]) => value !== undefined);
+    if (invalidEntries.length > 0) {
+        logger.warn('Warning - Invalid keys in check settings (will be ignored):', { settings, invalidEntries });
+    }
+}

package/dist/ufg/utils/take-dom-snapshots.js

@@ -54,36 +54,50 @@ async function takeDomSnapshots({ driver, settings, hooks, provides, logger, })
         return Array(settings.environments.length).fill(snapshot);
     }
     const isStrictBreakpoints = utils.types.isArray((_b = settings.layoutBreakpoints) === null || _b === void 0 ? void 0 : _b.breakpoints);
-    const requiredWidths = await settings.environments.reduce(async (prev, environment, index) => {
-        const { name, width } = (await extractEnvironmentInfo(environment));
-        const requiredWidths = await prev;
-        const requiredWidth = isStrictBreakpoints
-            ? calculateBreakpoint({ breakpoints: settings.layoutBreakpoints.breakpoints, value: width })
-            : width;
-        let environments = requiredWidths.get(requiredWidth);
+    const initialViewportSize = await driver.getViewportSize();
+    const requiredSizes = await settings.environments.reduce(async (prev, environment, index) => {
+        var _a;
+        const { name, width, height } = (await extractEnvironmentInfo(environment));
+        const requiredSizes = await prev;
+        let requiredSize;
+        if ((_a = settings.layoutBreakpoints) === null || _a === void 0 ? void 0 : _a.heightBreakpoints) {
+            const requiredWidth = settings.layoutBreakpoints.breakpoints ? width : initialViewportSize.width;
+            requiredSize = `${requiredWidth},${height}`;
+        }
+        else {
+            const requiredWidth = isStrictBreakpoints
+                ? calculateBreakpoint({ breakpoints: settings.layoutBreakpoints.breakpoints, value: width })
+                : width;
+            requiredSize = `${requiredWidth},${initialViewportSize.height}`;
+        }
+        let environments = requiredSizes.get(requiredSize);
         if (!environments)
-            requiredWidths.set(requiredWidth, (environments = []));
-        environments.push({ name, width, index });
-        return requiredWidths;
+            requiredSizes.set(requiredSize, (environments = []));
+        environments.push({ name, width, height, index });
+        return requiredSizes;
     }, Promise.resolve(new Map()));
     const smallestBreakpoint = Math.min(...(isStrictBreakpoints ? settings.layoutBreakpoints.breakpoints : []));
-    if (isStrictBreakpoints && requiredWidths.has(smallestBreakpoint - 1)) {
+    const smallestSize = `${smallestBreakpoint - 1},${initialViewportSize.height}`;
+    if (isStrictBreakpoints && requiredSizes.has(smallestSize)) {
         const message = lang.requestedEnvironmentSmallerThenBreakpointWarning({
             breakpoint: smallestBreakpoint,
-            browsers: requiredWidths.get(smallestBreakpoint - 1),
+            browsers: requiredSizes.get(smallestSize),
         });
         logger.console.log(chalk_1.default.yellow(message));
     }
-    logger.log(`Taking multiple dom snapshots for the breakpoints`, settings.layoutBreakpoints);
-    const initialViewportSize = await driver.getViewportSize();
+    logger.log(`Taking multiple dom snapshots for the breakpoints`, isStrictBreakpoints ? settings.layoutBreakpoints : requiredSizes);
     const entries = [];
-    const initialsBrowsers = requiredWidths.get(initialViewportSize.width);
+    const initialSize = `${initialViewportSize.width},${initialViewportSize.height}`;
+    const initialsBrowsers = requiredSizes.get(initialSize);
     if (initialsBrowsers) {
         logger.log(`The first dom snapshot is going to be taking for initial viewport size`, initialViewportSize);
-        requiredWidths.delete(initialViewportSize.width);
+        requiredSizes.delete(initialSize);
         entries.push({ browsers: initialsBrowsers });
     }
-    requiredWidths.forEach((browsers, requiredWidth) => entries.push({ viewportSize: { width: requiredWidth, height: initialViewportSize.height }, browsers }));
+    requiredSizes.forEach((browsers, requiredSize) => {
+        const [requiredWidth, requiredHeight] = requiredSize.split(',').map(Number);
+        entries.push({ viewportSize: { width: requiredWidth, height: requiredHeight }, browsers });
+    });
     logger.log(`All entries to take dom snapshots for`, entries);
     // making safe selectors before any resize in order to make sure fallback selectors are created
     await (0, generate_safe_selectors_1.generateSafeSelectors)({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@applitools/core",
-  "version": "4.37.0-debug-20250429",
+  "version": "4.37.0",
   "homepage": "https://applitools.com",
   "bugs": {
     "url": "https://github.com/applitools/eyes.sdk.javascript1/issues"
@@ -75,7 +75,7 @@
     }
   },
   "dependencies": {
-    "@applitools/core-base": "1.24.0-debug-20250429",
+    "@applitools/core-base": "1.24.0",
     "@applitools/dom-capture": "11.5.5",
     "@applitools/dom-snapshot": "4.11.18",
     "@applitools/driver": "1.21.1",

package/types/ufg/check-and-close.d.ts

@@ -19,6 +19,7 @@ export declare function makeCheckAndClose<TSpec extends SpecType>({ eyes, target
         disableBrowserFetching?: boolean | undefined;
         layoutBreakpoints?: {
             breakpoints: boolean | number[];
+            heightBreakpoints?: boolean | undefined;
             reload?: boolean | undefined;
         } | undefined;
         hooks?: {

package/types/ufg/types.d.ts

@@ -54,6 +54,7 @@ export type SnapshotSettings<TSpec extends SpecType> = AutomationCore.CheckSetti
     disableBrowserFetching?: boolean;
     layoutBreakpoints?: {
         breakpoints: number[] | boolean;
+        heightBreakpoints?: boolean;
         reload?: boolean;
     };
     hooks?: {

package/types/ufg/utils/take-dom-snapshots.d.ts

@@ -10,6 +10,7 @@ export type DomSnapshotsSettings<TSpec extends SpecType> = DomSnapshotSettings<T
     waitBeforeCapture?: number | (() => Promise<void>);
     layoutBreakpoints?: {
         breakpoints: number[] | boolean;
+        heightBreakpoints?: boolean;
         reload?: boolean;
     };
     lazyLoad?: boolean | LazyLoadOptions;

package/dist/errors/test-error.js

@@ -1,32 +0,0 @@
-"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/types/errors/test-error.d.ts

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