capybara 3.3.1 → 3.4.0

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

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 Capybara::SpecHelper.spec '#response_headers' do
-  it "should return response headers", requires: [:response_headers] do
+  it 'should return response headers', requires: [:response_headers] do
     @session.visit('/with_simple_html')
     expect(@session.response_headers['Content-Type']).to match %r{text/html}
   end

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

@@ -1,12 +1,12 @@
 # frozen_string_literal: true
 
 Capybara::SpecHelper.spec '#html' do
-  it "should return the unmodified page body" do
+  it 'should return the unmodified page body' do
     @session.visit('/')
     expect(@session.html).to include('Hello world!')
   end
 
-  it "should return the current state of the page", requires: [:js] do
+  it 'should return the current state of the page', requires: [:js] do
     @session.visit('/with_js')
     expect(@session.html).to include('I changed it')
     expect(@session.html).not_to include('This is text')
@@ -14,12 +14,12 @@ Capybara::SpecHelper.spec '#html' do
 end
 
 Capybara::SpecHelper.spec '#source' do
-  it "should return the unmodified page source" do
+  it 'should return the unmodified page source' do
     @session.visit('/')
     expect(@session.source).to include('Hello world!')
   end
 
-  it "should return the current state of the page", requires: [:js] do
+  it 'should return the current state of the page', requires: [:js] do
     @session.visit('/with_js')
     expect(@session.source).to include('I changed it')
     expect(@session.source).not_to include('This is text')
@@ -27,12 +27,12 @@ Capybara::SpecHelper.spec '#source' do
 end
 
 Capybara::SpecHelper.spec '#body' do
-  it "should return the unmodified page source" do
+  it 'should return the unmodified page source' do
     @session.visit('/')
     expect(@session.body).to include('Hello world!')
   end
 
-  it "should return the current state of the page", requires: [:js] do
+  it 'should return the current state of the page', requires: [:js] do
     @session.visit('/with_js')
     expect(@session.body).to include('I changed it')
     expect(@session.body).not_to include('This is text')

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

@@ -1,11 +1,11 @@
 # frozen_string_literal: true
 
-Capybara::SpecHelper.spec "node" do
+Capybara::SpecHelper.spec 'node' do
   before do
     @session.visit('/with_html')
   end
 
-  it "should act like a session object" do
+  it 'should act like a session object' do
     @session.visit('/form')
     @form = @session.find(:css, '#get-form')
     expect(@form).to have_field('Middle Name')
@@ -15,70 +15,70 @@ Capybara::SpecHelper.spec "node" do
     expect(extract_results(@session)['middle_name']).to eq('Monkey')
   end
 
-  it "should scope CSS selectors" do
+  it 'should scope CSS selectors' do
     expect(@session.find(:css, '#second')).to have_no_css('h1')
   end
 
-  describe "#query_scope" do
-    it "should have a reference to the element the query was evaluated on if there is one" do
+  describe '#query_scope' do
+    it 'should have a reference to the element the query was evaluated on if there is one' do
       @node = @session.find(:css, '#first')
       expect(@node.query_scope).to eq(@node.session.document)
       expect(@node.find(:css, '#foo').query_scope).to eq(@node)
     end
   end
 
-  describe "#text" do
-    it "should extract node texts" do
+  describe '#text' do
+    it 'should extract node texts' do
       expect(@session.all('//a')[0].text).to eq('labore')
       expect(@session.all('//a')[1].text).to eq('ullamco')
     end
 
-    it "should return document text on /html selector" do
+    it 'should return document text on /html selector' do
       @session.visit('/with_simple_html')
       expect(@session.all('/html')[0].text).to eq('Bar')
     end
   end
 
-  describe "#[]" do
-    it "should extract node attributes" do
+  describe '#[]' do
+    it 'should extract node attributes' do
       expect(@session.all('//a')[0][:class]).to eq('simple')
       expect(@session.all('//a')[1][:id]).to eq('foo')
       expect(@session.all('//input')[0][:type]).to eq('text')
     end
 
-    it "should extract boolean node attributes" do
+    it 'should extract boolean node attributes' do
       expect(@session.find('//input[@id="checked_field"]')[:checked]).to be_truthy
     end
   end
 
-  describe "#style", requires: [:css] do
-    it "should return the computed style value" do
+  describe '#style', requires: [:css] do
+    it 'should return the computed style value' do
       expect(@session.find(:css, '#first').style('display')).to eq('display' => 'block')
       expect(@session.find(:css, '#second').style(:display)).to eq('display' => 'inline')
     end
 
-    it "should return multiple style values" do
+    it 'should return multiple style values' do
       expect(@session.find(:css, '#first').style('display', :'line-height')).to eq('display' => 'block', 'line-height' => '25px')
     end
   end
 
-  describe "#value" do
-    it "should allow retrieval of the value" do
+  describe '#value' do
+    it 'should allow retrieval of the value' do
       expect(@session.find('//textarea[@id="normal"]').value).to eq('banana')
     end
 
-    it "should not swallow extra newlines in textarea" do
+    it 'should not swallow extra newlines in textarea' do
       expect(@session.find('//textarea[@id="additional_newline"]').value).to eq("\nbanana")
     end
 
-    it "should not swallow leading newlines for set content in textarea" do
+    it 'should not swallow leading newlines for set content in textarea' do
       @session.find('//textarea[@id="normal"]').set("\nbanana")
       expect(@session.find('//textarea[@id="normal"]').value).to eq("\nbanana")
     end
 
-    it "return any HTML content in textarea" do
-      @session.find('//textarea[1]').set("some <em>html</em> here")
-      expect(@session.find('//textarea[1]').value).to eq("some <em>html</em> here")
+    it 'return any HTML content in textarea' do
+      @session.find('//textarea[1]').set('some <em>html</em> here')
+      expect(@session.find('//textarea[1]').value).to eq('some <em>html</em> here')
     end
 
     it "defaults to 'on' for checkbox" do
@@ -92,24 +92,24 @@ Capybara::SpecHelper.spec "node" do
     end
   end
 
-  describe "#set" do
-    it "should allow assignment of field value" do
+  describe '#set' do
+    it 'should allow assignment of field value' do
       expect(@session.first('//input').value).to eq('monkey')
       @session.first('//input').set('gorilla')
       expect(@session.first('//input').value).to eq('gorilla')
     end
 
-    it "should fill the field even if the caret was not at the end", requires: [:js] do
+    it 'should fill the field even if the caret was not at the end', requires: [:js] do
       @session.execute_script("var el = document.getElementById('test_field'); el.focus(); el.setSelectionRange(0, 0);")
       @session.first('//input').set('')
       expect(@session.first('//input').value).to eq('')
     end
 
-    it "should raise if the text field is readonly" do
+    it 'should raise if the text field is readonly' do
       expect { @session.first('//input[@readonly]').set('changed') }.to raise_error(Capybara::ReadOnlyElementError)
     end
 
-    it "should raise if the textarea is readonly" do
+    it 'should raise if the textarea is readonly' do
       expect { @session.first('//textarea[@readonly]').set('changed') }.to raise_error(Capybara::ReadOnlyElementError)
     end
 
@@ -121,7 +121,7 @@ Capybara::SpecHelper.spec "node" do
       expect(element.base).to have_received(:set).with('gorilla', clear: :backspace)
     end
 
-    context "with a contenteditable element", requires: [:js] do
+    context 'with a contenteditable element', requires: [:js] do
       it 'should allow me to change the contents' do
         @session.visit('/with_js')
         @session.find(:css, '#existing_content_editable').set('WYSIWYG')
@@ -143,75 +143,81 @@ Capybara::SpecHelper.spec "node" do
     end
   end
 
-  describe "#tag_name" do
-    it "should extract node tag name" do
+  describe '#tag_name' do
+    it 'should extract node tag name' do
       expect(@session.all('//a')[0].tag_name).to eq('a')
       expect(@session.all('//a')[1].tag_name).to eq('a')
       expect(@session.all('//p')[1].tag_name).to eq('p')
     end
   end
 
-  describe "#disabled?" do
-    it "should extract disabled node" do
+  describe '#disabled?' do
+    it 'should extract disabled node' do
       @session.visit('/form')
       expect(@session.find('//input[@id="customer_name"]')).to be_disabled
       expect(@session.find('//input[@id="customer_email"]')).not_to be_disabled
     end
 
-    it "should see disabled options as disabled" do
+    it 'should see disabled options as disabled' do
       @session.visit('/form')
       expect(@session.find('//select[@id="form_title"]/option[1]')).not_to be_disabled
       expect(@session.find('//select[@id="form_title"]/option[@disabled]')).to be_disabled
     end
 
-    it "should see enabled options in disabled select as disabled" do
+    it 'should see enabled options in disabled select as disabled' do
       @session.visit('/form')
       expect(@session.find('//select[@id="form_disabled_select"]/option')).to be_disabled
       expect(@session.find('//select[@id="form_disabled_select"]/optgroup/option')).to be_disabled
       expect(@session.find('//select[@id="form_title"]/option[1]')).not_to be_disabled
     end
 
-    it "should see enabled options in disabled optgroup as disabled" do
+    it 'should see enabled options in disabled optgroup as disabled' do
       @session.visit('/form')
-      expect(@session.find('//option', text: "A.B.1")).to be_disabled
-      expect(@session.find('//option', text: "A.2")).not_to be_disabled
+      expect(@session.find('//option', text: 'A.B.1')).to be_disabled
+      expect(@session.find('//option', text: 'A.2')).not_to be_disabled
     end
 
-    it "should see a disabled fieldset as disabled" do
+    it 'should see a disabled fieldset as disabled' do
       @session.visit('/form')
       expect(@session.find(:css, '#form_disabled_fieldset')).to be_disabled
     end
 
-    context "in a disabled fieldset" do
+    context 'in a disabled fieldset' do
       # https://html.spec.whatwg.org/#the-fieldset-element
-      it "should see elements not in first legend as disabled" do
+      it 'should see elements not in first legend as disabled' do
         @session.visit('/form')
         expect(@session.find('//input[@id="form_disabled_fieldset_child"]')).to be_disabled
         expect(@session.find('//input[@id="form_disabled_fieldset_second_legend_child"]')).to be_disabled
         expect(@session.find('//input[@id="form_enabled_fieldset_child"]')).not_to be_disabled
       end
 
-      it "should see elements in first legend as enabled" do
+      it 'should see elements in first legend as enabled' do
         @session.visit('/form')
         expect(@session.find('//input[@id="form_disabled_fieldset_legend_child"]')).not_to be_disabled
       end
 
-      it "should sees options not in first legend as disabled" do
+      it 'should sees options not in first legend as disabled' do
         @session.visit('/form')
         expect(@session.find('//option', text: 'Disabled Child Option')).to be_disabled
       end
     end
 
-    it "should be boolean" do
+    it 'should be boolean' do
       @session.visit('/form')
       expect(@session.find('//select[@id="form_disabled_select"]/option').disabled?).to be true
       expect(@session.find('//select[@id="form_disabled_select2"]/option').disabled?).to be true
       expect(@session.find('//select[@id="form_title"]/option[1]').disabled?).to be false
     end
+
+    it 'should be disabled for all elements that are CSS :disabled' do
+      @session.visit('/form')
+      # sanity check
+      expect(@session.all(:css, ':disabled')).to all(be_disabled)
+    end
   end
 
-  describe "#visible?" do
-    it "should extract node visibility" do
+  describe '#visible?' do
+    it 'should extract node visibility' do
       Capybara.ignore_hidden_elements = false
       expect(@session.first('//a')).to be_visible
 
@@ -222,22 +228,22 @@ Capybara::SpecHelper.spec "node" do
       expect(@session.find('//input[@id="hidden_input"]')).not_to be_visible
     end
 
-    it "should be boolean" do
+    it 'should be boolean' do
       Capybara.ignore_hidden_elements = false
       expect(@session.first('//a').visible?).to be true
       expect(@session.find('//div[@id="hidden"]').visible?).to be false
     end
   end
 
-  describe "#checked?" do
-    it "should extract node checked state" do
+  describe '#checked?' do
+    it 'should extract node checked state' do
       @session.visit('/form')
       expect(@session.find('//input[@id="gender_female"]')).to be_checked
       expect(@session.find('//input[@id="gender_male"]')).not_to be_checked
       expect(@session.first('//h1')).not_to be_checked
     end
 
-    it "should be boolean" do
+    it 'should be boolean' do
       @session.visit('/form')
       expect(@session.find('//input[@id="gender_female"]').checked?).to be true
       expect(@session.find('//input[@id="gender_male"]').checked?).to be false
@@ -245,15 +251,15 @@ Capybara::SpecHelper.spec "node" do
     end
   end
 
-  describe "#selected?" do
-    it "should extract node selected state" do
+  describe '#selected?' do
+    it 'should extract node selected state' do
       @session.visit('/form')
       expect(@session.find('//option[@value="en"]')).to be_selected
       expect(@session.find('//option[@value="sv"]')).not_to be_selected
       expect(@session.first('//h1')).not_to be_selected
     end
 
-    it "should be boolean" do
+    it 'should be boolean' do
       @session.visit('/form')
       expect(@session.find('//option[@value="en"]').selected?).to be true
       expect(@session.find('//option[@value="sv"]').selected?).to be false
@@ -261,33 +267,33 @@ Capybara::SpecHelper.spec "node" do
     end
   end
 
-  describe "#==" do
-    it "preserve object identity" do
+  describe '#==' do
+    it 'preserve object identity' do
       expect(@session.find('//h1') == @session.find('//h1')).to be true # rubocop:disable Lint/UselessComparison
       expect(@session.find('//h1') === @session.find('//h1')).to be true # rubocop:disable Style/CaseEquality, Lint/UselessComparison
       expect(@session.find('//h1').eql?(@session.find('//h1'))).to be false
     end
 
-    it "returns false for unrelated object" do
-      expect(@session.find('//h1') == "Not Capybara::Node::Base").to be false
+    it 'returns false for unrelated object' do
+      expect(@session.find('//h1') == 'Not Capybara::Node::Base').to be false
     end
   end
 
-  describe "#path" do
+  describe '#path' do
     # Testing for specific XPaths here doesn't make sense since there
     # are many that can refer to the same element
     before do
       @session.visit('/path')
     end
 
-    it "returns xpath which points to itself" do
+    it 'returns xpath which points to itself' do
       element = @session.find(:link, 'Second Link')
       expect(@session.find(:xpath, element.path)).to eq(element)
     end
   end
 
-  describe "#trigger", requires: %i[js trigger] do
-    it "should allow triggering of custom JS events" do
+  describe '#trigger', requires: %i[js trigger] do
+    it 'should allow triggering of custom JS events' do
       @session.visit('/with_js')
       @session.find(:css, '#with_focus_event').trigger(:focus)
       expect(@session).to have_css('#focus_event_triggered')
@@ -295,7 +301,7 @@ Capybara::SpecHelper.spec "node" do
   end
 
   describe '#drag_to', requires: %i[js drag] do
-    it "should drag and drop an object" do
+    it 'should drag and drop an object' do
       @session.visit('/with_js')
       element = @session.find('//div[@id="drag"]')
       target = @session.find('//div[@id="drop"]')
@@ -303,7 +309,7 @@ Capybara::SpecHelper.spec "node" do
       expect(@session.find('//div[contains(., "Dropped!")]')).not_to be_nil
     end
 
-    it "should drag and drop if scrolling is needed" do
+    it 'should drag and drop if scrolling is needed' do
       @session.visit('/with_js')
       element = @session.find('//div[@id="drag_scroll"]')
       target = @session.find('//div[@id="drop_scroll"]')
@@ -313,14 +319,14 @@ Capybara::SpecHelper.spec "node" do
   end
 
   describe '#hover', requires: [:hover] do
-    it "should allow hovering on an element" do
+    it 'should allow hovering on an element' do
       @session.visit('/with_hover')
       expect(@session.find(:css, '.wrapper:not(.scroll_needed) .hidden_until_hover', visible: false)).not_to be_visible
       @session.find(:css, '.wrapper:not(.scroll_needed)').hover
       expect(@session.find(:css, '.wrapper:not(.scroll_needed) .hidden_until_hover', visible: false)).to be_visible
     end
 
-    it "should allow hovering on an element that needs to be scrolled into view" do
+    it 'should allow hovering on an element that needs to be scrolled into view' do
       @session.visit('/with_hover')
       expect(@session.find(:css, '.wrapper.scroll_needed .hidden_until_hover', visible: false)).not_to be_visible
       @session.find(:css, '.wrapper.scroll_needed').hover
@@ -329,18 +335,18 @@ Capybara::SpecHelper.spec "node" do
   end
 
   describe '#click' do
-    it "should not follow a link if no href" do
+    it 'should not follow a link if no href' do
       @session.find(:css, '#link_placeholder').click
       expect(@session.current_url).to match(%r{/with_html$})
     end
 
-    it "should go to the same page if href is blank" do
+    it 'should go to the same page if href is blank' do
       @session.find(:css, '#link_blank_href').click
       sleep 1
       expect(@session).to have_current_path('/with_html')
     end
 
-    it "should be able to check a checkbox" do
+    it 'should be able to check a checkbox' do
       @session.visit('form')
       cbox = @session.find(:checkbox, 'form_terms_of_use')
       expect(cbox).not_to be_checked
@@ -348,7 +354,7 @@ Capybara::SpecHelper.spec "node" do
       expect(cbox).to be_checked
     end
 
-    it "should be able to uncheck a checkbox" do
+    it 'should be able to uncheck a checkbox' do
       @session.visit('/form')
       cbox = @session.find(:checkbox, 'form_pets_dog')
       expect(cbox).to be_checked
@@ -356,7 +362,7 @@ Capybara::SpecHelper.spec "node" do
       expect(cbox).not_to be_checked
     end
 
-    it "should be able to select a radio button" do
+    it 'should be able to select a radio button' do
       @session.visit('/form')
       radio = @session.find(:radio_button, 'gender_male')
       expect(radio).not_to be_checked
@@ -364,20 +370,20 @@ Capybara::SpecHelper.spec "node" do
       expect(radio).to be_checked
     end
 
-    it "should allow modifiers", requires: [:js] do
+    it 'should allow modifiers', requires: [:js] do
       @session.visit('/with_js')
       @session.find(:css, '#click-test').click(:shift)
       expect(@session).to have_link('Has been shift clicked')
     end
 
-    it "should allow multiple modifiers", requires: [:js] do
+    it 'should allow multiple modifiers', requires: [:js] do
       @session.visit('with_js')
       @session.find(:css, '#click-test').click(:control, :alt, :meta, :shift)
       # Selenium with Chrome on OSX ctrl-click generates a right click so just verify all keys but not click type
-      expect(@session).to have_link("alt control meta shift")
+      expect(@session).to have_link('alt control meta shift')
     end
 
-    it "should allow to adjust the click offset", requires: [:js] do
+    it 'should allow to adjust the click offset', requires: [:js] do
       @session.visit('with_js')
       @session.find(:css, '#click-test').click(x: 5, y: 5)
       link = @session.find(:link, 'has-been-clicked')
@@ -388,7 +394,7 @@ Capybara::SpecHelper.spec "node" do
       expect(locations[:y].to_f).to be_within(1).of(5)
     end
 
-    it "should be able to click a table row", requires: [:js] do
+    it 'should be able to click a table row', requires: [:js] do
       @session.visit('/tables')
       tr = @session.find(:css, '#agent_table tr:first-child').click
       expect(tr).to have_css('label', text: 'Clicked')
@@ -396,23 +402,19 @@ Capybara::SpecHelper.spec "node" do
   end
 
   describe '#double_click', requires: [:js] do
-    before do
-      pending "selenium-webdriver/geckodriver doesn't generate double click event" if marionette_lt?(59, @session)
-    end
-
-    it "should double click an element" do
+    it 'should double click an element' do
       @session.visit('/with_js')
       @session.find(:css, '#click-test').double_click
       expect(@session.find(:css, '#has-been-double-clicked')).to be_truthy
     end
 
-    it "should allow modifiers", requires: [:js] do
+    it 'should allow modifiers', requires: [:js] do
       @session.visit('/with_js')
       @session.find(:css, '#click-test').double_click(:alt)
       expect(@session).to have_link('Has been alt double clicked')
     end
 
-    it "should allow to adjust the offset", requires: [:js] do
+    it 'should allow to adjust the offset', requires: [:js] do
       @session.visit('with_js')
       @session.find(:css, '#click-test').double_click(x: 10, y: 5)
       link = @session.find(:link, 'has-been-double-clicked')
@@ -425,19 +427,19 @@ Capybara::SpecHelper.spec "node" do
   end
 
   describe '#right_click', requires: [:js] do
-    it "should right click an element" do
+    it 'should right click an element' do
       @session.visit('/with_js')
       @session.find(:css, '#click-test').right_click
       expect(@session.find(:css, '#has-been-right-clicked')).to be_truthy
     end
 
-    it "should allow modifiers", requires: [:js] do
+    it 'should allow modifiers', requires: [:js] do
       @session.visit('/with_js')
       @session.find(:css, '#click-test').right_click(:meta)
       expect(@session).to have_link('Has been meta right clicked')
     end
 
-    it "should allow to adjust the offset", requires: [:js] do
+    it 'should allow to adjust the offset', requires: [:js] do
       @session.visit('with_js')
       @session.find(:css, '#click-test').right_click(x: 10, y: 10)
       link = @session.find(:link, 'has-been-right-clicked')
@@ -450,96 +452,100 @@ Capybara::SpecHelper.spec "node" do
   end
 
   describe '#send_keys', requires: [:send_keys] do
-    it "should send a string of keys to an element" do
+    it 'should send a string of keys to an element' do
       @session.visit('/form')
       @session.find(:css, '#address1_city').send_keys('Oceanside')
       expect(@session.find(:css, '#address1_city').value).to eq 'Oceanside'
     end
 
-    it "should send special characters" do
-      pending "selenium-webdriver/geckodriver doesn't support complex sets of characters" if marionette?(@session)
+    it 'should send special characters' do
       @session.visit('/form')
       @session.find(:css, '#address1_city').send_keys('Ocean', :space, 'sie', :left, 'd')
       expect(@session.find(:css, '#address1_city').value).to eq 'Ocean side'
     end
 
-    it "should allow for multiple simultaneous keys" do
-      pending "selenium-webdriver/geckodriver doesn't support complex sets of characters" if marionette?(@session)
+    it 'should allow for multiple simultaneous keys' do
       @session.visit('/form')
       @session.find(:css, '#address1_city').send_keys([:shift, 'o'], 'ceanside')
       expect(@session.find(:css, '#address1_city').value).to eq 'Oceanside'
     end
 
-    it "should generate key events", requires: %i[send_keys js] do
-      pending "selenium-webdriver/geckodriver doesn't support complex sets of characters" if marionette?(@session)
+    it 'should generate key events', requires: %i[send_keys js] do
       @session.visit('/with_js')
       @session.find(:css, '#with-key-events').send_keys([:shift, 't'], [:shift, 'w'])
       expect(@session.find(:css, '#key-events-output')).to have_text('keydown:16 keydown:84 keydown:16 keydown:87')
     end
   end
 
-  describe "#execute_script", requires: %i[js es_args] do
-    it "should execute the given script in the context of the element and return nothing" do
+  describe '#execute_script', requires: %i[js es_args] do
+    it 'should execute the given script in the context of the element and return nothing' do
       @session.visit('/with_js')
       expect(@session.find(:css, '#change').execute_script("this.textContent = 'Funky Doodle'")).to be_nil
       expect(@session).to have_css('#change', text: 'Funky Doodle')
     end
 
-    it "should pass arguments to the script" do
+    it 'should pass arguments to the script' do
       @session.visit('/with_js')
-      @session.find(:css, '#change').execute_script("this.textContent = arguments[0]", "Doodle Funk")
+      @session.find(:css, '#change').execute_script('this.textContent = arguments[0]', 'Doodle Funk')
       expect(@session).to have_css('#change', text: 'Doodle Funk')
     end
   end
 
-  describe "#evaluate_script", requires: %i[js es_args] do
-    it "should evaluate the given script in the context of the element and  return whatever it produces" do
+  describe '#evaluate_script', requires: %i[js es_args] do
+    it 'should evaluate the given script in the context of the element and  return whatever it produces' do
       @session.visit('/with_js')
       el = @session.find(:css, '#with_change_event')
-      expect(el.evaluate_script("this.value")).to eq('default value')
+      expect(el.evaluate_script('this.value')).to eq('default value')
+    end
+
+    it 'should ignore leading whitespace' do
+      @session.visit('/with_js')
+      expect(@session.find(:css, '#change').evaluate_script('
+        2 + 3
+      ')).to eq 5
     end
 
-    it "should pass arguments to the script" do
+    it 'should pass arguments to the script' do
       @session.visit('/with_js')
-      @session.find(:css, '#change').evaluate_script("this.textContent = arguments[0]", "Doodle Funk")
+      @session.find(:css, '#change').evaluate_script('this.textContent = arguments[0]', 'Doodle Funk')
       expect(@session).to have_css('#change', text: 'Doodle Funk')
     end
 
-    it "should pass multiple arguments" do
+    it 'should pass multiple arguments' do
       @session.visit('/with_js')
       change = @session.find(:css, '#change')
-      expect(change.evaluate_script("arguments[0] + arguments[1]", 2, 3)).to eq 5
+      expect(change.evaluate_script('arguments[0] + arguments[1]', 2, 3)).to eq 5
     end
 
-    it "should support returning elements" do
+    it 'should support returning elements' do
       @session.visit('/with_js')
       change = @session.find(:css, '#change') # ensure page has loaded and element is available
-      el = change.evaluate_script("this")
+      el = change.evaluate_script('this')
       expect(el).to be_instance_of(Capybara::Node::Element)
       expect(el).to eq(change)
     end
   end
 
-  describe "#evaluate_async_script", requires: %i[js es_args] do
-    it "should evaluate the given script in the context of the element" do
+  describe '#evaluate_async_script', requires: %i[js es_args] do
+    it 'should evaluate the given script in the context of the element' do
       @session.visit('/with_js')
       el = @session.find(:css, '#with_change_event')
-      expect(el.evaluate_async_script("arguments[0](this.value)")).to eq('default value')
+      expect(el.evaluate_async_script('arguments[0](this.value)')).to eq('default value')
     end
 
-    it "should support returning elements after asynchronous operation" do
+    it 'should support returning elements after asynchronous operation' do
       @session.visit('/with_js')
       change = @session.find(:css, '#change') # ensure page has loaded and element is available
-      el = change.evaluate_async_script("var cb = arguments[0]; setTimeout(function(el){ cb(el) }, 100, this)")
+      el = change.evaluate_async_script('var cb = arguments[0]; setTimeout(function(el){ cb(el) }, 100, this)')
       expect(el).to eq(change)
     end
   end
 
   describe '#reload', requires: [:js] do
-    context "without automatic reload" do
+    context 'without automatic reload' do
       before { Capybara.automatic_reload = false }
 
-      it "should reload the current context of the node" do
+      it 'should reload the current context of the node' do
         @session.visit('/with_js')
         node = @session.find(:css, '#reload-me')
         @session.click_link('Reload!')
@@ -548,7 +554,7 @@ Capybara::SpecHelper.spec "node" do
         expect(node.text).to eq('has been reloaded')
       end
 
-      it "should reload a parent node" do
+      it 'should reload a parent node' do
         @session.visit('/with_js')
         node = @session.find(:css, '#reload-me').find(:css, 'em')
         @session.click_link('Reload!')
@@ -557,7 +563,7 @@ Capybara::SpecHelper.spec "node" do
         expect(node.text).to eq('has been reloaded')
       end
 
-      it "should not automatically reload" do
+      it 'should not automatically reload' do
         @session.visit('/with_js')
         node = @session.find(:css, '#reload-me')
         @session.click_link('Reload!')
@@ -571,12 +577,12 @@ Capybara::SpecHelper.spec "node" do
       after { Capybara.automatic_reload = true }
     end
 
-    context "with automatic reload" do
+    context 'with automatic reload' do
       before do
         Capybara.default_max_wait_time = 4
       end
 
-      it "should reload the current context of the node automatically" do
+      it 'should reload the current context of the node automatically' do
         @session.visit('/with_js')
         node = @session.find(:css, '#reload-me')
         @session.click_link('Reload!')
@@ -584,7 +590,7 @@ Capybara::SpecHelper.spec "node" do
         expect(node.text).to eq('has been reloaded')
       end
 
-      it "should reload a parent node automatically" do
+      it 'should reload a parent node automatically' do
         @session.visit('/with_js')
         node = @session.find(:css, '#reload-me').find(:css, 'em')
         @session.click_link('Reload!')
@@ -592,7 +598,7 @@ Capybara::SpecHelper.spec "node" do
         expect(node.text).to eq('has been reloaded')
       end
 
-      it "should reload a node automatically when using find" do
+      it 'should reload a node automatically when using find' do
         @session.visit('/with_js')
         node = @session.find(:css, '#reload-me')
         @session.click_link('Reload!')
@@ -614,9 +620,9 @@ Capybara::SpecHelper.spec "node" do
         end.to(raise_error { |error| expect(error).to be_an_invalid_element_error(@session) })
       end
 
-      it "should reload nodes with options" do
+      it 'should reload nodes with options' do
         @session.visit('/with_js')
-        node = @session.find(:css, 'em', text: "reloaded")
+        node = @session.find(:css, 'em', text: 'reloaded')
         @session.click_link('Reload!')
         sleep(1)
         expect(node.text).to eq('has been reloaded')
@@ -624,9 +630,9 @@ Capybara::SpecHelper.spec "node" do
     end
   end
 
-  context "when #synchronize raises server errors" do
-    it "sets an explanatory exception as the cause of server exceptions", requires: %i[server js] do
-      quietly { @session.visit("/error") }
+  context 'when #synchronize raises server errors' do
+    it 'sets an explanatory exception as the cause of server exceptions', requires: %i[server js] do
+      quietly { @session.visit('/error') }
       expect do
         @session.find(:css, 'span')
       end.to(raise_error(TestApp::TestAppError) do |e|
@@ -635,8 +641,8 @@ Capybara::SpecHelper.spec "node" do
       end)
     end
 
-    it "sets an explanatory exception as the cause of server exceptions with errors with initializers", requires: %i[server js] do
-      quietly { @session.visit("/other_error") }
+    it 'sets an explanatory exception as the cause of server exceptions with errors with initializers', requires: %i[server js] do
+      quietly { @session.visit('/other_error') }
       expect do
         @session.find(:css, 'span')
       end.to(raise_error(TestApp::TestAppOtherError) do |e|