From 78e5cb1a9606402ff087196cc15cbcf7f17263ac Mon Sep 17 00:00:00 2001 From: Teemu Suo-Anttila Date: Mon, 9 Feb 2015 16:25:24 +0200 Subject: Fix RpcDataProviderExtension value change listener setup (#16550) This patch changes value change listener mapping from itemid based to index based mapping. This makes removing rows much less error prone Change-Id: I77e9078de4ae61ce5d752cc394aa47bccd505e70 --- .../grid/basicfeatures/server/GridStructureTest.java | 12 ++++++++++++ 1 file changed, 12 insertions(+) (limited to 'uitest/src/com/vaadin/tests/components') diff --git a/uitest/src/com/vaadin/tests/components/grid/basicfeatures/server/GridStructureTest.java b/uitest/src/com/vaadin/tests/components/grid/basicfeatures/server/GridStructureTest.java index 08f903b3fe..e83031f227 100644 --- a/uitest/src/com/vaadin/tests/components/grid/basicfeatures/server/GridStructureTest.java +++ b/uitest/src/com/vaadin/tests/components/grid/basicfeatures/server/GridStructureTest.java @@ -232,6 +232,18 @@ public class GridStructureTest extends GridBasicFeaturesTest { .findElements(By.tagName("tr")).size()); } + @Test + public void testRemoveFirstRowTwice() { + openTestURL(); + + selectMenuPath("Component", "Body rows", "Remove first row"); + selectMenuPath("Component", "Body rows", "Remove first row"); + + getGridElement().scrollToRow(50); + assertFalse("Listener setup problem occurred.", + logContainsText("AssertionError: Value change listeners")); + } + @Test public void testVerticalScrollBarVisibilityWhenEnoughRows() throws Exception { -- cgit v1.2.3