watir 6.17.0 → 6.18.0

data/lib/watirspec.rb

@@ -77,6 +77,7 @@ module WatirSpec
 
       info << caps.browser_name.to_s
       info << caps.version.to_s
+      info << @implementation.driver_info
 
       Watir.logger.warn "running watirspec against #{info.join ' '} using:\n#{WatirSpec.implementation.inspect_args}",
                         ids: [:browser_info]

data/lib/watirspec/implementation.rb

@@ -1,16 +1,14 @@
 module WatirSpec
   class Implementation
     attr_writer :name, :guard_proc, :browser_class
-    attr_accessor :browser_args
+    attr_accessor :browser_args, :driver_info
 
     def initialize
       @guard_proc = nil
     end
 
-    def initialize_copy(orig)
-      super
-      # Backward compatibility < Ruby 2.4
-      @browser_args = browser_args.map { |arg| arg.is_a?(Symbol) ? arg : arg.dup }
+    def initialize_copy(_orig)
+      @browser_args = browser_args.map(&:dup)
     end
 
     def browser_class

data/lib/watirspec/runner.rb

@@ -33,7 +33,7 @@ module WatirSpec
     end
 
     def execute_if_necessary
-      execute if !@executed && @execute
+      execute if (!defined?(@executed) || !@executed) && @execute
     end
 
     def configure

data/lib/watirspec/server.rb

@@ -24,7 +24,7 @@ module WatirSpec
       private
 
       def running?
-        @running
+        defined?(@running) && @running
       end
 
       def run_server

data/spec/spec_helper.rb

@@ -1,7 +1,7 @@
 $LOAD_PATH.unshift(File.dirname(__FILE__))
 $LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
 
-if ENV['TRAVIS'] || ENV['COVERAGE']
+if ENV['GITHUB_ACTIONS'] || ENV['COVERAGE']
   require 'coveralls'
   require 'simplecov'
   require 'simplecov-console'
@@ -19,11 +19,10 @@ if ENV['TRAVIS'] || ENV['COVERAGE']
 end
 
 require 'watir'
-require 'webdrivers'
 require 'locator_spec_helper'
 require 'rspec'
 
-if ENV['TRAVIS']
+if ENV['GITHUB_ACTIONS']
   require 'rspec/retry'
   RSpec.configure do |config|
     config.verbose_retry = true
@@ -36,7 +35,3 @@ end
 SELENIUM_SELECTORS = %i[css tag_name xpath link_text partial_link_text link].freeze
 
 Watir.relaxed_locate = false if ENV['RELAXED_LOCATE'] == 'false'
-
-ENV['DISPLAY'] = ':99.0' if ENV['TRAVIS']
-
-raise 'DISPLAY not set' if Selenium::WebDriver::Platform.linux? && ENV['DISPLAY'].nil?

data/spec/unit/match_elements/element_spec.rb

@@ -377,6 +377,8 @@ describe Watir::Locators::Element::Matcher do
       it 'not thrown when still matched by text content' do
         @values_to_match = {text: /some visible/}
         allow(browser).to receive(:execute_script).and_return('some visible some hidden')
+        allow(browser).to receive(:timer).and_return(Watir::Wait::Timer.new)
+        allow(browser).to receive(:timer=).and_return(Watir::Wait::Timer.new)
 
         expect {
           expect(matcher.match(elements, values_to_match, filter)).to eq elements[1]
@@ -386,31 +388,31 @@ describe Watir::Locators::Element::Matcher do
       it 'not thrown with complex regexp matched by text content' do
         @values_to_match = {text: /some (in|)visible/}
         allow(browser).to receive(:execute_script).and_return('some visible some hidden')
+        allow(browser).to receive(:timer).and_return(Watir::Wait::Timer.new)
+        allow(browser).to receive(:timer=).and_return(Watir::Wait::Timer.new)
 
         expect {
           expect(matcher.match(elements, values_to_match, filter)).to eq elements[1]
         }.not_to have_deprecated_text_regexp
       end
 
-      not_compliant_on :watigiri do
-        it 'thrown when no longer matched by text content' do
-          @values_to_match = {text: /some visible$/}
-          allow(browser).to receive(:execute_script).and_return('some visible some hidden')
+      it 'thrown when no longer matched by text content' do
+        @values_to_match = {text: /some visible$/}
+        allow(browser).to receive(:execute_script).and_return('some visible some hidden')
+        allow(browser).to receive(:timer).and_return(Watir::Wait::Timer.new)
+        allow(browser).to receive(:timer=).and_return(Watir::Wait::Timer.new)
 
-          expect {
-            expect(matcher.match(elements, values_to_match, filter)).to eq elements[1]
-          }.to have_deprecated_text_regexp
-        end
+        expect {
+          expect(matcher.match(elements, values_to_match, filter)).to eq elements[1]
+        }.to have_deprecated_text_regexp
       end
 
-      not_compliant_on :watigiri do
-        it 'not thrown when element does not exist' do
-          @values_to_match = {text: /definitely not there/}
+      it 'not thrown when element does not exist' do
+        @values_to_match = {text: /definitely not there/}
 
-          expect {
-            expect(matcher.match(elements, values_to_match, filter)).to eq elements[1]
-          }.to_not have_deprecated_text_regexp
-        end
+        expect {
+          expect(matcher.match(elements, values_to_match, filter)).to eq elements[1]
+        }.to_not have_deprecated_text_regexp
       end
 
       # Note: This will work after:text_regexp deprecation removed

data/spec/unit/unit_helper.rb

@@ -1,4 +1,2 @@
-require 'watirspec'
-WatirSpec::Runner.execute = false
-
-require_relative '../watirspec_helper'
+require 'spec_helper'
+require 'watirspec/support/rspec_matchers'

data/spec/watirspec/after_hooks_spec.rb

@@ -50,6 +50,8 @@ describe 'Browser::AfterHooks' do
   end
 
   describe '#run' do
+    before { @yield = nil }
+
     after(:each) do
       browser.original_window.use
       browser.after_hooks.delete @page_after_hook
@@ -186,17 +188,19 @@ describe 'Browser::AfterHooks' do
     end
 
     bug 'Clicking an Element that Closes a Window is returning NoMatchingWindowFoundException', :safari do
-      it 'does not raise error when running error checks on closed window' do
-        url = WatirSpec.url_for('window_switching.html')
-        @page_after_hook = proc { browser.url }
-        browser.after_hooks.add @page_after_hook
-        browser.goto url
-        browser.a(id: 'open').click
-
-        window = browser.window(title: 'closeable window')
-        window.use
-        expect { browser.a(id: 'close').click }.to_not raise_error
-        browser.original_window.use
+      bug 'https://github.com/mozilla/geckodriver/issues/1847', :firefox do
+        it 'does not raise error when running error checks on closed window' do
+          url = WatirSpec.url_for('window_switching.html')
+          @page_after_hook = proc { browser.url }
+          browser.after_hooks.add @page_after_hook
+          browser.goto url
+          browser.a(id: 'open').click
+
+          window = browser.window(title: 'closeable window')
+          window.use
+          expect { browser.a(id: 'close').click }.to_not raise_error
+          browser.original_window.use
+        end
       end
     end
   end

data/spec/watirspec/browser_spec.rb

@@ -148,8 +148,9 @@ describe 'Browser' do
     end
   end
 
-  describe '#new' do
-    not_compliant_on :remote, :appveyor do
+  # TODO: Temporarily disabling this before moving it to unit tests
+  xdescribe '#new' do
+    not_compliant_on :remote do
       context 'with parameters' do
         let(:url) { 'http://localhost:4544/wd/hub/' }
 

data/spec/watirspec/elements/element_spec.rb

@@ -709,10 +709,13 @@ describe 'Element' do
   describe '#size' do
     it 'returns size of element' do
       size = browser.button(name: 'new_user_image').size
-
       expect(size).to be_a Selenium::WebDriver::Dimension
-      expect(size['width']).to eq 104.0
-      expect(size['height']).to eq 70.0
+
+      expected_width = browser.name == :safari ? 105 : 104
+      expected_height = browser.name == :safari ? 71 : 70
+
+      expect(size['width']).to eq expected_width
+      expect(size['height']).to eq expected_height
     end
   end
 
@@ -720,7 +723,8 @@ describe 'Element' do
     it 'returns height of element' do
       height = browser.button(name: 'new_user_image').height
 
-      expect(height).to eq 70.0
+      expected_height = browser.name == :safari ? 71 : 70
+      expect(height).to eq expected_height
     end
   end
 
@@ -728,7 +732,8 @@ describe 'Element' do
     it 'returns width of element' do
       width = browser.button(name: 'new_user_image').width
 
-      expect(width).to eq 104.0
+      expected_width = browser.name == :safari ? 105 : 104
+      expect(width).to eq expected_width
     end
   end
 
@@ -905,12 +910,10 @@ describe 'Element' do
       end
     end
 
-    not_compliant_on %i[firefox appveyor] do
-      it 'returns false if element center is surrounded by non-descendants' do
-        btn = browser.button(id: 'surrounded')
-        expect(btn).not_to be_obscured
-        expect { btn.click }.not_to raise_exception
-      end
+    it 'returns false if element center is surrounded by non-descendants' do
+      btn = browser.button(id: 'surrounded')
+      expect(btn).not_to be_obscured
+      expect { btn.click }.not_to raise_exception
     end
 
     it 'scrolls interactive element into view before checking if obscured' do

data/spec/watirspec/elements/filefield_spec.rb

@@ -124,9 +124,9 @@ describe 'FileField' do
 
     not_compliant_on :internet_explorer, %i[chrome windows] do
       it 'does not alter its argument' do
-        value = File.expand_path '.travis.yml'
+        value = File.expand_path '.rubocop.yml'
         browser.file_field.value = value
-        expect(value).to match(/\.travis\.yml$/)
+        expect(value).to match(/\.rubocop\.yml$/)
       end
     end
   end

data/spec/watirspec/elements/iframe_spec.rb

@@ -11,7 +11,7 @@ describe 'IFrame' do
 
   not_compliant_on :safari do
     bug 'Firefox 58 broke this, appears to be working in Nightly',
-        %i[firefox linux], %i[firefox appveyor] do
+        %i[firefox linux] do
       it 'handles crossframe javascript' do
         browser.goto WatirSpec.url_for('iframes.html')
 
@@ -144,12 +144,10 @@ describe 'IFrame' do
     end
   end
 
-  bug 'Safari returns NoSuchElementError instead of Stale Error', :safari do
-    it 'switches between iframe and parent when needed' do
-      browser.iframe(id: 'iframe_1').elements.each do |element|
-        element.text
-        browser.h1.text
-      end
+  it 'switches between iframe and parent when needed' do
+    browser.iframe(id: 'iframe_1').elements.each do |element|
+      element.text
+      browser.h1.text
     end
   end
 

data/spec/watirspec/elements/link_spec.rb

@@ -134,9 +134,11 @@ describe 'Link' do
       expect(browser.text.include?('User administration')).to be true
     end
 
-    it 'finds an existing link by (index: Integer) and clicks it' do
-      browser.link(index: 2).click
-      expect(browser.text.include?('User administration')).to be true
+    bug 'sometimes safari does not work on the first click', :safari do
+      it 'finds an existing link by (index: Integer) and clicks it' do
+        browser.link(index: 2).click
+        expect(browser.text.include?('User administration')).to be true
+      end
     end
 
     it "raises an UnknownObjectException if the link doesn't exist" do

data/spec/watirspec/elements/select_list_spec.rb

@@ -81,6 +81,19 @@ describe 'SelectList' do
       expect(browser.select_list(index: 0).value).to eq '3'
     end
 
+    it 'returns the value of the selected options' do
+      browser.select_list(name: 'new_user_languages').select('1')
+      expect(browser.select_list(name: 'new_user_languages').value).to eq '1'
+      browser.select_list(name: 'new_user_languages').clear
+      browser.select_list(name: 'new_user_languages').select('NO')
+      expect(browser.select_list(name: 'new_user_languages').value).to eq '3'
+    end
+
+    it 'returns null when no values selected' do
+      browser.select_list(name: 'new_user_languages').clear
+      expect(browser.select_list(name: 'new_user_languages').value).to be_nil
+    end
+
     it "raises UnknownObjectException if the select list doesn't exist" do
       expect { browser.select_list(index: 1337).value }.to raise_unknown_object_exception
     end
@@ -241,7 +254,9 @@ describe 'SelectList' do
 
       it 'selects an option with a Regexp' do
         browser.select_list(name: 'new_user_languages').clear
-        expect { browser.select_list(name: 'new_user_languages').select(/1|3/) }.to have_deprecated_select_by
+        expect {
+          browser.select_list(name: 'new_user_languages').select(/1|3/)
+        }.to have_deprecated_select_by
         expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq %w[Danish NO]
       end
     end
@@ -272,25 +287,13 @@ describe 'SelectList' do
       end
     end
 
-    it 'selects multiple options successiveley' do
+    it 'selects multiple options successively' do
       browser.select_list(name: 'new_user_languages').clear
       browser.select_list(name: 'new_user_languages').select('Danish')
       browser.select_list(name: 'new_user_languages').select('Swedish')
       expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq %w[Danish Swedish]
     end
 
-    it 'selects each item in an Array' do
-      browser.select_list(name: 'new_user_languages').clear
-      browser.select_list(name: 'new_user_languages').select(%w[Danish Swedish])
-      expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq %w[Danish Swedish]
-    end
-
-    it 'selects each item in a parameter list' do
-      browser.select_list(name: 'new_user_languages').clear
-      browser.select_list(name: 'new_user_languages').select('Danish', 'Swedish')
-      expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq %w[Danish Swedish]
-    end
-
     bug 'Safari is returning click intercepted error', :safari do
       it 'selects empty options' do
         browser.select_list(id: 'delete_user_username').select('')
@@ -353,6 +356,59 @@ describe 'SelectList' do
     it 'raises a TypeError if argument is not a String, Regexp or Numeric' do
       expect { browser.select_list(id: 'new_user_languages').select({}) }.to raise_error(TypeError)
     end
+
+    context 'multiple options' do
+      it 'in an Array' do
+        browser.select_list(name: 'new_user_languages').clear
+        browser.select_list(name: 'new_user_languages').select(%w[Danish Swedish])
+        expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq %w[Danish Swedish]
+      end
+
+      it 'in a parameter list' do
+        browser.select_list(name: 'new_user_languages').clear
+        browser.select_list(name: 'new_user_languages').select('Danish', 'Swedish')
+        expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq %w[Danish Swedish]
+      end
+
+      it 'based on text' do
+        browser.select_list(name: 'new_user_languages').clear
+        browser.select_list(name: 'new_user_languages').select([/ish/])
+        list = %w[Danish EN Swedish]
+        expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq list
+      end
+
+      it 'based on label and single regexp' do
+        browser.select_list(name: 'new_user_languages').clear
+        browser.select_list(name: 'new_user_languages').select([/NO|EN/])
+        list = %w[EN NO]
+        expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq list
+      end
+
+      it 'based on label and multiple regexp' do
+        browser.select_list(name: 'new_user_languages').clear
+        browser.select_list(name: 'new_user_languages').select([/NO/, /EN/])
+        list = %w[EN NO]
+        expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq list
+      end
+
+      it 'from an Array' do
+        browser.select_list(name: 'new_user_languages').clear
+        browser.select_list(name: 'new_user_languages').select([/ish/, /Latin/])
+        list = ['Danish', 'EN', 'Swedish', 'Azeri - Latin', 'Latin']
+        expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq list
+      end
+
+      it 'from multiple arguments' do
+        browser.select_list(name: 'new_user_languages').clear
+        browser.select_list(name: 'new_user_languages').select(/ish/, /Latin/)
+        list = ['Danish', 'EN', 'Swedish', 'Azeri - Latin', 'Latin']
+        expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq list
+      end
+
+      it 'returns the first matching value if there are multiple matches' do
+        expect(browser.select_list(name: 'new_user_languages').select([/ish/])).to eq 'Danish'
+      end
+    end
   end
 
   describe '#select!' do
@@ -403,18 +459,6 @@ describe 'SelectList' do
       expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq %w[Danish Swedish]
     end
 
-    it 'selects each item in an Array' do
-      browser.select_list(name: 'new_user_languages').clear
-      browser.select_list(name: 'new_user_languages').select!(%w[Danish Swedish])
-      expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq %w[Danish Swedish]
-    end
-
-    it 'selects each item in a parameter list' do
-      browser.select_list(name: 'new_user_languages').clear
-      browser.select_list(name: 'new_user_languages').select!('Danish', 'Swedish')
-      expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq %w[Danish Swedish]
-    end
-
     it 'selects empty options' do
       browser.select_list(id: 'delete_user_username').select!('')
       expect(browser.select_list(id: 'delete_user_username').selected_options.map(&:text)).to eq ['']
@@ -456,80 +500,155 @@ describe 'SelectList' do
       browser.select_list(id: 'new_user_languages').clear
       expect { browser.select_list(id: 'new_user_languages').select!({}) }.to raise_error(TypeError)
     end
+
+    context 'multiple options' do
+      it 'in an Array' do
+        browser.select_list(name: 'new_user_languages').clear
+        browser.select_list(name: 'new_user_languages').select!(%w[Danish Swedish])
+        expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq %w[Danish Swedish]
+      end
+
+      it 'in a parameter list' do
+        browser.select_list(name: 'new_user_languages').clear
+        browser.select_list(name: 'new_user_languages').select!('Danish', 'Swedish')
+        expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq %w[Danish Swedish]
+      end
+
+      it 'based on text' do
+        browser.select_list(name: 'new_user_languages').clear
+        browser.select_list(name: 'new_user_languages').select!([/ish/])
+        list = %w[Danish EN Swedish]
+        expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq list
+      end
+
+      it 'based on label and single regexp' do
+        browser.select_list(name: 'new_user_languages').clear
+        browser.select_list(name: 'new_user_languages').select!([/NO|EN/])
+        list = %w[EN NO]
+        expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq list
+      end
+
+      it 'based on label and multiple regexp' do
+        browser.select_list(name: 'new_user_languages').clear
+        browser.select_list(name: 'new_user_languages').select!([/NO/, /EN/])
+        list = %w[EN NO]
+        expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq list
+      end
+
+      it 'from an Array' do
+        browser.select_list(name: 'new_user_languages').clear
+        browser.select_list(name: 'new_user_languages').select!([/ish/, /Latin/])
+        list = ['Danish', 'EN', 'Swedish', 'Azeri - Latin', 'Latin']
+        expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq list
+      end
+
+      it 'from multiple arguments' do
+        browser.select_list(name: 'new_user_languages').clear
+        browser.select_list(name: 'new_user_languages').select!(/ish/, /Latin/)
+        list = ['Danish', 'EN', 'Swedish', 'Azeri - Latin', 'Latin']
+        expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq list
+      end
+
+      it 'returns the first matching value if there are multiple matches' do
+        expect(browser.select_list(name: 'new_user_languages').select!([/ish/])).to eq 'Danish'
+      end
+    end
   end
 
   describe '#select_all' do
     it 'selects multiple options based on text' do
       browser.select_list(name: 'new_user_languages').clear
-      browser.select_list(name: 'new_user_languages').select_all(/ish/)
+      expect {
+        browser.select_list(name: 'new_user_languages').select_all(/ish/)
+      }.to have_deprecated_select_all
       list = %w[Danish EN Swedish]
       expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq list
     end
 
     it 'selects multiple options based on labels' do
       browser.select_list(name: 'new_user_languages').clear
-      browser.select_list(name: 'new_user_languages').select_all(/NO|EN/)
+      expect {
+        browser.select_list(name: 'new_user_languages').select_all(/NO|EN/)
+      }.to have_deprecated_select_all
       list = %w[EN NO]
       expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq list
     end
 
     it 'selects all options in an Array' do
       browser.select_list(name: 'new_user_languages').clear
-      browser.select_list(name: 'new_user_languages').select_all([/ish/, /Latin/])
+      expect {
+        browser.select_list(name: 'new_user_languages').select_all([/ish/, /Latin/])
+      }.to have_deprecated_select_all
       list = ['Danish', 'EN', 'Swedish', 'Azeri - Latin', 'Latin']
       expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq list
     end
 
     it 'selects all options in a parameter list' do
       browser.select_list(name: 'new_user_languages').clear
-      browser.select_list(name: 'new_user_languages').select_all(/ish/, /Latin/)
+      expect {
+        browser.select_list(name: 'new_user_languages').select_all(/ish/, /Latin/)
+      }.to have_deprecated_select_all
       list = ['Danish', 'EN', 'Swedish', 'Azeri - Latin', 'Latin']
       expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq list
     end
 
     it 'returns the first matching value if there are multiple matches' do
-      expect(browser.select_list(name: 'new_user_languages').select_all(/ish/)).to eq 'Danish'
+      expect {
+        expect(browser.select_list(name: 'new_user_languages').select_all(/ish/)).to eq 'Danish'
+      }.to have_deprecated_select_all
     end
   end
 
   describe '#select_all!' do
     it 'selects multiple options based on value' do
       browser.select_list(name: 'new_user_languages').clear
-      browser.select_list(name: 'new_user_languages').select_all!(/\d+/)
+      expect {
+        browser.select_list(name: 'new_user_languages').select_all!(/\d+/)
+      }.to have_deprecated_select_all
       list = %w[Danish EN NO Russian]
       expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq list
     end
 
     it 'selects multiple options based on text' do
       browser.select_list(name: 'new_user_languages').clear
-      browser.select_list(name: 'new_user_languages').select_all!(/ish/)
+      expect {
+        browser.select_list(name: 'new_user_languages').select_all!(/ish/)
+      }.to have_deprecated_select_all
       list = %w[Danish EN Swedish]
       expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq list
     end
 
     it 'selects multiple options based on labels' do
       browser.select_list(name: 'new_user_languages').clear
-      browser.select_list(name: 'new_user_languages').select_all!(/NO|EN/)
+      expect {
+        browser.select_list(name: 'new_user_languages').select_all!(/NO|EN/)
+      }.to have_deprecated_select_all
       list = %w[EN NO]
       expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq list
     end
 
     it 'selects all options in an Array' do
       browser.select_list(name: 'new_user_languages').clear
-      browser.select_list(name: 'new_user_languages').select_all!([/ish/, /Latin/])
+      expect {
+        browser.select_list(name: 'new_user_languages').select_all!([/ish/, /Latin/])
+      }.to have_deprecated_select_all
       list = ['Danish', 'EN', 'Swedish', 'Azeri - Latin', 'Latin']
       expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq list
     end
 
     it 'selects all options in a parameter list' do
       browser.select_list(name: 'new_user_languages').clear
-      browser.select_list(name: 'new_user_languages').select_all!(/ish/, /Latin/)
+      expect {
+        browser.select_list(name: 'new_user_languages').select_all!(/ish/, /Latin/)
+      }.to have_deprecated_select_all
       list = ['Danish', 'EN', 'Swedish', 'Azeri - Latin', 'Latin']
       expect(browser.select_list(name: 'new_user_languages').selected_options.map(&:text)).to eq list
     end
 
     it 'returns the first matching value if there are multiple matches' do
-      expect(browser.select_list(name: 'new_user_languages').select_all!(/ish/)).to eq 'Danish'
+      expect {
+        expect(browser.select_list(name: 'new_user_languages').select_all!(/ish/)).to eq 'Danish'
+      }.to have_deprecated_select_all
     end
   end
 end