From 37e42d097e8507b24a1edea2c97502971b98b68f Mon Sep 17 00:00:00 2001 From: Teemu Suo-Anttila Date: Fri, 29 Jun 2018 12:18:45 +0300 Subject: Fix ComboboxScrollableWindowTest to use assertion (#11014) --- .../components/window/ComboboxScrollableWindowTest.java | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) (limited to 'uitest/src') diff --git a/uitest/src/test/java/com/vaadin/tests/components/window/ComboboxScrollableWindowTest.java b/uitest/src/test/java/com/vaadin/tests/components/window/ComboboxScrollableWindowTest.java index c4cb78eeef..62a6925282 100644 --- a/uitest/src/test/java/com/vaadin/tests/components/window/ComboboxScrollableWindowTest.java +++ b/uitest/src/test/java/com/vaadin/tests/components/window/ComboboxScrollableWindowTest.java @@ -2,8 +2,10 @@ package com.vaadin.tests.components.window; import static com.vaadin.tests.components.window.ComboboxScrollableWindow.COMBOBOX_ID; import static com.vaadin.tests.components.window.ComboboxScrollableWindow.WINDOW_ID; +import static org.junit.Assert.assertEquals; import org.junit.Test; +import org.openqa.selenium.Point; import org.openqa.selenium.WebElement; import com.vaadin.testbench.By; @@ -29,11 +31,20 @@ public class ComboboxScrollableWindowTest extends MultiBrowserTest { TestBenchElementCommands scrollable = testBenchElement( scrollableElement); scrollable.scroll(1000); + + int beforeClick = getScrollTop(scrollableElement); ComboBoxElement comboBox = $(ComboBoxElement.class).id(COMBOBOX_ID); + Point location = comboBox.getLocation(); + comboBox.openPopup(); waitForElementPresent(By.className("v-filterselect-suggestpopup")); - compareScreen("combobox-open"); + assertEquals("Clicking should not cause scrolling", beforeClick, + getScrollTop(scrollableElement)); + assertEquals("ComboBox should not move along x-axis", location.getX(), + comboBox.getLocation().getX()); + assertEquals("ComboBox should not move along y-axis", location.getY(), + comboBox.getLocation().getY()); } } -- cgit v1.2.3