eyes_selenium 2.19.0 → 2.20.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e2e26548fc33b2ea447338b4d1143d74c192d8d4
-  data.tar.gz: a9890f6b2c2571b83e31c71977ad15a07b451726
+  metadata.gz: f2a03af87e605624000861188061a24e4a5a0c17
+  data.tar.gz: cc98a04b4d9d8e3f3b4a1a01b844f8cd5d2a6e50
 SHA512:
-  metadata.gz: 1b52dab2d476f1abd32fb5482945a676be560b94c64b795de45d600c75b5a46b8961f8d071826d9d048f0fdfa3613c079540a49b2518e6c3e3f9fec15f4d1eb4
-  data.tar.gz: 3dacfafa1340774a5377d74641263ca1b28abc6f249aed902ce066d3cd9f29b9303b1f9d6947192b2f54514dc58648c40a176f7a2bd1e02aff10a32131a5a9c6
+  metadata.gz: aa6cec3f2ff9eac384a78900262852c6801b22bddec37839594e2669aba0a5aa4d2c5331e43e172b34e32ecbd903fb61341e5841e923f31df553015607d7e380
+  data.tar.gz: 86df288b0117f79ab91dedf90b051190e94048e679b1486ba4ea5b7b347c06c39d0d9976d3765c8bdbb3b742700c7c9d8a0b39af875e28947ba503ecfa132f47

data/lib/applitools/selenium/browser.rb

@@ -161,16 +161,16 @@ module Applitools::Selenium
 
       # NOTE: this is required! Since when calculating the screenshot parts for full size, we use a screenshot size
       # which is a bit smaller (see comment below).
-      if @eyes.force_fullpage_screenshot && (screenshot.width < page_size.width || screenshot.height < page_size.height)
+      if screenshot.width < page_size.width || screenshot.height < page_size.height
         # We use a smaller size than the actual screenshot size in order to eliminate duplication of bottom scroll bars,
         # as well as footer-like elements with fixed position.
         max_scrollbar_size = @eyes.use_css_transition ? 0 : MAX_SCROLLBAR_SIZE
         height = [screenshot.height - (max_scrollbar_size * size_factor), MIN_SCREENSHOT_PART_HEIGHT * size_factor].max
         screenshot_part_size = Applitools::Base::Dimension.new(screenshot.width, height)
 
-        subregios = Applitools::Base::Region.new(0, 0, page_size.width,
+        sub_regions = Applitools::Base::Region.new(0, 0, page_size.width,
           page_size.height).subregions(screenshot_part_size)
-        parts = subregios.map do |screenshot_part|
+        parts = sub_regions.map do |screenshot_part|
           # Skip (0,0), as we already got the screenshot.
           if screenshot_part.left == 0 && screenshot_part.top == 0
             next Applitools::Base::ImagePosition.new(screenshot, Applitools::Base::Point::TOP_LEFT)

data/lib/applitools/selenium/driver.rb

@@ -79,7 +79,7 @@ module Applitools::Selenium
     #
     # Returns: +String+ A screenshot in the requested format.
     def screenshot_as(output_type, rotation = nil)
-      image = mobile_device? ? visible_screenshot : @browser.fullpage_screenshot
+      image = mobile_device? || !@eyes.force_fullpage_screenshot ? visible_screenshot : @browser.fullpage_screenshot
 
       Applitools::Selenium::Driver.normalize_image(self, image, rotation)
 
@@ -145,7 +145,7 @@ module Applitools::Selenium
       when 1
         arg = args.first
 
-        raise Argu mentError, "expected #{arg.inspect}:#{arg.class} to respond to #shift" unless arg.respond_to?(:shift)
+        raise ArgumentError, "expected #{arg.inspect}:#{arg.class} to respond to #shift" unless arg.respond_to?(:shift)
 
         # This will be a single-entry hash, so use #shift over #first or #[].
         arg.dup.shift.tap do |arr|

data/lib/applitools/version.rb

@@ -1,3 +1,3 @@
 module Applitools
-  VERSION = '2.19.0'.freeze
+  VERSION = '2.20.0'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: eyes_selenium
 version: !ruby/object:Gem::Version
-  version: 2.19.0
+  version: 2.20.0
 platform: ruby
 authors:
 - Applitools Team
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-06-02 00:00:00.000000000 Z
+date: 2015-06-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: selenium-webdriver