Skip to content

Commit

Permalink
[rb] be more exact about expected sizes of screenshots
Browse files Browse the repository at this point in the history
  • Loading branch information
titusfortner committed May 7, 2022
1 parent 0b9d238 commit 469970f
Showing 1 changed file with 9 additions and 2 deletions.
11 changes: 9 additions & 2 deletions rb/spec/integration/selenium/webdriver/takes_screenshot_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -81,13 +81,20 @@ def save_screenshot_and_assert(source, path)
end

it 'takes viewport screenshot by default' do
viewport_width = driver.execute_script("return window.innerWidth;") * 2
viewport_height = driver.execute_script("return window.innerHeight;") * 2

screenshot = driver.save_screenshot path
expect(File.read(screenshot)[0x10..0x18].unpack('NN').last).to be < 2600
expect(File.read(screenshot)[0x10..0x18].unpack('NN')).to eq([viewport_width, viewport_height])
end

it 'takes full page screenshot', exclusive: {browser: :firefox} do
viewport_width = driver.execute_script("return window.innerWidth;") * 2
viewport_height = driver.execute_script("return window.innerHeight;") * 2

screenshot = driver.save_screenshot path, full_page: true
expect(File.read(screenshot)[0x10..0x18].unpack('NN').last).to be > 2600
expect(File.read(screenshot)[0x10..0x18].unpack1('NN')).to eq viewport_width
expect(File.read(screenshot)[0x10..0x18].unpack('NN').last).to be > viewport_height
end

it 'does not take full page screenshot', exclude: {browser: :firefox} do
Expand Down

0 comments on commit 469970f

Please sign in to comment.