Make AbstractListing implement Focusable (#7965)

* Make AbstractListing implement Focusable

Fixes vaadin/framework8-issues#552

* Add tests for other components that inherit from AbstractListing

* Fix setTabIndex in NativeSelect, ListSelect, RadioButtonGroup

Also adds a test for TwinColSelect.

1 files changed, 45 insertions, 0 deletions

diff --git a/uitest/src/test/java/com/vaadin/tests/focusable/AbstractFocusableComponentTest.java b/uitest/src/test/java/com/vaadin/tests/focusable/AbstractFocusableComponentTest.java
new file mode 100644
index 0000000000..61d22ae647
--- /dev/null
+++ b/uitest/src/test/java/com/vaadin/tests/focusable/AbstractFocusableComponentTest.java
@@ -0,0 +1,45 @@
+package com.vaadin.tests.focusable;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertTrue;
+
+import org.junit.Before;
+import org.junit.Test;
+import org.openqa.selenium.WebElement;
+
+import com.vaadin.tests.tb3.MultiBrowserTest;
+
+public abstract class AbstractFocusableComponentTest extends MultiBrowserTest {
+
+    @Before
+    public void setUp() {
+        openTestURL();
+    }
+
+    @Test
+    public void testProgrammaticFocus() {
+        selectMenuPath("Component", "State", "Set focus");
+        assertTrue("Component should be focused", isFocused());
+    }
+
+    @Test
+    public void testTabIndex() {
+        assertEquals("0", getTabIndex());
+
+        selectMenuPath("Component", "State", "Tab index", "-1");
+        assertEquals("-1", getTabIndex());
+
+        selectMenuPath("Component", "State", "Tab index", "10");
+        assertEquals("10", getTabIndex());
+    }
+
+    protected String getTabIndex() {
+        return getFocusElement().getAttribute("tabindex");
+    }
+
+    protected boolean isFocused() {
+        return getFocusElement().equals(getDriver().switchTo().activeElement());
+    }
+
+    protected abstract WebElement getFocusElement();
+}