@applitools/screenshoter 3.3.12 → 3.3.15

package/index.js

@@ -1,2 +1,6 @@
-module.exports = require('./src/take-screenshot')
-exports.takeScreenshot = require('./src/take-screenshot')
+const takeScreenshot = require('./src/take-screenshot')
+const getTarget = require('./src/getTarget')
+module.exports = {
+  takeScreenshot,
+  getTarget,
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@applitools/screenshoter",
-  "version": "3.3.12",
+  "version": "3.3.15",
   "description": "Applitools universal screenshoter for web and native applications",
   "keywords": [
     "applitools",
@@ -35,8 +35,8 @@
     "test": "yarn test:it && yarn test:e2e",
     "test:unit": "mocha ./test/unit/*.spec.js --no-timeouts",
     "test:it": "mocha ./test/it/*.spec.js --no-timeouts",
-    "test:e2e": "mocha ./test/e2e/**/*.spec.js --no-timeouts --parallel --jobs ${MOCHA_JOBS:-1}",
-    "test:e2e:web": "mocha ./test/e2e/web/*.spec.js --no-timeouts -r @applitools/test-utils/mocha-hooks/docker --parallel --jobs ${MOCHA_JOBS:-15}",
+    "test:e2e": "yarn test:e2e:web && yarn && test:e2e:android && yarn test:e2e:ios",
+    "test:e2e:web": "APPLITOOLS_SHOW_LOGS=true mocha ./test/e2e/web/*.spec.js --no-timeouts -r @applitools/test-utils/mocha-hooks/docker",
     "test:e2e:android": "APPLITOOLS_TEST_REMOTE=sauce mocha ./test/e2e/android*/*.spec.js --no-timeouts --parallel --jobs ${MOCHA_JOBS:-2}",
     "test:e2e:ios": "APPLITOOLS_TEST_REMOTE=sauce mocha ./test/e2e/ios*/*.spec.js --no-timeouts --parallel --jobs ${MOCHA_JOBS:-2}",
     "setup:web": "yarn docker:setup",
@@ -65,11 +65,11 @@
     "png-async": "0.9.4"
   },
   "devDependencies": {
-    "@applitools/driver": "1.5.4",
+    "@applitools/bongo": "^2.0.1",
+    "@applitools/driver": "1.6.1",
     "@applitools/scripts": "1.1.0",
-    "@applitools/sdk-release-kit": "0.13.11",
-    "@applitools/spec-driver-webdriverio": "1.2.7",
-    "@applitools/test-utils": "1.1.5",
+    "@applitools/spec-driver-webdriverio": "1.2.8",
+    "@applitools/test-utils": "1.3.0",
     "appium": "^1.22.2",
     "chromedriver": "^95.0.0",
     "eslint": "^7.9.0",

package/src/getTarget.js

@@ -0,0 +1,94 @@
+'use strict'
+
+const utils = require('@applitools/utils')
+
+function isNavigationBar(region) {
+  if (
+    (region.value && region.value.includes(`type == "XCUIElementTypeNavigationBar"`)) ||
+    (region.selector && region.selector.includes(`type == "XCUIElementTypeNavigationBar"`))
+  ) {
+    return true
+  } else {
+    return false
+  }
+}
+
+function isTabBar(region) {
+  if (
+    (region.value && region.value.includes('type == "XCUIElementTypeTabBar"')) ||
+    (region.selector && region.selector.includes(`type == "XCUIElementTypeTabBar"`))
+  ) {
+    return true
+  } else {
+    return false
+  }
+}
+
+async function getTarget({window, context, region, fully, scrollingMode}) {
+  if (window) {
+    // window/app
+    const scrollingElement = await context.main.getScrollingElement()
+    return {
+      context: context.main,
+      scrollingElement,
+    }
+  } else if (region) {
+    if (utils.types.has(region, ['x', 'y', 'width', 'height'])) {
+      // region by coordinates
+      const scrollingElement = await context.getScrollingElement()
+      return {
+        context,
+        region,
+        scrollingElement,
+      }
+    } else {
+      // region by element or selector
+      const element = await context.element(region)
+      if (!element) throw new Error('Element not found!')
+
+      const elementContext = element.context
+
+      if (fully) {
+        const isScrollable = await element.isScrollable()
+        // if element is scrollable, then take screenshot of the full element content, otherwise take screenshot of full element
+        const region = isScrollable ? null : await element.getRegion()
+        const scrollingElement = isScrollable ? element : await elementContext.getScrollingElement()
+        // css stitching could be applied only to root element of its context
+        scrollingMode = scrollingMode === 'css' && !(await scrollingElement.isRoot()) ? 'mixed+' : scrollingMode
+        return {
+          context: elementContext,
+          region,
+          scrollingMode,
+          scrollingElement,
+        }
+      } else {
+        const scrollingElement = await context.getScrollingElement()
+        const navBar = isNavigationBar(region)
+        const tabBar = isTabBar(region)
+        return {
+          context: elementContext,
+          region: await element.getRegion(navBar || tabBar),
+          scrollingElement,
+        }
+      }
+    }
+  } else if (!context.isMain) {
+    // context
+    if (fully) {
+      const scrollingElement = await context.getScrollingElement()
+      return {
+        context,
+        scrollingElement,
+      }
+    } else {
+      const scrollingElement = await context.parent.getScrollingElement()
+      const element = await context.getContextElement()
+      return {
+        context: context.parent,
+        region: await element.getRegion(), // IMHO we should use CLIENT (without borders) region here
+        scrollingElement,
+      }
+    }
+  }
+}
+module.exports = getTarget

package/src/take-screenshot.js

@@ -1,6 +1,6 @@
-const utils = require('@applitools/utils')
-const makeScroller = require('./scroller')
 const scrollIntoViewport = require('./scroll-into-viewport')
+const getTarget = require('./getTarget')
+const makeScroller = require('./scroller')
 const takeStitchedScreenshot = require('./take-stitched-screenshot')
 const takeSimpleScreenshot = require('./take-simple-screenshot')
 
@@ -48,32 +48,48 @@ async function takeScreenshot({
   // blur active element in target context
   const activeElement = driver.isWeb && hideCaret ? await context.blurElement() : null
 
-  const target = await getTarget({window, context, region, fully, scrollingMode, logger})
+  const target = await getTarget({window, context, region, fully, scrollingMode})
+  // in some cases getTarget logic manipulates the 'scrollingMode'
+  scrollingMode = target.scrollingMode || scrollingMode
 
-  if (target.scroller) {
-    await target.scroller.preserveState()
-    if (driver.isWeb && hideScrollbars) await target.scroller.hideScrollbars()
+  const scroller = target.scrollingElement
+    ? makeScroller({element: target.scrollingElement, scrollingMode, logger})
+    : null
+
+  if (scroller) {
+    await scroller.preserveState()
+    if (driver.isWeb && hideScrollbars) await scroller.hideScrollbars()
   }
 
   try {
-    if (!window) await scrollIntoViewport({...target, logger})
+    if (!window) await scrollIntoViewport({context: target.context, region: target.region, scroller, logger})
 
     const screenshot =
-      fully && target.scroller
-        ? await takeStitchedScreenshot({...target, withStatusBar, overlap, framed, wait, stabilization, debug, logger})
-        : await takeSimpleScreenshot({...target, withStatusBar, wait, stabilization, debug, logger})
+      fully && scroller
+        ? await takeStitchedScreenshot({
+            ...target,
+            scroller,
+            withStatusBar,
+            overlap,
+            framed,
+            wait,
+            stabilization,
+            debug,
+            logger,
+          })
+        : await takeSimpleScreenshot({...target, scroller, withStatusBar, wait, stabilization, debug, logger})
 
     screenshot.image.scale(driver.viewportScale)
 
     if (hooks && hooks.afterScreenshot) {
-      await hooks.afterScreenshot({driver, scroller: target.scroller, screenshot})
+      await hooks.afterScreenshot({driver, scroller, screenshot})
     }
 
     return screenshot
   } finally {
-    if (target.scroller) {
-      await target.scroller.restoreScrollbars()
-      await target.scroller.restoreState()
+    if (scroller) {
+      await scroller.restoreScrollbars()
+      await scroller.restoreState()
     }
 
     // if there was active element and we have blurred it, then restore focus
@@ -93,69 +109,4 @@ async function takeScreenshot({
   }
 }
 
-async function getTarget({window, context, region, fully, scrollingMode, logger}) {
-  if (window) {
-    // window/app
-    const scrollingElement = await context.main.getScrollingElement()
-    return {
-      context: context.main,
-      scroller: scrollingElement ? makeScroller({element: scrollingElement, scrollingMode, logger}) : null,
-    }
-  } else if (region) {
-    if (utils.types.has(region, ['x', 'y', 'width', 'height'])) {
-      // region by coordinates
-      const scrollingElement = await context.getScrollingElement()
-      return {
-        context,
-        region,
-        scroller: scrollingElement ? makeScroller({element: scrollingElement, scrollingMode, logger}) : null,
-      }
-    } else {
-      // region by element or selector
-      const element = await context.element(region)
-      if (!element) throw new Error('Element not found!')
-
-      const elementContext = element.context
-
-      if (fully) {
-        const isScrollable = await element.isScrollable()
-        // if element is scrollable, then take screenshot of the full element content, otherwise take screenshot of full element
-        const region = isScrollable ? null : await element.getRegion()
-        const scrollingElement = isScrollable ? element : await elementContext.getScrollingElement()
-        // css stitching could be applied only to root element of its context
-        scrollingMode = scrollingMode === 'css' && !(await scrollingElement.isRoot()) ? 'mixed+' : scrollingMode
-        return {
-          context: elementContext,
-          region,
-          scroller: scrollingElement ? makeScroller({element: scrollingElement, scrollingMode, logger}) : null,
-        }
-      } else {
-        const scrollingElement = await context.getScrollingElement()
-        return {
-          context: elementContext,
-          region: await element.getRegion(),
-          scroller: scrollingElement ? makeScroller({element: scrollingElement, scrollingMode, logger}) : null,
-        }
-      }
-    }
-  } else if (!context.isMain) {
-    // context
-    if (fully) {
-      const scrollingElement = await context.getScrollingElement()
-      return {
-        context,
-        scroller: scrollingElement ? makeScroller({logger, element: scrollingElement, scrollingMode}) : null,
-      }
-    } else {
-      const scrollingElement = await context.parent.getScrollingElement()
-      const element = await context.getContextElement()
-      return {
-        context: context.parent,
-        region: await element.getRegion(), // IMHO we should use CLIENT (without borders) region here
-        scroller: scrollingElement ? makeScroller({logger, element: scrollingElement, scrollingMode}) : null,
-      }
-    }
-  }
-}
-
 module.exports = takeScreenshot

package/src/take-viewport-screenshot.js

@@ -163,7 +163,18 @@ function makeTakeNativeScreenshot({driver, stabilization = {}, debug, logger}) {
       const viewportSize = await driver.getViewportSize()
       const cropRegion = withStatusBar
         ? {x: 0, y: 0, width: viewportSize.width, height: viewportSize.height + driver.statusBarHeight}
-        : {top: driver.statusBarHeight, bottom: driver.navigationBarHeight, left: 0, right: 0}
+        : {
+            top: driver.statusBarHeight,
+            bottom: driver.orientation === 'landscape' ? 0 : driver.navigationBarHeight,
+            left:
+              driver.isAndroid && driver.orientation === 'landscape' && driver.platformVersion > 7
+                ? driver.navigationBarHeight
+                : 0,
+            right:
+              driver.isAndroid && driver.orientation === 'landscape' && driver.platformVersion < 8
+                ? driver.navigationBarHeight
+                : 0,
+          }
       image.crop(cropRegion)
     }
 

package/CHANGELOG.md

@@ -1,187 +0,0 @@
-
-# Change Log
-
-## Unreleased
-
-
-## 3.3.12 - 2022/3/14
-
-- update snippets
-- updated to @applitools/snippets@2.2.2 (from 2.2.1)
-
-## 3.3.11 - 2022/3/12
-
-- disable scroll to 0,0 before taking stitched screenshot
-- add better error handling when taking a stitched screenshot
-- updated to @applitools/snippets@2.2.1 (from 2.2.0)
-
-## 3.3.10 - 2022/2/16
-
-- do not perform initial scrolling in `takeStitchedScreenshot`
-
-## 3.3.9 - 2022/2/16
-
-- remove prevention of translation in `mixed` scrolling mode
-- updated to @applitools/snippets@2.1.15 (from 2.1.14)
-
-## 3.3.8 - 2022/2/16
-
-- updated to @applitools/snippets@2.1.14 (from 2.1.13)
-
-## 3.3.7 - 2022/2/15
-
-- fix image scaling on pages without viewport metatag
-- fix safari's viewport detection on iOS devices
-- updated to @applitools/snippets@2.1.13 (from 2.1.12)
-- updated to @applitools/utils@1.2.13 (from 1.2.12)
-
-## 3.3.6 - 2022/2/9
-
-- fix testing
-- updated to @applitools/utils@1.2.12 (from 1.2.11)
-
-## 3.3.5 - 2022/2/8
-
-- add "files" field to the package.json to avoid unnecessary files to be published
-- updated to @applitools/utils@1.2.11 (from 1.2.5)
-
-## 3.3.4 - 2021/12/23
-
-- updated to @applitools/snippets@2.1.12 (from 2.1.11)
-
-## 3.3.3 - 2021/12/22
-
-- improve default rotation and scaling logic
-- improve scroll into viewport algorithm
-- fix lazy handling of image rotation
-- updated to @applitools/snippets@2.1.11 (from 2.1.10)
-- updated to @applitools/utils@1.2.5 (from 1.2.4)
-
-## 3.3.2 - 2021/12/20
-
-- add basic support of webview screenshots on ios
-- updated to @applitools/snippets@2.1.10 (from 2.1.8)
-
-## 3.3.1 - 2021/12/17
-
-- no changes
-
-## 3.3.0 - 2021/12/16
-
-- fix ios web screenshots on pages without viewport meta tag
-- improve native apps support
-- updated to @applitools/snippets@2.1.8 (from 2.1.7)
-
-## 3.2.9 - 2021/11/14
-
-- add support of scrollable elements that change its size during scrolling
-
-## 3.2.8 - 2021/10/30
-
-- updated to @applitools/utils@1.2.4 (from 1.2.3)
-
-## 3.2.7 - 2021/10/13
-
-- handle a case when scrolling element does not exist
-
-## 3.2.6 - 2021/10/12
-
-- handle a case when scrolling element does not exist
-
-## 3.2.5 - 2021/10/5
-
-- fix issue with fractional image size after scaling
-
-## 3.2.4 - 2021/9/9
-
-- handle selectors that evaluate to elements from a different context
-- updated to @applitools/snippets@2.1.7 (from 2.1.4)
-- updated to @applitools/utils@1.2.3 (from 1.2.2)
-
-## 3.2.3 - 2021/8/13
-
-- remove base64 sanitizing
-
-## 3.2.2 - 2021/8/13
-
-- add `withStatusBar` capability property to take app and full app screenshots with status bar
-- fix ios web screenshots
-
-## 3.2.1 - 2021/8/8
-
-- no changes
-
-## 3.2.0 - 2021/8/7
-
-- change image processing order and improve general algorithm
-- updated to @applitools/utils@1.2.2 (from 1.2.1)
-
-## 3.1.0 - 2021/8/4
-
-- improve support of native devices
-- updated to @applitools/snippets@2.1.4 (from 2.1.3)
-- updated to @applitools/utils@1.2.1 (from 1.2.0)
-
-## 3.0.8 - 2021/5/24
-
-- updated to @applitools/utils@1.2.0 (from 1.1.3)
-
-## 3.0.7 - 2021/5/13
-
-- fixed image cropping algorithm to not copy data into a heap
-- optimized image rotation and image copping algorithms
-
-## 3.0.6 - 2021/5/11
-
-- updated to @applitools/utils@1.1.3 (from 1.0.1)
-
- ## 3.0.5 - 2021/2/18
-
-- fix bug with wrong stitching due to fractional offset 
-
-## 3.0.4 - 2021/2/15
-
-- fix scaling issue
-- handle firefox buggy versions
-
-## 3.0.3 - 2021/1/27
-
-- no changes
-- updated to @applitools/utils@1.0.1 (from 1.0.0)
-- updated to @applitools/utils@1.0.1 (from 1.0.0)
-## 3.0.2 - 2021/1/27
-
-- no changes
-## 3.0.1 - 2021/1/27
-- no changes
-## 3.0.0 - 2021/1/26
-
-- `crop`, `scale` and `rotate` now should be placed in `stabilization` object
-- rename `context` to `frames`
-- rename `isFully` to `fully`
-- integrate dom-capture
-- chore: add husky
-- fix bug when screenshots on iPad were taken with Safari navigation bar and iOS status bar
-
-## 2.1.1 - 2021/1/15
-
-- fix bug when `toPng` method returns empty buffer
-- fix bug when `toPng` method returns empty buffer
-## 2.1.0 - 2021/1/15
-
-- remove native module dependency (`sharp`)
-
-## 2.0.0 - 2021/1/7
-
-- return image location relative to viewport
-- return image location relative to viewport
-## 1.0.2 - 2020/12/29
-
-- fix saveScreenshot
-## 1.0.1 - 2020/12/1
-
-- round coordinate and size before image operations
-
-## 1.0.0 - 2020/12/1
-
-- Provide functionality to take screenshots (viewport and full) using Applitools driver wrapper