[rb] element index is a property and not an attribute

titusfortner · titusfortner · commit 5a3b3721700b · 2021-10-19T20:16:08.000-05:00
diff --git a/rb/lib/selenium/webdriver/support/select.rb b/rb/lib/selenium/webdriver/support/select.rb
@@ -258,7 +258,7 @@ def find_by_text(text)
         end
 
         def find_by_index(index)
-          options.select { |option| option.dom_attribute(:index) == index.to_s }
+          options.select { |option| option.property(:index) == index }
         end
 
         def find_by_value(value)
diff --git a/rb/spec/unit/selenium/webdriver/support/select_spec.rb b/rb/spec/unit/selenium/webdriver/support/select_spec.rb
@@ -126,19 +126,19 @@ module Support
           first_option = instance_double(Element, selected?: true)
           second_option = instance_double(Element, selected?: false)
 
-          allow(first_option).to receive(:dom_attribute).with(:index).and_return '0'
+          allow(first_option).to receive(:property).with(:index).and_return 0
           expect(first_option).not_to receive(:click)
 
-          allow(second_option).to receive(:dom_attribute).with(:index).and_return '1'
+          allow(second_option).to receive(:property).with(:index).and_return 1
           expect(second_option).to receive(:click).once
 
           allow(multi_select).to receive(:find_elements)
             .with(tag_name: 'option')
             .and_return([first_option, second_option])
 
           Select.new(multi_select).select_by(:index, 1)
-          expect(first_option).to have_received(:dom_attribute).with(:index)
-          expect(second_option).to have_received(:dom_attribute).with(:index)
+          expect(first_option).to have_received(:property).with(:index)
+          expect(second_option).to have_received(:property).with(:index)
           expect(multi_select).to have_received(:find_elements).with(tag_name: 'option')
         end
 
@@ -198,8 +198,8 @@ module Support
             .with(tag_name: 'option')
             .and_return([first_option, second_option])
 
-          allow(first_option).to receive(:dom_attribute).with(:index).and_return('2')
-          allow(second_option).to receive(:dom_attribute).with(:index).and_return('1')
+          allow(first_option).to receive(:property).with(:index).and_return(2)
+          allow(second_option).to receive(:property).with(:index).and_return(1)
 
           expect(first_option).to receive(:click).once
           expect(second_option).not_to receive(:click)
