@applitools/driver 1.9.2 → 1.9.3

package/dist/element.js

@@ -289,7 +289,7 @@ class Element {
                     .catch(err => {
                     this._logger.warn(`Unable to get the attribute 'contentSize' when looking up 'touchPadding' due to the following error: '${err.message}'`);
                 });
-                this._state.touchPadding = touchPadding !== null && touchPadding !== void 0 ? touchPadding : 20;
+                this._state.touchPadding = touchPadding !== null && touchPadding !== void 0 ? touchPadding : 21;
                 this._logger.log('Touch padding set:', this._state.touchPadding);
             }
         }
@@ -311,8 +311,11 @@ class Element {
     async getAttribute(name) {
         var _a;
         // we assumes that attributes are not changed during the session
-        if ((_a = this._state.attributes) === null || _a === void 0 ? void 0 : _a[name])
+        if ((_a = this._state.attributes) === null || _a === void 0 ? void 0 : _a[name]) {
+            if (this._state.attributes[name] instanceof Error)
+                throw this._state.attributes[name];
             return this._state.attributes[name];
+        }
         const value = await this.withRefresh(async () => {
             var _a;
             var _b;
@@ -322,17 +325,24 @@ class Element {
             }
             else {
                 this._logger.log(`Extracting "${name}" attribute of native element with selector`, this.selector);
-                const value = await this._spec.getElementAttribute(this.driver.target, this.target, name);
                 (_a = (_b = this._state).attributes) !== null && _a !== void 0 ? _a : (_b.attributes = {});
-                this._state.attributes[name] = value;
-                if (this.driver.isAndroid && name === 'contentSize') {
-                    // android has a bug when after extracting 'contentSize' attribute the element is being scrolled by undetermined number of pixels
-                    this._logger.log('Stabilizing android scroll offset');
-                    const originalScrollOffset = await this.getScrollOffset();
-                    await this.scrollTo({ x: 0, y: 0 }, { force: true });
-                    await this.scrollTo(originalScrollOffset);
+                try {
+                    this._state.attributes[name] = await this._spec.getElementAttribute(this.driver.target, this.target, name);
+                    return this._state.attributes[name];
+                }
+                catch (err) {
+                    this._state.attributes[name] = err;
+                    throw err;
+                }
+                finally {
+                    if (this.driver.isAndroid && name === 'contentSize') {
+                        // android has a bug when after extracting 'contentSize' attribute the element is being scrolled by undetermined number of pixels
+                        this._logger.log('Stabilizing android scroll offset');
+                        const originalScrollOffset = await this.getScrollOffset();
+                        await this.scrollTo({ x: 0, y: 0 }, { force: true });
+                        await this.scrollTo(originalScrollOffset);
+                    }
                 }
-                return value;
             }
         });
         this._logger.log(`Extracted element "${name}" attribute:`, value);
@@ -370,7 +380,7 @@ class Element {
                     : utils.geometry.offsetNegative(requiredOffset, currentScrollOffset);
                 if (this.driver.isAndroid) {
                     remainingOffset = utils.geometry.scale(remainingOffset, this.driver.pixelRatio);
-                    effectiveRegion = utils.geometry.scale(scrollableRegion, this.driver.pixelRatio);
+                    effectiveRegion = utils.geometry.scale(effectiveRegion, this.driver.pixelRatio);
                 }
                 const actions = [];
                 const touchPadding = await this.getTouchPadding();
@@ -464,10 +474,10 @@ class Element {
                     await this._spec.performAction(this.driver.target, [].concat(...actions));
                 }
                 const actualScrollableRegion = await this.getClientRegion();
-                this._state.scrollOffset = utils.geometry.offsetNegative(requiredOffset, {
+                this._state.scrollOffset = utils.geometry.round(utils.geometry.offsetNegative(requiredOffset, {
                     x: scrollableRegion.x - actualScrollableRegion.x,
                     y: scrollableRegion.y - actualScrollableRegion.y,
-                });
+                }));
                 return this._state.scrollOffset;
             }
         });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@applitools/driver",
-  "version": "1.9.2",
+  "version": "1.9.3",
   "description": "Applitools universal framework wrapper",
   "keywords": [
     "applitools",
@@ -24,6 +24,9 @@
     "name": "Applitools Team",
     "email": "team@applitools.com"
   },
+  "aliases": [
+    "driver"
+  ],
   "exports": {
     ".": {
       "types": "./types/index.d.ts",
@@ -73,13 +76,13 @@
     }
   },
   "dependencies": {
-    "@applitools/logger": "1.1.10",
-    "@applitools/snippets": "2.2.3",
-    "@applitools/types": "1.4.7",
-    "@applitools/utils": "1.3.6"
+    "@applitools/logger": "1.1.11",
+    "@applitools/snippets": "2.4.1",
+    "@applitools/types": "1.5.2",
+    "@applitools/utils": "1.3.7"
   },
   "devDependencies": {
-    "@applitools/bongo": "^2.1.4",
+    "@applitools/bongo": "^2.1.5",
     "@types/mocha": "^9.1.1",
     "@types/node": "^17.0.31",
     "@typescript-eslint/eslint-plugin": "^5.22.0",

package/types/element.d.ts

@@ -5,7 +5,7 @@ export declare type ElementState<TElement> = {
     contentSize?: types.Size;
     scrollOffset?: types.Location;
     transforms?: any;
-    attributes?: Record<string, string>;
+    attributes?: Record<string, string | Error>;
     touchPadding?: number;
     containedElements?: Map<TElement, boolean>;
 };