Set value selected by mouse when pressing Enter in ComboBox (#16981)

With this patch pressing Enter in ComboBox sets value selected by mouse.
Selection by keyboard is not given higher priority.

Change-Id: I2e5f610923a40f67be2c1504a004af0d431a67a3

2 files changed, 171 insertions, 0 deletions

diff --git a/uitest/src/com/vaadin/tests/components/combobox/ComboBoxMouseSelectEnter.java b/uitest/src/com/vaadin/tests/components/combobox/ComboBoxMouseSelectEnter.java
new file mode 100644
index 0000000000..5af4749349
--- /dev/null
+++ b/uitest/src/com/vaadin/tests/components/combobox/ComboBoxMouseSelectEnter.java
@@ -0,0 +1,66 @@
+/*
+ * Copyright 2000-2014 Vaadin Ltd.
+ * 
+ * Licensed under the Apache License, Version 2.0 (the "License"); you may not
+ * use this file except in compliance with the License. You may obtain a copy of
+ * the License at
+ * 
+ * http://www.apache.org/licenses/LICENSE-2.0
+ * 
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
+ * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
+ * License for the specific language governing permissions and limitations under
+ * the License.
+ */
+package com.vaadin.tests.components.combobox;
+
+import com.vaadin.data.Property;
+import com.vaadin.server.VaadinRequest;
+import com.vaadin.tests.components.AbstractTestUI;
+import com.vaadin.ui.ComboBox;
+import com.vaadin.ui.Label;
+
+public class ComboBoxMouseSelectEnter extends AbstractTestUI {
+    protected ComboBox comboBox;
+
+    @Override
+    protected void setup(VaadinRequest request) {
+        comboBox = new ComboBox();
+        final Label label = new Label();
+        label.setId("value");
+
+        comboBox.setTextInputAllowed(true);
+        comboBox.setNullSelectionAllowed(true);
+        comboBox.setNullSelectionItemId(null);
+
+        for (int i = 0; i < 10; i++) {
+            comboBox.addItem("a" + i);
+        }
+
+        comboBox.addValueChangeListener(new Property.ValueChangeListener() {
+            @Override
+            public void valueChange(Property.ValueChangeEvent event) {
+                Object value = event.getProperty().getValue();
+                if (value != null) {
+                    label.setValue(value.toString());
+                } else {
+                    label.setValue("null");
+                }
+            }
+        });
+
+        addComponents(comboBox);
+        addComponent(label);
+    }
+
+    @Override
+    protected String getTestDescription() {
+        return "Pressing Enter should set value highlighted from mouse position after using arrow keys";
+    }
+
+    @Override
+    protected Integer getTicketNumber() {
+        return 16981;
+    }
+}
diff --git a/uitest/src/com/vaadin/tests/components/combobox/ComboBoxMouseSelectEnterTest.java b/uitest/src/com/vaadin/tests/components/combobox/ComboBoxMouseSelectEnterTest.java
new file mode 100644
index 0000000000..d3ba37682d
--- /dev/null
+++ b/uitest/src/com/vaadin/tests/components/combobox/ComboBoxMouseSelectEnterTest.java
@@ -0,0 +1,105 @@
+/*
+ * Copyright 2000-2014 Vaadin Ltd.
+ * 
+ * Licensed under the Apache License, Version 2.0 (the "License"); you may not
+ * use this file except in compliance with the License. You may obtain a copy of
+ * the License at
+ * 
+ * http://www.apache.org/licenses/LICENSE-2.0
+ * 
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
+ * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
+ * License for the specific language governing permissions and limitations under
+ * the License.
+ */
+package com.vaadin.tests.components.combobox;
+
+import static org.hamcrest.CoreMatchers.is;
+import static org.hamcrest.MatcherAssert.assertThat;
+
+import java.util.List;
+
+import org.junit.Test;
+import org.openqa.selenium.Keys;
+import org.openqa.selenium.WebDriver;
+import org.openqa.selenium.WebElement;
+import org.openqa.selenium.interactions.Actions;
+import org.openqa.selenium.support.ui.ExpectedCondition;
+
+import com.vaadin.testbench.By;
+import com.vaadin.testbench.elements.LabelElement;
+import com.vaadin.testbench.parallel.BrowserUtil;
+import com.vaadin.tests.tb3.MultiBrowserTest;
+import com.vaadin.tests.tb3.newelements.ComboBoxElement;
+
+public class ComboBoxMouseSelectEnterTest extends MultiBrowserTest {
+
+    private ComboBoxElement comboBoxElement;
+
+    @Override
+    public void setup() throws Exception {
+
+        super.setup();
+        openTestURL();
+        waitForElementPresent(By.className("v-filterselect"));
+        comboBoxElement = $(ComboBoxElement.class).first();
+    }
+
+    @Test
+    public void enterSetsValueSelectedByMouseOver() {
+        comboBoxElement.openPopup();
+        comboBoxElement.sendKeys(Keys.DOWN, Keys.DOWN);
+        String selectedItemText = findElement(
+                By.className("gwt-MenuItem-selected")).getText();
+        assertThat("Item selected by arrows should be a1", selectedItemText,
+                is("a1"));
+        new Actions(driver).moveToElement(getWebElementForItem("a5")).build()
+                .perform();
+        comboBoxElement.sendKeys(getReturn());
+        assertThat("Item selected by mouse should be a5",
+                comboBoxElement.getText(), is("a5"));
+        checkLabelValue("a5");
+    }
+
+    private WebElement getWebElementForItem(String wantedText) {
+        WebElement wantedItem = null;
+        List<WebElement> items = findElements(By.className("gwt-MenuItem"));
+        for (WebElement item : items) {
+            if (item.getText().equals(wantedText)) {
+                wantedItem = item;
+                break;
+            }
+        }
+        return wantedItem;
+    }
+
+    private Keys getReturn() {
+        if (BrowserUtil.isPhantomJS(getDesiredCapabilities())) {
+            return Keys.ENTER;
+        } else {
+            return Keys.RETURN;
+        }
+    }
+
+    private void checkLabelValue(final String expectedValue) {
+
+        waitUntil(new ExpectedCondition<Boolean>() {
+            private String actualValue;
+
+            @Override
+            public Boolean apply(WebDriver input) {
+                actualValue = $(LabelElement.class).id("value").getText();
+                return actualValue.equals(expectedValue);
+            }
+
+            @Override
+            public String toString() {
+                // Timed out after 10 seconds waiting for ...
+                return String.format("Label value to match '%s' (was: '%s')",
+                        expectedValue, actualValue);
+            }
+        });
+    }
+
+}
\ No newline at end of file