capybara 2.7.0 → 3.35.3

data/spec/minitest_spec.rb

@@ -0,0 +1,164 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+require 'capybara/minitest'
+
+class MinitestTest < Minitest::Test
+  include Capybara::DSL
+  include Capybara::Minitest::Assertions
+
+  def setup
+    visit('/form')
+  end
+
+  def teardown
+    Capybara.reset_sessions!
+  end
+
+  def test_assert_text
+    assert_text('Form', normalize_ws: false)
+    assert_no_text('Not on the page')
+    refute_text('Also Not on the page')
+  end
+
+  def test_assert_title
+    visit('/with_title')
+    assert_title('Test Title')
+    assert_no_title('Not the title')
+    refute_title('Not the title')
+  end
+
+  def test_assert_current_path
+    assert_current_path('/form')
+    assert_current_path('/form') { |url| url.query.nil? }
+    assert_no_current_path('/not_form')
+    refute_current_path('/not_form')
+  end
+
+  def test_assert_xpath
+    assert_xpath('.//select[@id="form_title"]')
+    assert_xpath('.//select', count: 1) { |el| el[:id] == 'form_title' }
+    assert_no_xpath('.//select[@id="not_form_title"]')
+    assert_no_xpath('.//select') { |el| el[:id] == 'not_form_title' }
+    refute_xpath('.//select[@id="not_form_title"]')
+  end
+
+  def test_assert_css
+    assert_css('select#form_title')
+    assert_no_css('select#not_form_title')
+  end
+
+  def test_assert_selector
+    assert_selector(:css, 'select#form_title')
+    assert_selector(:xpath, './/select[@id="form_title"]')
+    assert_no_selector(:css, 'select#not_form_title')
+    assert_no_selector(:xpath, './/select[@id="not_form_title"]')
+    refute_selector(:css, 'select#not_form_title')
+  end
+
+  def test_assert_link
+    visit('/with_html')
+    assert_link('A link')
+    assert_link(count: 1) { |el| el.text == 'A link' }
+    assert_no_link('Not on page')
+  end
+
+  def test_assert_button
+    assert_button('fresh_btn')
+    assert_button(count: 1) { |el| el[:id] == 'fresh_btn' }
+    assert_no_button('not_btn')
+  end
+
+  def test_assert_field
+    assert_field('customer_email')
+    assert_no_field('not_on_the_form')
+  end
+
+  def test_assert_select
+    assert_select('form_title')
+    assert_no_select('not_form_title')
+  end
+
+  def test_assert_checked_field
+    assert_checked_field('form_pets_dog')
+    assert_no_checked_field('form_pets_cat')
+    refute_checked_field('form_pets_snake')
+  end
+
+  def test_assert_unchecked_field
+    assert_unchecked_field('form_pets_cat')
+    assert_no_unchecked_field('form_pets_dog')
+    refute_unchecked_field('form_pets_snake')
+  end
+
+  def test_assert_table
+    visit('/tables')
+    assert_table('agent_table')
+    assert_no_table('not_on_form')
+    refute_table('not_on_form')
+  end
+
+  def test_assert_all_of_selectors
+    assert_all_of_selectors(:css, 'select#form_other_title', 'input#form_last_name')
+  end
+
+  def test_assert_none_of_selectors
+    assert_none_of_selectors(:css, 'input#not_on_page', 'input#also_not_on_page')
+  end
+
+  def test_assert_any_of_selectors
+    assert_any_of_selectors(:css, 'input#not_on_page', 'select#form_other_title')
+  end
+
+  def test_assert_matches_selector
+    assert_matches_selector(find(:field, 'customer_email'), :field, 'customer_email')
+    assert_not_matches_selector(find(:select, 'form_title'), :field, 'customer_email')
+    refute_matches_selector(find(:select, 'form_title'), :field, 'customer_email')
+  end
+
+  def test_assert_matches_css
+    assert_matches_css(find(:select, 'form_title'), 'select#form_title')
+    refute_matches_css(find(:select, 'form_title'), 'select#form_other_title')
+  end
+
+  def test_assert_matches_xpath
+    assert_matches_xpath(find(:select, 'form_title'), './/select[@id="form_title"]')
+    refute_matches_xpath(find(:select, 'form_title'), './/select[@id="form_other_title"]')
+  end
+
+  def test_assert_matches_style
+    skip "Rack test doesn't support style" if Capybara.current_driver == :rack_test
+    visit('/with_html')
+    assert_matches_style(find(:css, '#second'), display: 'inline')
+  end
+
+  def test_assert_ancestor
+    option = find(:option, 'Finnish')
+    assert_ancestor(option, :css, '#form_locale')
+  end
+
+  def test_assert_sibling
+    option = find(:css, '#form_title').find(:option, 'Mrs')
+    assert_sibling(option, :option, 'Mr')
+  end
+end
+
+RSpec.describe 'capybara/minitest' do
+  before do
+    Capybara.current_driver = :rack_test
+    Capybara.app = TestApp
+  end
+
+  after do
+    Capybara.use_default_driver
+  end
+
+  it 'should support minitest' do
+    output = StringIO.new
+    reporter = Minitest::SummaryReporter.new(output)
+    reporter.start
+    MinitestTest.run reporter, {}
+    reporter.report
+    expect(output.string).to include('22 runs, 53 assertions, 0 failures, 0 errors, 1 skips')
+  end
+end

data/spec/minitest_spec_spec.rb

@@ -0,0 +1,162 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+require 'capybara/minitest'
+require 'capybara/minitest/spec'
+
+class MinitestSpecTest < Minitest::Spec
+  include ::Capybara::DSL
+  include ::Capybara::Minitest::Assertions
+
+  before do
+    visit('/form')
+  end
+
+  after do
+    Capybara.reset_sessions!
+  end
+
+  it 'supports text expectations' do
+    _(page).must_have_text('Form', minimum: 1)
+    _(page).wont_have_text('Not a form')
+    form = find(:css, 'form', text: 'Title')
+    _(form).must_have_text('Customer Email')
+    _(form).wont_have_text('Some other email')
+  end
+
+  it 'supports current_path expectations' do
+    _(page).must_have_current_path('/form')
+    _(page).wont_have_current_path('/form2')
+  end
+
+  it 'supports title expectations' do
+    visit('/with_title')
+    _(page).must_have_title('Test Title')
+    _(page).wont_have_title('Not the title')
+  end
+
+  it 'supports xpath expectations' do
+    _(page).must_have_xpath('.//input[@id="customer_email"]')
+    _(page).wont_have_xpath('.//select[@id="not_form_title"]')
+    _(page).wont_have_xpath('.//input[@id="customer_email"]') { |el| el[:id] == 'not_customer_email' }
+    select = find(:select, 'form_title')
+    _(select).must_have_xpath('.//option[@class="title"]')
+    _(select).must_have_xpath('.//option', count: 1) { |option| option[:class] != 'title' && !option.disabled? }
+    _(select).wont_have_xpath('.//input[@id="customer_email"]')
+  end
+
+  it 'support css expectations' do
+    _(page).must_have_css('input#customer_email')
+    _(page).wont_have_css('select#not_form_title')
+    el = find(:select, 'form_title')
+    _(el).must_have_css('option.title')
+    _(el).wont_have_css('input#customer_email')
+  end
+
+  it 'supports link expectations' do
+    visit('/with_html')
+    _(page).must_have_link('A link')
+    _(page).wont_have_link('Not on page')
+  end
+
+  it 'supports button expectations' do
+    _(page).must_have_button('fresh_btn')
+    _(page).wont_have_button('not_btn')
+  end
+
+  it 'supports field expectations' do
+    _(page).must_have_field('customer_email')
+    _(page).wont_have_field('not_on_the_form')
+  end
+
+  it 'supports select expectations' do
+    _(page).must_have_select('form_title')
+    _(page).wont_have_select('not_form_title')
+  end
+
+  it 'supports checked_field expectations' do
+    _(page).must_have_checked_field('form_pets_dog')
+    _(page).wont_have_checked_field('form_pets_cat')
+  end
+
+  it 'supports unchecked_field expectations' do
+    _(page).must_have_unchecked_field('form_pets_cat')
+    _(page).wont_have_unchecked_field('form_pets_dog')
+  end
+
+  it 'supports table expectations' do
+    visit('/tables')
+    _(page).must_have_table('agent_table')
+    _(page).wont_have_table('not_on_form')
+  end
+
+  it 'supports all_of_selectors expectations' do
+    _(page).must_have_all_of_selectors(:css, 'select#form_other_title', 'input#form_last_name')
+  end
+
+  it 'supports none_of_selectors expectations' do
+    _(page).must_have_none_of_selectors(:css, 'input#not_on_page', 'input#also_not_on_page')
+  end
+
+  it 'supports any_of_selectors expectations' do
+    _(page).must_have_any_of_selectors(:css, 'select#form_other_title', 'input#not_on_page')
+  end
+
+  it 'supports match_selector expectations' do
+    _(find(:field, 'customer_email')).must_match_selector(:field, 'customer_email')
+    _(find(:select, 'form_title')).wont_match_selector(:field, 'customer_email')
+  end
+
+  it 'supports match_css expectations' do
+    _(find(:select, 'form_title')).must_match_css('select#form_title')
+    _(find(:select, 'form_title')).wont_match_css('select#form_other_title')
+  end
+
+  it 'supports match_xpath expectations' do
+    _(find(:select, 'form_title')).must_match_xpath('.//select[@id="form_title"]')
+    _(find(:select, 'form_title')).wont_match_xpath('.//select[@id="not_on_page"]')
+  end
+
+  it 'handles failures' do
+    _(page).must_have_select('non_existing_form_title')
+  end
+
+  it 'supports style expectations' do
+    skip "Rack test doesn't support style" if Capybara.current_driver == :rack_test
+    visit('/with_html')
+    _(find(:css, '#second')).must_have_style('display' => 'inline') # deprecated
+    _(find(:css, '#second')).must_match_style('display' => 'inline')
+  end
+
+  it 'supports ancestor expectations' do
+    option = find(:option, 'Finnish')
+    _(option).must_have_ancestor(:css, '#form_locale')
+  end
+
+  it 'supports sibling expectations' do
+    option = find(:css, '#form_title').find(:option, 'Mrs')
+    _(option).must_have_sibling(:option, 'Mr')
+  end
+end
+
+RSpec.describe 'capybara/minitest/spec' do
+  before do
+    Capybara.current_driver = :rack_test
+    Capybara.app = TestApp
+  end
+
+  after do
+    Capybara.use_default_driver
+  end
+
+  it 'should support minitest spec' do
+    output = StringIO.new
+    reporter = Minitest::SummaryReporter.new(output)
+    reporter.start
+    MinitestSpecTest.run reporter, {}
+    reporter.report
+    expect(output.string).to include('22 runs, 44 assertions, 1 failures, 0 errors, 1 skips')
+    # Make sure error messages are displayed
+    expect(output.string).to match(/expected to find select box "non_existing_form_title" .*but there were no matches/)
+  end
+end

data/spec/per_session_config_spec.rb

@@ -0,0 +1,68 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+require 'capybara/dsl'
+
+RSpec.describe Capybara::SessionConfig do
+  describe 'threadsafe' do
+    it 'defaults to global session options' do
+      Capybara.threadsafe = true
+      session = Capybara::Session.new(:rack_test, TestApp)
+      %i[default_host app_host always_include_port run_server
+         default_selector default_max_wait_time ignore_hidden_elements
+         automatic_reload match exact raise_server_errors visible_text_only
+         automatic_label_click enable_aria_label save_path
+         asset_host].each do |m|
+           expect(session.config.public_send(m)).to eq Capybara.public_send(m)
+         end
+    end
+
+    it "doesn't change global session when changed" do
+      Capybara.threadsafe = true
+      host = 'http://my.example.com'
+      session = Capybara::Session.new(:rack_test, TestApp) do |config|
+        config.default_host = host
+        config.automatic_label_click = !config.automatic_label_click
+        config.server_errors << ArgumentError
+      end
+      expect(Capybara.default_host).not_to eq host
+      expect(session.config.default_host).to eq host
+      expect(Capybara.automatic_label_click).not_to eq session.config.automatic_label_click
+      expect(Capybara.server_errors).not_to eq session.config.server_errors
+    end
+
+    it "doesn't allow session configuration block when false" do
+      Capybara.threadsafe = false
+      expect do
+        Capybara::Session.new(:rack_test, TestApp) { |config| }
+      end.to raise_error 'A configuration block is only accepted when Capybara.threadsafe == true'
+    end
+
+    it "doesn't allow session config when false" do
+      Capybara.threadsafe = false
+      session = Capybara::Session.new(:rack_test, TestApp)
+      expect { session.config.default_selector = :title }.to raise_error(/Per session settings are only supported when Capybara.threadsafe == true/)
+      expect do
+        session.configure do |config|
+          config.exact = true
+        end
+      end.to raise_error(/Session configuration is only supported when Capybara.threadsafe == true/)
+    end
+
+    it 'uses the config from the session' do
+      Capybara.threadsafe = true
+      session = Capybara::Session.new(:rack_test, TestApp) do |config|
+        config.default_selector = :link
+      end
+      session.visit('/with_html')
+      expect(session.find('foo').tag_name).to eq 'a'
+    end
+
+    it "won't change threadsafe once a session is created" do
+      Capybara.threadsafe = true
+      Capybara.threadsafe = false
+      Capybara::Session.new(:rack_test, TestApp)
+      expect { Capybara.threadsafe = true }.to raise_error(/Threadsafe setting cannot be changed once a session is created/)
+    end
+  end
+end

data/spec/rack_test_spec.rb

@@ -1,186 +1,279 @@
 # frozen_string_literal: true
+
 require 'spec_helper'
+nokogumbo_required = begin
+  require 'nokogumbo'
+  true
+rescue LoadError
+  false
+end
 
 module TestSessions
   RackTest = Capybara::Session.new(:rack_test, TestApp)
 end
 
-Capybara::SpecHelper.run_specs TestSessions::RackTest, "RackTest", :capybara_skip => [
-  :js,
-  :modals,
-  :screenshot,
-  :frames,
-  :windows,
-  :send_keys,
-  :server,
-  :hover,
-  :about_scheme,
+skipped_tests = %i[
+  js
+  modals
+  screenshot
+  frames
+  windows
+  send_keys
+  server
+  hover
+  about_scheme
+  download
+  css
+  scroll
+  spatial
+  html_validation
+  shadow_dom
 ]
+Capybara::SpecHelper.run_specs TestSessions::RackTest, 'RackTest', capybara_skip: skipped_tests do |example|
+  case example.metadata[:full_description]
+  when /has_css\? should support case insensitive :class and :id options/
+    skip "Nokogiri doesn't support case insensitive CSS attribute matchers"
+  when /#click_button should follow permanent redirects that maintain method/
+    skip "Rack < 2 doesn't support 308" if Gem.loaded_specs['rack'].version < Gem::Version.new('2.0.0')
+  end
+end
 
-RSpec.describe Capybara::Session do
+RSpec.describe Capybara::Session do # rubocop:disable RSpec/MultipleDescribes
+  include Capybara::RSpecMatchers
   context 'with rack test driver' do
-    before do
-      @session = TestSessions::RackTest
-    end
+    let(:session) { TestSessions::RackTest }
 
     describe '#driver' do
-      it "should be a rack test driver" do
-        expect(@session.driver).to be_an_instance_of(Capybara::RackTest::Driver)
+      it 'should be a rack test driver' do
+        expect(session.driver).to be_an_instance_of(Capybara::RackTest::Driver)
       end
     end
 
     describe '#mode' do
-      it "should remember the mode" do
-        expect(@session.mode).to eq(:rack_test)
+      it 'should remember the mode' do
+        expect(session.mode).to eq(:rack_test)
       end
     end
 
     describe '#click_link' do
-      it "should use data-method if option is true" do
-        @session.driver.options[:respect_data_method] = true
-        @session.visit "/with_html"
-        @session.click_link "A link with data-method"
-        expect(@session.html).to include('The requested object was deleted')
+      after do
+        session.driver.options[:respect_data_method] = false
+      end
+
+      it 'should use data-method if option is true' do
+        session.driver.options[:respect_data_method] = true
+        session.visit '/with_html'
+        session.click_link 'A link with data-method'
+        expect(session.html).to include('The requested object was deleted')
       end
 
-      it "should not use data-method if option is false" do
-        @session.driver.options[:respect_data_method] = false
-        @session.visit "/with_html"
-        @session.click_link "A link with data-method"
-        expect(@session.html).to include('Not deleted')
+      it 'should not use data-method if option is false' do
+        session.driver.options[:respect_data_method] = false
+        session.visit '/with_html'
+        session.click_link 'A link with data-method'
+        expect(session.html).to include('Not deleted')
       end
 
       it "should use data-method if available even if it's capitalized" do
-        @session.driver.options[:respect_data_method] = true
-        @session.visit "/with_html"
-        @session.click_link "A link with capitalized data-method"
-        expect(@session.html).to include('The requested object was deleted')
+        session.driver.options[:respect_data_method] = true
+        session.visit '/with_html'
+        session.click_link 'A link with capitalized data-method'
+        expect(session.html).to include('The requested object was deleted')
       end
+    end
 
-      after do
-        @session.driver.options[:respect_data_method] = false
+    describe '#fill_in' do
+      it 'should warn that :fill_options are not supported' do
+        session.visit '/with_html'
+        expect { session.fill_in 'test_field', with: 'not_monkey', fill_options: { random: true } }.to \
+          output(/^Options passed to Node#set but the RackTest driver doesn't support any - ignoring/).to_stderr
+        expect(session).to have_field('test_field', with: 'not_monkey')
       end
     end
 
-    describe "#fill_in" do
-      it "should warn that :fill_options are not supported" do
-        expect_any_instance_of(Capybara::Node::Element).to receive(:warn)
-          .with("Options passed to Capybara::Node#set but the driver doesn't support them")
-        @session.visit "/with_html"
-        @session.fill_in 'test_field', with: 'not_moneky', fill_options: { random: true }
+    describe '#attach_file' do
+      context 'with multipart form' do
+        it 'should submit an empty form-data section if no file is submitted' do
+          session.visit('/form')
+          session.click_button('Upload Empty')
+          expect(session.html).to include('Successfully ignored empty file field.')
+        end
+      end
+
+      it 'should not submit an obsolete mime type' do
+        test_jpg_file_path = File.expand_path('fixtures/capybara.csv', File.dirname(__FILE__))
+        session.visit('/form')
+        session.attach_file 'form_document', test_jpg_file_path
+        session.click_button('Upload Single')
+        expect(session).to have_content('Content-type: text/csv')
       end
     end
 
-    describe "#attach_file" do
-      context "with multipart form" do
-        it "should submit an empty form-data section if no file is submitted" do
-          @session.visit("/form")
-          @session.click_button("Upload Empty")
-          expect(@session.html).to include('Successfully ignored empty file field.')
+    describe '#click' do
+      context 'on a label' do
+        it 'should toggle the associated checkbox' do
+          session.visit('/form')
+          expect(session).to have_unchecked_field('form_pets_cat')
+          session.find(:label, 'Cat').click
+          expect(session).to have_checked_field('form_pets_cat')
+          session.find(:label, 'Cat').click
+          expect(session).to have_unchecked_field('form_pets_cat')
+          session.find(:label, 'McLaren').click
+          expect(session).to have_checked_field('form_cars_mclaren', visible: :hidden)
+        end
+
+        it 'should toggle the associated radio' do
+          session.visit('/form')
+          expect(session).to have_unchecked_field('gender_male')
+          session.find(:label, 'Male').click
+          expect(session).to have_checked_field('gender_male')
+          session.find(:label, 'Female').click
+          expect(session).to have_unchecked_field('gender_male')
+        end
+
+        it 'should rewrite the forms action query for get submission' do
+          session.visit('/form')
+          session.click_button('mediocre')
+          expect(session).not_to have_current_path(/foo|bar/)
+        end
+
+        it 'should rewrite the submit buttons formaction query for get submission' do
+          session.visit('/form')
+          session.click_button('mediocre2')
+          expect(session).not_to have_current_path(/foo|bar/)
         end
       end
     end
+
+    describe '#send_keys' do
+      it 'raises an UnsupportedMethodError' do
+        session.visit('/form')
+
+        expect { session.send_keys(:tab) }.to raise_error(Capybara::NotSupportedByDriverError)
+      end
+    end
+
+    describe '#text' do
+      it 'should return original text content for textareas' do
+        session.visit('/with_html')
+        session.find_field('normal', type: 'textarea', with: 'banana').set('hello')
+        normal = session.find(:css, '#normal')
+        expect(normal.value).to eq 'hello'
+        expect(normal.text).to eq 'banana'
+      end
+    end
+
+    describe '#style' do
+      it 'should raise an error' do
+        session.visit('/with_html')
+        el = session.find(:css, '#first')
+        expect { el.style('display') }.to raise_error NotImplementedError, /not process CSS/
+      end
+    end
   end
 end
 
 RSpec.describe Capybara::RackTest::Driver do
-  before do
-    @driver = TestSessions::RackTest.driver
-  end
+  let(:driver) { TestSessions::RackTest.driver }
 
   describe ':headers option' do
     it 'should always set headers' do
-      @driver = Capybara::RackTest::Driver.new(TestApp, :headers => {'HTTP_FOO' => 'foobar'})
-      @driver.visit('/get_header')
-      expect(@driver.html).to include('foobar')
+      driver = described_class.new(TestApp, headers: { 'HTTP_FOO' => 'foobar' })
+      driver.visit('/get_header')
+      expect(driver.html).to include('foobar')
     end
 
     it 'should keep headers on link clicks' do
-      @driver = Capybara::RackTest::Driver.new(TestApp, :headers => {'HTTP_FOO' => 'foobar'})
-      @driver.visit('/header_links')
-      @driver.find_xpath('.//a').first.click
-      expect(@driver.html).to include('foobar')
+      driver = described_class.new(TestApp, headers: { 'HTTP_FOO' => 'foobar' })
+      driver.visit('/header_links')
+      driver.find_xpath('.//a').first.click
+      expect(driver.html).to include('foobar')
     end
 
     it 'should keep headers on form submit' do
-      @driver = Capybara::RackTest::Driver.new(TestApp, :headers => {'HTTP_FOO' => 'foobar'})
-      @driver.visit('/header_links')
-      @driver.find_xpath('.//input').first.click
-      expect(@driver.html).to include('foobar')
+      driver = described_class.new(TestApp, headers: { 'HTTP_FOO' => 'foobar' })
+      driver.visit('/header_links')
+      driver.find_xpath('.//input').first.click
+      expect(driver.html).to include('foobar')
     end
 
     it 'should keep headers on redirects' do
-      @driver = Capybara::RackTest::Driver.new(TestApp, :headers => {'HTTP_FOO' => 'foobar'})
-      @driver.visit('/get_header_via_redirect')
-      expect(@driver.html).to include('foobar')
+      driver = described_class.new(TestApp, headers: { 'HTTP_FOO' => 'foobar' })
+      driver.visit('/get_header_via_redirect')
+      expect(driver.html).to include('foobar')
     end
   end
 
   describe ':follow_redirects option' do
-    it "defaults to following redirects" do
-      @driver = Capybara::RackTest::Driver.new(TestApp)
+    it 'defaults to following redirects' do
+      driver = described_class.new(TestApp)
 
-      @driver.visit('/redirect')
-      expect(@driver.response.header['Location']).to be_nil
-      expect(@driver.current_url).to match %r{/landed$}
+      driver.visit('/redirect')
+      expect(driver.response.header['Location']).to be_nil
+      expect(driver.current_url).to match %r{/landed$}
     end
 
-    it "is possible to not follow redirects" do
-      @driver = Capybara::RackTest::Driver.new(TestApp, :follow_redirects => false)
+    it 'is possible to not follow redirects' do
+      driver = described_class.new(TestApp, follow_redirects: false)
 
-      @driver.visit('/redirect')
-      expect(@driver.response.header['Location']).to match %r{/redirect_again$}
-      expect(@driver.current_url).to match %r{/redirect$}
+      driver.visit('/redirect')
+      expect(driver.response.header['Location']).to match %r{/redirect_again$}
+      expect(driver.current_url).to match %r{/redirect$}
     end
   end
 
   describe ':redirect_limit option' do
-    context "with default redirect limit" do
-      before do
-        @driver = Capybara::RackTest::Driver.new(TestApp)
-      end
+    context 'with default redirect limit' do
+      let(:driver) { described_class.new(TestApp) }
 
-      it "should follow 5 redirects" do
-        @driver.visit("/redirect/5/times")
-        expect(@driver.html).to include('redirection complete')
+      it 'should follow 5 redirects' do
+        driver.visit('/redirect/5/times')
+        expect(driver.html).to include('redirection complete')
       end
 
-      it "should not follow more than 6 redirects" do
+      it 'should not follow more than 6 redirects' do
         expect do
-          @driver.visit("/redirect/6/times")
+          driver.visit('/redirect/6/times')
         end.to raise_error(Capybara::InfiniteRedirectError)
       end
     end
 
-    context "with 21 redirect limit" do
-      before do
-        @driver = Capybara::RackTest::Driver.new(TestApp, :redirect_limit => 21)
-      end
+    context 'with 21 redirect limit' do
+      let(:driver) { described_class.new(TestApp, redirect_limit: 21) }
 
-      it "should follow 21 redirects" do
-        @driver.visit("/redirect/21/times")
-        expect(@driver.html).to include('redirection complete')
+      it 'should follow 21 redirects' do
+        driver.visit('/redirect/21/times')
+        expect(driver.html).to include('redirection complete')
       end
 
-      it "should not follow more than 21 redirects" do
+      it 'should not follow more than 21 redirects' do
         expect do
-          @driver.visit("/redirect/22/times")
+          driver.visit('/redirect/22/times')
         end.to raise_error(Capybara::InfiniteRedirectError)
       end
     end
   end
 end
 
+RSpec.describe 'Capybara::String' do
+  it 'should use gumbo' do
+    skip 'Only  valid if gumbo is included' unless nokogumbo_required
+    allow(Nokogiri).to receive(:HTML5).and_call_original
+    Capybara.string('<div id=test_div></div>')
+    expect(Nokogiri).to have_received(:HTML5)
+  end
+end
+
 module CSSHandlerIncludeTester
   def dont_extend_css_handler
     raise 'should never be called'
   end
 end
-include CSSHandlerIncludeTester
 
-RSpec.describe  Capybara::RackTest::CSSHandlers do
-  it "should not be extended by global includes" do
-    expect(Capybara::RackTest::CSSHandlers.new).not_to respond_to(:dont_extend_css_handler)
+RSpec.describe Capybara::RackTest::CSSHandlers do
+  include CSSHandlerIncludeTester
+
+  it 'should not be extended by global includes' do
+    expect(described_class.new).not_to respond_to(:dont_extend_css_handler)
   end
 end
-