capybara 2.15.0 → 3.0.0

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#has_select?' do
   before { @session.visit('/form') }
 
@@ -6,7 +7,7 @@ Capybara::SpecHelper.spec '#has_select?' do
     expect(@session).to have_select('Locale')
     expect(@session).to have_select('form_region')
     expect(@session).to have_select('Languages')
-    expect(@session).to have_select(:'Languages')
+    expect(@session).to have_select(:Languages)
   end
 
   it "should be false if the field is not on the page" do
@@ -27,13 +28,13 @@ Capybara::SpecHelper.spec '#has_select?' do
       expect(@session).not_to have_select('Does not exist', selected: 'John')
       expect(@session).not_to have_select('City', selected: 'Not there')
       expect(@session).not_to have_select('Underwear', selected: [
-        'Boxerbriefs', 'Briefs', 'Commando', "Frenchman's Pantalons", 'Long Johns', 'Nonexistant'
+        'Boxerbriefs', 'Briefs', 'Commando', "Frenchman's Pantalons", 'Long Johns', 'Nonexistent'
       ])
       expect(@session).not_to have_select('Underwear', selected: [
         'Boxerbriefs', 'Briefs', 'Boxers', 'Commando', "Frenchman's Pantalons", 'Long Johns'
       ])
       expect(@session).not_to have_select('Underwear', selected: [
-        'Boxerbriefs', 'Briefs','Commando', "Frenchman's Pantalons"
+        'Boxerbriefs', 'Briefs', 'Commando', "Frenchman's Pantalons"
       ])
     end
 
@@ -69,25 +70,25 @@ Capybara::SpecHelper.spec '#has_select?' do
 
   context 'with partial select' do
     it "should be true if a field with the given partial values is on the page" do
-      expect(@session).to have_select('Underwear', with_selected: ['Boxerbriefs', 'Briefs'])
+      expect(@session).to have_select('Underwear', with_selected: %w[Boxerbriefs Briefs])
     end
 
     it "should be false if a field with the given partial values is not on the page" do
-      expect(@session).not_to have_select('Underwear', with_selected: ['Boxerbriefs', 'Boxers'])
+      expect(@session).not_to have_select('Underwear', with_selected: %w[Boxerbriefs Boxers])
     end
 
     it "should be true after the given partial value is selected" do
       @session.select('Boxers', from: 'Underwear')
-      expect(@session).to have_select('Underwear', with_selected: ['Boxerbriefs', 'Boxers'])
+      expect(@session).to have_select('Underwear', with_selected: %w[Boxerbriefs Boxers])
     end
 
     it "should be false after one of the given partial values is unselected" do
       @session.unselect('Briefs', from: 'Underwear')
-      expect(@session).not_to have_select('Underwear', with_selected: ['Boxerbriefs', 'Briefs'])
+      expect(@session).not_to have_select('Underwear', with_selected: %w[Boxerbriefs Briefs])
     end
 
     it "should be true even when the selected values are invisible, regardless of the select's visibility" do
-      expect(@session).to have_select('Dessert', visible: false, with_options: ['Pudding', 'Tiramisu'])
+      expect(@session).to have_select('Dessert', visible: false, with_options: %w[Pudding Tiramisu])
       expect(@session).to have_select('Cake', with_selected: ['Chocolate Cake', 'Sponge Cake'])
     end
 
@@ -99,7 +100,7 @@ Capybara::SpecHelper.spec '#has_select?' do
 
   context 'with exact options' do
     it "should be true if a field with the given options is on the page" do
-      expect(@session).to have_select('Region', options: ['Norway', 'Sweden', 'Finland'])
+      expect(@session).to have_select('Region', options: %w[Norway Sweden Finland])
       expect(@session).to have_select('Tendency', options: [])
     end
 
@@ -107,29 +108,29 @@ Capybara::SpecHelper.spec '#has_select?' do
       expect(@session).not_to have_select('Locale', options: ['Swedish'])
       expect(@session).not_to have_select('Does not exist', options: ['John'])
       expect(@session).not_to have_select('City', options: ['London', 'Made up city'])
-      expect(@session).not_to have_select('Region', options: ['Norway', 'Sweden'])
-      expect(@session).not_to have_select('Region', options: ['Norway', 'Norway', 'Norway'])
+      expect(@session).not_to have_select('Region', options: %w[Norway Sweden])
+      expect(@session).not_to have_select('Region', options: %w[Norway Norway Norway])
     end
 
     it "should be true even when the options are invisible, if the select itself is invisible" do
-      expect(@session).to have_select("Icecream", visible: false, options: ['Chocolate', 'Vanilla', 'Strawberry'])
+      expect(@session).to have_select("Icecream", visible: false, options: %w[Chocolate Vanilla Strawberry])
     end
   end
 
   context 'with partial options' do
     it "should be true if a field with the given partial options is on the page" do
-      expect(@session).to have_select('Region', with_options: ['Norway', 'Sweden'])
+      expect(@session).to have_select('Region', with_options: %w[Norway Sweden])
       expect(@session).to have_select('City', with_options: ['London'])
     end
 
     it "should be false if a field with the given partial options is not on the page" do
       expect(@session).not_to have_select('Locale', with_options: ['Uruguayan'])
       expect(@session).not_to have_select('Does not exist', with_options: ['John'])
-      expect(@session).not_to have_select('Region', with_options: ['Norway', 'Sweden', 'Finland', 'Latvia'])
+      expect(@session).not_to have_select('Region', with_options: %w[Norway Sweden Finland Latvia])
     end
 
     it "should be true even when the options are invisible, if the select itself is invisible" do
-      expect(@session).to have_select("Icecream", visible: false, with_options: ['Vanilla', 'Strawberry'])
+      expect(@session).to have_select("Icecream", visible: false, with_options: %w[Vanilla Strawberry])
     end
   end
 
@@ -151,9 +152,9 @@ Capybara::SpecHelper.spec '#has_select?' do
   end
 
   it "should support locator-less usage" do
-    expect(@session.has_select?(with_options: ['Norway', 'Sweden'])).to eq true
+    expect(@session.has_select?(with_options: %w[Norway Sweden])).to eq true
     expect(@session).to have_select(with_options: ['London'])
-    expect(@session.has_select?(with_selected: ['Commando', 'Boxerbriefs'])).to eq true
+    expect(@session.has_select?(with_selected: %w[Commando Boxerbriefs])).to eq true
     expect(@session).to have_select(with_selected: ['Briefs'])
   end
 end
@@ -185,13 +186,13 @@ Capybara::SpecHelper.spec '#has_no_select?' do
       expect(@session).to have_no_select('Does not exist', selected: 'John')
       expect(@session).to have_no_select('City', selected: 'Not there')
       expect(@session).to have_no_select('Underwear', selected: [
-        'Boxerbriefs', 'Briefs', 'Commando', "Frenchman's Pantalons", 'Long Johns', 'Nonexistant'
+        'Boxerbriefs', 'Briefs', 'Commando', "Frenchman's Pantalons", 'Long Johns', 'Nonexistent'
       ])
       expect(@session).to have_no_select('Underwear', selected: [
         'Boxerbriefs', 'Briefs', 'Boxers', 'Commando', "Frenchman's Pantalons", 'Long Johns'
       ])
       expect(@session).to have_no_select('Underwear', selected: [
-        'Boxerbriefs', 'Briefs','Commando', "Frenchman's Pantalons"
+        'Boxerbriefs', 'Briefs', 'Commando', "Frenchman's Pantalons"
       ])
     end
 
@@ -222,21 +223,21 @@ Capybara::SpecHelper.spec '#has_no_select?' do
 
   context 'with partial select' do
     it "should be false if a field with the given partial values is on the page" do
-      expect(@session).not_to have_no_select('Underwear', with_selected: ['Boxerbriefs', 'Briefs'])
+      expect(@session).not_to have_no_select('Underwear', with_selected: %w[Boxerbriefs Briefs])
     end
 
     it "should be true if a field with the given partial values is not on the page" do
-      expect(@session).to have_no_select('Underwear', with_selected: ['Boxerbriefs', 'Boxers'])
+      expect(@session).to have_no_select('Underwear', with_selected: %w[Boxerbriefs Boxers])
     end
 
     it "should be false after the given partial value is selected" do
       @session.select('Boxers', from: 'Underwear')
-      expect(@session).not_to have_no_select('Underwear', with_selected: ['Boxerbriefs', 'Boxers'])
+      expect(@session).not_to have_no_select('Underwear', with_selected: %w[Boxerbriefs Boxers])
     end
 
     it "should be true after one of the given partial values is unselected" do
       @session.unselect('Briefs', from: 'Underwear')
-      expect(@session).to have_no_select('Underwear', with_selected: ['Boxerbriefs', 'Briefs'])
+      expect(@session).to have_no_select('Underwear', with_selected: %w[Boxerbriefs Briefs])
     end
 
     it "should support non array partial values" do
@@ -247,35 +248,35 @@ Capybara::SpecHelper.spec '#has_no_select?' do
 
   context 'with exact options' do
     it "should be false if a field with the given options is on the page" do
-      expect(@session).not_to have_no_select('Region', options: ['Norway', 'Sweden', 'Finland'])
+      expect(@session).not_to have_no_select('Region', options: %w[Norway Sweden Finland])
     end
 
     it "should be true if the given field is not on the page" do
       expect(@session).to have_no_select('Locale', options: ['Swedish'])
       expect(@session).to have_no_select('Does not exist', options: ['John'])
       expect(@session).to have_no_select('City', options: ['London', 'Made up city'])
-      expect(@session).to have_no_select('Region', options: ['Norway', 'Sweden'])
-      expect(@session).to have_no_select('Region', options: ['Norway', 'Norway', 'Norway'])
+      expect(@session).to have_no_select('Region', options: %w[Norway Sweden])
+      expect(@session).to have_no_select('Region', options: %w[Norway Norway Norway])
     end
   end
 
   context 'with partial options' do
     it "should be false if a field with the given partial options is on the page" do
-      expect(@session).not_to have_no_select('Region', with_options: ['Norway', 'Sweden'])
+      expect(@session).not_to have_no_select('Region', with_options: %w[Norway Sweden])
       expect(@session).not_to have_no_select('City', with_options: ['London'])
     end
 
     it "should be true if a field with the given partial options is not on the page" do
       expect(@session).to have_no_select('Locale', with_options: ['Uruguayan'])
       expect(@session).to have_no_select('Does not exist', with_options: ['John'])
-      expect(@session).to have_no_select('Region', with_options: ['Norway', 'Sweden', 'Finland', 'Latvia'])
+      expect(@session).to have_no_select('Region', with_options: %w[Norway Sweden Finland Latvia])
     end
   end
 
   it "should support locator-less usage" do
-    expect(@session.has_no_select?(with_options: ['Norway', 'Sweden', 'Finland', 'Latvia'])).to eq true
-    expect(@session).to have_no_select(with_options: ['New London'] )
+    expect(@session.has_no_select?(with_options: %w[Norway Sweden Finland Latvia])).to eq true
+    expect(@session).to have_no_select(with_options: ['New London'])
     expect(@session.has_no_select?(with_selected: ['Boxers'])).to eq true
-    expect(@session).to have_no_select(with_selected: ['Commando', 'Boxers'])
+    expect(@session).to have_no_select(with_selected: %w[Commando Boxers])
   end
 end

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#has_selector?' do
   before do
     @session.visit('/with_html')
@@ -72,9 +73,10 @@ Capybara::SpecHelper.spec '#has_selector?' do
       expect(@session).not_to have_selector("//p//a", text: /Red$/)
     end
 
-    it "should warn when extra parameters passed" do
-      expect_any_instance_of(Kernel).to receive(:warn).with(/extra/)
-      expect(@session).to have_selector(:css, "p a#foo", 'extra')
+    it "should raise when extra parameters passed" do
+      expect do
+        expect(@session).to have_selector(:css, "p a#foo", 'extra')
+      end.to raise_error ArgumentError, /extra/
     end
   end
 
@@ -131,7 +133,6 @@ Capybara::SpecHelper.spec '#has_no_selector?' do
   end
 
   it "should accept a filter block" do
-    skip "RSpec < 3 doesn't pass the block along to the matcher for the Builtin::Has matcher" if rspec2?
     expect(@session).to have_no_selector(:css, "a#foo") { |el| el[:id] != "foo" }
   end
 

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#has_table?' do
   before do
     @session.visit('/tables')
@@ -7,7 +8,7 @@ Capybara::SpecHelper.spec '#has_table?' do
   it "should be true if the table is on the page" do
     expect(@session).to have_table('Villain')
     expect(@session).to have_table('villain_table')
-    expect(@session).to have_table(:'villain_table')
+    expect(@session).to have_table(:villain_table)
   end
 
   it "should be false if the table is not on the page" do

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

@@ -1,11 +1,12 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#has_text?' do
   it "should be true if the given text is on the page at least once" do
     @session.visit('/with_html')
     expect(@session).to have_text('est')
     expect(@session).to have_text('Lorem')
     expect(@session).to have_text('Redirect')
-    expect(@session).to have_text(:'Redirect')
+    expect(@session).to have_text(:Redirect)
   end
 
   it "should be true if scoped to an element which has the text" do
@@ -28,14 +29,9 @@ Capybara::SpecHelper.spec '#has_text?' do
     expect(@session).to have_text('exercitation ullamco laboris')
   end
 
-  it "should ignore extra whitespace and newlines" do
-    @session.visit('/with_html')
-    expect(@session).to have_text('text with whitespace')
-  end
-
-  it "should ignore whitespace and newlines in the search string" do
+  it "should search correctly normalized text" do
     @session.visit('/with_html')
-    expect(@session).to have_text("text     with \n\n whitespace")
+    expect(@session).to have_text('text with   whitespace')
   end
 
   it "should be false if the given text is not on the page" do
@@ -51,12 +47,12 @@ Capybara::SpecHelper.spec '#has_text?' do
 
   it 'should handle double quotes in the text' do
     @session.visit('/with-quotes')
-    expect(@session).to have_text(%q{"No," he said})
+    expect(@session).to have_text('"No," he said')
   end
 
   it 'should handle mixed single and double quotes in the text' do
     @session.visit('/with-quotes')
-    expect(@session).to have_text(%q{"you can't do that."})
+    expect(@session).to have_text(%q("you can't do that."))
   end
 
   it 'should be false if text is in the title tag in the head' do
@@ -265,12 +261,12 @@ Capybara::SpecHelper.spec '#has_no_text?' do
 
   it 'should handle double quotes in the text' do
     @session.visit('/with-quotes')
-    expect(@session).not_to have_no_text(%q{"No," he said})
+    expect(@session).not_to have_no_text('"No," he said')
   end
 
   it 'should handle mixed single and double quotes in the text' do
     @session.visit('/with-quotes')
-    expect(@session).not_to have_no_text(%q{"you can't do that."})
+    expect(@session).not_to have_no_text(%q("you can't do that."))
   end
 
   it 'should be true if text is in the title tag in the head' do
@@ -325,7 +321,7 @@ Capybara::SpecHelper.spec '#has_no_text?' do
     it "should not find element if it appears after given wait duration" do
       @session.visit('/with_js')
       @session.click_link('Click me')
-      expect(@session).to have_no_text('Has been clicked', wait: 0.1)
+      expect(@session).to have_no_text('Has been clicked', wait: 0.05)
     end
   end
 end

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#has_title?' do
   before do
     @session.visit('/with_js')

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#has_xpath?' do
   before do
     @session.visit('/with_html')

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

@@ -1,7 +1,8 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#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)
+    expect(@session.response_headers['Content-Type']).to match %r{text/html}
   end
 end

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#html' do
   it "should return the unmodified page body" do
     @session.visit('/')

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

@@ -27,14 +27,6 @@ Capybara::SpecHelper.spec "node" do
     end
   end
 
-  describe "#parent" do
-    it "should be deprecated" do
-      @node = @session.find(:css, '#first')
-      expect(@node).to receive(:warn).with(/^DEPRECATED:/)
-      expect(@node.parent).to eq(@node.query_scope)
-    end
-  end
-
   describe "#text" do
     it "should extract node texts" do
       expect(@session.all('//a')[0].text).to eq('labore')
@@ -102,44 +94,32 @@ Capybara::SpecHelper.spec "node" do
       expect(@session.first('//input').value).to eq('')
     end
 
-    it "should not set if the text field is readonly" do
-      expect(@session.first('//input[@readonly]').value).to eq('should not change')
-      @session.first('//input[@readonly]').set('changed')
-      expect(@session.first('//input[@readonly]').value).to eq('should not change')
-    end
-
     it "should raise if the text field is readonly" do
-      expect(@session.first('//input[@readonly]').set('changed')).to raise_error(Capybara::ReadOnlyElementError)
-    end if Capybara::VERSION.to_f > 3.0
-
-    it "should not set if the textarea is readonly" do
-      expect(@session.first('//textarea[@readonly]').value).to eq('textarea should not change')
-      @session.first('//textarea[@readonly]').set('changed')
-      expect(@session.first('//textarea[@readonly]').value).to eq('textarea should not change')
+      expect { @session.first('//input[@readonly]').set('changed') }.to raise_error(Capybara::ReadOnlyElementError)
     end
 
     it "should raise if the text field is readonly" do
-      expect(@session.first('//textarea[@readonly]').set('changed')).to raise_error(Capybara::ReadOnlyElementError)
-    end if Capybara::VERSION.to_f > 3.0
+      expect { @session.first('//textarea[@readonly]').set('changed') }.to raise_error(Capybara::ReadOnlyElementError)
+    end
 
     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')
-        expect(@session.find(:css,'#existing_content_editable').text).to eq('WYSIWYG')
+        @session.find(:css, '#existing_content_editable').set('WYSIWYG')
+        expect(@session.find(:css, '#existing_content_editable').text).to eq('WYSIWYG')
       end
 
       it 'should allow me to set the contents' do
         @session.visit('/with_js')
-        @session.find(:css,'#blank_content_editable').set('WYSIWYG')
-        expect(@session.find(:css,'#blank_content_editable').text).to eq('WYSIWYG')
+        @session.find(:css, '#blank_content_editable').set('WYSIWYG')
+        expect(@session.find(:css, '#blank_content_editable').text).to eq('WYSIWYG')
       end
 
       it 'should allow me to change the contents of a child element' do
         @session.visit('/with_js')
-        @session.find(:css,'#existing_content_editable_child').set('WYSIWYG')
-        expect(@session.find(:css,'#existing_content_editable_child').text).to eq('WYSIWYG')
-        expect(@session.find(:css,'#existing_content_editable_child_parent').text).to eq('Some content WYSIWYG')
+        @session.find(:css, '#existing_content_editable_child').set('WYSIWYG')
+        expect(@session.find(:css, '#existing_content_editable_child').text).to eq('WYSIWYG')
+        expect(@session.find(:css, '#existing_content_editable_child_parent').text).to eq("Some content\nWYSIWYG")
       end
     end
   end
@@ -258,9 +238,9 @@ Capybara::SpecHelper.spec "node" do
 
   describe "#==" do
     it "preserve object identity" do
-      expect(@session.find('//h1') == @session.find('//h1')).to be true
-      expect(@session.find('//h1') === @session.find('//h1')).to be true
-      expect(@session.find('//h1').eql? @session.find('//h1')).to be false
+      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
@@ -281,7 +261,7 @@ Capybara::SpecHelper.spec "node" do
     end
   end
 
-  describe "#trigger", requires: [:js, :trigger] 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)
@@ -289,7 +269,7 @@ Capybara::SpecHelper.spec "node" do
     end
   end
 
-  describe '#drag_to', requires: [:js, :drag] do
+  describe '#drag_to', requires: %i[js drag] do
     it "should drag and drop an object" do
       @session.visit('/with_js')
       element = @session.find('//div[@id="drag"]')
@@ -297,14 +277,29 @@ Capybara::SpecHelper.spec "node" do
       element.drag_to(target)
       expect(@session.find('//div[contains(., "Dropped!")]')).not_to be_nil
     end
+
+    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"]')
+      element.drag_to(target)
+      expect(@session.find('//div[contains(., "Dropped!")]')).not_to be_nil
+    end
   end
 
   describe '#hover', requires: [:hover] do
     it "should allow hovering on an element" do
       @session.visit('/with_hover')
-      expect(@session.find(:css,'.hidden_until_hover', visible: false)).not_to be_visible
-      @session.find(:css,'.wrapper').hover
-      expect(@session.find(:css, '.hidden_until_hover', visible: false)).to be_visible
+      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
+      @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
+      expect(@session.find(:css, '.wrapper.scroll_needed .hidden_until_hover', visible: false)).to be_visible
     end
   end
 
@@ -343,24 +338,84 @@ Capybara::SpecHelper.spec "node" do
       radio.click
       expect(radio).to be_checked
     end
+
+    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
+      @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")
+    end
+
+    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')
+      locations = link.text.match(/^Has been clicked at (?<x>[\d\.-]+),(?<y>[\d\.-]+)$/)
+      # Resulting click location should be very close to 0, 0 relative to top left corner of the element, but may not be exact due to
+      # integer/float conversions and rounding.
+      expect(locations[:x].to_f).to be_within(1).of(5)
+      expect(locations[:y].to_f).to be_within(1).of(5)
+    end
   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
-      pending "selenium-webdriver/geckodriver doesn't support mouse move_to" if marionette?(@session)
       @session.visit('/with_js')
       @session.find(:css, '#click-test').double_click
       expect(@session.find(:css, '#has-been-double-clicked')).to be
     end
+
+    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
+      @session.visit('with_js')
+      @session.find(:css, '#click-test').double_click(x: 10, y: 5)
+      link = @session.find(:link, 'has-been-double-clicked')
+      locations = link.text.match(/^Has been double clicked at (?<x>[\d\.-]+),(?<y>[\d\.-]+)$/)
+      # Resulting click location should be very close to 10, 5 relative to top left corner of the element, but may not be exact due
+      # to integer/float conversions and rounding.
+      expect(locations[:x].to_f).to be_within(1).of(10)
+      expect(locations[:y].to_f).to be_within(1).of(5)
+    end
   end
 
   describe '#right_click', requires: [:js] do
     it "should right click an element" do
-      pending "selenium-webdriver/geckodriver doesn't support mouse move_to" if marionette?(@session)
       @session.visit('/with_js')
       @session.find(:css, '#click-test').right_click
       expect(@session.find(:css, '#has-been-right-clicked')).to be
     end
+
+    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
+      @session.visit('with_js')
+      @session.find(:css, '#click-test').right_click(x: 10, y: 10)
+      link = @session.find(:link, 'has-been-right-clicked')
+      locations = link.text.match(/^Has been right clicked at (?<x>[\d\.-]+),(?<y>[\d\.-]+)$/)
+      # Resulting click location should be very close to 10, 10 relative to top left corner of the element, but may not be exact due
+      # to integer/float conversions and rounding
+      expect(locations[:x].to_f).to be_within(1).of(10)
+      expect(locations[:y].to_f).to be_within(1).of(10)
+    end
   end
 
   describe '#send_keys', requires: [:send_keys] do
@@ -384,10 +439,10 @@ Capybara::SpecHelper.spec "node" do
       expect(@session.find(:css, '#address1_city').value).to eq 'Oceanside'
     end
 
-    it "should generate key events", requires: [:send_keys, :js] do
+    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)
       @session.visit('/with_js')
-      @session.find(:css, '#with-key-events').send_keys([:shift,'t'], [:shift,'w'])
+      @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
@@ -420,9 +475,9 @@ Capybara::SpecHelper.spec "node" do
         sleep(0.3)
         expect do
           expect(node).to have_text('has been reloaded')
-        end.to raise_error do |error|
+        end.to(raise_error do |error|
           expect(error).to be_an_invalid_element_error(@session)
-        end
+        end)
       end
       after { Capybara.automatic_reload = true }
     end
@@ -464,14 +519,10 @@ Capybara::SpecHelper.spec "node" do
 
         expect do
           expect(node).to have_text('Foo')
-        end.to raise_error { |error|
-          expect(error).to be_an_invalid_element_error(@session)
-        }
+        end.to(raise_error { |error| expect(error).to be_an_invalid_element_error(@session) })
         expect do
           expect(node).to have_text('Bar')
-        end.to raise_error { |error|
-          expect(error).to be_an_invalid_element_error(@session)
-        }
+        end.to(raise_error { |error| expect(error).to be_an_invalid_element_error(@session) })
       end
 
       it "should reload nodes with options" do
@@ -485,26 +536,24 @@ Capybara::SpecHelper.spec "node" do
   end
 
   context "when #synchronize raises server errors" do
-    it "sets an explanatory exception as the cause of server exceptions", requires: [:server, :js] do
-      skip "This version of ruby doesn't support exception causes" unless Exception.instance_methods.include? :cause
+    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|
+      end.to(raise_error(TestApp::TestAppError) do |e|
         expect(e.cause).to be_a Capybara::CapybaraError
         expect(e.cause.message).to match(/Your application server raised an error/)
-      end
+      end)
     end
 
-    it "sets an explanatory exception as the cause of server exceptions with errors with initializers", requires: [:server, :js] do
-      skip "This version of ruby doesn't support exception causes" unless Exception.instance_methods.include? :cause
+    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|
+      end.to(raise_error(TestApp::TestAppOtherError) do |e|
         expect(e.cause).to be_a Capybara::CapybaraError
         expect(e.cause.message).to match(/Your application server raised an error/)
-      end
+      end)
     end
   end