capybara 2.18.0 → 3.0.0

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

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

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec "#check" do
   before do
     @session.visit('/form')
@@ -18,7 +19,7 @@ Capybara::SpecHelper.spec "#check" do
   it "should trigger associated events", requires: [:js] do
     @session.visit('/with_js')
     @session.check('checkbox_with_event')
-    expect(@session).to have_css('#checkbox_event_triggered');
+    expect(@session).to have_css('#checkbox_event_triggered')
   end
 
   describe "checking" do
@@ -62,7 +63,7 @@ Capybara::SpecHelper.spec "#check" do
   end
 
   it "casts to string" do
-    @session.check(:"form_pets_cat")
+    @session.check(:form_pets_cat)
     @session.click_button('awesome')
     expect(extract_results(@session)['pets']).to include('dog', 'cat', 'hamster')
   end
@@ -86,14 +87,14 @@ Capybara::SpecHelper.spec "#check" do
 
   context "with :exact option" do
     it "should accept partial matches when false" do
-      @session.check('Ham', exact:  false)
+      @session.check('Ham', exact: false)
       @session.click_button('awesome')
       expect(extract_results(@session)['pets']).to include('hamster')
     end
 
     it "should not accept partial matches when true" do
       expect do
-        @session.check('Ham', exact:  true)
+        @session.check('Ham', exact: true)
       end.to raise_error(Capybara::ElementNotFound)
     end
   end
@@ -146,7 +147,7 @@ Capybara::SpecHelper.spec "#check" do
       end
 
       it "should raise error if not allowed to click label" do
-        expect{@session.check('form_cars_mclaren', allow_label_click: false)}.to raise_error(Capybara::ElementNotFound, 'Unable to find visible checkbox "form_cars_mclaren" that is not disabled')
+        expect { @session.check('form_cars_mclaren', allow_label_click: false) }.to raise_error(Capybara::ElementNotFound, 'Unable to find visible checkbox "form_cars_mclaren" that is not disabled')
       end
     end
 
@@ -158,7 +159,7 @@ Capybara::SpecHelper.spec "#check" do
       end
 
       it "should raise error if checkbox not visible" do
-        expect{@session.check('form_cars_mclaren')}.to raise_error(Capybara::ElementNotFound, 'Unable to find visible checkbox "form_cars_mclaren" that is not disabled')
+        expect { @session.check('form_cars_mclaren') }.to raise_error(Capybara::ElementNotFound, 'Unable to find visible checkbox "form_cars_mclaren" that is not disabled')
       end
 
       context "with allow_label_click == true" do

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec "#choose" do
   before do
     @session.visit('/form')
@@ -18,7 +19,7 @@ Capybara::SpecHelper.spec "#choose" do
 
   it "casts to string" do
     @session.choose("Both")
-    @session.click_button(:'awesome')
+    @session.click_button(:awesome)
     expect(extract_results(@session)['gender']).to eq('both')
   end
 
@@ -41,14 +42,14 @@ Capybara::SpecHelper.spec "#choose" do
 
   context "with :exact option" do
     it "should accept partial matches when false" do
-      @session.choose("Mal", exact:  false)
+      @session.choose("Mal", exact: false)
       @session.click_button('awesome')
       expect(extract_results(@session)['gender']).to eq('male')
     end
 
     it "should not accept partial matches when true" do
       expect do
-        @session.choose("Mal", exact:  true)
+        @session.choose("Mal", exact: true)
       end.to raise_error(Capybara::ElementNotFound)
     end
   end
@@ -82,7 +83,7 @@ Capybara::SpecHelper.spec "#choose" do
       end
 
       it "should raise error if not allowed to click label" do
-        expect{@session.choose("party_democrat", allow_label_click: false)}.to raise_error(Capybara::ElementNotFound, 'Unable to find visible radio button "party_democrat" that is not disabled')
+        expect { @session.choose("party_democrat", allow_label_click: false) }.to raise_error(Capybara::ElementNotFound, 'Unable to find visible radio button "party_democrat" that is not disabled')
       end
     end
   end

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#click_button' do
   before do
     @session.visit('/form')
@@ -151,31 +152,24 @@ Capybara::SpecHelper.spec '#click_button' do
     end
   end
 
-  context "with id given on a submit button" do
-    it "should submit the associated form" do
+  context "input type=submit button" do
+    it "should submit by button id" do
       @session.click_button('awe123')
       expect(extract_results(@session)['first_name']).to eq('John')
     end
 
-    it "should work with partial matches" do
-      @session.click_button('Go')
-      expect(@session).to have_content('You landed')
-    end
-  end
-
-  context "with title given on a submit button" do
-    it "should submit the associated form" do
+    it "should submit by button title" do
       @session.click_button('What an Awesome Button')
       expect(extract_results(@session)['first_name']).to eq('John')
     end
 
-    it "should work with partial matches" do
+    it "should submit by partial title", :exact_false do
       @session.click_button('What an Awesome')
       expect(extract_results(@session)['first_name']).to eq('John')
     end
   end
 
-  context "with fields associated with the form using the form attribute" do
+  context "with fields associated with the form using the form attribute", requires: [:form_attribute] do
     before do
       @session.click_button('submit_form1')
       @results = extract_results(@session)
@@ -198,8 +192,7 @@ Capybara::SpecHelper.spec '#click_button' do
     end
   end
 
-
-  context "with submit button outside the form defined by <button> tag" do
+  context "with submit button outside the form defined by <button> tag", requires: [:form_attribute] do
     before do
       @session.click_button('outside_button')
       @results = extract_results(@session)
@@ -215,7 +208,7 @@ Capybara::SpecHelper.spec '#click_button' do
     end
   end
 
-  context "with submit button outside the form defined by <input type='submit'> tag" do
+  context "with submit button outside the form defined by <input type='submit'> tag", requires: [:form_attribute] do
     before do
       @session.click_button('outside_submit')
       @results = extract_results(@session)
@@ -231,7 +224,7 @@ Capybara::SpecHelper.spec '#click_button' do
     end
   end
 
-  context "with submit button for form1 located within form2" do
+  context "with submit button for form1 located within form2", requires: [:form_attribute] do
     it "should submit the form associated with the button" do
       @session.click_button('other_form_button')
       expect(extract_results(@session)['which_form']).to eq("form1")
@@ -250,20 +243,19 @@ Capybara::SpecHelper.spec '#click_button' do
       expect(extract_results(@session)['first_name']).to eq('John')
     end
 
-    it "should work with partial matches" do
+    it "should work with partial matches", :exact_false do
       @session.click_button('hai')
       expect(extract_results(@session)['first_name']).to eq('John')
     end
   end
 
-
   context "with value given on an image button" do
     it "should submit the associated form" do
       @session.click_button('okay')
       expect(extract_results(@session)['first_name']).to eq('John')
     end
 
-    it "should work with partial matches" do
+    it "should work with partial matches", :exact_false do
       @session.click_button('kay')
       expect(extract_results(@session)['first_name']).to eq('John')
     end
@@ -282,7 +274,7 @@ Capybara::SpecHelper.spec '#click_button' do
       expect(extract_results(@session)['first_name']).to eq('John')
     end
 
-    it "should work with partial matches" do
+    it "should work with partial matches", :exact_false do
       @session.click_button('Okay 556')
       expect(extract_results(@session)['first_name']).to eq('John')
     end
@@ -290,17 +282,17 @@ Capybara::SpecHelper.spec '#click_button' do
 
   context "with text given on a button defined by <button> tag" do
     it "should submit the associated form" do
-      @session.click_button('Click me')
+      @session.click_button('Click me!')
       expect(extract_results(@session)['first_name']).to eq('John')
     end
 
-    it "should work with partial matches" do
+    it "should work with partial matches", :exact_false do
       @session.click_button('Click')
       expect(extract_results(@session)['first_name']).to eq('John')
     end
   end
 
- context "with id given on a button defined by <button> tag" do
+  context "with id given on a button defined by <button> tag" do
     it "should submit the associated form" do
       @session.click_button('click_me_123')
       expect(extract_results(@session)['first_name']).to eq('John')
@@ -315,13 +307,13 @@ Capybara::SpecHelper.spec '#click_button' do
     end
   end
 
- context "with value given on a button defined by <button> tag" do
+  context "with value given on a button defined by <button> tag" do
     it "should submit the associated form" do
       @session.click_button('click_me')
       expect(extract_results(@session)['first_name']).to eq('John')
     end
 
-    it "should work with partial matches" do
+    it "should work with partial matches", :exact_false do
       @session.click_button('ck_me')
       expect(extract_results(@session)['first_name']).to eq('John')
     end
@@ -333,7 +325,7 @@ Capybara::SpecHelper.spec '#click_button' do
       expect(extract_results(@session)['first_name']).to eq('John')
     end
 
-    it "should work with partial matches" do
+    it "should work with partial matches", :exact_false do
       @session.click_button('Click Title')
       expect(extract_results(@session)['first_name']).to eq('John')
     end
@@ -345,7 +337,7 @@ Capybara::SpecHelper.spec '#click_button' do
       expect(extract_results(@session)['first_name']).to eq('John')
     end
 
-    it "should work with partial matches" do
+    it "should work with partial matches", :exact_false do
       @session.click_button('se eating h')
       expect(extract_results(@session)['first_name']).to eq('John')
     end
@@ -424,15 +416,15 @@ Capybara::SpecHelper.spec '#click_button' do
 
   it "should encode complex field names, like array[][value]" do
     @session.visit('/form')
-    @session.fill_in('address1_city', :with =>'Paris')
-    @session.fill_in('address1_street', :with =>'CDG')
+    @session.fill_in('address1_city', with: 'Paris')
+    @session.fill_in('address1_street', with: 'CDG')
 
     @session.fill_in('address2_city', with: 'Mikolaiv')
     @session.fill_in('address2_street', with: 'PGS')
 
     @session.click_button "awesome"
 
-    addresses=extract_results(@session)["addresses"]
+    addresses = extract_results(@session)["addresses"]
     expect(addresses.size).to eq(2)
 
     expect(addresses[0]["street"]).to eq('CDG')
@@ -446,13 +438,13 @@ Capybara::SpecHelper.spec '#click_button' do
 
   context "with :exact option" do
     it "should accept partial matches when false" do
-      @session.click_button('What an Awesome', exact:  false)
+      @session.click_button('What an Awesome', exact: false)
       expect(extract_results(@session)['first_name']).to eq('John')
     end
 
     it "should not accept partial matches when true" do
       expect do
-        @session.click_button('What an Awesome', exact:  true)
+        @session.click_button('What an Awesome', exact: true)
       end.to raise_error(Capybara::ElementNotFound)
     end
   end

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#click_link_or_button' do
   it "should click on a link" do
     @session.visit('/with_html')
@@ -32,31 +33,31 @@ Capybara::SpecHelper.spec '#click_link_or_button' do
 
   it "casts to string" do
     @session.visit('/form')
-    @session.click_link_or_button(:'awe123')
+    @session.click_link_or_button(:awe123)
     expect(extract_results(@session)['first_name']).to eq('John')
   end
 
   context "with :exact option" do
-    context "when `true`" do
+    context "when `false`" do
       it "clicks on approximately matching link" do
         @session.visit('/with_html')
-        @session.click_link_or_button('abore', exact:  false)
+        @session.click_link_or_button('abore', exact: false)
         expect(@session).to have_content('Bar')
       end
 
       it "clicks on approximately matching button" do
         @session.visit('/form')
-        @session.click_link_or_button('awe')
+        @session.click_link_or_button('awe', exact: false)
         expect(extract_results(@session)['first_name']).to eq('John')
       end
     end
 
-    context "when `false`" do
+    context "when `true`" do
       it "does not click on link which matches approximately" do
         @session.visit('/with_html')
         msg = "Unable to find visible link or button \"abore\""
         expect do
-          @session.click_link_or_button('abore', exact:  true)
+          @session.click_link_or_button('abore', exact: true)
         end.to raise_error(Capybara::ElementNotFound, msg)
       end
 
@@ -65,7 +66,7 @@ Capybara::SpecHelper.spec '#click_link_or_button' do
         msg = "Unable to find visible link or button \"awe\""
 
         expect do
-          @session.click_link_or_button('awe', exact:  true)
+          @session.click_link_or_button('awe', exact: true)
         end.to raise_error(Capybara::ElementNotFound, msg)
       end
     end

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#click_link' do
   before do
     @session.visit('/with_html')
@@ -11,7 +12,7 @@ Capybara::SpecHelper.spec '#click_link' do
   end
 
   it "casts to string" do
-    @session.click_link(:'foo')
+    @session.click_link(:foo)
     expect(@session).to have_content('Another World')
   end
 
@@ -35,7 +36,7 @@ Capybara::SpecHelper.spec '#click_link' do
       expect(@session).to have_content('Bar')
     end
 
-    it "should accept partial matches" do
+    it "should accept partial matches", :exact_false do
       @session.click_link('abo')
       expect(@session).to have_content('Bar')
     end
@@ -47,7 +48,7 @@ Capybara::SpecHelper.spec '#click_link' do
       expect(@session).to have_content('Bar')
     end
 
-    it "should accept partial matches" do
+    it "should accept partial matches", :exact_false do
       @session.click_link('some titl')
       expect(@session).to have_content('Bar')
     end
@@ -59,7 +60,7 @@ Capybara::SpecHelper.spec '#click_link' do
       expect(@session).to have_content('Bar')
     end
 
-    it "should accept partial matches" do
+    it "should accept partial matches", :exact_false do
       @session.click_link('some imag')
       expect(@session).to have_content('Bar')
     end
@@ -85,7 +86,7 @@ Capybara::SpecHelper.spec '#click_link' do
     end
   end
 
-  context "with a regex :href option given"  do
+  context "with a regex :href option given" do
     it "should find a link matching an all-matching regex pattern" do
       @session.click_link('labore', href: /.+/)
       expect(@session).to have_content('Bar')
@@ -180,13 +181,13 @@ Capybara::SpecHelper.spec '#click_link' do
 
   context "with :exact option" do
     it "should accept partial matches when false" do
-      @session.click_link('abo', exact:  false)
+      @session.click_link('abo', exact: false)
       expect(@session).to have_content('Bar')
     end
 
     it "should not accept partial matches when true" do
       expect do
-        @session.click_link('abo', exact:  true)
+        @session.click_link('abo', exact: true)
       end.to raise_error(Capybara::ElementNotFound)
     end
   end

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

@@ -1,15 +1,16 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#current_scope' do
   before do
     @session.visit('/with_scope')
   end
-  
+
   context "when not in a #within block" do
     it "should return the document" do
       expect(@session.current_scope).to be_kind_of Capybara::Node::Document
     end
   end
-  
+
   context "when in a #within block" do
     it "should return the element in scope" do
       @session.within(:css, "#simple_first_name") do
@@ -17,7 +18,7 @@ Capybara::SpecHelper.spec '#current_scope' do
       end
     end
   end
-  
+
   context "when in a nested #within block" do
     it "should return the element in scope" do
       @session.within("//div[@id='for_bar']") do

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

@@ -1,24 +1,25 @@
 # frozen_string_literal: true
+
 require "capybara/spec/test_app"
 
 Capybara::SpecHelper.spec '#current_url, #current_path, #current_host' do
   before :all do
-    @servers = 2.times.map { Capybara::Server.new(TestApp.new).boot }
+    @servers = Array.new(2) { Capybara::Server.new(TestApp.new).boot }
     # sanity check
     expect(@servers[0].port).not_to eq(@servers[1].port)
-    expect(@servers.map { |s| s.port }).not_to include 80
+    expect(@servers.map(&:port)).not_to include 80
   end
 
   def bases
     @servers.map { |s| "http://#{s.host}:#{s.port}" }
   end
 
-  def should_be_on server_index, path="/host", scheme="http"
-    #This delay is to give fully async drivers (selenium w/chromedriver) time for the browser
-    #to get to its destination - should be removed when we have a waiting current_url matcher
-    sleep 0.1  # remove and adjust tests when a waiting current_url/path matcher is implemented
+  def should_be_on(server_index, path = "/host", scheme = "http")
     # Check that we are on /host on the given server
     s = @servers[server_index]
+
+    expect(@session).to have_current_path("#{scheme}://#{s.host}:#{s.port}#{path}", url: true)
+
     expect(@session.current_url.chomp('?')).to eq("#{scheme}://#{s.host}:#{s.port}#{path}")
     expect(@session.current_host).to eq("#{scheme}://#{s.host}") # no port
     expect(@session.current_path).to eq(path)
@@ -104,4 +105,14 @@ Capybara::SpecHelper.spec '#current_url, #current_path, #current_host' do
     expect { @session.current_url }.not_to raise_exception
     expect { @session.current_path }.not_to raise_exception
   end
+
+  context "within iframe", requires: [:frames] do
+    it "should get the url of the top level browsing context" do
+      @session.visit('/within_frames')
+      expect(@session.current_url).to match(/within_frames\z/)
+      @session.within_frame('frameOne') do
+        expect(@session.current_url).to match(/within_frames\z/)
+      end
+    end
+  end
 end

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#dismiss_confirm', requires: [:modals] do
   before do
     @session.visit('/with_js')
@@ -26,7 +27,6 @@ Capybara::SpecHelper.spec '#dismiss_confirm', requires: [:modals] do
     end.to raise_error(Capybara::ModalNotFound)
   end
 
-
   it "should return the message presented" do
     message = @session.dismiss_confirm do
       @session.click_link('Open confirm')

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#dismiss_prompt', requires: [:modals] do
   before do
     @session.visit('/with_js')

data/lib/capybara/spec/session/element/assert_match_selector.rb

@@ -31,6 +31,6 @@ Capybara::SpecHelper.spec '#assert_matches_selector' do
 
   it "should accept a filter block" do
     @element.assert_matches_selector(:css, 'span') { |el| el[:class] == "number" }
-    @element.assert_not_matches_selector(:css,'span') { |el| el[:class] == "not_number" }
+    @element.assert_not_matches_selector(:css, 'span') { |el| el[:class] == "not_number" }
   end
 end

data/lib/capybara/spec/session/element/match_xpath_spec.rb

@@ -20,4 +20,4 @@ Capybara::SpecHelper.spec '#match_xpath?' do
     expect(@element).not_to have_xpath("//span[@class='not_a_number']")
     expect(@element).not_to have_xpath("//div[@class='number']")
   end
-end
+end

data/lib/capybara/spec/session/element/matches_selector_spec.rb

@@ -52,10 +52,10 @@ Capybara::SpecHelper.spec '#match_selector?' do
   it 'should accept a custom filter block' do
     @session.visit('/form')
     cbox = @session.find(:css, '#form_pets_dog')
-    expect(cbox).to match_selector(:checkbox){ |node| node[:id] == "form_pets_dog"}
-    expect(cbox).not_to match_selector(:checkbox){ |node| node[:id] != "form_pets_dog"}
-    expect(cbox.matches_selector?(:checkbox){ |node| node[:id] == "form_pets_dog"}).to be true
-    expect(cbox.matches_selector?(:checkbox){ |node| node[:id] != "form_pets_dog"}).to be false
+    expect(cbox).to match_selector(:checkbox) { |node| node[:id] == "form_pets_dog" }
+    expect(cbox).not_to match_selector(:checkbox) { |node| node[:id] != "form_pets_dog" }
+    expect(cbox.matches_selector?(:checkbox) { |node| node[:id] == "form_pets_dog" }).to be true
+    expect(cbox.matches_selector?(:checkbox) { |node| node[:id] != "form_pets_dog" }).to be false
   end
 end
 
@@ -103,4 +103,4 @@ Capybara::SpecHelper.spec '#not_matches_selector?' do
     expect(@element.not_matches_xpath?("//span", text: "42")).to be false
     expect(@element.not_matches_xpath?("//span", text: "Doesnotexist")).to be true
   end
-end if Gem::Version.new(RSpec::Expectations::Version::STRING) >= Gem::Version.new('3.1')
+end

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

@@ -1,18 +1,19 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec "#evaluate_async_script", requires: [:js] do
   it "should evaluate the given script and return whatever it produces" do
     @session.visit('/with_js')
     expect(@session.evaluate_async_script("arguments[0](4)")).to eq(4)
   end
 
-  it "should support passing elements as arguments to the script", requires: [:js, :es_args] do
+  it "should support passing elements as arguments to the script", requires: %i[js es_args] do
     @session.visit('/with_js')
     el = @session.find(:css, '#drag p')
     result = @session.evaluate_async_script("arguments[2]([arguments[0].innerText, arguments[1]])", el, "Doodle Funk")
     expect(result).to eq ["This is a draggable element.", "Doodle Funk"]
   end
 
-  it "should support returning elements after asynchronous operation", requires: [:js, :es_args] do
+  it "should support returning elements after asynchronous operation", requires: %i[js es_args] do
     @session.visit('/with_js')
     @session.find(:css, '#change') # ensure page has loaded and element is available
     el = @session.evaluate_async_script("var cb = arguments[0]; setTimeout(function(){ cb(document.getElementById('change')) }, 100)")

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

@@ -1,24 +1,25 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec "#evaluate_script", requires: [:js] do
   it "should evaluate the given script and return whatever it produces" do
     @session.visit('/with_js')
     expect(@session.evaluate_script("1+3")).to eq(4)
   end
 
-  it "should pass arguments to the script", requires: [:js, :es_args] do
+  it "should pass arguments to the script", requires: %i[js es_args] do
     @session.visit('/with_js')
     @session.evaluate_script("document.getElementById('change').textContent = arguments[0]", "Doodle Funk")
     expect(@session).to have_css('#change', text: 'Doodle Funk')
   end
 
-  it "should support passing elements as arguments to the script", requires: [:js, :es_args] do
+  it "should support passing elements as arguments to the script", requires: %i[js es_args] do
     @session.visit('/with_js')
     el = @session.find(:css, '#change')
     @session.evaluate_script("arguments[0].textContent = arguments[1]", el, "Doodle Funk")
     expect(@session).to have_css('#change', text: 'Doodle Funk')
   end
 
-  it "should support returning elements", requires: [:js, :es_args] do
+  it "should support returning elements", requires: %i[js es_args] do
     @session.visit('/with_js')
     @session.find(:css, '#change') # ensure page has loaded and element is available
     el = @session.evaluate_script("document.getElementById('change')")

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

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec "#execute_script", requires: [:js] do
   it "should execute the given script and return nothing" do
     @session.visit('/with_js')
@@ -8,16 +9,16 @@ Capybara::SpecHelper.spec "#execute_script", requires: [:js] do
 
   it "should be able to call functions defined in the page" do
     @session.visit('/with_js')
-    expect{ @session.execute_script("$('#change').text('Funky Doodle')") }.not_to raise_error
+    expect { @session.execute_script("$('#change').text('Funky Doodle')") }.not_to raise_error
   end
 
-  it "should pass arguments to the script", requires: [:js, :es_args] do
+  it "should pass arguments to the script", requires: %i[js es_args] do
     @session.visit('/with_js')
     @session.execute_script("document.getElementById('change').textContent = arguments[0]", "Doodle Funk")
     expect(@session).to have_css('#change', text: 'Doodle Funk')
   end
 
-  it "should support passing elements as arguments to the script", requires: [:js, :es_args] do
+  it "should support passing elements as arguments to the script", requires: %i[js es_args] do
     @session.visit('/with_js')
     el = @session.find(:css, '#change')
     @session.execute_script("arguments[1].textContent = arguments[0]", "Doodle Funk", el)