fix: continue searching for visible optiones before throwing in selectByVisibleText

Author: SantiagoCecena

java/src/org/openqa/selenium/support/ui/Select.java

@@ -124,11 +124,14 @@ public WebElement getFirstSelectedOption() {
   public void selectByVisibleText(String text) {
     assertSelectIsEnabled();
     assertSelectIsVisible();
+
     // try to find the option via XPATH ...
     List<WebElement> options =
         element.findElements(
             By.xpath(".//option[normalize-space(.) = " + Quotes.escape(text) + "]"));
 
+    boolean selectedAnyVisible = false;
+
     for (WebElement option : options) {
       if (!hasCssPropertyAndVisible(option)) {
         throw new NoSuchElementException("Invisible option with text: " + text);
@@ -139,6 +142,11 @@ public void selectByVisibleText(String text) {
       }
     }
 
+    if (!selectedAnyVisible && !options.isEmpty()) {
+      // if we found options, but none of them was visible, we throw an exception
+      throw new NoSuchElementException("No visible option with text: " + text);
+    }
+
     boolean matched = !options.isEmpty();
     if (!matched && text.contains(" ")) {
       String subStringWithoutSpace = getLongestSubstringWithoutSpace(text);