capybara 3.3.1 → 3.4.0

data/spec/selenium_spec_chrome.rb

@@ -13,13 +13,7 @@ CHROME_DRIVER = ENV['HEADLESS'] ? :selenium_chrome_headless : :selenium_chrome
 
 Capybara.register_driver :selenium_chrome do |app|
   driver = Capybara::Selenium::Driver.new(app, browser: :chrome)
-  # TODO: Fix this when selenium with `download_path =` support is released
-  bridge = driver.browser.send(:bridge)
-  params = { cmd: 'Page.setDownloadBehavior', params: { behavior: 'allow', downloadPath: Capybara.save_path } }
-  command = +'/session/:session_id/chromium/send_command'
-  command[':session_id'] = bridge.session_id
-  bridge.http.call(:post, command, params)
-
+  driver.browser.download_path = Capybara.save_path
   driver
 end
 
@@ -28,14 +22,7 @@ Capybara.register_driver :selenium_chrome_headless do |app|
   browser_options.args << '--headless'
   browser_options.args << '--disable-gpu' if Gem.win_platform?
   driver = Capybara::Selenium::Driver.new(app, browser: :chrome, options: browser_options)
-
-  # TODO: Fix this when selenium with `download_path =` support is released
-  bridge = driver.browser.send(:bridge)
-  params = { cmd: 'Page.setDownloadBehavior', params: { behavior: 'allow', downloadPath: Capybara.save_path } }
-  command = +'/session/:session_id/chromium/send_command'
-  command[':session_id'] = bridge.session_id
-  bridge.http.call(:post, command, params)
-
+  driver.browser.download_path = Capybara.save_path
   driver
 end
 
@@ -58,16 +45,21 @@ skipped_tests << :windows if ENV['TRAVIS'] && (ENV['SKIP_WINDOW'] || ENV['HEADLE
 
 $stdout.puts `#{Selenium::WebDriver::Chrome.driver_path} --version` if ENV['CI']
 
-Capybara::SpecHelper.run_specs TestSessions::Chrome, CHROME_DRIVER.to_s, capybara_skip: skipped_tests
+Capybara::SpecHelper.run_specs TestSessions::Chrome, CHROME_DRIVER.to_s, capybara_skip: skipped_tests do |example|
+  case example.metadata[:description]
+  when /should fetch a response when sequentially visiting same destination with a target/
+    skip 'Chrome/chromedriver has an issue visiting URL with target after visiting same URL without target' if chrome_lt?(65.0, @session)
+  end
+end
 
-RSpec.describe "Capybara::Session with chrome" do
+RSpec.describe 'Capybara::Session with chrome' do
   include Capybara::SpecHelper
-  include_examples  "Capybara::Session", TestSessions::Chrome, CHROME_DRIVER
+  include_examples  'Capybara::Session', TestSessions::Chrome, CHROME_DRIVER
   include_examples  Capybara::RSpecMatchers, TestSessions::Chrome, CHROME_DRIVER
 
-  context "storage" do
-    describe "#reset!" do
-      it "does not clear either storage by default" do
+  context 'storage' do
+    describe '#reset!' do
+      it 'does not clear either storage by default' do
         @session = TestSessions::Chrome
         @session.visit('/with_js')
         @session.find(:css, '#set-storage').click
@@ -79,7 +71,7 @@ RSpec.describe "Capybara::Session with chrome" do
         expect(@session.evaluate_script('Object.keys(sessionStorage)')).not_to be_empty
       end
 
-      it "clears storage when set" do
+      it 'clears storage when set' do
         @session = Capybara::Session.new(:selenium_chrome_clear_storage, TestApp)
         @session.visit('/with_js')
         @session.find(:css, '#set-storage').click

data/spec/selenium_spec_chrome_remote.rb

@@ -14,11 +14,20 @@ def selenium_port
 end
 
 def ensure_selenium_running!
-  TCPSocket.open(selenium_host, selenium_port)
-rescue
-  raise 'Selenium is not running. ' \
-        "You can run a selenium server easily with: \n" \
-        '  $ docker-compose up -d selenium'
+  timer = Capybara::Helpers.timer(expire_in: 20)
+  begin
+    TCPSocket.open(selenium_host, selenium_port)
+  rescue StandardError
+    if timer.expired?
+      raise 'Selenium is not running. ' \
+          "You can run a selenium server easily with: \n" \
+          '  $ docker-compose up -d selenium_chrome'
+    else
+      puts 'Waiting for Selenium docker instance...'
+      sleep 1
+      retry
+    end
+  end
 end
 
 Capybara.register_driver :selenium_chrome_remote do |app|
@@ -39,15 +48,26 @@ module TestSessions
   Chrome = Capybara::Session.new(CHROME_REMOTE_DRIVER, TestApp)
 end
 
+TestSessions::Chrome.driver.browser.file_detector = lambda do |args|
+  # args => ["/path/to/file"]
+  str = args.first.to_s
+  str if File.exist?(str)
+end
+
 skipped_tests = %i[response_headers status_code trigger download]
 # skip window tests when headless for now - closing a window not supported by chromedriver/chrome
 skipped_tests << :windows if ENV['TRAVIS'] && (ENV['SKIP_WINDOW'] || ENV['HEADLESS'])
 
-Capybara::SpecHelper.run_specs TestSessions::Chrome, CHROME_REMOTE_DRIVER.to_s, capybara_skip: skipped_tests
+Capybara::SpecHelper.run_specs TestSessions::Chrome, CHROME_REMOTE_DRIVER.to_s, capybara_skip: skipped_tests do |example|
+  case example.metadata[:full_description]
+  when 'Capybara::Session selenium_chrome_remote #attach_file with multipart form should not break when using HTML5 multiple file input uploading multiple files'
+    pending "Selenium with Remote Chrome doesn't support multiple file upload"
+  end
+end
 
-RSpec.describe "Capybara::Session with chrome" do
+RSpec.describe 'Capybara::Session with remote Chrome' do
   include Capybara::SpecHelper
-  include_examples  "Capybara::Session", TestSessions::Chrome, CHROME_REMOTE_DRIVER
+  include_examples  'Capybara::Session', TestSessions::Chrome, CHROME_REMOTE_DRIVER
   include_examples  Capybara::RSpecMatchers, TestSessions::Chrome, CHROME_REMOTE_DRIVER
 
   it 'is considered to be chrome' do

data/spec/selenium_spec_edge.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 require 'spec_helper'
-require "selenium-webdriver"
+require 'selenium-webdriver'
 require 'shared_selenium_session'
 require 'rspec/shared_spec_matchers'
 
@@ -18,10 +18,15 @@ skipped_tests = %i[response_headers status_code trigger modals]
 
 $stdout.puts `#{Selenium::WebDriver::Edge.driver_path} --version` if ENV['CI']
 
-Capybara::SpecHelper.run_specs TestSessions::SeleniumEdge, "selenium", capybara_skip: skipped_tests
+Capybara::SpecHelper.run_specs TestSessions::SeleniumEdge, 'selenium', capybara_skip: skipped_tests do |example|
+  case example.metadata[:description]
+  when /#refresh it reposts$/
+    skip 'Edge insists on prompting without providing a way to suppress'
+  end
+end
 
-RSpec.describe "Capybara::Session with Edge", capybara_skip: skipped_tests do
+RSpec.describe 'Capybara::Session with Edge', capybara_skip: skipped_tests do
   include Capybara::SpecHelper
-  include_examples  "Capybara::Session", TestSessions::SeleniumEdge, :selenium_edge
+  include_examples  'Capybara::Session', TestSessions::SeleniumEdge, :selenium_edge
   include_examples  Capybara::RSpecMatchers, TestSessions::SeleniumEdge, :selenium_edge
 end

data/spec/selenium_spec_firefox_remote.rb

@@ -0,0 +1,87 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+require 'selenium-webdriver'
+require 'shared_selenium_session'
+require 'rspec/shared_spec_matchers'
+
+def selenium_host
+  ENV.fetch('SELENIUM_HOST', '0.0.0.0')
+end
+
+def selenium_port
+  ENV.fetch('SELENIUM_PORT', 4445)
+end
+
+def ensure_selenium_running!
+  timer = Capybara::Helpers.timer(expire_in: 20)
+  begin
+    TCPSocket.open(selenium_host, selenium_port)
+  rescue StandardError
+    if timer.expired?
+      raise 'Selenium is not running. ' \
+          "You can run a selenium server easily with: \n" \
+          '  $ docker-compose up -d selenium_firefox'
+    else
+      puts 'Waiting for Selenium docker instance...'
+      sleep 1
+      retry
+    end
+  end
+end
+
+Capybara.register_driver :selenium_firefox_remote do |app|
+  ensure_selenium_running!
+
+  url = "http://#{selenium_host}:#{selenium_port}/wd/hub"
+  caps = Selenium::WebDriver::Remote::Capabilities.firefox
+
+  Capybara::Selenium::Driver.new app,
+                                 browser: :remote,
+                                 desired_capabilities: caps,
+                                 url: url
+end
+
+FIREFOX_REMOTE_DRIVER = :selenium_firefox_remote
+
+module TestSessions
+  RemoteFirefox = Capybara::Session.new(FIREFOX_REMOTE_DRIVER, TestApp)
+end
+
+TestSessions::RemoteFirefox.driver.browser.file_detector = lambda do |args|
+  # args => ["/path/to/file"]
+  str = args.first.to_s
+  str if File.exist?(str)
+end
+
+skipped_tests = %i[response_headers status_code trigger download]
+# skip window tests when headless for now - closing a window not supported by chromedriver/chrome
+skipped_tests << :windows if ENV['TRAVIS'] && (ENV['SKIP_WINDOW'] || ENV['HEADLESS'])
+
+Capybara::SpecHelper.run_specs TestSessions::RemoteFirefox, FIREFOX_REMOTE_DRIVER.to_s, capybara_skip: skipped_tests do |example|
+  case example.metadata[:full_description]
+  when 'Capybara::Session selenium_firefox_remote node #send_keys should generate key events',
+       'Capybara::Session selenium_firefox_remote node #send_keys should allow for multiple simultaneous keys',
+       'Capybara::Session selenium_firefox_remote node #send_keys should send special characters'
+    pending "selenium-webdriver/geckodriver doesn't support complex sets of characters"
+  when 'Capybara::Session selenium_firefox_remote node #click should allow multiple modifiers'
+    pending "Firefox doesn't generate an event for shift+control+click" if marionette_gte?(62, @session)
+  when /^Capybara::Session selenium node #double_click/
+    pending "selenium-webdriver/geckodriver doesn't generate double click event" if marionette_lt?(59, @session)
+  when 'Capybara::Session selenium_firefox_remote #refresh it reposts'
+    skip 'Firefox insists on prompting without providing a way to suppress'
+  when 'Capybara::Session selenium_firefox_remote #accept_prompt should accept the prompt with a blank response when there is a default'
+    pending "Geckodriver doesn't set a blank response currently"
+  end
+end
+
+RSpec.describe 'Capybara::Session with remote firefox' do
+  include Capybara::SpecHelper
+  include_examples  'Capybara::Session', TestSessions::RemoteFirefox, FIREFOX_REMOTE_DRIVER
+  include_examples  Capybara::RSpecMatchers, TestSessions::RemoteFirefox, FIREFOX_REMOTE_DRIVER
+
+  it 'is considered to be firefox' do
+    expect(session.driver.send(:firefox?)).to be_truthy
+    expect(session.driver.send(:marionette?)).to be_truthy
+  end
+end

data/spec/selenium_spec_ie.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 require 'spec_helper'
-require "selenium-webdriver"
+require 'selenium-webdriver'
 require 'shared_selenium_session'
 require 'rspec/shared_spec_matchers'
 
@@ -22,10 +22,15 @@ skipped_tests = %i[response_headers status_code trigger modals hover form_attrib
 
 $stdout.puts `#{Selenium::WebDriver::IE.driver_path} --version` if ENV['CI']
 
-Capybara::SpecHelper.run_specs TestSessions::SeleniumIE, "selenium", capybara_skip: skipped_tests
+Capybara::SpecHelper.run_specs TestSessions::SeleniumIE, 'selenium', capybara_skip: skipped_tests do |example|
+  case example.metadata[:description]
+  when /#refresh it reposts$/
+    skip 'Firefox and Edge insist on prompting without providing a way to suppress'
+  end
+end
 
-RSpec.describe "Capybara::Session with Internet Explorer", capybara_skip: skipped_tests do
+RSpec.describe 'Capybara::Session with Internet Explorer', capybara_skip: skipped_tests do
   include Capybara::SpecHelper
-  include_examples "Capybara::Session", TestSessions::SeleniumIE, :selenium_ie
+  include_examples 'Capybara::Session', TestSessions::SeleniumIE, :selenium_ie
   include_examples Capybara::RSpecMatchers, TestSessions::SeleniumIE, :selenium_ie
 end

data/spec/selenium_spec_marionette.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 require 'spec_helper'
-require "selenium-webdriver"
+require 'selenium-webdriver'
 require 'shared_selenium_session'
 require 'rspec/shared_spec_matchers'
 
@@ -48,11 +48,26 @@ skipped_tests << :windows if ENV['TRAVIS'] && ENV['SKIP_WINDOW']
 
 $stdout.puts `#{Selenium::WebDriver::Firefox.driver_path} --version` if ENV['CI']
 
-Capybara::SpecHelper.run_specs TestSessions::SeleniumMarionette, "selenium", capybara_skip: skipped_tests
+Capybara::SpecHelper.run_specs TestSessions::SeleniumMarionette, 'selenium', capybara_skip: skipped_tests do |example|
+  case example.metadata[:full_description]
+  when 'Capybara::Session selenium node #send_keys should generate key events',
+       'Capybara::Session selenium node #send_keys should allow for multiple simultaneous keys',
+       'Capybara::Session selenium node #send_keys should send special characters'
+    pending "selenium-webdriver/geckodriver doesn't support complex sets of characters"
+  when 'Capybara::Session selenium node #click should allow multiple modifiers'
+    pending "Firefox doesn't generate an event for shift+control+click" if marionette_gte?(62, @session)
+  when /^Capybara::Session selenium node #double_click/
+    pending "selenium-webdriver/geckodriver doesn't generate double click event" if marionette_lt?(59, @session)
+  when 'Capybara::Session selenium #refresh it reposts'
+    skip 'Firefox insists on prompting without providing a way to suppress'
+  when 'Capybara::Session selenium #accept_prompt should accept the prompt with a blank response when there is a default'
+    pending "Geckodriver doesn't set a blank response currently"
+  end
+end
 
-RSpec.describe "Capybara::Session with firefox" do # rubocop:disable RSpec/MultipleDescribes
+RSpec.describe 'Capybara::Session with firefox' do # rubocop:disable RSpec/MultipleDescribes
   include Capybara::SpecHelper
-  include_examples  "Capybara::Session", TestSessions::SeleniumMarionette, :selenium_marionette
+  include_examples  'Capybara::Session', TestSessions::SeleniumMarionette, :selenium_marionette
   include_examples  Capybara::RSpecMatchers, TestSessions::SeleniumMarionette, :selenium_marionette
 end
 
@@ -62,14 +77,14 @@ RSpec.describe Capybara::Selenium::Driver do
   end
 
   describe '#quit' do
-    it "should reset browser when quit" do
+    it 'should reset browser when quit' do
       expect(@driver.browser).to be_truthy
       @driver.quit
       # access instance variable directly so we don't create a new browser instance
       expect(@driver.instance_variable_get(:@browser)).to be_nil
     end
 
-    context "with errors" do
+    context 'with errors' do
       before do
         @original_browser = @driver.browser
       end
@@ -80,11 +95,11 @@ RSpec.describe Capybara::Selenium::Driver do
         @original_browser.quit
       end
 
-      it "warns UnknownError returned during quit because the browser is probably already gone" do
+      it 'warns UnknownError returned during quit because the browser is probably already gone' do
         allow(@driver).to receive(:warn)
         allow(@driver.browser).to(
           receive(:quit)
-          .and_raise(Selenium::WebDriver::Error::UnknownError, "random message")
+          .and_raise(Selenium::WebDriver::Error::UnknownError, 'random message')
         )
 
         expect { @driver.quit }.not_to raise_error
@@ -92,11 +107,11 @@ RSpec.describe Capybara::Selenium::Driver do
         expect(@driver).to have_received(:warn).with(/random message/)
       end
 
-      it "ignores silenced UnknownError returned during quit because the browser is almost definitely already gone" do
+      it 'ignores silenced UnknownError returned during quit because the browser is almost definitely already gone' do
         allow(@driver).to receive(:warn)
         allow(@driver.browser).to(
           receive(:quit)
-          .and_raise(Selenium::WebDriver::Error::UnknownError, "Error communicating with the remote browser")
+          .and_raise(Selenium::WebDriver::Error::UnknownError, 'Error communicating with the remote browser')
         )
 
         expect { @driver.quit }.not_to raise_error
@@ -106,9 +121,9 @@ RSpec.describe Capybara::Selenium::Driver do
     end
   end
 
-  context "storage" do
-    describe "#reset!" do
-      it "does not clear either storage by default" do
+  context 'storage' do
+    describe '#reset!' do
+      it 'does not clear either storage by default' do
         @session = TestSessions::SeleniumMarionette
         @session.visit('/with_js')
         @session.find(:css, '#set-storage').click
@@ -118,7 +133,7 @@ RSpec.describe Capybara::Selenium::Driver do
         expect(@session.driver.browser.session_storage.keys).not_to be_empty
       end
 
-      it "clears storage when set" do
+      it 'clears storage when set' do
         @session = Capybara::Session.new(:selenium_marionette_clear_storage, TestApp)
         @session.visit('/with_js')
         @session.find(:css, '#set-storage').click
@@ -130,13 +145,13 @@ RSpec.describe Capybara::Selenium::Driver do
     end
   end
 
-  context "#refresh" do
+  context '#refresh' do
     def extract_results(session)
       expect(session).to have_xpath("//pre[@id='results']")
       YAML.load Nokogiri::HTML(session.body).xpath("//pre[@id='results']").first.inner_html.lstrip
     end
 
-    it "can repost by accepting confirm" do
+    it 'can repost by accepting confirm' do
       @session = TestSessions::SeleniumMarionette
       @session.visit('/form')
       @session.select('Sweden', from: 'form_region')
@@ -154,8 +169,8 @@ RSpec.describe Capybara::Selenium::Driver do
 end
 
 RSpec.describe Capybara::Selenium::Node do
-  context "#click" do
-    it "warns when attempting on a table row" do
+  context '#click' do
+    it 'warns when attempting on a table row' do
       session = TestSessions::SeleniumMarionette
       session.visit('/tables')
       tr = session.find(:css, '#agent_table tr:first-child')
@@ -163,5 +178,14 @@ RSpec.describe Capybara::Selenium::Node do
       tr.click
       expect(tr.base).to have_received(:warn).with(/Clicking the first cell in the row instead/)
     end
+
+    it 'should allow multiple modifiers', requires: [:js] do
+      session = TestSessions::SeleniumMarionette
+      session.visit('with_js')
+      # Firefox v62+ doesn't generate an event for control+shift+click
+      session.find(:css, '#click-test').click(:alt, :ctrl, :meta)
+      # it also triggers a contextmenu event when control is held so don't check click type
+      expect(session).to have_link('Has been alt control meta')
+    end
   end
 end

data/spec/server_spec.rb

@@ -3,8 +3,8 @@
 require 'spec_helper'
 
 RSpec.describe Capybara::Server do
-  it "should spool up a rack server" do
-    @app = proc { |_env| [200, {}, ["Hello Server!"]] }
+  it 'should spool up a rack server' do
+    @app = proc { |_env| [200, {}, ['Hello Server!']] }
     @server = Capybara::Server.new(@app).boot
 
     @res = Net::HTTP.start(@server.host, @server.port) { |http| http.get('/') }
@@ -12,14 +12,16 @@ RSpec.describe Capybara::Server do
     expect(@res.body).to include('Hello Server')
   end
 
-  it "should do nothing when no server given" do
+  it 'should do nothing when no server given' do
     expect do
       @server = Capybara::Server.new(nil).boot
     end.not_to raise_error
   end
 
-  it "should bind to the specified host" do
+  it 'should bind to the specified host' do
     # TODO: travis with jruby in container mode has an issue with this test
+    skip 'This platform has an issue with this test' if (ENV['TRAVIS'] && (RUBY_ENGINE == 'jruby')) || Gem.win_platform?
+
     begin
       app = proc { |_env| [200, {}, ['Hello Server!']] }
 
@@ -35,12 +37,12 @@ RSpec.describe Capybara::Server do
     ensure
       Capybara.server_host = nil
     end
-  end unless (ENV['TRAVIS'] && (RUBY_ENGINE == 'jruby')) || Gem.win_platform?
+  end
 
-  it "should use specified port" do
+  it 'should use specified port' do
     Capybara.server_port = 22789
 
-    @app = proc { |_env| [200, {}, ["Hello Server!"]] }
+    @app = proc { |_env| [200, {}, ['Hello Server!']] }
     @server = Capybara::Server.new(@app).boot
 
     @res = Net::HTTP.start(@server.host, 22789) { |http| http.get('/') }
@@ -49,8 +51,8 @@ RSpec.describe Capybara::Server do
     Capybara.server_port = nil
   end
 
-  it "should use given port" do
-    @app = proc { |_env| [200, {}, ["Hello Server!"]] }
+  it 'should use given port' do
+    @app = proc { |_env| [200, {}, ['Hello Server!']] }
     @server = Capybara::Server.new(@app, port: 22790).boot
 
     @res = Net::HTTP.start(@server.host, 22790) { |http| http.get('/') }
@@ -59,9 +61,9 @@ RSpec.describe Capybara::Server do
     Capybara.server_port = nil
   end
 
-  it "should find an available port" do
-    @app1 = proc { |_env| [200, {}, ["Hello Server!"]] }
-    @app2 = proc { |_env| [200, {}, ["Hello Second Server!"]] }
+  it 'should find an available port' do
+    @app1 = proc { |_env| [200, {}, ['Hello Server!']] }
+    @app2 = proc { |_env| [200, {}, ['Hello Second Server!']] }
 
     @server1 = Capybara::Server.new(@app1).boot
     @server2 = Capybara::Server.new(@app2).boot
@@ -73,10 +75,10 @@ RSpec.describe Capybara::Server do
     expect(@res2.body).to include('Hello Second Server')
   end
 
-  it "should support SSL" do
+  it 'should support SSL' do
     begin
-      key = File.join(Dir.pwd, "spec", "fixtures", "key.pem")
-      cert = File.join(Dir.pwd, "spec", "fixtures", "certificate.pem")
+      key = File.join(Dir.pwd, 'spec', 'fixtures', 'key.pem')
+      cert = File.join(Dir.pwd, 'spec', 'fixtures', 'certificate.pem')
       Capybara.server = :puma, { Host: "ssl://#{Capybara.server_host}?key=#{key}&cert=#{cert}" }
       app = proc { |_env| [200, {}, ['Hello SSL Server!']] }
       server = Capybara::Server.new(app).boot
@@ -95,7 +97,7 @@ RSpec.describe Capybara::Server do
     end
   end
 
-  context "When Capybara.reuse_server is true" do
+  context 'When Capybara.reuse_server is true' do
     before do
       @old_reuse_server = Capybara.reuse_server
       Capybara.reuse_server = true
@@ -105,8 +107,8 @@ RSpec.describe Capybara::Server do
       Capybara.reuse_server = @old_reuse_server
     end
 
-    it "should use the existing server if it already running" do
-      @app = proc { |_env| [200, {}, ["Hello Server!"]] }
+    it 'should use the existing server if it already running' do
+      @app = proc { |_env| [200, {}, ['Hello Server!']] }
 
       @server1 = Capybara::Server.new(@app).boot
       @server2 = Capybara::Server.new(@app).boot
@@ -120,14 +122,14 @@ RSpec.describe Capybara::Server do
       expect(@server1.port).to eq(@server2.port)
     end
 
-    it "detects and waits for all reused server sessions pending requests" do
+    it 'detects and waits for all reused server sessions pending requests' do
       done = 0
 
       app = proc do |env|
         request = Rack::Request.new(env)
         sleep request.params['wait_time'].to_f
         done += 1
-        [200, {}, ["Hello Server!"]]
+        [200, {}, ['Hello Server!']]
       end
 
       server1 = Capybara::Server.new(app).boot
@@ -142,7 +144,7 @@ RSpec.describe Capybara::Server do
     end
   end
 
-  context "When Capybara.reuse_server is false" do
+  context 'When Capybara.reuse_server is false' do
     before do
       @old_reuse_server = Capybara.reuse_server
       Capybara.reuse_server = false
@@ -152,8 +154,8 @@ RSpec.describe Capybara::Server do
       Capybara.reuse_server = @old_reuse_server
     end
 
-    it "should not reuse an already running server" do
-      @app = proc { |_env| [200, {}, ["Hello Server!"]] }
+    it 'should not reuse an already running server' do
+      @app = proc { |_env| [200, {}, ['Hello Server!']] }
 
       @server1 = Capybara::Server.new(@app).boot
       @server2 = Capybara::Server.new(@app).boot
@@ -167,14 +169,14 @@ RSpec.describe Capybara::Server do
       expect(@server1.port).not_to eq(@server2.port)
     end
 
-    it "detects and waits for only one sessions pending requests" do
+    it 'detects and waits for only one sessions pending requests' do
       done = 0
 
       app = proc do |env|
         request = Rack::Request.new(env)
         sleep request.params['wait_time'].to_f
         done += 1
-        [200, {}, ["Hello Server!"]]
+        [200, {}, ['Hello Server!']]
       end
 
       server1 = Capybara::Server.new(app).boot
@@ -192,7 +194,7 @@ RSpec.describe Capybara::Server do
     end
   end
 
-  it "should raise server errors when the server errors before the timeout" do
+  it 'should raise server errors when the server errors before the timeout' do
     begin
       Capybara.register_server :kaboom do
         sleep 0.1
@@ -208,7 +210,7 @@ RSpec.describe Capybara::Server do
     end
   end
 
-  it "is not #responsive? when Net::HTTP raises a SystemCallError" do
+  it 'is not #responsive? when Net::HTTP raises a SystemCallError' do
     app = -> { [200, {}, ['Hello, world']] }
     server = Capybara::Server.new(app)
     allow(Net::HTTP).to receive(:start).and_raise(SystemCallError.allocate)