add spec: 'resizes to 'normal' from 'fullscreen' window state'

Author: Mifrill

lib/ferrum/page.rb

@@ -90,14 +90,12 @@ def close
     end
 
     def resize(width: nil, height: nil, fullscreen: false)
-      @window_id = @browser.command("Browser.getWindowForTarget", targetId: @target_id)["windowId"]
-
       if fullscreen
         width, height = document_size
-        @browser.command("Browser.setWindowBounds", windowId: @window_id, bounds: { windowState: "fullscreen" })
+        set_window_bounds(windowState: "fullscreen")
       else
-        @browser.command("Browser.setWindowBounds", windowId: @window_id, bounds: { windowState: "normal" })
-        @browser.command("Browser.setWindowBounds", windowId: @window_id, bounds: { width: width, height: height })
+        set_window_bounds(windowState: "normal")
+        set_window_bounds(width: width, height: height)
       end
 
       command("Emulation.setDeviceMetricsOverride", slowmoable: true,
@@ -137,6 +135,14 @@ def bypass_csp(value = true)
       enabled
     end
 
+    def window_id
+      @browser.command("Browser.getWindowForTarget", targetId: @target_id)["windowId"]
+    end
+
+    def set_window_bounds(bounds = {})
+      @browser.command("Browser.setWindowBounds", windowId: window_id, bounds: bounds)
+    end
+
     def command(method, wait: 0, slowmoable: false, **params)
       iteration = @event.reset if wait > 0
       sleep(@browser.slowmo) if slowmoable && @browser.slowmo > 0

spec/browser_spec.rb

@@ -99,13 +99,11 @@ module Ferrum
 
     context 'fullscreen' do
       shared_examples 'resize viewport by fullscreen' do
-        it "allows the viewport to be resized to fullscreen and back" do
+        it "allows the viewport to be resized by fullscreen" do
           expect(browser.viewport_size).to eq([1024, 768])
           browser.go_to(path)
           browser.resize(fullscreen: true)
           expect(browser.viewport_size).to eq(viewport_size)
-          browser.resize(width: 200, height: 400)
-          expect(browser.viewport_size).to eq([200, 400])
         end
       end
 
@@ -118,6 +116,13 @@ module Ferrum
         let(:path) { "/ferrum/custom_html_size_100%" }
         let(:viewport_size) { [1272, 1008] }
       end
+
+      it "resizes to 'normal' from 'fullscreen' window state" do
+        browser.go_to(path)
+        browser.resize(fullscreen: true)
+        browser.resize(width: 200, height: 400)
+        expect(browser.viewport_size).to eq([200, 400])
+      end
     end
 
     it "allows the page to be scrolled" do