eyes_appium 3.17.15 → 3.17.20

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 65f533d59787d0642f7a120c1780064bae76e393d8660f7255551b97656346db
-  data.tar.gz: 5306b26b0f6a4389187a9917a79024c5da7ad8184a93c710dd324ebcb778c330
+  metadata.gz: 7f57d3b2870a62f52e8bf4d833419905c2bf4ae1b6cc6f0727cea1f7320125d0
+  data.tar.gz: d8a7bc1206792e52e6729f91c4f8af12f33d0227412a82d2b625bdff0886030c
 SHA512:
-  metadata.gz: 1b717b79a7b8255543222ad5e4cd2bdb5b881dded27e19f7b491ddd965a94b528aaa51f6b2fef3d8d8f68cb509b4827083c00d9156b0187aa0882c396d6fe535
-  data.tar.gz: c22e339a2476fea292f63a55de51498216870123ce34f9262a5643a61b25925832dfe7c8c129258d6600db5f9e79b0b29d83fcc1b132f9d44466cf9dfa9da2a3
+  metadata.gz: 4ef68f3c8f3a6d4f2fe4c9b649c07e32e8d8383dfcdcd6f0d3e42b97ad8bc64c9593806bac07b40acf176921e1abc99a55c8df5601a3dbf4a7d38c61713a0000
+  data.tar.gz: 70ff3025b0fbba86486b392e1905f3971a93d3362899353b11a6d036d4ec6f937e0feaf659d7ddcf9e9ff9105697e385823c49afcd0e897f0ca1712523762dc9

data/lib/applitools/appium/android_region_provider.rb

@@ -0,0 +1,24 @@
+# frozen_string_literal: true
+
+require_relative 'region_provider'
+module Applitools
+  module Appium
+    class AndroidRegionProvider < ::Applitools::Appium::RegionProvider
+      private
+
+      def convert_element_coordinates
+        Applitools::Region.from_location_size(eye_region.location, eye_region.size)
+      end
+
+      def convert_viewport_rect_coordinates
+        region  = viewport_rect
+        Applitools::Region.new(
+            region['left'],
+            region['top'],
+            region['width'],
+            region['height']
+        )
+      end
+    end
+  end
+end

data/lib/applitools/appium/eyes.rb

@@ -2,6 +2,7 @@
 
 class Applitools::Appium::Eyes < Applitools::Selenium::SeleniumEyes
   attr_accessor :status_bar_height
+
   def perform_driver_settings_for_appium_driver
     self.region_visibility_strategy = Applitools::Selenium::NopRegionVisibilityStrategy.new
     self.force_driver_resolution_as_viewport_size = true
@@ -51,7 +52,7 @@ class Applitools::Appium::Eyes < Applitools::Selenium::SeleniumEyes
 
     eyes_element = target_to_check.region_to_check.call(driver)
     self.eyes_element_to_check = eyes_element
-    region_provider = Applitools::Appium::RegionProvider.new(driver, eyes_element)
+    region_provider = region_provider_class.new(driver, eyes_element)
     match_data.read_target(target_to_check, driver)
 
     check_window_base(
@@ -119,16 +120,19 @@ class Applitools::Appium::Eyes < Applitools::Selenium::SeleniumEyes
 
   def viewport_screenshot
     logger.info 'Viewport screenshot requested...'
-
-    self.screenshot = screenshot_class.new(
-      Applitools::Screenshot.from_datastream(driver.screenshot_as(:png))
-    )
+    obtain_viewport_screenshot
   end
 
   def element_screenshot
     logger.info 'Element screenshot requested...'
+    obtain_viewport_screenshot
+  end
+
+  def obtain_viewport_screenshot
     self.screenshot = screenshot_class.new(
-      Applitools::Screenshot.from_datastream(driver.element_screenshot_as(eyes_element_to_check, :png))
+        Applitools::Screenshot.from_datastream(driver.screenshot_as(:png)),
+        status_bar_height: Applitools::Utils::EyesSeleniumUtils.status_bar_height(driver),
+        device_pixel_ratio: Applitools::Utils::EyesSeleniumUtils.device_pixel_ratio(driver)
     )
   end
 
@@ -137,4 +141,10 @@ class Applitools::Appium::Eyes < Applitools::Selenium::SeleniumEyes
     return Applitools::Appium::AndroidScreenshot if Applitools::Utils::EyesSeleniumUtils.android?(Applitools::Appium::Driver::AppiumLib)
     raise Applitools::EyesError, 'Unknown device type'
   end
+
+  def region_provider_class
+    return Applitools::Appium::IosRegionProvider if Applitools::Utils::EyesSeleniumUtils.ios?(Applitools::Appium::Driver::AppiumLib)
+    return Applitools::Appium::AndroidRegionProvider if Applitools::Utils::EyesSeleniumUtils.android?(Applitools::Appium::Driver::AppiumLib)
+    raise Applitools::EyesError, 'Unknown device type'
+  end
 end

data/lib/applitools/appium/ios_region_provider.rb

@@ -0,0 +1,25 @@
+# frozen_string_literal: true
+#
+require_relative 'region_provider'
+module Applitools
+  module Appium
+    class IosRegionProvider < ::Applitools::Appium::RegionProvider
+      private
+
+      def convert_element_coordinates
+        Applitools::Region.from_location_size(eye_region.location, eye_region.size).
+            scale_it!(scale_factor)
+      end
+
+      def convert_viewport_rect_coordinates
+        region  = viewport_rect
+        Applitools::Region.new(
+          region['left'],
+          region['top'],
+          region['width'],
+          region['height']
+        )
+      end
+    end
+  end
+end

data/lib/applitools/appium/region_provider.rb

@@ -3,23 +3,48 @@
 module Applitools
   module Appium
     class RegionProvider
-      attr_accessor :driver, :eye_region, :coordinate_type
+      attr_accessor :driver, :eye_region, :region_to_check
 
       def initialize(driver, eye_region)
         self.driver = driver
         self.eye_region = eye_region
+        self.region_to_check = Applitools::Region::EMPTY
+        convert_region_coordinates
       end
 
       def region
-        return Applitools::Region::EMPTY if
-            [::Selenium::WebDriver::Element, Applitools::Selenium::Element].include? eye_region.class
-        region = driver.session_capabilities['viewportRect']
-        Applitools::Region.new(
-          region['left'],
-          region['top'],
-          region['width'],
-          region['height']
-        )
+        region_to_check
+      end
+
+      def coordinate_type
+        nil
+      end
+
+      private
+
+      def viewport_rect
+        Applitools::Utils::EyesSeleniumUtils.viewport_rect(driver)
+      end
+
+      def convert_region_coordinates
+        self.region_to_check = case eye_region
+                               when ::Selenium::WebDriver::Element, Applitools::Selenium::Element
+                                  convert_element_coordinates
+                               else
+                                  convert_viewport_rect_coordinates
+                               end
+      end
+
+      def convert_element_coordinates
+        raise Applitools::AbstractMethodCalled.new(:convert_region_coordinates, 'Applitools::Appium::RegionProvider')
+      end
+
+      def convert_viewport_rect_coordinates
+        raise Applitools::AbstractMethodCalled.new(:convert_viewport_rect_coordinates, 'Applitools::Appium::RegionProvider')
+      end
+
+      def scale_factor
+        Applitools::Utils::EyesSeleniumUtils.device_pixel_ratio(driver)
       end
     end
   end

data/lib/applitools/appium/utils.rb

@@ -28,7 +28,7 @@ module Applitools::Appium
     def device_pixel_ratio(executor)
       session_info = session_capabilities(executor)
       return session_info['pixelRatio'].to_f if session_info['pixelRatio']
-      Applitools::Selenium::Eyes::UNKNOWN_DEVICE_PIXEL_RATIO
+      1
     end
 
     def status_bar_height(executor)
@@ -37,6 +37,17 @@ module Applitools::Appium
       0
     end
 
+    def viewport_rect(executor)
+      session_info = session_capabilities(executor)
+      return session_info['viewportRect'] if session_info['viewportRect']
+      {
+        'left' => 0,
+        'top' => 0,
+        'width' => 0,
+        'height' => 0
+      }
+    end
+
     def session_capabilities(executor)
        executor.session_capabilities if executor.respond_to? :session_capabilities
     end

data/lib/applitools/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: false
 
 module Applitools
-  VERSION = '3.17.15'.freeze
+  VERSION = '3.17.20'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: eyes_appium
 version: !ruby/object:Gem::Version
-  version: 3.17.15
+  version: 3.17.20
 platform: ruby
 authors:
 - Applitools Team
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-07-16 00:00:00.000000000 Z
+date: 2020-08-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: eyes_selenium
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 3.17.15
+        version: 3.17.20
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 3.17.15
+        version: 3.17.20
 - !ruby/object:Gem::Dependency
   name: appium_lib
   requirement: !ruby/object:Gem::Requirement
@@ -45,11 +45,13 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- lib/applitools/appium/android_region_provider.rb
 - lib/applitools/appium/android_screenshot.rb
 - lib/applitools/appium/driver.rb
 - lib/applitools/appium/eyes.rb
 - lib/applitools/appium/initialize_1.9.rb
 - lib/applitools/appium/initialize_2.0.rb
+- lib/applitools/appium/ios_region_provider.rb
 - lib/applitools/appium/ios_screenshot.rb
 - lib/applitools/appium/region_provider.rb
 - lib/applitools/appium/screenshot.rb