summaryrefslogtreecommitdiffstats
path: root/uitest
diff options
context:
space:
mode:
Diffstat (limited to 'uitest')
-rw-r--r--uitest/src/com/vaadin/tests/components/tree/TreeItemClickListening.html81
-rw-r--r--uitest/src/com/vaadin/tests/components/tree/TreeItemClickListening.java19
-rw-r--r--uitest/src/com/vaadin/tests/components/tree/TreeItemClickListeningTest.java (renamed from uitest/src/com/vaadin/tests/components/tree/TreeItemSelectionWithoutImmediateTest.java)45
-rw-r--r--uitest/src/com/vaadin/tests/components/tree/TreeItemSelectionWithoutImmediate.java46
-rw-r--r--uitest/src/com/vaadin/tests/tb3/AbstractTB3Test.java35
5 files changed, 60 insertions, 166 deletions
diff --git a/uitest/src/com/vaadin/tests/components/tree/TreeItemClickListening.html b/uitest/src/com/vaadin/tests/components/tree/TreeItemClickListening.html
deleted file mode 100644
index 841565df2f..0000000000
--- a/uitest/src/com/vaadin/tests/components/tree/TreeItemClickListening.html
+++ /dev/null
@@ -1,81 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
-<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
-<head profile="http://selenium-ide.openqa.org/profiles/test-case">
-<meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />
-<link rel="selenium.base" href="" />
-<title>New Test</title>
-</head>
-<body>
-<table cellpadding="1" cellspacing="1" border="1">
-<thead>
-<tr><td rowspan="1" colspan="3">New Test</td></tr>
-</thead><tbody>
-<tr>
- <td>open</td>
- <td>/run/com.vaadin.tests.components.tree.TreeItemClickListening?restartApplication</td>
- <td></td>
-</tr>
-<tr>
- <td>mouseClick</td>
- <td>vaadin=runcomvaadintestscomponentstreeTreeItemClickListening::/VVerticalLayout[0]/ChildComponentContainer[1]/VVerticalLayout[0]/ChildComponentContainer[0]/VTree[0]#n[0]</td>
- <td>36,7</td>
-</tr>
-<tr>
- <td>assertText</td>
- <td>vaadin=runcomvaadintestscomponentstreeTreeItemClickListening::PID_SLog_row_0</td>
- <td>1. Left Click</td>
-</tr>
-<tr>
- <td>mouseClick</td>
- <td>vaadin=runcomvaadintestscomponentstreeTreeItemClickListening::/VVerticalLayout[0]/ChildComponentContainer[1]/VVerticalLayout[0]/ChildComponentContainer[0]/VTree[0]#n[1]</td>
- <td>45,7</td>
-</tr>
-<tr>
- <td>assertText</td>
- <td>vaadin=runcomvaadintestscomponentstreeTreeItemClickListening::PID_SLog_row_0</td>
- <td>2. Left Click</td>
-</tr>
-<tr>
- <td>mouseClick</td>
- <td>vaadin=runcomvaadintestscomponentstreeTreeItemClickListening::/VVerticalLayout[0]/ChildComponentContainer[1]/VVerticalLayout[0]/ChildComponentContainer[0]/VTree[0]#n[2]</td>
- <td>37,13</td>
-</tr>
-<tr>
- <td>assertText</td>
- <td>vaadin=runcomvaadintestscomponentstreeTreeItemClickListening::PID_SLog_row_0</td>
- <td>3. Left Click</td>
-</tr>
-<tr>
- <td>mouseDownRight</td>
- <td>vaadin=runcomvaadintestscomponentstreeTreeItemClickListening::/VVerticalLayout[0]/ChildComponentContainer[1]/VVerticalLayout[0]/ChildComponentContainer[0]/VTree[0]#n[3]</td>
- <td></td>
-</tr>
-<tr>
- <td>mouseUpRight</td>
- <td>vaadin=runcomvaadintestscomponentstreeTreeItemClickListening::/VVerticalLayout[0]/ChildComponentContainer[1]/VVerticalLayout[0]/ChildComponentContainer[0]/VTree[0]#n[3]</td>
- <td></td>
-</tr>
-<tr>
- <td>assertText</td>
- <td>vaadin=runcomvaadintestscomponentstreeTreeItemClickListening::PID_SLog_row_0</td>
- <td>4. Right Click</td>
-</tr>
-<tr>
- <td>mouseDownRight</td>
- <td>vaadin=runcomvaadintestscomponentstreeTreeItemClickListening::/VVerticalLayout[0]/ChildComponentContainer[1]/VVerticalLayout[0]/ChildComponentContainer[0]/VTree[0]#n[4]</td>
- <td></td>
-</tr>
-<tr>
- <td>mouseUpRight</td>
- <td>vaadin=runcomvaadintestscomponentstreeTreeItemClickListening::/VVerticalLayout[0]/ChildComponentContainer[1]/VVerticalLayout[0]/ChildComponentContainer[0]/VTree[0]#n[4]</td>
- <td></td>
-</tr>
-<tr>
- <td>assertText</td>
- <td>vaadin=runcomvaadintestscomponentstreeTreeItemClickListening::PID_SLog_row_0</td>
- <td>5. Right Click</td>
-</tr>
-</tbody></table>
-</body>
-</html>
diff --git a/uitest/src/com/vaadin/tests/components/tree/TreeItemClickListening.java b/uitest/src/com/vaadin/tests/components/tree/TreeItemClickListening.java
index 724c1c6f62..a3f2b113cd 100644
--- a/uitest/src/com/vaadin/tests/components/tree/TreeItemClickListening.java
+++ b/uitest/src/com/vaadin/tests/components/tree/TreeItemClickListening.java
@@ -1,24 +1,22 @@
package com.vaadin.tests.components.tree;
import com.vaadin.event.ItemClickEvent;
-import com.vaadin.tests.components.TestBase;
-import com.vaadin.tests.util.Log;
+import com.vaadin.server.VaadinRequest;
+import com.vaadin.tests.components.AbstractTestUIWithLog;
import com.vaadin.ui.Tree;
-public class TreeItemClickListening extends TestBase {
+public class TreeItemClickListening extends AbstractTestUIWithLog {
private int clickCounter = 0;
- private Log log = new Log(5);
-
@Override
- protected void setup() {
+ protected void setup(VaadinRequest request) {
Tree tree = new Tree();
tree.setImmediate(true);
tree.addContainerProperty("caption", String.class, "");
- for (int i = 1; i <= 10; i++) {
+ for (int i = 1; i <= 2; i++) {
String item = "Node " + i;
tree.addItem(item);
tree.getContainerProperty(item, "caption").setValue("Caption " + i);
@@ -38,19 +36,15 @@ public class TreeItemClickListening extends TestBase {
case RIGHT:
log.log("Right Click");
break;
- case MIDDLE:
- log.log("Middle Click");
- break;
}
}
});
addComponent(tree);
- addComponent(log);
}
@Override
- protected String getDescription() {
+ protected String getTestDescription() {
return "Item click event should be triggered from all mouse button clicks";
}
@@ -58,5 +52,4 @@ public class TreeItemClickListening extends TestBase {
protected Integer getTicketNumber() {
return 6845;
}
-
}
diff --git a/uitest/src/com/vaadin/tests/components/tree/TreeItemSelectionWithoutImmediateTest.java b/uitest/src/com/vaadin/tests/components/tree/TreeItemClickListeningTest.java
index 2fb35b1680..2567fdb16d 100644
--- a/uitest/src/com/vaadin/tests/components/tree/TreeItemSelectionWithoutImmediateTest.java
+++ b/uitest/src/com/vaadin/tests/components/tree/TreeItemClickListeningTest.java
@@ -16,7 +16,6 @@
package com.vaadin.tests.components.tree;
import static org.hamcrest.MatcherAssert.assertThat;
-import static org.hamcrest.Matchers.is;
import org.junit.Test;
import org.openqa.selenium.WebElement;
@@ -25,37 +24,37 @@ import org.openqa.selenium.interactions.Actions;
import com.vaadin.testbench.By;
import com.vaadin.tests.tb3.MultiBrowserTest;
-public class TreeItemSelectionWithoutImmediateTest extends MultiBrowserTest {
+public class TreeItemClickListeningTest extends MultiBrowserTest {
- private static final long serialVersionUID = 1L;
+ private WebElement getTreeNode(String caption) {
+ return getDriver().findElement(
+ By.xpath("//span[text() = '" + caption + "']"));
+ }
@Test
- public void testSelectTreeWithItemClickListenerNotImmediate()
- throws InterruptedException {
+ public void test() throws InterruptedException {
openTestURL();
- // click on item i (in circle we select next item and check if it is
- // selected in tree)
- for (int i = 1; i <= 4; i++) {
- WebElement treeItem = getTreeNode(String.format(
- TreeItemSelectionWithoutImmediate.MENU_ITEM_TEMPLATE, i));
-
- new Actions(getDriver()).moveToElement(treeItem).click().perform();
- Thread.sleep(100);
+ performLeftClick();
+ assertEventFired("1. Left Click");
- WebElement selectedElement = driver.findElement(By
- .className("v-tree-node-selected"));
+ performRightClick();
+ assertEventFired("2. Right Click");
+ }
- treeItem = getTreeNode(String.format(
- TreeItemSelectionWithoutImmediate.MENU_ITEM_TEMPLATE, i));
+ private void assertEventFired(String text) {
+ assertThat(
+ String.format("Couldn't find text '%s' from the log.", text),
+ logContainsText(text));
+ }
- assertThat("Clicked element should be selected", selectedElement
- .getText().equals(treeItem.getText()), is(true));
- }
+ private void performLeftClick() {
+ new Actions(driver).click(getTreeNode("Caption 1")).build().perform();
}
- private WebElement getTreeNode(String caption) {
- return getDriver().findElement(
- By.xpath("//span[text() = '" + caption + "']"));
+ private void performRightClick() {
+ new Actions(driver).contextClick(getTreeNode("Caption 2")).build()
+ .perform();
}
+
}
diff --git a/uitest/src/com/vaadin/tests/components/tree/TreeItemSelectionWithoutImmediate.java b/uitest/src/com/vaadin/tests/components/tree/TreeItemSelectionWithoutImmediate.java
deleted file mode 100644
index 8e0f39be99..0000000000
--- a/uitest/src/com/vaadin/tests/components/tree/TreeItemSelectionWithoutImmediate.java
+++ /dev/null
@@ -1,46 +0,0 @@
-package com.vaadin.tests.components.tree;
-
-import com.vaadin.event.ItemClickEvent;
-import com.vaadin.server.VaadinRequest;
-import com.vaadin.tests.components.AbstractTestUIWithLog;
-import com.vaadin.ui.Tree;
-
-@SuppressWarnings("serial")
-public class TreeItemSelectionWithoutImmediate extends AbstractTestUIWithLog {
-
- protected static final String TREE_ID = "TreeId";
-
- protected static final String MENU_ITEM_TEMPLATE = "Menu Item %d";
-
- @Override
- protected void setup(VaadinRequest request) {
- Tree tree = new Tree("With ItemClickListener not Immediate");
- tree.setId(TREE_ID);
- tree.setImmediate(false);
-
- for (int i = 1; i <= 4; i++) {
- tree.addItem(String.format(MENU_ITEM_TEMPLATE, i));
- }
-
- tree.addItemClickListener(new ItemClickEvent.ItemClickListener() {
-
- @Override
- public void itemClick(ItemClickEvent event) {
- log("ItemClickEvent = " + event.getItemId());
- }
- });
-
- addComponent(tree);
- }
-
- @Override
- protected String getTestDescription() {
- return "Test for ensuring that selection of tree items works correctly if immediate == false "
- + "and ItemClickListener is added to Tree";
- }
-
- @Override
- protected Integer getTicketNumber() {
- return 14388;
- }
-}
diff --git a/uitest/src/com/vaadin/tests/tb3/AbstractTB3Test.java b/uitest/src/com/vaadin/tests/tb3/AbstractTB3Test.java
index 1e1cbedbd6..8e6370a6fd 100644
--- a/uitest/src/com/vaadin/tests/tb3/AbstractTB3Test.java
+++ b/uitest/src/com/vaadin/tests/tb3/AbstractTB3Test.java
@@ -27,11 +27,10 @@ import java.lang.annotation.RetentionPolicy;
import java.lang.annotation.Target;
import java.lang.reflect.Field;
import java.net.URL;
+import java.util.ArrayList;
import java.util.Collections;
import java.util.List;
-import elemental.json.JsonObject;
-import elemental.json.impl.JsonUtil;
import org.apache.commons.io.IOUtils;
import org.apache.http.HttpHost;
import org.apache.http.HttpResponse;
@@ -66,10 +65,15 @@ import com.vaadin.testbench.TestBench;
import com.vaadin.testbench.TestBenchDriverProxy;
import com.vaadin.testbench.TestBenchElement;
import com.vaadin.testbench.TestBenchTestCase;
+import com.vaadin.testbench.elements.LabelElement;
+import com.vaadin.testbench.elements.VerticalLayoutElement;
import com.vaadin.tests.components.AbstractTestUIWithLog;
import com.vaadin.tests.tb3.MultiBrowserTest.Browser;
import com.vaadin.ui.UI;
+import elemental.json.JsonObject;
+import elemental.json.impl.JsonUtil;
+
/**
* Base class for TestBench 3+ tests. All TB3+ tests in the project should
* extend this class.
@@ -1258,7 +1262,32 @@ public abstract class AbstractTB3Test extends TestBenchTestCase {
return null;
}
- private static JsonObject extractObject(HttpResponse resp) throws IOException {
+ protected boolean logContainsText(String string) {
+ List<String> logs = getLogs();
+
+ for (String text : logs) {
+ if (text.contains(string)) {
+ return true;
+ }
+ }
+
+ return false;
+ }
+
+ protected List<String> getLogs() {
+ VerticalLayoutElement log = $(VerticalLayoutElement.class).id("Log");
+ List<LabelElement> logLabels = log.$(LabelElement.class).all();
+ List<String> logTexts = new ArrayList<String>();
+
+ for (LabelElement label : logLabels) {
+ logTexts.add(label.getText());
+ }
+
+ return logTexts;
+ }
+
+ private static JsonObject extractObject(HttpResponse resp)
+ throws IOException {
InputStream contents = resp.getEntity().getContent();
StringWriter writer = new StringWriter();
IOUtils.copy(contents, writer, "UTF8");
generated by cgit v1.2.3 (git 2.39.1) at 2025-03-12 16:52:51 +0000