diff options
author | Denis Anisimov <denis@vaadin.com> | 2014-04-09 21:27:00 +0300 |
---|---|---|
committer | Vaadin Code Review <review@vaadin.com> | 2014-04-09 18:48:24 +0000 |
commit | 901a0bb7527a4c8b33dae7997c7754d01b33be7a (patch) | |
tree | 0991d1c012a475e76e64f44107d0a98d73bf1919 /uitest | |
parent | 52db8e5457e1afcb56a5047adbddf15c44754e8c (diff) | |
download | vaadin-framework-901a0bb7527a4c8b33dae7997c7754d01b33be7a.tar.gz vaadin-framework-901a0bb7527a4c8b33dae7997c7754d01b33be7a.zip |
Don't use WindowConnector as a content connector in UI (#13127).
Change-Id: I17b963c49ba2d2de05341cf60dca0d65187314b0
Diffstat (limited to 'uitest')
-rw-r--r-- | uitest/src/com/vaadin/tests/components/window/WindowInUiWithNoContent.java | 49 | ||||
-rw-r--r-- | uitest/src/com/vaadin/tests/components/window/WindowInUiWithNoContentTest.java | 44 |
2 files changed, 93 insertions, 0 deletions
diff --git a/uitest/src/com/vaadin/tests/components/window/WindowInUiWithNoContent.java b/uitest/src/com/vaadin/tests/components/window/WindowInUiWithNoContent.java new file mode 100644 index 0000000000..720f751cef --- /dev/null +++ b/uitest/src/com/vaadin/tests/components/window/WindowInUiWithNoContent.java @@ -0,0 +1,49 @@ +/* + * Copyright 2000-2013 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.window; + +import com.vaadin.server.VaadinRequest; +import com.vaadin.tests.components.AbstractTestUI; +import com.vaadin.ui.Window; + +/** + * Test UI for Window attached to the UI with not content. + * + * @since 7.2 + * @author Vaadin Ltd + */ +public class WindowInUiWithNoContent extends AbstractTestUI { + + @Override + protected void setup(VaadinRequest request) { + // This is requires for the test + setContent(null); + + Window window = new Window("window"); + addWindow(window); + } + + @Override + protected String getTestDescription() { + return "Client UI component should not use VWindow as a content component"; + } + + @Override + protected Integer getTicketNumber() { + return 13127; + } + +} diff --git a/uitest/src/com/vaadin/tests/components/window/WindowInUiWithNoContentTest.java b/uitest/src/com/vaadin/tests/components/window/WindowInUiWithNoContentTest.java new file mode 100644 index 0000000000..8378c8cf7b --- /dev/null +++ b/uitest/src/com/vaadin/tests/components/window/WindowInUiWithNoContentTest.java @@ -0,0 +1,44 @@ +/* + * Copyright 2000-2013 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.window; + +import org.junit.Assert; +import org.junit.Test; +import org.openqa.selenium.WebElement; + +import com.vaadin.testbench.By; +import com.vaadin.tests.tb3.MultiBrowserTest; + +/** + * Test for Window attached to the UI with not content. + * + * @since 7.2 + * @author Vaadin Ltd + */ +public class WindowInUiWithNoContentTest extends MultiBrowserTest { + + @Test + public void testWindowInEmptyUI() { + openTestURL(); + + WebElement window = driver.findElement(By.className("v-window")); + String position = window.getCssValue("position"); + + Assert.assertEquals("Window element has non-absolute position and " + + "is broken in the UI", "absolute", position); + } + +} |