Fix subparts for merged cells (#13334)

Change-Id: Ifa7fc4584cd74d0b3675e6e82a9ad98db8c46d5f

5 files changed, 20 insertions, 9 deletions

diff --git a/uitest/src/com/vaadin/tests/components/grid/GridAddAndRemoveDataOnInitTest.java b/uitest/src/com/vaadin/tests/components/grid/GridAddAndRemoveDataOnInitTest.java
index 2f333698bf..c108aaecf3 100644
--- a/uitest/src/com/vaadin/tests/components/grid/GridAddAndRemoveDataOnInitTest.java
+++ b/uitest/src/com/vaadin/tests/components/grid/GridAddAndRemoveDataOnInitTest.java
@@ -19,8 +19,10 @@ import org.junit.Assert;
 import org.junit.Test;
 
 import com.vaadin.testbench.By;
+import com.vaadin.tests.annotations.TestCategory;
 import com.vaadin.tests.tb3.MultiBrowserTest;
 
+@TestCategory("grid")
 public class GridAddAndRemoveDataOnInitTest extends MultiBrowserTest {
 
     @Test
diff --git a/uitest/src/com/vaadin/tests/components/grid/GridGeneratedPropertiesTest.java b/uitest/src/com/vaadin/tests/components/grid/GridGeneratedPropertiesTest.java
index de7fa40eab..21cc66c8a3 100644
--- a/uitest/src/com/vaadin/tests/components/grid/GridGeneratedPropertiesTest.java
+++ b/uitest/src/com/vaadin/tests/components/grid/GridGeneratedPropertiesTest.java
@@ -21,9 +21,11 @@ import static org.junit.Assert.assertTrue;
 
 import org.junit.Test;
 
+import com.vaadin.tests.annotations.TestCategory;
 import com.vaadin.tests.components.grid.GridElement.GridCellElement;
 import com.vaadin.tests.tb3.MultiBrowserTest;
 
+@TestCategory("grid")
 public class GridGeneratedPropertiesTest extends MultiBrowserTest {
 
     @Test
diff --git a/uitest/src/com/vaadin/tests/components/grid/WidgetRenderersTest.java b/uitest/src/com/vaadin/tests/components/grid/WidgetRenderersTest.java
index c6649326c8..d130eb643c 100644
--- a/uitest/src/com/vaadin/tests/components/grid/WidgetRenderersTest.java
+++ b/uitest/src/com/vaadin/tests/components/grid/WidgetRenderersTest.java
@@ -22,6 +22,7 @@ import org.junit.Test;
 import org.openqa.selenium.WebElement;
 
 import com.vaadin.testbench.By;
+import com.vaadin.tests.annotations.TestCategory;
 import com.vaadin.tests.components.grid.GridElement.GridCellElement;
 import com.vaadin.tests.tb3.MultiBrowserTest;
 
@@ -31,6 +32,7 @@ import com.vaadin.tests.tb3.MultiBrowserTest;
  * @since
  * @author Vaadin Ltd
  */
+@TestCategory("grid")
 public class WidgetRenderersTest extends MultiBrowserTest {
 
     @Test
diff --git a/uitest/src/com/vaadin/tests/components/grid/basicfeatures/client/GridFooterTest.java b/uitest/src/com/vaadin/tests/components/grid/basicfeatures/client/GridFooterTest.java
index 8124e5361f..7c0aa0deb0 100644
--- a/uitest/src/com/vaadin/tests/components/grid/basicfeatures/client/GridFooterTest.java
+++ b/uitest/src/com/vaadin/tests/components/grid/basicfeatures/client/GridFooterTest.java
@@ -16,7 +16,6 @@
 package com.vaadin.tests.components.grid.basicfeatures.client;
 
 import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertNotEquals;
 import static org.junit.Assert.assertTrue;
 
@@ -105,8 +104,9 @@ public class GridFooterTest extends GridStaticSectionTest {
         assertTrue(spannedCell.isDisplayed());
         assertEquals("2", spannedCell.getAttribute("colspan"));
 
-        GridCellElement hiddenCell = getGridElement().getFooterCell(0, 1);
-        assertFalse(hiddenCell.isDisplayed());
+        // TestBench returns the spanned cell for all columns
+        assertEquals(spannedCell.getText(), getGridElement()
+                .getFooterCell(0, 1).getText());
     }
 
     @Test
@@ -121,8 +121,9 @@ public class GridFooterTest extends GridStaticSectionTest {
         assertTrue(spannedCell.isDisplayed());
         assertEquals("2", spannedCell.getAttribute("colspan"));
 
-        GridCellElement hiddenCell = getGridElement().getFooterCell(0, 2);
-        assertFalse(hiddenCell.isDisplayed());
+        // TestBench returns the spanned cell for all columns
+        assertEquals(spannedCell.getText(), getGridElement()
+                .getFooterCell(0, 2).getText());
     }
 
     @Test
@@ -141,7 +142,8 @@ public class GridFooterTest extends GridStaticSectionTest {
         for (int columnIndex = 1; columnIndex < GridBasicFeatures.COLUMNS; columnIndex++) {
             GridCellElement hiddenCell = getGridElement().getFooterCell(0,
                     columnIndex);
-            assertFalse(hiddenCell.isDisplayed());
+            // TestBench returns the spanned cell for all columns
+            assertEquals(spannedCell.getText(), hiddenCell.getText());
         }
     }
 
diff --git a/uitest/src/com/vaadin/tests/components/grid/basicfeatures/client/GridHeaderTest.java b/uitest/src/com/vaadin/tests/components/grid/basicfeatures/client/GridHeaderTest.java
index c528571a2e..59a98899f2 100644
--- a/uitest/src/com/vaadin/tests/components/grid/basicfeatures/client/GridHeaderTest.java
+++ b/uitest/src/com/vaadin/tests/components/grid/basicfeatures/client/GridHeaderTest.java
@@ -175,8 +175,9 @@ public class GridHeaderTest extends GridStaticSectionTest {
         assertTrue(spannedCell.isDisplayed());
         assertEquals("2", spannedCell.getAttribute("colspan"));
 
+        // TestBench returns the spanned cell for all spanned columns
         GridCellElement hiddenCell = getGridElement().getHeaderCell(1, 1);
-        assertFalse(hiddenCell.isDisplayed());
+        assertEquals(spannedCell.getText(), hiddenCell.getText());
     }
 
     @Test
@@ -191,8 +192,9 @@ public class GridHeaderTest extends GridStaticSectionTest {
         assertTrue(spannedCell.isDisplayed());
         assertEquals("2", spannedCell.getAttribute("colspan"));
 
+        // TestBench returns the spanned cell for all spanned columns
         GridCellElement hiddenCell = getGridElement().getHeaderCell(1, 2);
-        assertFalse(hiddenCell.isDisplayed());
+        assertEquals(spannedCell.getText(), hiddenCell.getText());
     }
 
     @Test
@@ -209,9 +211,10 @@ public class GridHeaderTest extends GridStaticSectionTest {
                 spannedCell.getAttribute("colspan"));
 
         for (int columnIndex = 1; columnIndex < GridBasicFeatures.COLUMNS; columnIndex++) {
+            // TestBench returns the spanned cell for all spanned columns
             GridCellElement hiddenCell = getGridElement().getHeaderCell(1,
                     columnIndex);
-            assertFalse(hiddenCell.isDisplayed());
+            assertEquals(spannedCell.getText(), hiddenCell.getText());
         }
     }