Add a client test for row focus moving with reset size (#16607)

Change-Id: I937ea36cbb21366e559fcb76aae79e42c039ccfc

diff --git a/uitest/src/com/vaadin/tests/components/grid/GridCellFocusOnResetSize.java b/uitest/src/com/vaadin/tests/components/grid/GridCellFocusOnResetSize.java
new file mode 100644 (file)
index 0000000..c0e3c9d
--- /dev/null
+++ b/uitest/src/com/vaadin/tests/components/grid/GridCellFocusOnResetSize.java
@@ -0,0 +1,34 @@
+/*
+ * 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.grid;
+
+import com.vaadin.annotations.Widgetset;
+import com.vaadin.server.VaadinRequest;
+import com.vaadin.tests.components.AbstractTestUI;
+import com.vaadin.tests.widgetset.TestingWidgetSet;
+import com.vaadin.tests.widgetset.client.grid.GridCellFocusOnResetSizeWidget;
+import com.vaadin.tests.widgetset.server.TestWidgetComponent;
+
+@Widgetset(TestingWidgetSet.NAME)
+public class GridCellFocusOnResetSize extends AbstractTestUI {
+
+    @Override
+    protected void setup(VaadinRequest request) {
+        addComponent(new TestWidgetComponent(
+                GridCellFocusOnResetSizeWidget.class));
+    }
+
+}

diff --git a/uitest/src/com/vaadin/tests/components/grid/GridCellFocusOnResetSizeTest.java b/uitest/src/com/vaadin/tests/components/grid/GridCellFocusOnResetSizeTest.java
new file mode 100644 (file)
index 0000000..31ccb4e
--- /dev/null
+++ b/uitest/src/com/vaadin/tests/components/grid/GridCellFocusOnResetSizeTest.java
@@ -0,0 +1,57 @@
+/*
+ * 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.grid;
+
+import static org.junit.Assert.assertTrue;
+
+import java.io.IOException;
+
+import org.junit.Test;
+import org.openqa.selenium.By;
+
+import com.vaadin.testbench.elements.GridElement;
+import com.vaadin.testbench.elementsbase.ServerClass;
+import com.vaadin.tests.tb3.MultiBrowserTest;
+
+public class GridCellFocusOnResetSizeTest extends MultiBrowserTest {
+
+    @ServerClass("com.vaadin.tests.widgetset.server.TestWidgetComponent")
+    public static class MyGridElement extends GridElement {
+    }
+
+    @Test
+    public void testCellFocusOnSizeReset() throws IOException {
+        openTestURL();
+
+        GridElement grid = $(MyGridElement.class).first();
+        int rowIndex = 9;
+        grid.getCell(rowIndex, 0).click();
+        assertTrue("Row was not focused after click.", grid.getRow(rowIndex)
+                .isFocused());
+
+        // Clicking the button decreases size until it is down to 5 rows.
+        while (rowIndex > 4) {
+            findElement(By.tagName("button")).click();
+            assertTrue("Row focus was not moved when size decreased", grid
+                    .getRow(--rowIndex).isFocused());
+        }
+
+        // Next click increases size back to 10, this should not move focus.
+        findElement(By.tagName("button")).click();
+        assertTrue("Row focus should not have moved when size increased", grid
+                .getRow(4).isFocused());
+    }
+}

diff --git a/uitest/src/com/vaadin/tests/widgetset/client/grid/GridCellFocusOnResetSizeWidget.java b/uitest/src/com/vaadin/tests/widgetset/client/grid/GridCellFocusOnResetSizeWidget.java
new file mode 100644 (file)
index 0000000..b289346
--- /dev/null
+++ b/uitest/src/com/vaadin/tests/widgetset/client/grid/GridCellFocusOnResetSizeWidget.java
@@ -0,0 +1,114 @@
+/*
+ * 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.widgetset.client.grid;
+
+import java.util.ArrayList;
+import java.util.List;
+
+import com.google.gwt.event.dom.client.ClickEvent;
+import com.google.gwt.event.dom.client.ClickHandler;
+import com.google.gwt.user.client.ui.Button;
+import com.vaadin.client.data.DataChangeHandler;
+import com.vaadin.client.data.DataSource;
+import com.vaadin.client.renderers.HtmlRenderer;
+import com.vaadin.client.widgets.Grid;
+import com.vaadin.client.widgets.Grid.SelectionMode;
+
+public class GridCellFocusOnResetSizeWidget extends
+        PureGWTTestApplication<Grid<String[]>> {
+
+    private Grid<String[]> grid;
+
+    private final class MyDataSource implements DataSource<String[]> {
+        List<String[]> rows = new ArrayList<String[]>();
+        int ROWS_MAX = 10;
+        int size = ROWS_MAX;
+        DataChangeHandler handler = null;
+        {
+            for (int i = 0; i < ROWS_MAX; ++i) {
+                rows.add(new String[] { "Foo " + i });
+            }
+        }
+
+        @Override
+        public void ensureAvailability(int firstRowIndex, int numberOfRows) {
+            handler.dataAvailable(firstRowIndex, numberOfRows);
+        }
+
+        @Override
+        public int size() {
+            return size;
+        }
+
+        @Override
+        public void setDataChangeHandler(DataChangeHandler dataChangeHandler) {
+            handler = dataChangeHandler;
+        }
+
+        @Override
+        public RowHandle<String[]> getHandle(final String[] rowData) {
+            return null;
+        }
+
+        @Override
+        public String[] getRow(int rowIndex) {
+            if (rowIndex < size && rowIndex >= 0) {
+                return rows.get(rowIndex);
+            }
+            return null;
+        }
+
+        public void changeSize() {
+            size--;
+            if (size < ROWS_MAX / 2) {
+                size = ROWS_MAX;
+            }
+            handler.resetDataAndSize(size);
+        }
+    }
+
+    private class Col extends Grid.Column<String, String[]> {
+        public Col(String header) {
+            super(header, new HtmlRenderer());
+        }
+
+        @Override
+        public String getValue(String[] row) {
+            int index = grid.getColumns().indexOf(this);
+            return "<span>" + String.valueOf(row[index]) + "</span>";
+        }
+    }
+
+    public GridCellFocusOnResetSizeWidget() {
+        super(new Grid<String[]>());
+        grid = getTestedWidget();
+        grid.setSelectionMode(SelectionMode.NONE);
+        grid.setWidth("300px");
+        grid.addColumn(new Col("Foo"));
+        final MyDataSource dataSource = new MyDataSource();
+        grid.setDataSource(dataSource);
+        Button widget = new Button("Change Container Size");
+        widget.addClickHandler(new ClickHandler() {
+
+            @Override
+            public void onClick(ClickEvent event) {
+                dataSource.changeSize();
+            }
+        });
+        addNorth(grid, 400);
+        addNorth(widget, 50);
+    }
+}