Fix CustomLayout: component not added at the default location (#14340)

Change-Id: I5eb76915b52f6b73ff0799631d98d5711041eeb5

2 files changed, 88 insertions, 0 deletions

diff --git a/uitest/src/com/vaadin/tests/layouts/customlayout/DefaultLocationInCustomLayout.java b/uitest/src/com/vaadin/tests/layouts/customlayout/DefaultLocationInCustomLayout.java
new file mode 100644
index 0000000000..18ec491c0b
--- /dev/null
+++ b/uitest/src/com/vaadin/tests/layouts/customlayout/DefaultLocationInCustomLayout.java
@@ -0,0 +1,49 @@
+/*
+ * 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.layouts.customlayout;
+
+import com.vaadin.server.VaadinRequest;
+import com.vaadin.tests.components.AbstractTestUI;
+import com.vaadin.ui.Button;
+import com.vaadin.ui.CustomLayout;
+
+@SuppressWarnings("serial")
+public class DefaultLocationInCustomLayout extends AbstractTestUI {
+
+    protected static final String BUTTON_ID = "DefaultLocationInCustomLayoutTestButtonId";
+
+    @Override
+    protected Integer getTicketNumber() {
+        return 14340;
+    }
+
+    @Override
+    protected String getTestDescription() {
+        return "A test for adding a component at the default location in a "
+                + "CustomLayout: a button should be visible.";
+    }
+
+    @Override
+    protected void setup(VaadinRequest request) {
+        setTheme("tests-tickets");
+        CustomLayout customLayout = new CustomLayout("Ticket14340");
+        final Button button = new Button("Button");
+        button.setId(BUTTON_ID);
+        customLayout.addComponent(button);
+        addComponent(customLayout);
+    }
+
+}
diff --git a/uitest/src/com/vaadin/tests/layouts/customlayout/DefaultLocationInCustomLayoutTest.java b/uitest/src/com/vaadin/tests/layouts/customlayout/DefaultLocationInCustomLayoutTest.java
new file mode 100644
index 0000000000..f4ac7419f7
--- /dev/null
+++ b/uitest/src/com/vaadin/tests/layouts/customlayout/DefaultLocationInCustomLayoutTest.java
@@ -0,0 +1,39 @@
+/*
+ * 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.layouts.customlayout;
+
+import static org.hamcrest.CoreMatchers.is;
+import static org.hamcrest.MatcherAssert.assertThat;
+
+import org.junit.Test;
+import org.openqa.selenium.By;
+
+import com.vaadin.tests.tb3.MultiBrowserTest;
+
+public class DefaultLocationInCustomLayoutTest extends MultiBrowserTest {
+
+    @Test
+    public void buttonExistsInLayout() {
+        openTestURL();
+
+        // We don't use TestBench's ElementQuery here because we need to check
+        // the DOM for buttons existence.
+        assertThat(
+                driver.findElements(
+                        By.id(DefaultLocationInCustomLayout.BUTTON_ID)).size(),
+                is(1));
+    }
+}