capybara 3.3.1 → 3.4.0

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

@@ -5,17 +5,17 @@ class NodeWrapper
   def to_capybara_node(); @element end
 end
 
-Capybara::SpecHelper.spec "#to_capybara_node" do
+Capybara::SpecHelper.spec '#to_capybara_node' do
   before do
     @session.visit('/with_html')
   end
 
-  it "should support have_xxx expectations" do
+  it 'should support have_xxx expectations' do
     para = NodeWrapper.new(@session.find(:css, '#first'))
-    expect(para).to have_link("ullamco")
+    expect(para).to have_link('ullamco')
   end
 
-  it "should support within" do
+  it 'should support within' do
     para = NodeWrapper.new(@session.find(:css, '#first'))
     expect(@session).to have_css('#second')
     @session.within(para) do
@@ -24,13 +24,16 @@ Capybara::SpecHelper.spec "#to_capybara_node" do
     end
   end
 
-  it "should generate correct errors" do
+  it 'should generate correct errors' do
     para = NodeWrapper.new(@session.find(:css, '#first'))
     expect do
-      expect(para).to have_text("Header Class Test One")
+      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.to raise_error(/^expected to find css "#second" within #<Capybara::Node::Element/)
+    expect do
+      expect(para).to have_link(href: %r{/without_simple_html})
+    end.to raise_error(%r{^expected to find visible link nil with href matching /\\/without_simple_html/ within #<Capybara::Node::Element})
   end
 end

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

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 Capybara::SpecHelper.spec '#refresh' do
-  it "reload the page" do
+  it 'reload the page' do
     @session.visit('/form')
     expect(@session).to have_select('form_locale', selected: 'English')
     @session.select('Swedish', from: 'form_locale')
@@ -10,17 +10,14 @@ Capybara::SpecHelper.spec '#refresh' do
     expect(@session).to have_select('form_locale', selected: 'English')
   end
 
-  it "raises any errors caught inside the server", requires: [:server] do
-    quietly { @session.visit("/error") }
+  it 'raises any errors caught inside the server', requires: [:server] do
+    quietly { @session.visit('/error') }
     expect do
       @session.refresh
     end.to raise_error(TestApp::TestAppError)
   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
+  it 'it reposts' do
     @session.visit('/form')
     @session.select('Sweden', from: 'form_region')
     @session.click_button('awesome')

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

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 Capybara::SpecHelper.spec '#reset_session!' do
-  it "removes cookies" do
+  it 'removes cookies' do
     @session.visit('/set_cookie')
     @session.visit('/get_cookie')
     expect(@session).to have_content('test_cookie')
@@ -11,7 +11,7 @@ Capybara::SpecHelper.spec '#reset_session!' do
     expect(@session.body).not_to include('test_cookie')
   end
 
-  it "resets current url, host, path" do
+  it 'resets current url, host, path' do
     @session.visit '/foo'
     expect(@session.current_url).not_to be_empty
     expect(@session.current_host).not_to be_empty
@@ -23,7 +23,7 @@ Capybara::SpecHelper.spec '#reset_session!' do
     expect(@session.current_host).to be_nil
   end
 
-  it "resets page body" do
+  it 'resets page body' do
     @session.visit('/with_html')
     expect(@session).to have_content('This is a test')
     expect(@session.find('.//h1').text).to include('This is a test')
@@ -33,37 +33,37 @@ Capybara::SpecHelper.spec '#reset_session!' do
     expect(@session).to have_no_selector('.//h1')
   end
 
-  it "is synchronous" do
-    @session.visit("/with_slow_unload")
+  it 'is synchronous' do
+    @session.visit('/with_slow_unload')
     expect(@session).to have_selector(:css, 'div')
     @session.reset_session!
-    expect(@session).to have_no_selector :xpath, "/html/body/*", wait: false
+    expect(@session).to have_no_selector :xpath, '/html/body/*', wait: false
   end
 
-  it "handles modals during unload", requires: [:modals] do
+  it 'handles modals during unload', requires: [:modals] do
     @session.visit('/with_unload_alert')
     expect(@session).to have_selector(:css, 'div')
     expect { @session.reset_session! }.not_to raise_error
-    expect(@session).to have_no_selector :xpath, "/html/body/*", wait: false
+    expect(@session).to have_no_selector :xpath, '/html/body/*', wait: false
   end
 
-  it "handles already open modals", requires: [:modals] do
+  it 'handles already open modals', requires: [:modals] do
     @session.visit('/with_unload_alert')
     @session.click_link('Go away')
     expect { @session.reset_session! }.not_to raise_error
-    expect(@session).to have_no_selector :xpath, "/html/body/*", wait: false
+    expect(@session).to have_no_selector :xpath, '/html/body/*', wait: false
   end
 
-  it "raises any standard errors caught inside the server", requires: [:server] do
-    quietly { @session.visit("/error") }
+  it 'raises any standard errors caught inside the server', requires: [:server] do
+    quietly { @session.visit('/error') }
     expect do
       @session.reset_session!
     end.to raise_error(TestApp::TestAppError)
-    @session.visit("/")
-    expect(@session.current_path).to eq("/")
+    @session.visit('/')
+    expect(@session.current_path).to eq('/')
   end
 
-  it "closes extra windows", requires: [:windows] do
+  it 'closes extra windows', requires: [:windows] do
     @session.visit('/with_html')
     @session.open_new_window
     @session.open_new_window
@@ -72,7 +72,7 @@ Capybara::SpecHelper.spec '#reset_session!' do
     expect(@session.windows.size).to eq 1
   end
 
-  it "closes extra windows when not on the first window", requires: [:windows] do
+  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
@@ -81,22 +81,22 @@ Capybara::SpecHelper.spec '#reset_session!' do
     expect(@session.windows.size).to eq 1
   end
 
-  context "When reuse_server == false" do
+  context 'When reuse_server == false' do
     before do
       @reuse_server = Capybara.reuse_server
       Capybara.reuse_server = false
     end
 
-    it "raises any standard errors caught inside the server during a second session", requires: [:server] do
+    it 'raises any standard errors caught inside the server during a second session', requires: [:server] do
       Capybara.using_driver(@session.mode) do
         Capybara.using_session(:another_session) do
           @another_session = Capybara.current_session
-          quietly { @another_session.visit("/error") }
+          quietly { @another_session.visit('/error') }
           expect do
             @another_session.reset_session!
           end.to raise_error(TestApp::TestAppError)
-          @another_session.visit("/")
-          expect(@another_session.current_path).to eq("/")
+          @another_session.visit('/')
+          expect(@another_session.current_path).to eq('/')
         end
       end
     end
@@ -106,16 +106,16 @@ Capybara::SpecHelper.spec '#reset_session!' do
     end
   end
 
-  it "raises configured errors caught inside the server", requires: [:server] do
+  it 'raises configured errors caught inside the server', requires: [:server] do
     prev_errors = Capybara.server_errors.dup
 
     Capybara.server_errors = [LoadError]
-    quietly { @session.visit("/error") }
+    quietly { @session.visit('/error') }
     expect do
       @session.reset_session!
     end.not_to raise_error
 
-    quietly { @session.visit("/load_error") }
+    quietly { @session.visit('/load_error') }
     expect do
       @session.reset_session!
     end.to raise_error(LoadError)
@@ -123,11 +123,11 @@ Capybara::SpecHelper.spec '#reset_session!' do
     Capybara.server_errors = prev_errors
   end
 
-  it "ignores server errors when `Capybara.raise_server_errors = false`", requires: [:server] do
+  it 'ignores server errors when `Capybara.raise_server_errors = false`', requires: [:server] do
     Capybara.raise_server_errors = false
-    quietly { @session.visit("/error") }
+    quietly { @session.visit('/error') }
     @session.reset_session!
-    @session.visit("/")
-    expect(@session.current_path).to eq("/")
+    @session.visit('/')
+    expect(@session.current_path).to eq('/')
   end
 end

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

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 Capybara::SpecHelper.spec '#status_code' do
-  it "should return response codes", requires: [:status_code] do
+  it 'should return response codes', requires: [:status_code] do
     @session.visit('/with_simple_html')
     expect(@session.status_code).to eq(200)
   end

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

@@ -8,12 +8,12 @@ Capybara::SpecHelper.spec '#save_and_open_page' do
   end
 
   after do
-    Dir.glob("capybara-*.html").each do |file|
+    Dir.glob('capybara-*.html').each do |file|
       FileUtils.rm(file)
     end
   end
 
-  it "sends open method to launchy" do
+  it 'sends open method to launchy' do
     allow(Launchy).to receive(:open)
     @session.save_and_open_page
     expect(Launchy).to have_received(:open).with(/capybara-\d+\.html/)

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

@@ -1,79 +1,79 @@
 # frozen_string_literal: true
 
 Capybara::SpecHelper.spec '#save_page' do
-  let(:alternative_path) { File.join(Dir.pwd, "save_and_open_page_tmp") }
+  let(:alternative_path) { File.join(Dir.pwd, 'save_and_open_page_tmp') }
   before do
     @old_save_path = Capybara.save_path
     Capybara.save_path = nil
-    @session.visit("/foo")
+    @session.visit('/foo')
   end
 
   after do
     Capybara.save_path = @old_save_path
-    Dir.glob("capybara-*.html").each do |file|
+    Dir.glob('capybara-*.html').each do |file|
       FileUtils.rm(file)
     end
     FileUtils.rm_rf alternative_path
   end
 
-  it "saves the page in the root directory" do
+  it 'saves the page in the root directory' do
     @session.save_page
-    path = Dir.glob("capybara-*.html").first
-    expect(File.read(path)).to include("Another World")
+    path = Dir.glob('capybara-*.html').first
+    expect(File.read(path)).to include('Another World')
   end
 
-  it "generates a sensible filename" do
+  it 'generates a sensible filename' do
     @session.save_page
-    filename = Dir.glob("capybara-*.html").first
+    filename = Dir.glob('capybara-*.html').first
     expect(filename).to match(/^capybara-\d+\.html$/)
   end
 
-  it "can store files in a specified directory" do
+  it 'can store files in a specified directory' do
     Capybara.save_path = alternative_path
     @session.save_page
-    path = Dir.glob(alternative_path + "/capybara-*.html").first
-    expect(File.read(path)).to include("Another World")
+    path = Dir.glob(alternative_path + '/capybara-*.html').first
+    expect(File.read(path)).to include('Another World')
   end
 
-  it "uses the given filename" do
-    @session.save_page("capybara-001122.html")
-    expect(File.read("capybara-001122.html")).to include("Another World")
+  it 'uses the given filename' do
+    @session.save_page('capybara-001122.html')
+    expect(File.read('capybara-001122.html')).to include('Another World')
   end
 
-  it "can store files in a specified directory with a given filename" do
+  it 'can store files in a specified directory with a given filename' do
     Capybara.save_path = alternative_path
-    @session.save_page("capybara-001133.html")
-    path = alternative_path + "/capybara-001133.html"
-    expect(File.read(path)).to include("Another World")
+    @session.save_page('capybara-001133.html')
+    path = alternative_path + '/capybara-001133.html'
+    expect(File.read(path)).to include('Another World')
   end
 
-  it "can store files in a specified directory with a given relative filename" do
+  it 'can store files in a specified directory with a given relative filename' do
     Capybara.save_path = alternative_path
-    @session.save_page("tmp/capybara-001144.html")
-    path = alternative_path + "/tmp/capybara-001144.html"
-    expect(File.read(path)).to include("Another World")
+    @session.save_page('tmp/capybara-001144.html')
+    path = alternative_path + '/tmp/capybara-001144.html'
+    expect(File.read(path)).to include('Another World')
   end
 
-  it "returns an absolute path in pwd" do
+  it 'returns an absolute path in pwd' do
     result = @session.save_page
-    path = File.expand_path(Dir.glob("capybara-*.html").first, Dir.pwd)
+    path = File.expand_path(Dir.glob('capybara-*.html').first, Dir.pwd)
     expect(result).to eq(path)
   end
 
-  it "returns an absolute path in given directory" do
+  it 'returns an absolute path in given directory' do
     Capybara.save_path = alternative_path
     result = @session.save_page
-    path = File.expand_path(Dir.glob(alternative_path + "/capybara-*.html").first, alternative_path)
+    path = File.expand_path(Dir.glob(alternative_path + '/capybara-*.html').first, alternative_path)
     expect(result).to eq(path)
   end
 
-  context "asset_host contains a string" do
-    before { Capybara.asset_host = "http://example.com" }
+  context 'asset_host contains a string' do
+    before { Capybara.asset_host = 'http://example.com' }
 
     after { Capybara.asset_host = nil }
 
-    it "prepends base tag with value from asset_host to the head" do
-      @session.visit("/with_js")
+    it 'prepends base tag with value from asset_host to the head' do
+      @session.visit('/with_js')
       path = @session.save_page
 
       result = File.read(path)
@@ -82,29 +82,29 @@ Capybara::SpecHelper.spec '#save_page' do
 
     it "doesn't prepend base tag to pages when asset_host is nil" do
       Capybara.asset_host = nil
-      @session.visit("/with_js")
+      @session.visit('/with_js')
       path = @session.save_page
 
       result = File.read(path)
       expect(result).to include('<html')
-      expect(result).not_to include("http://example.com")
+      expect(result).not_to include('http://example.com')
     end
 
     it "doesn't prepend base tag to pages which already have it" do
-      @session.visit("/with_base_tag")
+      @session.visit('/with_base_tag')
       path = @session.save_page
 
       result = File.read(path)
       expect(result).to include('<html')
-      expect(result).not_to include("http://example.com")
+      expect(result).not_to include('http://example.com')
     end
 
-    it "executes successfully even if the page is missing a <head>" do
-      @session.visit("/with_simple_html")
+    it 'executes successfully even if the page is missing a <head>' do
+      @session.visit('/with_simple_html')
       path = @session.save_page
 
       result = File.read(path)
-      expect(result).to include("Bar")
+      expect(result).to include('Bar')
     end
   end
 end

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

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 Capybara::SpecHelper.spec '#save_screenshot', requires: [:screenshot] do
-  let(:alternative_path) { File.join(Dir.pwd, "save_screenshot_tmp") }
+  let(:alternative_path) { File.join(Dir.pwd, 'save_screenshot_tmp') }
   before do
     @old_save_path = Capybara.save_path
     Capybara.save_path = nil
@@ -13,7 +13,7 @@ Capybara::SpecHelper.spec '#save_screenshot', requires: [:screenshot] do
     FileUtils.rm_rf alternative_path
   end
 
-  it "generates sensible filename" do
+  it 'generates sensible filename' do
     allow(@session.driver).to receive(:save_screenshot)
 
     @session.save_screenshot
@@ -22,7 +22,7 @@ Capybara::SpecHelper.spec '#save_screenshot', requires: [:screenshot] do
     expect(@session.driver).to have_received(:save_screenshot).with(regexp, {})
   end
 
-  it "allows to specify another path" do
+  it 'allows to specify another path' do
     allow(@session.driver).to receive(:save_screenshot)
 
     custom_path = 'screenshots/1.png'
@@ -31,8 +31,8 @@ Capybara::SpecHelper.spec '#save_screenshot', requires: [:screenshot] do
     expect(@session.driver).to have_received(:save_screenshot).with(/#{custom_path}$/, {})
   end
 
-  context "with Capybara.save_path" do
-    it "file is generated in the correct location" do
+  context 'with Capybara.save_path' do
+    it 'file is generated in the correct location' do
       Capybara.save_path = alternative_path
       allow(@session.driver).to receive(:save_screenshot)
 
@@ -42,7 +42,7 @@ Capybara::SpecHelper.spec '#save_screenshot', requires: [:screenshot] do
       expect(@session.driver).to have_received(:save_screenshot).with(regexp, {})
     end
 
-    it "relative paths are relative to save_path" do
+    it 'relative paths are relative to save_path' do
       Capybara.save_path = alternative_path
       allow(@session.driver).to receive(:save_screenshot)
 

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

@@ -1,14 +1,14 @@
 # coding: US-ASCII
 # frozen_string_literal: true
 
-Capybara::SpecHelper.spec "#save_screenshot" do
+Capybara::SpecHelper.spec '#save_screenshot' do
   let(:image_path) { File.join(Dir.tmpdir, 'capybara-screenshot.png') }
 
   before do
     @session.visit '/'
   end
 
-  it "should generate PNG file", requires: [:screenshot] do
+  it 'should generate PNG file', requires: [:screenshot] do
     path = @session.save_screenshot image_path
 
     magic = File.read(image_path, 4)