capybara 3.0.0 → 3.40.0

data/lib/capybara/spec/session/{assert_current_path.rb → assert_current_path_spec.rb}

@@ -5,39 +5,40 @@ Capybara::SpecHelper.spec '#assert_current_path' do
     @session.visit('/with_js')
   end
 
-  it "should not raise if the page has the given current path" do
+  it 'should not raise if the page has the given current path' do
     expect { @session.assert_current_path('/with_js') }.not_to raise_error
   end
 
-  it "should allow regexp matches" do
+  it 'should allow regexp matches' do
     expect { @session.assert_current_path(/w[a-z]{3}_js/) }.not_to raise_error
   end
 
-  it "should wait for current_path", requires: [:js] do
-    @session.click_link("Change page")
-    expect { @session.assert_current_path("/with_html") }.not_to raise_error
+  it 'should wait for current_path', requires: [:js] do
+    @session.click_link('Change page')
+    expect { @session.assert_current_path('/with_html') }.not_to raise_error
   end
 
-  it "should raise if the page has not the given current_path" do
+  it 'should raise if the page has not the given current_path' do
     expect { @session.assert_current_path('/with_html') }.to raise_error(Capybara::ExpectationNotMet, 'expected "/with_js" to equal "/with_html"')
   end
 
-  it "should check query options" do
+  it 'should check query options' do
     @session.visit('/with_js?test=test')
     expect { @session.assert_current_path('/with_js?test=test') }.not_to raise_error
   end
 
-  it "should compare the full url" do
+  it 'should compare the full url' do
     expect { @session.assert_current_path(%r{\Ahttp://[^/]*/with_js\Z}, url: true) }.not_to raise_error
   end
 
-  it "should ignore the query" do
+  it 'should ignore the query' do
     @session.visit('/with_js?test=test')
     expect { @session.assert_current_path('/with_js', ignore_query: true) }.not_to raise_error
   end
 
-  it "should not cause an exception when current_url is nil" do
-    allow_any_instance_of(Capybara::Session).to receive(:current_url) { nil }
+  it 'should not cause an exception when current_url is nil' do
+    allow(@session).to receive(:current_url).and_return(nil)
+    allow(@session.page).to receive(:current_url).and_return(nil) if @session.respond_to? :page
 
     expect { @session.assert_current_path(nil) }.not_to raise_error
   end
@@ -48,25 +49,26 @@ Capybara::SpecHelper.spec '#assert_no_current_path?' do
     @session.visit('/with_js')
   end
 
-  it "should raise if the page has the given current_path" do
+  it 'should raise if the page has the given current_path' do
     expect { @session.assert_no_current_path('/with_js') }.to raise_error(Capybara::ExpectationNotMet)
   end
 
-  it "should allow regexp matches" do
+  it 'should allow regexp matches' do
     expect { @session.assert_no_current_path(/monkey/) }.not_to raise_error
   end
 
-  it "should wait for current_path to disappear", requires: [:js] do
-    @session.click_link("Change page")
+  it 'should wait for current_path to disappear', requires: [:js] do
+    @session.click_link('Change page')
     expect { @session.assert_no_current_path('/with_js') }.not_to raise_error
   end
 
-  it "should not raise if the page has not the given current_path" do
+  it 'should not raise if the page has not the given current_path' do
     expect { @session.assert_no_current_path('/with_html') }.not_to raise_error
   end
 
-  it "should not cause an exception when current_url is nil" do
-    allow_any_instance_of(Capybara::Session).to receive(:current_url) { nil }
+  it 'should not cause an exception when current_url is nil' do
+    allow(@session).to receive(:current_url).and_return(nil)
+    allow(@session.page).to receive(:current_url).and_return(nil) if @session.respond_to? :page
 
     expect { @session.assert_no_current_path('/with_html') }.not_to raise_error
   end

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

@@ -0,0 +1,143 @@
+# frozen_string_literal: true
+
+Capybara::SpecHelper.spec '#assert_selector' do
+  before do
+    @session.visit('/with_html')
+  end
+
+  it 'should be true if the given selector is on the page' do
+    @session.assert_selector(:xpath, '//p')
+    @session.assert_selector(:css, 'p a#foo')
+    @session.assert_selector("//p[contains(.,'est')]")
+  end
+
+  it 'should be false if the given selector is not on the page' do
+    expect { @session.assert_selector(:xpath, '//abbr') }.to raise_error(Capybara::ElementNotFound)
+    expect { @session.assert_selector(:css, 'p a#doesnotexist') }.to raise_error(Capybara::ElementNotFound)
+    expect { @session.assert_selector("//p[contains(.,'thisstringisnotonpage')]") }.to raise_error(Capybara::ElementNotFound)
+  end
+
+  it 'should use default selector' do
+    Capybara.default_selector = :css
+    expect { @session.assert_selector('p a#doesnotexist') }.to raise_error(Capybara::ElementNotFound)
+    @session.assert_selector('p a#foo')
+  end
+
+  it 'should respect scopes' do
+    @session.within "//p[@id='first']" do
+      @session.assert_selector(".//a[@id='foo']")
+      expect { @session.assert_selector(".//a[@id='red']") }.to raise_error(Capybara::ElementNotFound)
+    end
+  end
+
+  context 'with count' do
+    it 'should be true if the content is on the page the given number of times' do
+      @session.assert_selector('//p', count: 3)
+      @session.assert_selector("//p//a[@id='foo']", count: 1)
+      @session.assert_selector("//p[contains(.,'est')]", count: 1)
+    end
+
+    it 'should be false if the content is on the page the given number of times' do
+      expect { @session.assert_selector('//p', count: 6) }.to raise_error(Capybara::ElementNotFound)
+      expect { @session.assert_selector("//p//a[@id='foo']", count: 2) }.to raise_error(Capybara::ElementNotFound)
+      expect { @session.assert_selector("//p[contains(.,'est')]", count: 5) }.to raise_error(Capybara::ElementNotFound)
+    end
+
+    it "should be false if the content isn't on the page at all" do
+      expect { @session.assert_selector('//abbr', count: 2) }.to raise_error(Capybara::ElementNotFound)
+      expect { @session.assert_selector("//p//a[@id='doesnotexist']", count: 1) }.to raise_error(Capybara::ElementNotFound)
+    end
+  end
+
+  context 'with text' do
+    it 'should discard all matches where the given string is not contained' do
+      @session.assert_selector('//p//a', text: 'Redirect', count: 1)
+      expect { @session.assert_selector('//p', text: 'Doesnotexist') }.to raise_error(Capybara::ElementNotFound)
+    end
+
+    it 'should discard all matches where the given regexp is not matched' do
+      @session.assert_selector('//p//a', text: /re[dab]i/i, count: 1)
+      expect { @session.assert_selector('//p//a', text: /Red$/) }.to raise_error(Capybara::ElementNotFound)
+    end
+  end
+
+  context 'with wait', requires: [:js] do
+    it 'should find element if it appears before given wait duration' do
+      Capybara.using_wait_time(0.1) do
+        @session.visit('/with_js')
+        @session.click_link('Click me')
+        @session.assert_selector(:css, 'a#has-been-clicked', text: 'Has been clicked', wait: 2)
+      end
+    end
+  end
+end
+
+Capybara::SpecHelper.spec '#assert_no_selector' do
+  before do
+    @session.visit('/with_html')
+  end
+
+  it 'should be false if the given selector is on the page' do
+    expect { @session.assert_no_selector(:xpath, '//p') }.to raise_error(Capybara::ElementNotFound)
+    expect { @session.assert_no_selector(:css, 'p a#foo') }.to raise_error(Capybara::ElementNotFound)
+    expect { @session.assert_no_selector("//p[contains(.,'est')]") }.to raise_error(Capybara::ElementNotFound)
+  end
+
+  it 'should be true if the given selector is not on the page' do
+    @session.assert_no_selector(:xpath, '//abbr')
+    @session.assert_no_selector(:css, 'p a#doesnotexist')
+    @session.assert_no_selector("//p[contains(.,'thisstringisnotonpage')]")
+  end
+
+  it 'should use default selector' do
+    Capybara.default_selector = :css
+    @session.assert_no_selector('p a#doesnotexist')
+    expect { @session.assert_no_selector('p a#foo') }.to raise_error(Capybara::ElementNotFound)
+  end
+
+  it 'should respect scopes' do
+    @session.within "//p[@id='first']" do
+      expect { @session.assert_no_selector(".//a[@id='foo']") }.to raise_error(Capybara::ElementNotFound)
+      @session.assert_no_selector(".//a[@id='red']")
+    end
+  end
+
+  context 'with count' do
+    it 'should be false if the content is on the page the given number of times' do
+      expect { @session.assert_no_selector('//p', count: 3) }.to raise_error(Capybara::ElementNotFound)
+      expect { @session.assert_no_selector("//p//a[@id='foo']", count: 1) }.to raise_error(Capybara::ElementNotFound)
+      expect { @session.assert_no_selector("//p[contains(.,'est')]", count: 1) }.to raise_error(Capybara::ElementNotFound)
+    end
+
+    it 'should be true if the content is on the page the wrong number of times' do
+      @session.assert_no_selector('//p', count: 6)
+      @session.assert_no_selector("//p//a[@id='foo']", count: 2)
+      @session.assert_no_selector("//p[contains(.,'est')]", count: 5)
+    end
+
+    it "should be true if the content isn't on the page at all" do
+      @session.assert_no_selector('//abbr', count: 2)
+      @session.assert_no_selector("//p//a[@id='doesnotexist']", count: 1)
+    end
+  end
+
+  context 'with text' do
+    it 'should discard all matches where the given string is contained' do
+      expect { @session.assert_no_selector('//p//a', text: 'Redirect', count: 1) }.to raise_error(Capybara::ElementNotFound)
+      @session.assert_no_selector('//p', text: 'Doesnotexist')
+    end
+
+    it 'should discard all matches where the given regexp is matched' do
+      expect { @session.assert_no_selector('//p//a', text: /re[dab]i/i, count: 1) }.to raise_error(Capybara::ElementNotFound)
+      @session.assert_no_selector('//p//a', text: /Red$/)
+    end
+  end
+
+  context 'with wait', requires: [:js] do
+    it 'should not find element if it appears after given wait duration' do
+      @session.visit('/with_js')
+      @session.click_link('Click me')
+      @session.assert_no_selector(:css, 'a#has-been-clicked', text: 'Has been clicked', wait: 0.1)
+    end
+  end
+end

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

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+Capybara::SpecHelper.spec '#assert_matches_style', requires: [:css] do
+  it 'should not raise if the elements style contains the given properties' do
+    @session.visit('/with_html')
+    expect do
+      @session.find(:css, '#first').assert_matches_style(display: 'block')
+    end.not_to raise_error
+  end
+
+  it "should raise error if the elements style doesn't contain the given properties" do
+    @session.visit('/with_html')
+    expect do
+      @session.find(:css, '#first').assert_matches_style(display: 'inline')
+    end.to raise_error(Capybara::ExpectationNotMet, 'Expected node to have styles {"display"=>"inline"}. Actual styles were {"display"=>"block"}')
+  end
+
+  it 'should wait for style', requires: %i[css js] do
+    @session.visit('/with_js')
+    el = @session.find(:css, '#change')
+    @session.click_link('Change size')
+    expect do
+      el.assert_matches_style({ 'font-size': '50px' }, wait: 3)
+    end.not_to raise_error
+  end
+end

data/lib/capybara/spec/session/{assert_text.rb → assert_text_spec.rb}

@@ -1,23 +1,42 @@
 # frozen_string_literal: true
 
 Capybara::SpecHelper.spec '#assert_text' do
-  it "should be true if the given text is on the page" do
+  it 'should be true if the given text is on the page' do
     @session.visit('/with_html')
-    expect(@session.assert_text('est')).to eq(true)
-    expect(@session.assert_text('Lorem')).to eq(true)
-    expect(@session.assert_text('Redirect')).to eq(true)
-    expect(@session.assert_text(:Redirect)).to eq(true)
-    expect(@session.assert_text('text with   whitespace')).to eq(true)
+    expect(@session.assert_text('est')).to be(true)
+    expect(@session.assert_text('Lorem')).to be(true)
+    expect(@session.assert_text('Redirect')).to be(true)
+    expect(@session.assert_text(:Redirect)).to be(true)
+    expect(@session.assert_text('text with   whitespace')).to be(true)
   end
 
-  it "should take scopes into account" do
+  it 'should support collapsing whitespace' do
+    @session.visit('/with_html')
+    expect(@session.assert_text('text with whitespace', normalize_ws: true)).to be(true)
+  end
+
+  context 'with enabled default collapsing whitespace' do
+    before { Capybara.default_normalize_ws = true }
+
+    it 'should be true if the given unnormalized text is on the page' do
+      @session.visit('/with_html')
+      expect(@session.assert_text('text with   whitespace', normalize_ws: false)).to be(true)
+    end
+
+    it 'should support collapsing whitespace' do
+      @session.visit('/with_html')
+      expect(@session.assert_text('text with whitespace')).to be(true)
+    end
+  end
+
+  it 'should take scopes into account' do
     @session.visit('/with_html')
     @session.within("//a[@title='awesome title']") do
-      expect(@session.assert_text('labore')).to eq(true)
+      expect(@session.assert_text('labore')).to be(true)
     end
   end
 
-  it "should raise if scoped to an element which does not have the text" do
+  it 'should raise if scoped to an element which does not have the text' do
     @session.visit('/with_html')
     @session.within("//a[@title='awesome title']") do
       expect do
@@ -26,18 +45,18 @@ Capybara::SpecHelper.spec '#assert_text' do
     end
   end
 
-  it "should be true if :all given and text is invisible." do
+  it 'should be true if :all given and text is invisible.' do
     @session.visit('/with_html')
-    expect(@session.assert_text(:all, 'Some of this text is hidden!')).to eq(true)
+    expect(@session.assert_text(:all, 'Some of this text is hidden!')).to be(true)
   end
 
-  it "should be true if `Capybara.ignore_hidden_elements = true` and text is invisible." do
+  it 'should be true if `Capybara.ignore_hidden_elements = true` and text is invisible.' do
     Capybara.ignore_hidden_elements = false
     @session.visit('/with_html')
-    expect(@session.assert_text('Some of this text is hidden!')).to eq(true)
+    expect(@session.assert_text('Some of this text is hidden!')).to be(true)
   end
 
-  it "should raise error with a helpful message if the requested text is present but invisible" do
+  it 'should raise error with a helpful message if the requested text is present but invisible' do
     @session.visit('/with_html')
     el = @session.find(:css, '#hidden-text')
     expect do
@@ -45,14 +64,14 @@ Capybara::SpecHelper.spec '#assert_text' do
     end.to raise_error(Capybara::ExpectationNotMet, /it was found 1 time including non-visible text/)
   end
 
-  it "should raise error with a helpful message if the requested text is present but with incorrect case" do
+  it 'should raise error with a helpful message if the requested text is present but with incorrect case' do
     @session.visit('/with_html')
     expect do
       @session.assert_text('Text With   Whitespace')
     end.to raise_error(Capybara::ExpectationNotMet, /it was found 1 time using a case insensitive search/)
   end
 
-  it "should raise error with helpful message if requested text is present but invisible and with incorrect case", requires: [:js] do
+  it 'should raise error with helpful message if requested text is present but invisible and with incorrect case', requires: [:js] do
     @session.visit('/with_html')
     el = @session.find(:css, '#uppercase')
     expect do
@@ -60,54 +79,55 @@ Capybara::SpecHelper.spec '#assert_text' do
     end.to raise_error(Capybara::ExpectationNotMet, /it was found 1 time using a case insensitive search and it was found 1 time including non-visible text/)
   end
 
-  it "should raise the correct error if requested text is missing but contains regex special characters" do
+  it 'should raise the correct error if requested text is missing but contains regex special characters' do
     @session.visit('/with_html')
     expect do
       @session.assert_text('[]*.')
     end.to raise_error(Capybara::ExpectationNotMet, /expected to find text "\[\]\*\."/)
   end
 
-  it "should be true if the text in the page matches given regexp" do
+  it 'should be true if the text in the page matches given regexp' do
     @session.visit('/with_html')
-    expect(@session.assert_text(/Lorem/)).to eq(true)
+    expect(@session.assert_text(/Lorem/)).to be(true)
   end
 
-  it "should be raise error if the text in the page doesn't match given regexp" do
+  it "should raise error if the text in the page doesn't match given regexp" do
     @session.visit('/with_html')
     expect do
       @session.assert_text(/xxxxyzzz/)
-    end.to raise_error(Capybara::ExpectationNotMet, /\Aexpected to find text matching \/xxxxyzzz\/ in "This is a test\\nHeader Class(.+)"\Z/)
+    end.to raise_error(Capybara::ExpectationNotMet, %r{\Aexpected to find text matching /xxxxyzzz/ in "This is a test\\nHeader Class(.+)"\Z})
   end
 
-  it "should escape any characters that would have special meaning in a regexp" do
+  it 'should escape any characters that would have special meaning in a regexp' do
     @session.visit('/with_html')
     expect do
       @session.assert_text('.orem')
     end.to raise_error(Capybara::ExpectationNotMet)
   end
 
-  it "should wait for text to appear", requires: [:js] do
+  it 'should wait for text to appear', requires: [:js] do
+    Capybara.default_max_wait_time = 2
     @session.visit('/with_js')
     @session.click_link('Click me')
-    expect(@session.assert_text('Has been clicked')).to eq(true)
+    expect(@session.assert_text('Has been clicked')).to be(true)
   end
 
-  context "with between" do
-    it "should be true if the text occurs within the range given" do
+  context 'with between' do
+    it 'should be true if the text occurs within the range given' do
       @session.visit('/with_count')
-      expect(@session.assert_text('count', between: 1..3)).to eq(true)
+      expect(@session.assert_text('count', between: 1..3)).to be(true)
     end
 
-    it "should be false if the text occurs more or fewer times than range" do
+    it 'should be false if the text occurs more or fewer times than range' do
       @session.visit('/with_html')
       expect do
         @session.find(:css, '.number').assert_text(/\d/, between: 0..1)
-      end.to raise_error(Capybara::ExpectationNotMet, "expected to find text matching /\\d/ between 0 and 1 times but found 2 times in \"42\"")
+      end.to raise_error(Capybara::ExpectationNotMet, 'expected to find text matching /\\d/ between 0 and 1 times but found 2 times in "42"')
     end
   end
 
-  context "with wait", requires: [:js] do
-    it "should find element if it appears before given wait duration" do
+  context 'with wait', requires: [:js] do
+    it 'should find element if it appears before given wait duration' do
       Capybara.using_wait_time(0) do
         @session.visit('/with_js')
         @session.find(:css, '#reload-list').click
@@ -115,7 +135,7 @@ Capybara::SpecHelper.spec '#assert_text' do
       end
     end
 
-    it "should raise error if it appears after given wait duration" do
+    it 'should raise error if it appears after given wait duration' do
       Capybara.using_wait_time(0) do
         @session.visit('/with_js')
         @session.find(:css, '#reload-list').click
@@ -128,7 +148,7 @@ Capybara::SpecHelper.spec '#assert_text' do
   end
 
   context 'with multiple count filters' do
-    before(:each) do
+    before do
       @session.visit('/with_html')
     end
 
@@ -137,58 +157,62 @@ Capybara::SpecHelper.spec '#assert_text' do
             minimum: 6,
             maximum: 0,
             between: 0..4 }
-      expect { @session.assert_text('Header', o) }.not_to raise_error
+      expect { @session.assert_text('Header', **o) }.not_to raise_error
     end
+
     context 'with no :count expectation' do
       it 'fails if :minimum is not met' do
         o = { minimum: 6,
               maximum: 5,
               between: 2..7 }
-        expect { @session.assert_text('Header', o) }.to raise_error(Capybara::ExpectationNotMet)
+        expect { @session.assert_text('Header', **o) }.to raise_error(Capybara::ExpectationNotMet)
       end
+
       it 'fails if :maximum is not met' do
         o = { minimum: 0,
               maximum: 0,
               between: 2..7 }
-        expect { @session.assert_text('Header', o) }.to raise_error(Capybara::ExpectationNotMet)
+        expect { @session.assert_text('Header', **o) }.to raise_error(Capybara::ExpectationNotMet)
       end
+
       it 'fails if :between is not met' do
         o = { minimum: 0,
               maximum: 5,
               between: 0..4 }
-        expect { @session.assert_text('Header', o) }.to raise_error(Capybara::ExpectationNotMet)
+        expect { @session.assert_text('Header', **o) }.to raise_error(Capybara::ExpectationNotMet)
       end
+
       it 'succeeds if all combineable expectations are met' do
         o = { minimum: 0,
               maximum: 5,
               between: 2..7 }
-        expect { @session.assert_text('Header', o) }.not_to raise_error
+        expect { @session.assert_text('Header', **o) }.not_to raise_error
       end
     end
   end
 end
 
 Capybara::SpecHelper.spec '#assert_no_text' do
-  it "should raise error if the given text is on the page at least once" do
+  it 'should raise error if the given text is on the page at least once' do
     @session.visit('/with_html')
     expect do
       @session.assert_no_text('Lorem')
     end.to raise_error(Capybara::ExpectationNotMet, /\Aexpected not to find text "Lorem" in "This is a test.*"\z/)
   end
 
-  it "should be true if scoped to an element which does not have the text" do
+  it 'should be true if scoped to an element which does not have the text' do
     @session.visit('/with_html')
     @session.within("//a[@title='awesome title']") do
-      expect(@session.assert_no_text('monkey')).to eq(true)
+      expect(@session.assert_no_text('monkey')).to be(true)
     end
   end
 
-  it "should be true if the given text is on the page but not visible" do
+  it 'should be true if the given text is on the page but not visible' do
     @session.visit('/with_html')
-    expect(@session.assert_no_text('Inside element with hidden ancestor')).to eq(true)
+    expect(@session.assert_no_text('Inside element with hidden ancestor')).to be(true)
   end
 
-  it "should raise error if :all given and text is invisible." do
+  it 'should raise error if :all given and text is invisible.' do
     @session.visit('/with_html')
     el = @session.find(:css, '#hidden-text', visible: false)
     expect do
@@ -196,7 +220,7 @@ Capybara::SpecHelper.spec '#assert_no_text' do
     end.to raise_error(Capybara::ExpectationNotMet, 'expected not to find text "Some of this text is hidden!" in "Some of this text is hidden!"')
   end
 
-  it "should raise error if :all given and text is invisible." do
+  it 'should raise error if :all given and text is invisible.' do
     @session.visit('/with_html')
     el = @session.find(:css, '#some-hidden-text', visible: false)
     expect do
@@ -209,22 +233,22 @@ Capybara::SpecHelper.spec '#assert_no_text' do
     @session.assert_no_text(/xxxxyzzz/)
   end
 
-  context "with count" do
-    it "should be true if the text occurs within the range given" do
+  context 'with count' do
+    it 'should be true if the text occurs within the range given' do
       @session.visit('/with_count')
-      expect(@session.assert_text('count', count: 2)).to eq(true)
+      expect(@session.assert_text('count', count: 2)).to be(true)
     end
 
-    it "should be false if the text occurs more or fewer times than range" do
+    it 'should be false if the text occurs more or fewer times than range' do
       @session.visit('/with_html')
       expect do
         @session.find(:css, '.number').assert_text(/\d/, count: 1)
-      end.to raise_error(Capybara::ExpectationNotMet, "expected to find text matching /\\d/ 1 time but found 2 times in \"42\"")
+      end.to raise_error(Capybara::ExpectationNotMet, 'expected to find text matching /\\d/ 1 time but found 2 times in "42"')
     end
   end
 
-  context "with wait", requires: [:js] do
-    it "should not find element if it appears after given wait duration" do
+  context 'with wait', requires: [:js] do
+    it 'should not find element if it appears after given wait duration' do
       @session.visit('/with_js')
       @session.click_link('Click me')
       @session.find(:css, '#reload-list').click

data/lib/capybara/spec/session/{assert_title.rb → assert_title_spec.rb}

@@ -11,13 +11,13 @@ Capybara::SpecHelper.spec '#assert_title' do
     end.not_to raise_error
   end
 
-  it "should not raise when given an empty string" do
+  it 'should not raise when given an empty string' do
     expect do
       @session.assert_title('')
     end.not_to raise_error
   end
 
-  it "should allow regexp matches" do
+  it 'should allow regexp matches' do
     expect do
       @session.assert_title(/w[a-z]{3}_js/)
     end.not_to raise_error
@@ -27,10 +27,10 @@ Capybara::SpecHelper.spec '#assert_title' do
     end.to raise_error(Capybara::ExpectationNotMet, 'expected "with_js" to match /w[a-z]{10}_js/')
   end
 
-  it "should wait for title", requires: [:js] do
-    @session.click_link("Change title")
+  it 'should wait for title', requires: [:js] do
+    @session.click_link('Change title')
     expect do
-      @session.assert_title("changed title", wait: 3)
+      @session.assert_title('changed title', wait: 3)
     end.not_to raise_error
   end
 
@@ -40,12 +40,12 @@ Capybara::SpecHelper.spec '#assert_title' do
     end.to raise_error(Capybara::ExpectationNotMet, 'expected "with_js" to include "monkey"')
   end
 
-  it "should not normalize given title" do
+  it 'should not normalize given title' do
     @session.visit('/with_js')
     expect { @session.assert_title('  with_js  ') }.to raise_error(Capybara::ExpectationNotMet)
   end
 
-  it "should match correctly normalized title" do
+  it 'should match correctly normalized title' do
     uri = Addressable::URI.parse('/with_title')
     uri.query_values = { title: '   with space  title   ' }
     @session.visit(uri.to_s)
@@ -53,7 +53,7 @@ Capybara::SpecHelper.spec '#assert_title' do
     expect { @session.assert_title('with space title') }.to raise_error(Capybara::ExpectationNotMet)
   end
 
-  it "should not normalize given title in error message" do
+  it 'should not normalize given title in error message' do
     expect do
       @session.assert_title(2)
     end.to raise_error(Capybara::ExpectationNotMet, 'expected "with_js" to include "2"')
@@ -65,21 +65,21 @@ Capybara::SpecHelper.spec '#assert_no_title' do
     @session.visit('/with_js')
   end
 
-  it "should raise error if the title contains the given string" do
+  it 'should raise error if the title contains the given string' do
     expect do
       @session.assert_no_title('with_j')
     end.to raise_error(Capybara::ExpectationNotMet, 'expected "with_js" not to include "with_j"')
   end
 
-  it "should allow regexp matches" do
+  it 'should allow regexp matches' do
     expect do
       @session.assert_no_title(/w[a-z]{3}_js/)
     end.to raise_error(Capybara::ExpectationNotMet, 'expected "with_js" not to match /w[a-z]{3}_js/')
     @session.assert_no_title(/monkey/)
   end
 
-  it "should wait for title to disappear", requires: [:js] do
-    @session.click_link("Change title")
+  it 'should wait for title to disappear', requires: [:js] do
+    @session.click_link('Change title')
     expect do
       @session.assert_no_title('with_js', wait: 3)
     end.not_to raise_error