[py] Fix select being able to select options hidden by css rules

Author: FFederi

py/selenium/webdriver/support/select.py

@@ -114,6 +114,8 @@ def select_by_visible_text(self, text: str) -> None:
         opts = self._el.find_elements(By.XPATH, xpath)
         matched = False
         for opt in opts:
+            if not self._has_css_property_and_visible(opt):
+                raise NoSuchElementException(f"Could not locate element with visible text: {text}")
             self._set_selected(opt)
             if not self.is_multiple:
                 return
@@ -202,6 +204,8 @@ def deselect_by_visible_text(self, text: str) -> None:
         xpath = f".//option[normalize-space(.) = {self._escape_string(text)}]"
         opts = self._el.find_elements(By.XPATH, xpath)
         for opt in opts:
+            if not self._has_css_property_and_visible(opt):
+                raise NoSuchElementException(f"Could not locate element with visible text: {text}")
             self._unset_selected(opt)
             matched = True
         if not matched:
@@ -241,3 +245,14 @@ def _get_longest_token(self, value: str) -> str:
             if len(item) > len(longest):
                 longest = item
         return longest
+
+    def _has_css_property_and_visible(self, option) -> bool:
+        css_value_candidates = ["hidden", "none", "0", "0.0"]
+        css_property_candidates = ["visibility", "display", "opacity"]
+
+        for property in css_property_candidates:
+            css_value = option.value_of_css_property(property)
+            if css_value in css_value_candidates:
+                return False
+        return True
+

py/test/selenium/webdriver/common/select_class_tests.py

@@ -25,6 +25,7 @@
 disabledSelect = {"name": "no-select", "values": ["Foo"]}
 disabledSingleSelect = {"name": "single_disabled", "values": ["Enabled", "Disabled"]}
 disabledMultiSelect = {"name": "multi_disabled", "values": ["Enabled", "Disabled"]}
+invisibleMultiSelect = {"id": "invisible_multi_select", "values": ["Apples", "Oranges", "Lemons"]}
 singleSelectValues1 = {
     "name": "selectomatic",
     "values": ["One", "Two", "Four", "Still learning how to count, apparently"],
@@ -160,6 +161,12 @@ def test_raises_exception_select_by_text_multiple_disabled(driver, pages):
     with pytest.raises(NotImplementedError):
         sel.select_by_visible_text(disabledMultiSelect["values"][1])
 
+def test_raises_exception_select_by_text_multiple_hidden(driver, pages):
+    pages.load("formPage.html")
+
+    sel = Select(driver.find_element(By.ID, invisibleMultiSelect["id"]))
+    with pytest.raises(NoSuchElementException):
+        sel.select_by_visible_text(invisibleMultiSelect["values"][0])
 
 def test_deselect_all_single(driver, pages):
     pages.load("formPage.html")