capybara 2.2.1 → 2.3.0

data/lib/capybara/spec/session/window/window_opened_by_spec.rb

@@ -0,0 +1,83 @@
+Capybara::SpecHelper.spec '#window_opened_by', requires: [:windows] do
+  before(:each) do
+    @window = @session.current_window
+    @session.visit('/with_windows')
+  end
+  after(:each) do
+    (@session.windows - [@window]).each do |w|
+      @session.switch_to_window w
+      w.close
+    end
+    @session.switch_to_window(@window)
+  end
+
+  let(:zero_windows_message) { "block passed to #window_opened_by opened 0 windows instead of 1" }
+  let(:two_windows_message) { "block passed to #window_opened_by opened 2 windows instead of 1" }
+
+  context 'with :wait option' do
+    it 'should raise error if value of :wait is less than timeout' do
+      Capybara.using_wait_time 1 do
+        expect do
+          @session.window_opened_by(wait: 0.4) do
+            @session.find(:css, '#openWindowWithTimeout').click
+          end
+        end.to raise_error(Capybara::WindowError, zero_windows_message)
+      end
+      @session.document.synchronize(2, errors: [Capybara::CapybaraError]) do
+        raise Capybara::CapybaraError if @session.windows.size != 2
+      end
+    end
+
+    it 'should find window if value of :wait is more than timeout' do
+      Capybara.using_wait_time 0.1 do
+        window = @session.window_opened_by(wait: 1.5) do
+          @session.find(:css, '#openWindowWithTimeout').click
+        end
+        expect(window).to be_instance_of(Capybara::Window)
+      end
+    end
+  end
+
+  context 'without :wait option' do
+    it 'should raise error if default_wait_time is less than timeout' do
+      Capybara.using_wait_time 0.4 do
+        expect do
+          @session.window_opened_by do
+            @session.find(:css, '#openWindowWithTimeout').click
+          end
+        end.to raise_error(Capybara::WindowError, zero_windows_message)
+      end
+      @session.document.synchronize(2, errors: [Capybara::CapybaraError]) do
+        raise Capybara::CapybaraError if @session.windows.size != 2
+      end
+    end
+
+    it 'should find window if default_wait_time is more than timeout' do
+      Capybara.using_wait_time 1.5 do
+        window = @session.window_opened_by do
+          @session.find(:css, '#openWindowWithTimeout').click
+        end
+        expect(window).to be_instance_of(Capybara::Window)
+      end
+    end
+  end
+
+  it 'should raise error when two windows have been opened by block' do
+    expect do
+      @session.window_opened_by do
+        @session.find(:css, '#openTwoWindows').click
+      end
+    end.to raise_error(Capybara::WindowError, two_windows_message)
+    @session.document.synchronize(2, errors: [Capybara::CapybaraError]) do
+      raise Capybara::CapybaraError if @session.windows.size != 3
+    end
+  end
+
+  it 'should raise error when no windows were opened by block' do
+    expect do
+      @session.window_opened_by do
+        @session.find(:css, '#doesNotOpenWindows').click
+      end
+    end.to raise_error(Capybara::WindowError, zero_windows_message)
+  end
+end

data/lib/capybara/spec/session/window/window_spec.rb

@@ -0,0 +1,141 @@
+Capybara::SpecHelper.spec Capybara::Window, requires: [:windows] do
+  before(:each) do
+    @window = @session.current_window
+    @session.visit('/with_windows')
+  end
+  after(:each) do
+    (@session.windows - [@window]).each do |w|
+      @session.switch_to_window w
+      w.close
+    end
+    @session.switch_to_window(@window)
+  end
+
+  describe '#exists?' do
+    before(:each) do
+      @other_window = @session.window_opened_by do
+        @session.find(:css, '#openWindow').click
+      end
+    end
+
+    it "should become false after window was closed" do
+      expect do
+        @session.switch_to_window @other_window
+        @other_window.close
+      end.to change { @other_window.exists? }.from(true).to(false)
+    end
+  end
+
+  describe '#closed?' do
+    it "should become true after window was closed" do
+      @other_window = @session.window_opened_by do
+        @session.find(:css, '#openWindow').click
+      end
+      expect do
+        @session.switch_to_window @other_window
+        @other_window.close
+      end.to change { @other_window.closed? }.from(false).to(true)
+    end
+  end
+
+  describe '#current?' do
+    before(:each) do
+      @other_window = @session.window_opened_by do
+        @session.find(:css, '#openWindow').click
+      end
+    end
+
+    it 'should become true after switching to window' do
+      expect do
+        @session.switch_to_window(@other_window)
+      end.to change { @other_window.current? }.from(false).to(true)
+    end
+
+    it 'should return false if window is closed' do
+      @session.switch_to_window(@other_window)
+      @other_window.close
+      expect(@other_window.current?).to eq(false)
+    end
+  end
+
+  describe '#close' do
+    before(:each) do
+      @other_window = @session.window_opened_by do
+        @session.find(:css, '#openWindow').click
+      end
+    end
+
+    it 'should switch to original window if invoked not for current window' do
+      expect(@session.windows.size).to eq(2)
+      expect(@session.current_window).to eq(@window)
+      @other_window.close
+      expect(@session.windows.size).to eq(1)
+      expect(@session.current_window).to eq(@window)
+    end
+
+    it 'should make subsequent invocations of other methods raise no_such_window_error if invoked for current window' do
+      @session.switch_to_window(@other_window)
+      expect(@session.current_window).to eq(@other_window)
+      @other_window.close
+      expect do
+        @session.find(:css, '#some_id')
+      end.to raise_error(@session.driver.no_such_window_error)
+      @session.switch_to_window(@window)
+    end
+  end
+
+  describe '#size' do
+    it 'should return size of whole window', requires: [:js] do
+      expect(@session.current_window.size).to eq @session.evaluate_script("[window.outerWidth, window.outerHeight];")
+    end
+
+    it 'should switch to original window if invoked not for current window' do
+      @other_window = @session.window_opened_by do
+        @session.find(:css, '#openWindow').click
+      end
+      size =
+        @session.within_window @other_window do
+          @session.evaluate_script("[window.outerWidth, window.outerHeight];")
+        end
+      expect(@other_window.size).to eq(size)
+      expect(@session.current_window).to eq(@window)
+    end
+  end
+
+  describe '#resize_to' do
+    it 'should be able to resize window', requires: [:js] do
+      width, height = @session.evaluate_script("[window.outerWidth, window.outerHeight];")
+      @session.current_window.resize_to(width-10, height-10)
+      expect(@session.evaluate_script("[window.outerWidth, window.outerHeight];")).to eq([width-10, height-10])
+    end
+
+    it 'should switch to original window if invoked not for current window' do
+      @other_window = @session.window_opened_by do
+        @session.find(:css, '#openWindow').click
+      end
+      @other_window.resize_to(400, 300)
+      expect(@session.current_window).to eq(@window)
+      expect(@other_window.size).to eq([400, 300])
+    end
+  end
+
+  describe '#maximize' do
+    it 'should be able to maximize window', requires: [:js] do
+      screen_width, screen_height = @session.evaluate_script("[window.screen.availWidth, window.screen.availHeight];")
+      window = @session.current_window
+      window.resize_to(screen_width-100, screen_height-100)
+      expect(@session.evaluate_script("[window.outerWidth, window.outerHeight];")).to eq([screen_width-100, screen_height-100])
+      window.maximize
+      expect(@session.evaluate_script("[window.outerWidth, window.outerHeight];")).to eq([screen_width, screen_height])
+    end
+
+    it 'should switch to original window if invoked not for current window' do
+      @other_window = @session.window_opened_by do
+        @session.find(:css, '#openWindow').click
+      end
+      @other_window.maximize
+      expect(@session.current_window).to eq(@window)
+      expect(@other_window.size).to eq(@session.evaluate_script("[window.screen.availWidth, window.screen.availHeight];"))
+    end
+  end
+end

data/lib/capybara/spec/session/window/windows_spec.rb

@@ -0,0 +1,31 @@
+Capybara::SpecHelper.spec '#windows', requires: [:windows] do
+  before(:each) do
+    @window = @session.current_window
+    @session.visit('/with_windows')
+    @session.find(:css, '#openTwoWindows').click
+
+    @session.document.synchronize(3, errors: [Capybara::CapybaraError]) do
+      raise Capybara::CapybaraError if @session.windows.size != 3
+    end
+  end
+  after(:each) do
+    (@session.windows - [@window]).each do |w|
+      @session.switch_to_window w
+      w.close
+    end
+    @session.switch_to_window(@window)
+  end
+
+  it 'should return objects of Capybara::Window class' do
+    expect(@session.windows.map { |window| window.instance_of?(Capybara::Window) }).to eq([true] * 3)
+  end
+
+  it 'should switchable windows' do
+    titles = @session.windows.map do |window|
+      @session.within_window(window) { @session.title }
+    end
+    expect(titles).to match_array([
+      'With Windows', 'Title of the first popup', 'Title of popup two'
+    ])
+  end
+end

data/lib/capybara/spec/session/window/within_window_spec.rb

@@ -0,0 +1,188 @@
+Capybara::SpecHelper.spec '#within_window', requires: [:windows] do
+  before(:each) do
+    @window = @session.current_window
+    @session.visit('/with_windows')
+    @session.find(:css, '#openTwoWindows').click
+
+    @session.document.synchronize(3, errors: [Capybara::CapybaraError]) do
+      raise Capybara::CapybaraError if @session.windows.size != 3
+    end
+  end
+  after(:each) do
+    (@session.windows - [@window]).each do |w|
+      @session.switch_to_window w
+      w.close
+    end
+    @session.switch_to_window(@window)
+  end
+
+  context "with an instance of Capybara::Window" do
+    it "should not invoke driver#switch_to_window when given current window" do
+      # switch_to_window is invoked in after hook
+      expect(@session.driver).to receive(:switch_to_window).exactly(3).times.and_call_original
+      @session.within_window @window do
+        expect(@session.title).to eq('With Windows')
+      end
+    end
+
+    it "should be able to switch to another window" do
+      window = (@session.windows - [@window]).first
+      expect(@session.driver).to receive(:switch_to_window).exactly(5).times.and_call_original
+      @session.within_window window do
+        expect(['Title of the first popup', 'Title of popup two']).to include(@session.title)
+      end
+      expect(@session.title).to eq('With Windows')
+    end
+
+    it "returns value from the block" do
+      window = (@session.windows - [@window]).first
+      value = @session.within_window window do
+                43252003274489856000
+              end
+      expect(value).to eq(43252003274489856000)
+    end
+
+    it "should switch back if exception was raised inside block" do
+      window = (@session.windows - [@window]).first
+      expect do
+        @session.within_window(window) do
+          @session.within 'html' do
+            raise 'some error'
+          end
+        end
+      end.to raise_error(StandardError, 'some error')
+      expect(@session.current_window).to eq(@window)
+      expect(@session).to have_css('#doesNotOpenWindows')
+      expect(@session.send(:scopes)).to eq([nil])
+    end
+
+    it "should leave correct scopes after execution in case of error" do
+      window = (@session.windows - [@window]).first
+      expect do
+        @session.within 'html' do
+          @session.within_window(window) {}
+        end
+      end.to raise_error(Capybara::ScopeError)
+      expect(@session.current_window).to eq(@window)
+      expect(@session).to have_css('#doesNotOpenWindows')
+      expect(@session.send(:scopes)).to eq([nil])
+    end
+
+    it 'should raise error if closed window was passed' do
+      other_window = (@session.windows - [@window]).first
+      @session.within_window other_window do
+        other_window.close
+      end
+      expect do
+        @session.within_window(other_window) do
+          raise 'should not be invoked'
+        end
+      end.to raise_error(@session.driver.no_such_window_error)
+      expect(@session.current_window).to eq(@window)
+      expect(@session).to have_css('#doesNotOpenWindows')
+      expect(@session.send(:scopes)).to eq([nil])
+    end
+  end
+
+  context "with lambda" do
+    it "should find the div in another window" do
+      @session.within_window(->{ @session.title == 'Title of the first popup'}) do
+        expect(@session).to have_css('#divInPopupOne')
+      end
+    end
+
+    it "should find divs in both windows" do
+      @session.within_window(->{ @session.title == 'Title of popup two'}) do
+        expect(@session).to have_css('#divInPopupTwo')
+      end
+      @session.within_window(->{ @session.title == 'Title of the first popup'}) do
+        expect(@session).to have_css('#divInPopupOne')
+      end
+      expect(@session.title).to eq('With Windows')
+    end
+
+    it "should raise error if window wasn't found" do
+      expect do
+        @session.within_window(->{ @session.title == 'Invalid title'}) do
+          expect(@session).to have_css('#divInPopupOne')
+        end
+      end.to raise_error(Capybara::WindowError, "Could not find a window matching block/lambda")
+      expect(@session.current_window).to eq(@window)
+      expect(@session).to have_css('#doesNotOpenWindows')
+      expect(@session.send(:scopes)).to eq([nil])
+    end
+
+    it "returns value from the block" do
+      value = @session.within_window(->{ @session.title == 'Title of popup two'}) do
+                42
+              end
+      expect(value).to eq(42)
+    end
+
+    it "should switch back if exception was raised inside block" do
+      expect do
+        @session.within_window(->{ @session.title == 'Title of popup two'}) do
+          raise 'some error'
+        end
+      end.to raise_error(StandardError, 'some error')
+      expect(@session.current_window).to eq(@window)
+      expect(@session.send(:scopes)).to eq([nil])
+    end
+  end
+
+  context "with string" do
+    it "should warn" do
+      expect(@session).to receive(:warn).with("DEPRECATION WARNING: Passing string argument "\
+        "to #within_window is deprecated. Pass window object or lambda. "\
+        "(called from #{__FILE__}:138)").and_call_original
+      @session.within_window('firstPopup') {}
+    end
+
+    it "should find window by handle" do
+      window = (@session.windows - [@window]).first
+      @session.within_window window.handle do
+        expect(['Title of the first popup', 'Title of popup two']).to include(@session.title)
+      end
+    end
+
+    it "should find the div in firstPopup" do
+      @session.within_window("firstPopup") do
+        expect(@session.find("//*[@id='divInPopupOne']").text).to eq 'This is the text of divInPopupOne'
+      end
+    end
+    it "should find the div in secondPopup" do
+      @session.within_window("secondPopup") do
+        expect(@session.find("//*[@id='divInPopupTwo']").text).to eq 'This is the text of divInPopupTwo'
+      end
+    end
+    it "should find the divs in both popups" do
+      @session.within_window("secondPopup") do
+        expect(@session.find("//*[@id='divInPopupTwo']").text).to eq 'This is the text of divInPopupTwo'
+      end
+      @session.within_window("firstPopup") do
+        expect(@session.find("//*[@id='divInPopupOne']").text).to eq 'This is the text of divInPopupOne'
+      end
+    end
+    it "should find the div in the main window after finding a div in a popup" do
+      @session.within_window("secondPopup") do
+        expect(@session.find("//*[@id='divInPopupTwo']").text).to eq 'This is the text of divInPopupTwo'
+      end
+      expect(@session.find("//*[@id='doesNotOpenWindows']").text).to eq 'Does not open windows'
+    end
+    it "should reset scope when switching windows" do
+      @session.within(:css, '#doesNotOpenWindows') do
+        @session.within_window("secondPopup") do
+          expect(@session.find("//*[@id='divInPopupTwo']").text).to eq 'This is the text of divInPopupTwo'
+        end
+      end
+    end
+    it "should switch back if exception was raised inside block" do
+      expect do
+        @session.within_window('secondPopup') do
+          raise 'some error'
+        end
+      end.to raise_error(StandardError, 'some error')
+      expect(@session.current_window).to eq(@window)
+    end
+  end
+end

data/lib/capybara/spec/session/within_frame_spec.rb

@@ -5,33 +5,33 @@ Capybara::SpecHelper.spec '#within_frame', :requires => [:frames] do
 
   it "should find the div in frameOne" do
     @session.within_frame("frameOne") do
-      @session.find("//*[@id='divInFrameOne']").text.should eql 'This is the text of divInFrameOne'
+      expect(@session.find("//*[@id='divInFrameOne']").text).to eql 'This is the text of divInFrameOne'
     end
   end
   it "should find the div in FrameTwo" do
     @session.within_frame("frameTwo") do
-      @session.find("//*[@id='divInFrameTwo']").text.should eql 'This is the text of divInFrameTwo'
+      expect(@session.find("//*[@id='divInFrameTwo']").text).to eql 'This is the text of divInFrameTwo'
     end
   end
   it "should find the text div in the main window after finding text in frameOne" do
     @session.within_frame("frameOne") do
-      @session.find("//*[@id='divInFrameOne']").text.should eql 'This is the text of divInFrameOne'
+      expect(@session.find("//*[@id='divInFrameOne']").text).to eql 'This is the text of divInFrameOne'
     end
-    @session.find("//*[@id='divInMainWindow']").text.should eql 'This is the text for divInMainWindow'
+    expect(@session.find("//*[@id='divInMainWindow']").text).to eql 'This is the text for divInMainWindow'
   end
   it "should find the text div in the main window after finding text in frameTwo" do
     @session.within_frame("frameTwo") do
-      @session.find("//*[@id='divInFrameTwo']").text.should eql 'This is the text of divInFrameTwo'
+      expect(@session.find("//*[@id='divInFrameTwo']").text).to eql 'This is the text of divInFrameTwo'
     end
-    @session.find("//*[@id='divInMainWindow']").text.should eql 'This is the text for divInMainWindow'
+    expect(@session.find("//*[@id='divInMainWindow']").text).to eql 'This is the text for divInMainWindow'
   end
   it "should return the result of executing the block" do
-    @session.within_frame("frameOne") { "return value" }.should eql "return value"
+    expect(@session.within_frame("frameOne") { "return value" }).to eql "return value"
   end
   it "should find the div given Element" do
     element = @session.find(:id, 'frameOne')
     @session.within_frame element do
-      @session.find("//*[@id='divInFrameOne']").text.should eql 'This is the text of divInFrameOne'
+      expect(@session.find("//*[@id='divInFrameOne']").text).to eql 'This is the text of divInFrameOne'
     end
   end
   it "should find multiple nested frames" do
@@ -45,7 +45,7 @@ Capybara::SpecHelper.spec '#within_frame', :requires => [:frames] do
   it "should reset scope when changing frames" do
     @session.within(:css, '#divInMainWindow') do
       @session.within_frame 'parentFrame' do
-        @session.has_selector?(:css, "iframe#childFrame").should be_true
+        expect(@session.has_selector?(:css, "iframe#childFrame")).to be true
       end
     end
   end