capybara 2.15.0 → 3.0.0

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

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+Capybara::SpecHelper.spec "#to_capybara_node" do
+  before do
+    @session.visit('/with_html')
+  end
+
+  class NodeWrapper
+    def initialize(element); @element = element end
+    def to_capybara_node(); @element end
+  end
+
+  it "should support have_xxx expectations" do
+    para = NodeWrapper.new(@session.find(:css, '#first'))
+    expect(para).to have_link("ullamco")
+  end
+
+  it "should support within" do
+    para = NodeWrapper.new(@session.find(:css, '#first'))
+    expect(@session).to have_css('#second')
+    @session.within(para) do
+      expect(@session).to have_link('ullamco')
+      expect(@session).not_to have_css('#second')
+    end
+  end
+
+  it "should generate correct errors" do
+    para = NodeWrapper.new(@session.find(:css, '#first'))
+    expect do
+      expect(para).to have_text("Header Class Test One")
+    end.to raise_error(/^expected to find text "Header Class Test One" in "Lore/)
+    expect do
+      expect(para).to have_css('#second')
+    end.to raise_error(/^expected to find visible css "#second" within #<Capybara::Node::Element/)
+  end
+end

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#refresh' do
   it "reload the page" do
     @session.visit('/form')
@@ -17,12 +18,15 @@ Capybara::SpecHelper.spec '#refresh' do
   end
 
   it "it reposts" do
+    if marionette?(@session) || edge?(@session) || ie?(@session)
+      skip "Firefox and Edge insist on prompting without providing a way to suppress"
+    end
     @session.visit('/form')
     @session.select('Sweden', from: 'form_region')
     @session.click_button('awesome')
-    expect {
+    expect do
       @session.refresh
       sleep 2
-    }.to change{ extract_results(@session)['post_count'] }.by(1)
+    end.to change { extract_results(@session)['post_count'] }.by(1)
   end
 end

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#reset_session!' do
   it "removes cookies" do
     @session.visit('/set_cookie')
@@ -62,6 +63,24 @@ Capybara::SpecHelper.spec '#reset_session!' do
     expect(@session.current_path).to eq("/")
   end
 
+  it "closes extra windows", requires: [:windows] do
+    @session.visit('/with_html')
+    @session.open_new_window
+    @session.open_new_window
+    expect(@session.windows.size).to eq 3
+    @session.reset_session!
+    expect(@session.windows.size).to eq 1
+  end
+
+  it "closes extra windows when not on the first window", requires: [:windows] do
+    @session.visit('/with_html')
+    @session.switch_to_window(@session.open_new_window)
+    @session.open_new_window
+    expect(@session.windows.size).to eq 3
+    @session.reset_session!
+    expect(@session.windows.size).to eq 1
+  end
+
   context "When reuse_server == false" do
     before do
       @reuse_server = Capybara.reuse_server

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#status_code' do
   it "should return response codes", requires: [:status_code] do
     @session.visit('/with_simple_html')

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'launchy'
 
 Capybara::SpecHelper.spec '#save_and_open_page' do

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'launchy'
 
 Capybara::SpecHelper.spec '#save_and_open_screenshot' do
@@ -13,8 +14,8 @@ Capybara::SpecHelper.spec '#save_and_open_screenshot' do
 
     @session.save_and_open_screenshot
 
-    expect(@session.driver).to have_received(:save_screenshot).
-      with(expected_file_regex, {})
+    expect(@session.driver).to have_received(:save_screenshot)
+      .with(expected_file_regex, {})
     expect(Launchy).to have_received(:open).with(expected_file_regex)
   end
 
@@ -25,8 +26,8 @@ Capybara::SpecHelper.spec '#save_and_open_screenshot' do
 
     @session.save_and_open_screenshot(custom_path)
 
-    expect(@session.driver).to have_received(:save_screenshot).
-      with(/#{custom_path}$/, {})
+    expect(@session.driver).to have_received(:save_screenshot)
+      .with(/#{custom_path}$/, {})
     expect(Launchy).to have_received(:open).with(/#{custom_path}$/)
   end
 
@@ -34,14 +35,8 @@ Capybara::SpecHelper.spec '#save_and_open_screenshot' do
     it 'prints out a correct warning message', requires: [:screenshot] do
       file_path = File.join(Dir.tmpdir, 'test.png')
       allow(@session).to receive(:require).with('launchy').and_raise(LoadError)
-      allow(@session).to receive(:warn)
-
+      expect(@session).to receive(:warn).with("File saved to #{file_path}.\nPlease install the launchy gem to open the file automatically.")
       @session.save_and_open_screenshot(file_path)
-
-      expect(@session).to have_received(:warn).
-        with("File saved to #{file_path}.")
-      expect(@session).to have_received(:warn).
-        with('Please install the launchy gem to open the file automatically.')
     end
   end
 end

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#save_page' do
   let(:alternative_path) { File.join(Dir.pwd, "save_and_open_page_tmp") }
   before do
@@ -6,7 +7,6 @@ Capybara::SpecHelper.spec '#save_page' do
   end
 
   after do
-    Capybara.save_and_open_page_path = nil
     Capybara.save_path = nil
     Dir.glob("capybara-*.html").each do |file|
       FileUtils.rm(file)
@@ -104,20 +104,4 @@ Capybara::SpecHelper.spec '#save_page' do
       expect(result).to include("Bar")
     end
   end
-
-  context "with deprecated save_and_open_page_path" do
-    it "can store files in a specified directory" do
-      Capybara.save_and_open_page_path = alternative_path
-      @session.save_page
-      path = Dir.glob(alternative_path + "/capybara-*.html").first
-      expect(File.read(path)).to include("Another World")
-    end
-
-    it "returns an absolute path in given directory" do
-      Capybara.save_and_open_page_path = alternative_path
-      result = @session.save_page
-      path = File.expand_path(Dir.glob(alternative_path + "/capybara-*.html").first, alternative_path)
-      expect(result).to eq(path)
-    end
-  end
 end

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#save_screenshot', requires: [:screenshot] do
   let(:alternative_path) { File.join(Dir.pwd, "save_screenshot_tmp") }
   before do
@@ -6,7 +7,6 @@ Capybara::SpecHelper.spec '#save_screenshot', requires: [:screenshot] do
   end
 
   after do
-    Capybara.save_and_open_page_path = nil
     Capybara.save_path = nil
     FileUtils.rm_rf alternative_path
   end
@@ -16,7 +16,7 @@ Capybara::SpecHelper.spec '#save_screenshot', requires: [:screenshot] do
 
     @session.save_screenshot
 
-    regexp = Regexp.new(File.expand_path('capybara-\d+\.png'))
+    regexp = Regexp.new(File.join(Dir.pwd, 'capybara-\d+\.png'))
     expect(@session.driver).to have_received(:save_screenshot).with(regexp, {})
   end
 
@@ -36,7 +36,7 @@ Capybara::SpecHelper.spec '#save_screenshot', requires: [:screenshot] do
 
       @session.save_screenshot
 
-      regexp = Regexp.new(File.expand_path('capybara-\d+\.png', alternative_path))
+      regexp = Regexp.new(File.join(alternative_path, 'capybara-\d+\.png'))
       expect(@session.driver).to have_received(:save_screenshot).with(regexp, {})
     end
 

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec "#select" do
   before do
     @session.visit('/form')
@@ -13,24 +14,24 @@ Capybara::SpecHelper.spec "#select" do
     expect(@session.find_field('Title').value).to eq('Miss')
   end
 
-  it "should allow selecting options where there are inexact matches" do
+  it "should allow selecting exact options where there are inexact matches", :exact_false do
     @session.select("Mr", from: 'Title')
     expect(@session.find_field('Title').value).to eq('Mr')
   end
 
-  it "should allow selecting options where they are the only inexact match" do
+  it "should allow selecting options where they are the only inexact match", :exact_false do
     @session.select("Mis", from: 'Title')
     expect(@session.find_field('Title').value).to eq('Miss')
   end
 
-  it "should not allow selecting options where they are the only inexact match if `Capybara.exact_options = true`" do
-    Capybara.exact_options = true
+  it "should not allow selecting options where they are the only inexact match if `exact: true` is specified" do
+    sel = @session.find(:select, 'Title')
     expect do
-      @session.select("Mis", from: 'Title')
+      sel.select("Mis", exact: true)
     end.to raise_error(Capybara::ElementNotFound)
   end
 
-  it "should not allow selecting an option if the match is ambiguous" do
+  it "should not allow selecting an option if the match is ambiguous", :exact_false do
     expect do
       @session.select("M", from: 'Title')
     end.to raise_error(Capybara::Ambiguous)
@@ -79,7 +80,7 @@ Capybara::SpecHelper.spec "#select" do
   end
 
   it "casts to string" do
-    @session.select(:"Miss", from: :'Title')
+    @session.select(:Miss, from: :Title)
     expect(@session.find_field('Title').value).to eq('Miss')
   end
 
@@ -94,7 +95,7 @@ Capybara::SpecHelper.spec "#select" do
 
   context "with an option that doesn't exist" do
     it "should raise an error" do
-      msg = "Unable to find visible option \"Does not Exist\""
+      msg = /^Unable to find visible option "Does not Exist" within/
       expect do
         @session.select('Does not Exist', from: 'form_locale')
       end.to raise_error(Capybara::ElementNotFound, msg)
@@ -123,32 +124,32 @@ Capybara::SpecHelper.spec "#select" do
 
   context "with multiple select" do
     it "should return an empty value" do
-      expect(@session.find_field('Language').value).to eq([])
+      expect(@session.find_field('Languages').value).to eq([])
     end
 
     it "should return value of the selected options" do
-      @session.select("Ruby",       from: 'Language')
-      @session.select("Javascript", from: 'Language')
-      expect(@session.find_field('Language').value).to include('Ruby', 'Javascript')
+      @session.select("Ruby",       from: 'Languages')
+      @session.select("Javascript", from: 'Languages')
+      expect(@session.find_field('Languages').value).to include('Ruby', 'Javascript')
     end
 
     it "should select one option" do
-      @session.select("Ruby", from: 'Language')
+      @session.select("Ruby", from: 'Languages')
       @session.click_button('awesome')
       expect(extract_results(@session)['languages']).to eq(['Ruby'])
     end
 
     it "should select multiple options" do
-      @session.select("Ruby",       from: 'Language')
-      @session.select("Javascript", from: 'Language')
+      @session.select("Ruby",       from: 'Languages')
+      @session.select("Javascript", from: 'Languages')
       @session.click_button('awesome')
       expect(extract_results(@session)['languages']).to include('Ruby', 'Javascript')
     end
 
     it "should remain selected if already selected" do
-      @session.select("Ruby",       from: 'Language')
-      @session.select("Javascript", from: 'Language')
-      @session.select("Ruby",       from: 'Language')
+      @session.select("Ruby",       from: 'Languages')
+      @session.select("Javascript", from: 'Languages')
+      @session.select("Ruby",       from: 'Languages')
       @session.click_button('awesome')
       expect(extract_results(@session)['languages']).to include('Ruby', 'Javascript')
     end
@@ -161,7 +162,7 @@ Capybara::SpecHelper.spec "#select" do
   context "with :exact option" do
     context "when `false`" do
       it "can match select box approximately" do
-        @session.select("Finish", from: "Loc", exact:  false)
+        @session.select("Finish", from: "Loc", exact: false)
         @session.click_button("awesome")
         expect(extract_results(@session)["locale"]).to eq("fi")
       end
@@ -194,7 +195,7 @@ Capybara::SpecHelper.spec "#select" do
 
       it "can match option approximately when :from not given" do
         expect do
-          @session.select("made-up language", exact:  true)
+          @session.select("made-up language", exact: true)
         end.to raise_error(Capybara::ElementNotFound)
       end
     end

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

@@ -59,7 +59,7 @@ Capybara::SpecHelper.spec Capybara::Selector do
     it "can find by class" do
       expect(@session.find(:field, class: 'confusion-checkbox')['id']).to eq 'confusion_checkbox'
       expect(@session).to have_selector(:field, class: 'confusion', count: 3)
-      expect(@session.find(:field, class: ['confusion','confusion-textarea'])['id']).to eq 'confusion_textarea'
+      expect(@session.find(:field, class: ['confusion', 'confusion-textarea'])['id']).to eq 'confusion_textarea'
     end
   end
-end
+end

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#sibling' do
   before do
     @session.visit('/with_html')
@@ -42,7 +43,6 @@ Capybara::SpecHelper.spec '#sibling' do
     end
   end
 
-
   it "should raise ElementNotFound with a useful default message if nothing was found" do
     el = @session.find(:css, '#child')
     expect do

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#text' do
   it "should print the text of the page" do
     @session.visit('/with_simple_html')
@@ -50,9 +51,22 @@ Capybara::SpecHelper.spec '#text' do
     after { Capybara.default_selector = :xpath }
   end
 
-  it "should strip whitespace" do
+  it "should be correctly normalized when visible" do
+    @session.visit('/with_html')
+    el = @session.find(:css, '#normalized')
+    expect(el.text).to eq "Some text\nMore text\nAnd more text\nEven more    text on multiple lines"
+  end
+
+  it "should be a textContent with irrelevant whitespace collapsed when non-visible" do
+    @session.visit('/with_html')
+    el = @session.find(:css, '#non_visible_normalized', visible: false)
+    expect(el.text(:all)).to eq "Some textMore text And more text Even more    text on multiple lines"
+  end
+
+  it "should strip correctly" do
     @session.visit('/with_html')
-    @session.find(:css, '#second')
-    expect(@session.find(:css, '#second').text).to match(/\ADuis aute .* text with whitespace .* est laborum\.\z/)
+    el = @session.find(:css, '#ws')
+    expect(el.text).to eq " "
+    expect(el.text(:all)).to eq " "
   end
 end

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

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
-Capybara::SpecHelper.spec '#title' do
 
+Capybara::SpecHelper.spec '#title' do
   it "should get the title of the page" do
     @session.visit('/with_title')
     expect(@session.title).to eq('Test Title')
@@ -14,4 +14,14 @@ Capybara::SpecHelper.spec '#title' do
     end
     after { Capybara.default_selector = :xpath }
   end
+
+  context "within iframe", requires: [:frames] do
+    it "should get the title of the top level browsing context" do
+      @session.visit('/within_frames')
+      expect(@session.title).to eq('With Frames')
+      @session.within_frame('frameOne') do
+        expect(@session.title).to eq('With Frames')
+      end
+    end
+  end
 end

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec "#uncheck" do
   before do
     @session.visit('/form')
@@ -19,7 +20,7 @@ Capybara::SpecHelper.spec "#uncheck" do
   end
 
   it "casts to string" do
-    @session.uncheck(:"form_pets_hamster")
+    @session.uncheck(:form_pets_hamster)
     @session.click_button('awesome')
     expect(extract_results(@session)['pets']).to include('dog')
     expect(extract_results(@session)['pets']).not_to include('hamster')
@@ -69,11 +70,11 @@ Capybara::SpecHelper.spec "#uncheck" do
       end
 
       it "should raise original error when no label available" do
-        expect { @session.uncheck('form_cars_ariel') }.to raise_error(Capybara::ElementNotFound, 'Unable to find visible checkbox "form_cars_ariel" that is not disabled')
+        expect { @session.uncheck('form_cars_porsche') }.to raise_error(Capybara::ElementNotFound, 'Unable to find visible checkbox "form_cars_porsche" that is not disabled')
       end
 
       it "should raise error if not allowed to click label" do
-        expect{@session.uncheck('form_cars_jaguar', allow_label_click: false)}.to raise_error(Capybara::ElementNotFound, 'Unable to find visible checkbox "form_cars_jaguar" that is not disabled')
+        expect { @session.uncheck('form_cars_jaguar', allow_label_click: false) }.to raise_error(Capybara::ElementNotFound, 'Unable to find visible checkbox "form_cars_jaguar" that is not disabled')
       end
     end
   end

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec "#unselect" do
   before do
     @session.visit('/form')
@@ -40,7 +41,7 @@ Capybara::SpecHelper.spec "#unselect" do
     end
 
     it "casts to string" do
-      @session.unselect(:"Briefs", from: :'Underwear')
+      @session.unselect(:Briefs, from: :Underwear)
       @session.click_button('awesome')
       expect(extract_results(@session)['underwear']).to include('Commando', 'Boxerbriefs')
       expect(extract_results(@session)['underwear']).not_to include('Briefs')
@@ -64,7 +65,7 @@ Capybara::SpecHelper.spec "#unselect" do
 
   context "with an option that doesn't exist" do
     it "should raise an error" do
-      msg = 'Unable to find visible option "Does not Exist"'
+      msg = /^Unable to find visible option "Does not Exist" within/
       expect do
         @session.unselect('Does not Exist', from: 'form_underwear')
       end.to raise_error(Capybara::ElementNotFound, msg)
@@ -74,19 +75,19 @@ Capybara::SpecHelper.spec "#unselect" do
   context "with :exact option" do
     context "when `false`" do
       it "can match select box approximately" do
-        @session.unselect("Boxerbriefs", from: "Under", exact:  false)
+        @session.unselect("Boxerbriefs", from: "Under", exact: false)
         @session.click_button("awesome")
         expect(extract_results(@session)["underwear"]).not_to include("Boxerbriefs")
       end
 
       it "can match option approximately" do
-        @session.unselect("Boxerbr", from: "Underwear", exact:  false)
+        @session.unselect("Boxerbr", from: "Underwear", exact: false)
         @session.click_button("awesome")
         expect(extract_results(@session)["underwear"]).not_to include("Boxerbriefs")
       end
 
       it "can match option approximately when :from not given" do
-        @session.unselect("Boxerbr", exact:  false)
+        @session.unselect("Boxerbr", exact: false)
         @session.click_button("awesome")
         expect(extract_results(@session)["underwear"]).not_to include("Boxerbriefs")
       end
@@ -107,7 +108,7 @@ Capybara::SpecHelper.spec "#unselect" do
 
       it "can match option approximately when :from not given" do
         expect do
-          @session.unselect("Boxerbr", exact:  true)
+          @session.unselect("Boxerbr", exact: true)
         end.to raise_error(Capybara::ElementNotFound)
       end
     end

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#visit' do
   it "should fetch a response from the driver with a relative url" do
     @session.visit('/')
@@ -27,6 +28,14 @@ Capybara::SpecHelper.spec '#visit' do
     expect(@session).to have_content('Hello world!')
   end
 
+  it "should fetch a response when sequentially visiting same destination with a target" do
+    skip "Chrome/chromedriver has an issue visiting URL with target after visiting same URL without target" if chrome_lt?(65.0, @session)
+    @session.visit('/form')
+    expect(@session).to have_css('#form_title')
+    @session.visit('/form#form_title')
+    expect(@session).to have_css('#form_title')
+  end
+
   it "raises any errors caught inside the server", requires: [:server] do
     quietly { @session.visit("/error") }
     expect do
@@ -40,7 +49,6 @@ Capybara::SpecHelper.spec '#visit' do
   end
 
   context "when Capybara.always_include_port is true" do
-
     let(:root_uri) do
       @session.visit('/')
       URI.parse(@session.current_url)
@@ -63,6 +71,55 @@ Capybara::SpecHelper.spec '#visit' do
       expect(URI.parse(@session.current_url).port).to eq(root_uri.port)
       expect(@session).to have_content('Another World')
     end
+
+    it "should add the server port to a visited url if no port specified", requires: [:server] do
+      expect(@session.driver).to receive(:visit).with("http://www.example.com:#{@session.server.port}")
+      @session.visit("http://www.example.com")
+    end
+
+    it "should not override the visit specified port even if default for scheme", requires: [:server] do
+      expect(@session.driver).to receive(:visit).with("http://www.example.com:80")
+      @session.visit('http://www.example.com:80')
+    end
+
+    it "should give preference to app_host port if specified", requires: [:server] do
+      Capybara.app_host = "http://www.example.com:6666"
+      expect(@session.driver).to receive(:visit).with("http://www.example.com:6666/random")
+      @session.visit('/random')
+    end
+
+    it "shouldn't override port if no server", requires: [:server] do
+      session = Capybara::Session.new(@session.mode, nil)
+      expect(session.driver).to receive(:visit).with("http://www.google.com")
+      session.visit("http://www.google.com")
+    end
+
+    it "shouldn't override port if no server but app_host is set", requires: [:server] do
+      session = Capybara::Session.new(@session.mode, nil)
+      Capybara.app_host = "http://www.example.com:6666"
+      expect(session.driver).to receive(:visit).with("http://www.google.com")
+      session.visit("http://www.google.com")
+    end
+  end
+
+  context "when Capybara.always_include_port is false" do
+    before(:each) do
+      Capybara.always_include_port = false
+    end
+
+    it "shouldn't overwrite port if app_host is set", requires: [:server] do
+      session = Capybara::Session.new(@session.mode, nil)
+      Capybara.app_host = "http://www.example.com:6666"
+      expect(session.driver).to receive(:visit).with("http://www.google.com")
+      session.visit("http://www.google.com")
+    end
+
+    it "shouldn't overwrite port if port specfified", requires: [:server] do
+      session = Capybara::Session.new(@session.mode, nil)
+      Capybara.app_host = "http://www.example.com:6666"
+      expect(session.driver).to receive(:visit).with("http://www.google.com:99")
+      session.visit("http://www.google.com:99")
+    end
   end
 
   context "without a server", requires: [:server] do
@@ -96,8 +153,13 @@ Capybara::SpecHelper.spec '#visit' do
       @session.visit('/times')
       expect(@session).to have_content('redirection complete')
     end
-  end
 
+    it "should work if `app_host` has a trailing /", requires: [:server] do
+      Capybara.app_host = "http://#{@session.server.host}:#{@session.server.port}/"
+      @session.visit('/')
+      expect(@session).to have_content('Hello world!')
+    end
+  end
 
   it "should send no referer when visiting a page" do
     @session.visit '/get_referer'
@@ -133,5 +195,4 @@ Capybara::SpecHelper.spec '#visit' do
     @session.visit('/get_cookie')
     expect(@session).to have_content('root cookie')
   end
-
 end

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

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
-Capybara::SpecHelper.spec '#become_closed', requires: [:windows, :js] do
+
+Capybara::SpecHelper.spec '#become_closed', requires: %i[windows js] do
   before(:each) do
     @window = @session.current_window
     @session.visit('/with_windows')

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#current_window', requires: [:windows] do
   before(:each) do
     @window = @session.current_window

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#open_new_window', requires: [:windows] do
   before(:each) do
     @window = @session.current_window

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#switch_to_window', requires: [:windows] do
   before(:each) do
     @window = @session.current_window
@@ -121,7 +122,7 @@ Capybara::SpecHelper.spec '#switch_to_window', requires: [:windows] do
   it "should wait for window to appear" do
     @session.find(:css, '#openWindowWithTimeout').click
     expect do
-      @session.switch_to_window(wait: 5) { @session.title == 'Title of the first popup'}
+      @session.switch_to_window(wait: 5) { @session.title == 'Title of the first popup' }
     end.not_to raise_error
   end
 end

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#window_opened_by', requires: [:windows] do
   before(:each) do
     @window = @session.current_window
@@ -20,7 +21,7 @@ Capybara::SpecHelper.spec '#window_opened_by', requires: [:windows] do
     it 'should raise error if value of :wait is less than timeout' do
       # So large value is used as `driver.window_handles` takes up to 800 ms on Travis
       Capybara.using_wait_time 2 do
-        button=@session.find(:css, '#openWindowWithLongerTimeout')
+        button = @session.find(:css, '#openWindowWithLongerTimeout')
         expect do
           @session.window_opened_by(wait: 0.3) do
             button.click