From 6e80f676f4c8d8db3ae3586b6b7184237e94ede3 Mon Sep 17 00:00:00 2001
From: Henri Sara <henri.sara@gmail.com>
Date: Mon, 6 Feb 2017 12:06:35 +0200
Subject: Add Firefox 45 workaround in ComboBoxElement (#8449)

---
 .../com/vaadin/testbench/elements/ComboBoxElement.java   | 16 +++++++++++++---
 1 file changed, 13 insertions(+), 3 deletions(-)

(limited to 'testbench-api/src/main/java/com')

diff --git a/testbench-api/src/main/java/com/vaadin/testbench/elements/ComboBoxElement.java b/testbench-api/src/main/java/com/vaadin/testbench/elements/ComboBoxElement.java
index dc6a59d360..64023c7d69 100644
--- a/testbench-api/src/main/java/com/vaadin/testbench/elements/ComboBoxElement.java
+++ b/testbench-api/src/main/java/com/vaadin/testbench/elements/ComboBoxElement.java
@@ -25,6 +25,7 @@ import org.openqa.selenium.WebDriverException;
 import org.openqa.selenium.WebElement;
 
 import com.vaadin.testbench.By;
+import com.vaadin.testbench.TestBenchElement;
 import com.vaadin.testbench.elementsbase.ServerClass;
 
 @ServerClass("com.vaadin.ui.ComboBox")
@@ -86,7 +87,7 @@ public class ComboBoxElement extends AbstractSelectElement {
     private boolean selectSuggestion(String text) {
         for (WebElement suggestion : getPopupSuggestionElements()) {
             if (text.equals(suggestion.getText())) {
-                suggestion.click();
+                clickElement(suggestion);
                 return true;
             }
         }
@@ -181,7 +182,7 @@ public class ComboBoxElement extends AbstractSelectElement {
      */
     public boolean openNextPage() {
         try {
-            getSuggestionPopup().findElement(byNextPage).click();
+            clickElement(getSuggestionPopup().findElement(byNextPage));
             return true;
         } catch (WebDriverException e) {
             // PhantomJS driver can throw WDE instead of the more specific
@@ -197,7 +198,7 @@ public class ComboBoxElement extends AbstractSelectElement {
      */
     public boolean openPrevPage() {
         try {
-            getSuggestionPopup().findElement(byPrevPage).click();
+            clickElement(getSuggestionPopup().findElement(byPrevPage));
             return true;
         } catch (WebDriverException e) {
             // PhantomJS driver can throw WDE instead of the more specific
@@ -275,4 +276,13 @@ public class ComboBoxElement extends AbstractSelectElement {
             }
         }
     }
+
+    private void clickElement(WebElement element) {
+        if (isFirefox()) {
+            // Workaround for Selenium/TB and Firefox 45 issue
+            ((TestBenchElement) (element)).clickHiddenElement();
+        } else {
+            element.click();
+        }
+    }
 }
-- 
cgit v1.2.3