capybara 2.18.0 → 3.39.2

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

@@ -1,21 +1,48 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#has_link?' do
   before do
     @session.visit('/with_html')
   end
 
-  it "should be true if the given link is on the page" do
+  it 'should be true if the given link is on the page' do
     expect(@session).to have_link('foo')
     expect(@session).to have_link('awesome title')
     expect(@session).to have_link('A link', href: '/with_simple_html')
     expect(@session).to have_link(:'A link', href: :'/with_simple_html')
-    expect(@session).to have_link('A link', href: /\/with_simple_html/)
+    expect(@session).to have_link('A link', href: %r{/with_simple_html})
+    expect(@session).to have_link('labore', target: '_self')
   end
 
-  it "should be false if the given link is not on the page" do
+  it 'should be false if the given link is not on the page' do
     expect(@session).not_to have_link('monkey')
     expect(@session).not_to have_link('A link', href: '/nonexistent-href')
     expect(@session).not_to have_link('A link', href: /nonexistent/)
+    expect(@session).not_to have_link('labore', target: '_blank')
+  end
+
+  it 'should notify if an invalid locator is specified' do
+    allow(Capybara::Helpers).to receive(:warn).and_return(nil)
+    @session.has_link?(@session)
+    expect(Capybara::Helpers).to have_received(:warn).with(/Called from: .+/)
+  end
+
+  context 'with focused:', requires: [:active_element] do
+    it 'should be true if the given link is on the page and has focus' do
+      @session.send_keys(:tab)
+
+      expect(@session).to have_link('labore', focused: true)
+    end
+
+    it 'should be false if the given link is on the page and does not have focus' do
+      expect(@session).to have_link('labore', focused: false)
+    end
+  end
+
+  it 'should raise an error if an invalid option is passed' do
+    expect do
+      expect(@session).to have_link('labore', invalid: true)
+    end.to raise_error(ArgumentError, 'Invalid option(s) :invalid, should be one of :above, :below, :left_of, :right_of, :near, :count, :minimum, :maximum, :between, :text, :id, :class, :style, :visible, :obscured, :exact, :exact_text, :normalize_ws, :match, :wait, :filter_set, :focused, :href, :alt, :title, :target, :download')
   end
 end
 
@@ -24,15 +51,29 @@ Capybara::SpecHelper.spec '#has_no_link?' do
     @session.visit('/with_html')
   end
 
-  it "should be false if the given link is on the page" do
+  it 'should be false if the given link is on the page' do
     expect(@session).not_to have_no_link('foo')
     expect(@session).not_to have_no_link('awesome title')
     expect(@session).not_to have_no_link('A link', href: '/with_simple_html')
+    expect(@session).not_to have_no_link('labore', target: '_self')
   end
 
-  it "should be true if the given link is not on the page" do
+  it 'should be true if the given link is not on the page' do
     expect(@session).to have_no_link('monkey')
     expect(@session).to have_no_link('A link', href: '/nonexistent-href')
-    expect(@session).to have_no_link('A link', href: /\/nonexistent-href/)
+    expect(@session).to have_no_link('A link', href: %r{/nonexistent-href})
+    expect(@session).to have_no_link('labore', target: '_blank')
+  end
+
+  context 'with focused:', requires: [:active_element] do
+    it 'should be true if the given link is on the page and has focus' do
+      expect(@session).to have_no_link('labore', focused: true)
+    end
+
+    it 'should be false if the given link is on the page and does not have focus' do
+      @session.send_keys(:tab)
+
+      expect(@session).to have_no_link('labore', focused: false)
+    end
   end
 end

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

@@ -1,76 +1,78 @@
+# frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#have_none_of_selectors' do
   before do
     @session.visit('/with_html')
   end
 
-  it "should be false if any of the given locators are on the page" do
-    expect {
-      expect(@session).to have_none_of_selectors(:xpath, "//p", "//a")
-    }.to raise_error ::RSpec::Expectations::ExpectationNotMetError
-    expect {
-      expect(@session).to have_none_of_selectors(:css, "p a#foo")
-    }.to raise_error ::RSpec::Expectations::ExpectationNotMetError
+  it 'should be false if any of the given locators are on the page' do
+    expect do
+      expect(@session).to have_none_of_selectors(:xpath, '//p', '//a')
+    end.to raise_error RSpec::Expectations::ExpectationNotMetError
+    expect do
+      expect(@session).to have_none_of_selectors(:css, 'p a#foo')
+    end.to raise_error RSpec::Expectations::ExpectationNotMetError
   end
 
-  it "should be true if none of the given locators are on the page" do
-    expect(@session).to have_none_of_selectors(:xpath, "//abbr", "//td" )
-    expect(@session).to have_none_of_selectors(:css, "p a#doesnotexist", "abbr")
+  it 'should be true if none of the given locators are on the page' do
+    expect(@session).to have_none_of_selectors(:xpath, '//abbr', '//td')
+    expect(@session).to have_none_of_selectors(:css, 'p a#doesnotexist', 'abbr')
   end
 
-  it "should use default selector" do
+  it 'should use default selector' do
     Capybara.default_selector = :css
-    expect(@session).to have_none_of_selectors("p a#doesnotexist", "abbr")
-    expect {
-      expect(@session).to have_none_of_selectors("abbr", "p a#foo")
-    }.to raise_error ::RSpec::Expectations::ExpectationNotMetError
+    expect(@session).to have_none_of_selectors('p a#doesnotexist', 'abbr')
+    expect do
+      expect(@session).to have_none_of_selectors('abbr', 'p a#foo')
+    end.to raise_error RSpec::Expectations::ExpectationNotMetError
   end
 
-  context "should respect scopes" do
-    it "when used with `within`" do
+  context 'should respect scopes' do
+    it 'when used with `within`' do
       @session.within "//p[@id='first']" do
-        expect {
+        expect do
           expect(@session).to have_none_of_selectors(".//a[@id='foo']")
-        }.to raise_error ::RSpec::Expectations::ExpectationNotMetError
+        end.to raise_error RSpec::Expectations::ExpectationNotMetError
         expect(@session).to have_none_of_selectors(".//a[@id='red']")
       end
     end
 
-    it "when called on an element" do
+    it 'when called on an element' do
       el = @session.find "//p[@id='first']"
-      expect {
+      expect do
         expect(el).to have_none_of_selectors(".//a[@id='foo']")
-      }.to raise_error ::RSpec::Expectations::ExpectationNotMetError
+      end.to raise_error RSpec::Expectations::ExpectationNotMetError
       expect(el).to have_none_of_selectors(".//a[@id='red']")
     end
   end
 
-  context "with options" do
-    it "should apply the options to all locators" do
-      expect {
-        expect(@session).to have_none_of_selectors("//p//a", text: "Redirect")
-      }.to raise_error ::RSpec::Expectations::ExpectationNotMetError
-      expect(@session).to have_none_of_selectors("//p", text: "Doesnotexist")
+  context 'with options' do
+    it 'should apply the options to all locators' do
+      expect do
+        expect(@session).to have_none_of_selectors('//p//a', text: 'Redirect')
+      end.to raise_error RSpec::Expectations::ExpectationNotMetError
+      expect(@session).to have_none_of_selectors('//p', text: 'Doesnotexist')
     end
 
-    it "should discard all matches where the given regexp is matched" do
-      expect {
-        expect(@session).to have_none_of_selectors("//p//a", text: /re[dab]i/i, count: 1)
-      }.to raise_error ::RSpec::Expectations::ExpectationNotMetError
-      expect(@session).to have_none_of_selectors("//p//a", text: /Red$/)
+    it 'should discard all matches where the given regexp is matched' do
+      expect do
+        expect(@session).to have_none_of_selectors('//p//a', text: /re[dab]i/i, count: 1)
+      end.to raise_error RSpec::Expectations::ExpectationNotMetError
+      expect(@session).to have_none_of_selectors('//p//a', text: /Red$/)
     end
   end
 
-  context "with wait", requires: [:js] do
-    it "should not find elements if they appear after given wait duration" do
+  context 'with wait', requires: [:js] do
+    it 'should not find elements if they appear after given wait duration' do
       @session.visit('/with_js')
       @session.click_link('Click me')
-      expect(@session).to have_none_of_selectors(:css, "#new_field", "a#has-been-clicked", wait: 0.1)
+      expect(@session).to have_none_of_selectors(:css, '#new_field', 'a#has-been-clicked', wait: 0.1)
     end
   end
 
-  it "cannot be negated" do
-    expect {
-      expect(@session).not_to have_none_of_selectors(:css, "p a#foo", "h2#h2one", "h2#h2two")
-    }.to raise_error ArgumentError
+  it 'cannot be negated' do
+    expect do
+      expect(@session).not_to have_none_of_selectors(:css, 'p a#foo', 'h2#h2one', 'h2#h2two')
+    end.to raise_error ArgumentError
   end
 end

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

@@ -1,20 +1,21 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#has_select?' do
   before { @session.visit('/form') }
 
-  it "should be true if the field is on the page" do
+  it 'should be true if the field is on the page' 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
+  it 'should be false if the field is not on the page' do
     expect(@session).not_to have_select('Monkey')
   end
 
   context 'with selected value' do
-    it "should be true if a field with the given value is on the page" do
+    it 'should be true if a field with the given value is on the page' do
       expect(@session).to have_select('form_locale', selected: 'English')
       expect(@session).to have_select('Region', selected: 'Norway')
       expect(@session).to have_select('Underwear', selected: [
@@ -22,7 +23,7 @@ Capybara::SpecHelper.spec '#has_select?' do
       ])
     end
 
-    it "should be false if the given field is not on the page" do
+    it 'should be false if the given field is not on the page' do
       expect(@session).not_to have_select('Locale', selected: 'Swedish')
       expect(@session).not_to have_select('Does not exist', selected: 'John')
       expect(@session).not_to have_select('City', selected: 'Not there')
@@ -33,28 +34,28 @@ Capybara::SpecHelper.spec '#has_select?' do
         '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
 
-    it "should be true after the given value is selected" do
+    it 'should be true after the given value is selected' do
       @session.select('Swedish', from: 'Locale')
       expect(@session).to have_select('Locale', selected: 'Swedish')
     end
 
-    it "should be false after a different value is selected" do
+    it 'should be false after a different value is selected' do
       @session.select('Swedish', from: 'Locale')
       expect(@session).not_to have_select('Locale', selected: 'English')
     end
 
-    it "should be true after the given values are selected" do
+    it 'should be true after the given values are selected' do
       @session.select('Boxers', from: 'Underwear')
       expect(@session).to have_select('Underwear', selected: [
         'Boxerbriefs', 'Briefs', 'Boxers', 'Commando', "Frenchman's Pantalons", 'Long Johns'
       ])
     end
 
-    it "should be false after one of the values is unselected" do
+    it 'should be false after one of the values is unselected' do
       @session.unselect('Briefs', from: 'Underwear')
       expect(@session).not_to have_select('Underwear', selected: [
         'Boxerbriefs', 'Briefs', 'Commando', "Frenchman's Pantalons", 'Long Johns'
@@ -62,98 +63,132 @@ Capybara::SpecHelper.spec '#has_select?' do
     end
 
     it "should be true even when the selected option invisible, regardless of the select's visibility" do
-      expect(@session).to have_select('Icecream', visible: false, selected: 'Chocolate')
+      expect(@session).to have_select('Icecream', visible: :hidden, selected: 'Chocolate')
       expect(@session).to have_select('Sorbet', selected: 'Vanilla')
     end
   end
 
   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'])
+    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: %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'])
+    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: %w[Boxerbriefs Boxers])
     end
 
-    it "should be true after the given partial value is selected" do
+    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
+    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: :hidden, with_options: %w[Pudding Tiramisu])
       expect(@session).to have_select('Cake', with_selected: ['Chocolate Cake', 'Sponge Cake'])
     end
 
-    it "should support non array partial values" do
+    it 'should support non array partial values' do
       expect(@session).to have_select('Underwear', with_selected: 'Briefs')
       expect(@session).not_to have_select('Underwear', with_selected: 'Boxers')
     end
   end
 
   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'])
+    it 'should be true if a field with the given options is on the page' do
+      expect(@session).to have_select('Region', options: %w[Norway Sweden Finland])
       expect(@session).to have_select('Tendency', options: [])
     end
 
-    it "should be false if the given field is not on the page" do
+    it 'should be false if the given field is not on the page' 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'])
+    it 'should be true even when the options are invisible, if the select itself is invisible' do
+      expect(@session).to have_select('Icecream', visible: :hidden, options: %w[Chocolate Vanilla Strawberry])
+    end
+  end
+
+  context 'with enabled options' do
+    it 'should be true if the listed options exist and are enabled' do
+      expect(@session).to have_select('form_title', enabled_options: %w[Mr Mrs Miss])
+    end
+
+    it 'should be false if the listed options do not exist' do
+      expect(@session).not_to have_select('form_title', enabled_options: ['Not there'])
+    end
+
+    it 'should be false if the listed option exists but is not enabled' do
+      expect(@session).not_to have_select('form_title', enabled_options: %w[Mr Mrs Miss Other])
+    end
+  end
+
+  context 'with disabled options' do
+    it 'should be true if the listed options exist and are disabled' do
+      expect(@session).to have_select('form_title', disabled_options: ['Other'])
+    end
+
+    it 'should be false if the listed options do not exist' do
+      expect(@session).not_to have_select('form_title', disabled_options: ['Not there'])
+    end
+
+    it 'should be false if the listed option exists but is not disabled' do
+      expect(@session).not_to have_select('form_title', disabled_options: %w[Other Mrs])
     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'])
+    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: %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
+    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'])
+    it 'should be true even when the options are invisible, if the select itself is invisible' do
+      expect(@session).to have_select('Icecream', visible: :hidden, with_options: %w[Vanilla Strawberry])
     end
   end
 
   context 'with multiple option' do
-    it "should find multiple selects if true" do
+    it 'should find multiple selects if true' do
       expect(@session).to have_select('form_languages', multiple: true)
       expect(@session).not_to have_select('form_other_title', multiple: true)
     end
 
-    it "should not find multiple selects if false" do
+    it 'should not find multiple selects if false' do
       expect(@session).not_to have_select('form_languages', multiple: false)
       expect(@session).to have_select('form_other_title', multiple: false)
     end
 
-    it "should find both if not specified" do
+    it 'should find both if not specified' do
       expect(@session).to have_select('form_languages')
       expect(@session).to have_select('form_other_title')
     end
   end
 
-  it "should support locator-less usage" do
-    expect(@session.has_select?(with_options: ['Norway', 'Sweden'])).to eq true
+  it 'should raise an error if an invalid option is passed' do
+    expect do
+      expect(@session).to have_select('form_languages', invalid: true)
+    end.to raise_error(ArgumentError, 'Invalid option(s) :invalid, should be one of :above, :below, :left_of, :right_of, :near, :count, :minimum, :maximum, :between, :text, :id, :class, :style, :visible, :obscured, :exact, :exact_text, :normalize_ws, :match, :wait, :filter_set, :focused, :disabled, :name, :placeholder, :options, :enabled_options, :disabled_options, :selected, :with_selected, :multiple, :with_options')
+  end
+
+  it 'should support locator-less usage' do
+    expect(@session.has_select?(with_options: %w[Norway Sweden])).to be 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 be true
     expect(@session).to have_select(with_selected: ['Briefs'])
   end
 end
@@ -161,18 +196,18 @@ end
 Capybara::SpecHelper.spec '#has_no_select?' do
   before { @session.visit('/form') }
 
-  it "should be false if the field is on the page" do
+  it 'should be false if the field is on the page' do
     expect(@session).not_to have_no_select('Locale')
     expect(@session).not_to have_no_select('form_region')
     expect(@session).not_to have_no_select('Languages')
   end
 
-  it "should be true if the field is not on the page" do
+  it 'should be true if the field is not on the page' do
     expect(@session).to have_no_select('Monkey')
   end
 
   context 'with selected value' do
-    it "should be false if a field with the given value is on the page" do
+    it 'should be false if a field with the given value is on the page' do
       expect(@session).not_to have_no_select('form_locale', selected: 'English')
       expect(@session).not_to have_no_select('Region', selected: 'Norway')
       expect(@session).not_to have_no_select('Underwear', selected: [
@@ -180,7 +215,7 @@ Capybara::SpecHelper.spec '#has_no_select?' do
       ])
     end
 
-    it "should be true if the given field is not on the page" do
+    it 'should be true if the given field is not on the page' do
       expect(@session).to have_no_select('Locale', selected: 'Swedish')
       expect(@session).to have_no_select('Does not exist', selected: 'John')
       expect(@session).to have_no_select('City', selected: 'Not there')
@@ -191,28 +226,28 @@ Capybara::SpecHelper.spec '#has_no_select?' do
         '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
 
-    it "should be false after the given value is selected" do
+    it 'should be false after the given value is selected' do
       @session.select('Swedish', from: 'Locale')
       expect(@session).not_to have_no_select('Locale', selected: 'Swedish')
     end
 
-    it "should be true after a different value is selected" do
+    it 'should be true after a different value is selected' do
       @session.select('Swedish', from: 'Locale')
       expect(@session).to have_no_select('Locale', selected: 'English')
     end
 
-    it "should be false after the given values are selected" do
+    it 'should be false after the given values are selected' do
       @session.select('Boxers', from: 'Underwear')
       expect(@session).not_to have_no_select('Underwear', selected: [
         'Boxerbriefs', 'Briefs', 'Boxers', 'Commando', "Frenchman's Pantalons", 'Long Johns'
       ])
     end
 
-    it "should be true after one of the values is unselected" do
+    it 'should be true after one of the values is unselected' do
       @session.unselect('Briefs', from: 'Underwear')
       expect(@session).to have_no_select('Underwear', selected: [
         'Boxerbriefs', 'Briefs', 'Commando', "Frenchman's Pantalons", 'Long Johns'
@@ -221,61 +256,61 @@ Capybara::SpecHelper.spec '#has_no_select?' do
   end
 
   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'])
+    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: %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'])
+    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: %w[Boxerbriefs Boxers])
     end
 
-    it "should be false after the given partial value is selected" do
+    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
+    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
+    it 'should support non array partial values' do
       expect(@session).not_to have_no_select('Underwear', with_selected: 'Briefs')
       expect(@session).to have_no_select('Underwear', with_selected: 'Boxers')
     end
   end
 
   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'])
+    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: %w[Norway Sweden Finland])
     end
 
-    it "should be true if the given field is not on the page" do
+    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'])
+    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: %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
+    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_selected: ['Boxers'])).to eq true
-    expect(@session).to have_no_select(with_selected: ['Commando', 'Boxers'])
+  it 'should support locator-less usage' do
+    expect(@session.has_no_select?(with_options: %w[Norway Sweden Finland Latvia])).to be true
+    expect(@session).to have_no_select(with_options: ['New London'])
+    expect(@session.has_no_select?(id: 'form_underwear', with_selected: ['Boxers'])).to be true
+    expect(@session).to have_no_select(id: 'form_underwear', with_selected: %w[Commando Boxers])
   end
 end