diff options
author | Patrick Tasse | 2019-06-18 20:41:02 +0000 |
---|---|---|
committer | Patrick Tasse | 2019-06-19 15:00:28 +0000 |
commit | e773a2fc0a1b05678efa38b6e9a70564dc13a1d5 (patch) | |
tree | a64a6196c0acce05fcf0e3afb5cc4d9d95059e0a | |
parent | 127c3293fb1d8da7d1c68a48114160775a128faf (diff) | |
download | org.eclipse.tracecompass-e773a2fc0a1b05678efa38b6e9a70564dc13a1d5.tar.gz org.eclipse.tracecompass-e773a2fc0a1b05678efa38b6e9a70564dc13a1d5.tar.xz org.eclipse.tracecompass-e773a2fc0a1b05678efa38b6e9a70564dc13a1d5.zip |
swtbot: Fix ConditionHelpers
Rename some static methods to start with a lower-case letter.
Fix viewIsClosed() to check that the view is closed, instead of checking
that the view is not active.
Rename viewIsOpened() to viewIsActive(), which is what the condition
checks. It is not possible to get a SWTBotView for a view that is not
open.
Add failure message for viewIsClosed() and viewIsActive().
Fix Javadoc @return tag for methods that return a condition.
Change-Id: I02103c4ede05a92ae30e43f54f13928e3120af6d
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/144397
Tested-by: CI Bot
Reviewed-by: Simon Delisle <simon.delisle@ericsson.com>
Tested-by: Simon Delisle <simon.delisle@ericsson.com>
17 files changed, 84 insertions, 54 deletions
diff --git a/ctf/org.eclipse.tracecompass.tmf.ctf.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/ctf/ui/swtbot/tests/AbstractImportAndReadSmokeTest.java b/ctf/org.eclipse.tracecompass.tmf.ctf.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/ctf/ui/swtbot/tests/AbstractImportAndReadSmokeTest.java index cc4a81b07f..f51ad56c85 100644 --- a/ctf/org.eclipse.tracecompass.tmf.ctf.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/ctf/ui/swtbot/tests/AbstractImportAndReadSmokeTest.java +++ b/ctf/org.eclipse.tracecompass.tmf.ctf.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/ctf/ui/swtbot/tests/AbstractImportAndReadSmokeTest.java @@ -124,12 +124,12 @@ public abstract class AbstractImportAndReadSmokeTest { SWTBotTree tree = fBot.tree(); assertNotNull(tree); final String tracingKey = "Tracing"; - fBot.waitUntil(ConditionHelpers.IsTreeNodeAvailable(tracingKey, tree)); + fBot.waitUntil(ConditionHelpers.isTreeNodeAvailable(tracingKey, tree)); final SWTBotTreeItem tracingNode = tree.expandNode(tracingKey); tracingNode.select(); final String projectKey = "Tracing Project"; - fBot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable(projectKey, tracingNode)); + fBot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable(projectKey, tracingNode)); final SWTBotTreeItem tracingProject = tracingNode.getNode(projectKey); assertNotNull(tracingProject); diff --git a/lttng/org.eclipse.tracecompass.lttng2.control.ui.swtbot.tests/src/org/eclipse/tracecompass/lttng2/control/ui/swtbot/tests/ControlViewAllEventsTest.java b/lttng/org.eclipse.tracecompass.lttng2.control.ui.swtbot.tests/src/org/eclipse/tracecompass/lttng2/control/ui/swtbot/tests/ControlViewAllEventsTest.java index 3e604a5d91..1ea866f87e 100644 --- a/lttng/org.eclipse.tracecompass.lttng2.control.ui.swtbot.tests/src/org/eclipse/tracecompass/lttng2/control/ui/swtbot/tests/ControlViewAllEventsTest.java +++ b/lttng/org.eclipse.tracecompass.lttng2.control.ui.swtbot.tests/src/org/eclipse/tracecompass/lttng2/control/ui/swtbot/tests/ControlViewAllEventsTest.java @@ -84,7 +84,7 @@ public class ControlViewAllEventsTest extends ControlViewTest { shell.bot().button(ControlViewSwtBotUtil.DIALOG_OK_BUTTON).click(); WaitUtils.waitForJobs(); - fBot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable(ControlViewSwtBotUtil.KERNEL_DOMAIN_NAME, sessionItem)); + fBot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable(ControlViewSwtBotUtil.KERNEL_DOMAIN_NAME, sessionItem)); SWTBotTreeItem channelItem = SWTBotUtils.getTreeItem(fBot, fTree, getNodeName(), diff --git a/lttng/org.eclipse.tracecompass.lttng2.control.ui.swtbot.tests/src/org/eclipse/tracecompass/lttng2/control/ui/swtbot/tests/ControlViewExcludeEventsTest.java b/lttng/org.eclipse.tracecompass.lttng2.control.ui.swtbot.tests/src/org/eclipse/tracecompass/lttng2/control/ui/swtbot/tests/ControlViewExcludeEventsTest.java index 6b14540429..cb8ac7771d 100644 --- a/lttng/org.eclipse.tracecompass.lttng2.control.ui.swtbot.tests/src/org/eclipse/tracecompass/lttng2/control/ui/swtbot/tests/ControlViewExcludeEventsTest.java +++ b/lttng/org.eclipse.tracecompass.lttng2.control.ui.swtbot.tests/src/org/eclipse/tracecompass/lttng2/control/ui/swtbot/tests/ControlViewExcludeEventsTest.java @@ -120,7 +120,7 @@ public class ControlViewExcludeEventsTest extends ControlViewTest { shell.bot().button(ControlViewSwtBotUtil.DIALOG_OK_BUTTON).click(); WaitUtils.waitForJobs(); - fBot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable(ControlViewSwtBotUtil.UST_DOMAIN_NAME, sessionItem)); + fBot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable(ControlViewSwtBotUtil.UST_DOMAIN_NAME, sessionItem)); // Assert that the domain is UST global SWTBotTreeItem ustGlobalDomainItem = SWTBotUtils.getTreeItem(fBot, fTree, diff --git a/lttng/org.eclipse.tracecompass.lttng2.control.ui.swtbot.tests/src/org/eclipse/tracecompass/lttng2/control/ui/swtbot/tests/ControlViewKernelFilterTest.java b/lttng/org.eclipse.tracecompass.lttng2.control.ui.swtbot.tests/src/org/eclipse/tracecompass/lttng2/control/ui/swtbot/tests/ControlViewKernelFilterTest.java index 697854e524..57b7ab376f 100644 --- a/lttng/org.eclipse.tracecompass.lttng2.control.ui.swtbot.tests/src/org/eclipse/tracecompass/lttng2/control/ui/swtbot/tests/ControlViewKernelFilterTest.java +++ b/lttng/org.eclipse.tracecompass.lttng2.control.ui.swtbot.tests/src/org/eclipse/tracecompass/lttng2/control/ui/swtbot/tests/ControlViewKernelFilterTest.java @@ -120,7 +120,7 @@ public class ControlViewKernelFilterTest extends ControlViewTest { shell.bot().button(ControlViewSwtBotUtil.DIALOG_OK_BUTTON).click(); WaitUtils.waitForJobs(); - fBot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable(ControlViewSwtBotUtil.KERNEL_DOMAIN_NAME, sessionItem)); + fBot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable(ControlViewSwtBotUtil.KERNEL_DOMAIN_NAME, sessionItem)); SWTBotTreeItem kernelDomainItem = SWTBotUtils.getTreeItem(fBot, fTree, getNodeName(), @@ -182,7 +182,7 @@ public class ControlViewKernelFilterTest extends ControlViewTest { ControlViewSwtBotUtil.SESSION_GROUP_NAME, getSessionName()); - fBot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable(ControlViewSwtBotUtil.KERNEL_DOMAIN_NAME, sessionItem)); + fBot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable(ControlViewSwtBotUtil.KERNEL_DOMAIN_NAME, sessionItem)); SWTBotTreeItem kernelDomainItem = SWTBotUtils.getTreeItem(fBot, fTree, getNodeName(), diff --git a/lttng/org.eclipse.tracecompass.lttng2.control.ui.swtbot.tests/src/org/eclipse/tracecompass/lttng2/control/ui/swtbot/tests/ControlViewLoggerTest.java b/lttng/org.eclipse.tracecompass.lttng2.control.ui.swtbot.tests/src/org/eclipse/tracecompass/lttng2/control/ui/swtbot/tests/ControlViewLoggerTest.java index 2844f87d65..f6d0fdb914 100644 --- a/lttng/org.eclipse.tracecompass.lttng2.control.ui.swtbot.tests/src/org/eclipse/tracecompass/lttng2/control/ui/swtbot/tests/ControlViewLoggerTest.java +++ b/lttng/org.eclipse.tracecompass.lttng2.control.ui.swtbot.tests/src/org/eclipse/tracecompass/lttng2/control/ui/swtbot/tests/ControlViewLoggerTest.java @@ -145,7 +145,7 @@ public class ControlViewLoggerTest extends ControlViewTest { // Click the Ok at the bottom of the dialog window shell.bot().button(ControlViewSwtBotUtil.DIALOG_OK_BUTTON).click(); WaitUtils.waitForJobs(); - fBot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable(domainName, sessionItem)); + fBot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable(domainName, sessionItem)); // Assert that the domain is correct SWTBotTreeItem domainItem = SWTBotUtils.getTreeItem(fBot, fTree, @@ -185,7 +185,7 @@ public class ControlViewLoggerTest extends ControlViewTest { // Click the Ok at the bottom of the dialog window shell.bot().button(ControlViewSwtBotUtil.DIALOG_OK_BUTTON).click(); WaitUtils.waitForJobs(); - fBot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable(domainName, sessionItem)); + fBot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable(domainName, sessionItem)); // Assert that the domain is correct domainItem = SWTBotUtils.getTreeItem(fBot, fTree, @@ -224,7 +224,7 @@ public class ControlViewLoggerTest extends ControlViewTest { // Click the Ok at the bottom of the dialog window shell.bot().button(ControlViewSwtBotUtil.DIALOG_OK_BUTTON).click(); WaitUtils.waitForJobs(); - fBot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable(domainName, sessionItem)); + fBot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable(domainName, sessionItem)); // Assert that the domain is correct domainItem = SWTBotUtils.getTreeItem(fBot, fTree, @@ -255,7 +255,7 @@ public class ControlViewLoggerTest extends ControlViewTest { // Click the Ok at the bottom of the dialog window shell.bot().button(ControlViewSwtBotUtil.DIALOG_OK_BUTTON).click(); WaitUtils.waitForJobs(); - fBot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable(domainName, sessionItem)); + fBot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable(domainName, sessionItem)); // Assert that the domain is correct domainItem = SWTBotUtils.getTreeItem(fBot, fTree, diff --git a/lttng/org.eclipse.tracecompass.lttng2.control.ui.swtbot.tests/src/org/eclipse/tracecompass/lttng2/control/ui/swtbot/tests/ControlViewProfileTest.java b/lttng/org.eclipse.tracecompass.lttng2.control.ui.swtbot.tests/src/org/eclipse/tracecompass/lttng2/control/ui/swtbot/tests/ControlViewProfileTest.java index 86d6d2b093..5feaf3a345 100644 --- a/lttng/org.eclipse.tracecompass.lttng2.control.ui.swtbot.tests/src/org/eclipse/tracecompass/lttng2/control/ui/swtbot/tests/ControlViewProfileTest.java +++ b/lttng/org.eclipse.tracecompass.lttng2.control.ui.swtbot.tests/src/org/eclipse/tracecompass/lttng2/control/ui/swtbot/tests/ControlViewProfileTest.java @@ -126,7 +126,7 @@ public class ControlViewProfileTest extends ControlViewTest { sessionGroupItem = SWTBotUtils.getTreeItem(fBot, fTree, getNodeName(), ControlViewSwtBotUtil.SESSION_GROUP_NAME); - fBot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable(SESSION_NAME, sessionGroupItem)); + fBot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable(SESSION_NAME, sessionGroupItem)); assertEquals(1, sessionGroupItem.getNodes().size()); } diff --git a/lttng/org.eclipse.tracecompass.lttng2.control.ui.swtbot.tests/src/org/eclipse/tracecompass/lttng2/control/ui/swtbot/tests/ControlViewSpecificEventTest.java b/lttng/org.eclipse.tracecompass.lttng2.control.ui.swtbot.tests/src/org/eclipse/tracecompass/lttng2/control/ui/swtbot/tests/ControlViewSpecificEventTest.java index ddd4056793..77e0205e0f 100644 --- a/lttng/org.eclipse.tracecompass.lttng2.control.ui.swtbot.tests/src/org/eclipse/tracecompass/lttng2/control/ui/swtbot/tests/ControlViewSpecificEventTest.java +++ b/lttng/org.eclipse.tracecompass.lttng2.control.ui.swtbot.tests/src/org/eclipse/tracecompass/lttng2/control/ui/swtbot/tests/ControlViewSpecificEventTest.java @@ -157,7 +157,7 @@ public class ControlViewSpecificEventTest extends ControlViewTest { WaitUtils.waitForJobs(); // Wait until the child of Sessions is activated - fBot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable(ControlViewSwtBotUtil.KERNEL_DOMAIN_NAME, sessionItem)); + fBot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable(ControlViewSwtBotUtil.KERNEL_DOMAIN_NAME, sessionItem)); // Assert that the new channel name is channel0 (which is the default name) SWTBotTreeItem channelItem = SWTBotUtils.getTreeItem(fBot, fTree, @@ -176,7 +176,7 @@ public class ControlViewSpecificEventTest extends ControlViewTest { if (event.equals(ControlViewSwtBotUtil.ALL_TREE_NODE)) { eventName = ControlViewSwtBotUtil.ALL_EVENTS_NAME; } - fBot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable(eventName, channelItem)); + fBot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable(eventName, channelItem)); // Assert that the event type in the channel node are correct SWTBotTreeItem eventItem = SWTBotUtils.getTreeItem(fBot, fTree, getNodeName(), diff --git a/lttng/org.eclipse.tracecompass.lttng2.control.ui.swtbot.tests/src/org/eclipse/tracecompass/lttng2/control/ui/swtbot/tests/ControlViewTest.java b/lttng/org.eclipse.tracecompass.lttng2.control.ui.swtbot.tests/src/org/eclipse/tracecompass/lttng2/control/ui/swtbot/tests/ControlViewTest.java index a49e843bab..fcc4cf0330 100644 --- a/lttng/org.eclipse.tracecompass.lttng2.control.ui.swtbot.tests/src/org/eclipse/tracecompass/lttng2/control/ui/swtbot/tests/ControlViewTest.java +++ b/lttng/org.eclipse.tracecompass.lttng2.control.ui.swtbot.tests/src/org/eclipse/tracecompass/lttng2/control/ui/swtbot/tests/ControlViewTest.java @@ -255,7 +255,7 @@ public class ControlViewTest { SWTBotMenu menuBot = nodeItem.contextMenu(ControlViewSwtBotUtil.CONNECT_MENU_ITEM); menuBot.click(); - fBot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable(ControlViewSwtBotUtil.SESSION_GROUP_NAME, nodeItem)); + fBot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable(ControlViewSwtBotUtil.SESSION_GROUP_NAME, nodeItem)); // Verify that node is connected fBot.waitUntil(ControlViewSwtBotUtil.isStateChanged(fNode, TargetNodeState.CONNECTED)); @@ -286,7 +286,7 @@ public class ControlViewTest { sessionGroupItem.expand(); - fBot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable(getSessionName(), sessionGroupItem)); + fBot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable(getSessionName(), sessionGroupItem)); assertEquals(1, sessionGroupItem.getNodes().size()); SWTBotTreeItem sessionItem = SWTBotUtils.getTreeItem(fBot, fTree, @@ -319,7 +319,7 @@ public class ControlViewTest { shell.bot().button(ControlViewSwtBotUtil.DIALOG_OK_BUTTON).click(); WaitUtils.waitForJobs(); - fBot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable(ControlViewSwtBotUtil.KERNEL_DOMAIN_NAME, sessionItem)); + fBot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable(ControlViewSwtBotUtil.KERNEL_DOMAIN_NAME, sessionItem)); SWTBotTreeItem channelItem = SWTBotUtils.getTreeItem(fBot, fTree, getNodeName(), @@ -369,7 +369,7 @@ public class ControlViewTest { shell.bot().button(ControlViewSwtBotUtil.DIALOG_OK_BUTTON).click(); WaitUtils.waitForJobs(); - fBot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable(ControlViewSwtBotUtil.KERNEL_DOMAIN_NAME, sessionItem)); + fBot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable(ControlViewSwtBotUtil.KERNEL_DOMAIN_NAME, sessionItem)); SWTBotTreeItem kernelDomainItem = SWTBotUtils.getTreeItem(fBot, fTree, getNodeName(), @@ -417,7 +417,7 @@ public class ControlViewTest { shell.bot().button(ControlViewSwtBotUtil.DIALOG_OK_BUTTON).click(); WaitUtils.waitForJobs(); - fBot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable(ControlViewSwtBotUtil.KERNEL_DOMAIN_NAME, sessionItem)); + fBot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable(ControlViewSwtBotUtil.KERNEL_DOMAIN_NAME, sessionItem)); kernelDomainItem = SWTBotUtils.getTreeItem(fBot, fTree, getNodeName(), @@ -482,7 +482,7 @@ public class ControlViewTest { shell.bot().radioInGroup(ControlViewSwtBotUtil.BUFFERTYPE_PER_UID, ControlViewSwtBotUtil.BUFFERTYPE_GROUP_NAME).click(); shell.bot().button(ControlViewSwtBotUtil.DIALOG_OK_BUTTON).click(); WaitUtils.waitForJobs(); - fBot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable(ControlViewSwtBotUtil.UST_DOMAIN_NAME, sessionItem)); + fBot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable(ControlViewSwtBotUtil.UST_DOMAIN_NAME, sessionItem)); } /** diff --git a/releng/org.eclipse.tracecompass.integration.swtbot.tests/src/org/eclipse/tracecompass/integration/swtbot/tests/projectexplorer/ProjectExplorerTraceActionsTest.java b/releng/org.eclipse.tracecompass.integration.swtbot.tests/src/org/eclipse/tracecompass/integration/swtbot/tests/projectexplorer/ProjectExplorerTraceActionsTest.java index b94beef468..152a573486 100644 --- a/releng/org.eclipse.tracecompass.integration.swtbot.tests/src/org/eclipse/tracecompass/integration/swtbot/tests/projectexplorer/ProjectExplorerTraceActionsTest.java +++ b/releng/org.eclipse.tracecompass.integration.swtbot.tests/src/org/eclipse/tracecompass/integration/swtbot/tests/projectexplorer/ProjectExplorerTraceActionsTest.java @@ -428,7 +428,7 @@ public class ProjectExplorerTraceActionsTest { fBot.waitUntil(new ConditionHelpers.ActiveEventsEditor(fBot, "Experiment")); SWTBotTreeItem experimentsItem = SWTBotUtils.getTraceProjectItem(fBot, SWTBotUtils.selectProject(fBot, TRACE_PROJECT_NAME), "Experiments"); experimentsItem.expand(); - fBot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable("Experiment [2]", experimentsItem)); + fBot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable("Experiment [2]", experimentsItem)); SWTBotTreeItem expItem = SWTBotUtils.getTraceProjectItem(fBot, experimentsItem, "Experiment"); @@ -466,7 +466,7 @@ public class ProjectExplorerTraceActionsTest { fBot.viewByTitle(PROJECT_EXPLORER_VIEW_NAME).setFocus(); SWTBotTreeItem experimentsItem = SWTBotUtils.getTraceProjectItem(fBot, SWTBotUtils.selectProject(fBot, TRACE_PROJECT_NAME), "Experiments"); experimentsItem.expand(); - fBot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable("ExampleCustomTxt.log [1]", experimentsItem)); + fBot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable("ExampleCustomTxt.log [1]", experimentsItem)); SWTBotTreeItem expItem = SWTBotUtils.getTraceProjectItem(fBot, experimentsItem, "ExampleCustomTxt.log [1]"); expItem.expand(); SWTBotTreeItem expTrace = expItem.getNode(TRACE_NAME); @@ -505,7 +505,7 @@ public class ProjectExplorerTraceActionsTest { fBot.viewByTitle(PROJECT_EXPLORER_VIEW_NAME).setFocus(); SWTBotTreeItem experimentsItem = SWTBotUtils.getTraceProjectItem(fBot, SWTBotUtils.selectProject(fBot, TRACE_PROJECT_NAME), "Experiments"); experimentsItem.expand(); - fBot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable("ExampleCustomTxt.log [1]", experimentsItem)); + fBot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable("ExampleCustomTxt.log [1]", experimentsItem)); SWTBotTreeItem expItem = SWTBotUtils.getTraceProjectItem(fBot, experimentsItem, "ExampleCustomTxt.log [1]"); expItem.expand(); SWTBotTreeItem expTrace = expItem.getNode(TRACE_NAME); @@ -701,11 +701,11 @@ public class ProjectExplorerTraceActionsTest { private static void testLinkStatus(SWTBotTreeItem traceItem, boolean isLinked) { SWTBotView viewBot = fBot.viewByTitle(PROPERTIES_VIEW_NAME); viewBot.show(); - fBot.waitUntil(ConditionHelpers.viewIsOpened(viewBot)); + fBot.waitUntil(ConditionHelpers.viewIsActive(viewBot)); traceItem.select(); SWTBotTree tree = viewBot.bot().tree(); SWTBotTreeItem resourcePropertiesItem = tree.getTreeItem(RESOURCE_PROPERTIES_ITEM_NAME); - fBot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable(LINKED_ITEM_NAME, resourcePropertiesItem)); + fBot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable(LINKED_ITEM_NAME, resourcePropertiesItem)); SWTBotTreeItem linkedNode = resourcePropertiesItem.getNode(LINKED_ITEM_NAME); String linkedValue = linkedNode.cell(1); assertEquals(Boolean.toString(isLinked), linkedValue); diff --git a/tmf/org.eclipse.tracecompass.tmf.analysis.xml.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/analysis/xml/ui/swtbot/tests/preferences/XMLAnalysesManagerPreferencePageTest.java b/tmf/org.eclipse.tracecompass.tmf.analysis.xml.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/analysis/xml/ui/swtbot/tests/preferences/XMLAnalysesManagerPreferencePageTest.java index 4a227110cd..15d482b311 100644 --- a/tmf/org.eclipse.tracecompass.tmf.analysis.xml.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/analysis/xml/ui/swtbot/tests/preferences/XMLAnalysesManagerPreferencePageTest.java +++ b/tmf/org.eclipse.tracecompass.tmf.analysis.xml.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/analysis/xml/ui/swtbot/tests/preferences/XMLAnalysesManagerPreferencePageTest.java @@ -407,7 +407,7 @@ public class XMLAnalysesManagerPreferencePageTest { SWTBotTreeItem treeNode = tree.getTreeItem("Tracing"); treeNode.select(); treeNode.expand(); - bot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable("XML Analyses", treeNode)); + bot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable("XML Analyses", treeNode)); treeNode = treeNode.getNode("XML Analyses"); treeNode.select(); return preferencesShell; diff --git a/tmf/org.eclipse.tracecompass.tmf.remote.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/remote/ui/swtbot/tests/fetch/FetchRemoteTracesTest.java b/tmf/org.eclipse.tracecompass.tmf.remote.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/remote/ui/swtbot/tests/fetch/FetchRemoteTracesTest.java index ae5a035a55..44283a264e 100644 --- a/tmf/org.eclipse.tracecompass.tmf.remote.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/remote/ui/swtbot/tests/fetch/FetchRemoteTracesTest.java +++ b/tmf/org.eclipse.tracecompass.tmf.remote.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/remote/ui/swtbot/tests/fetch/FetchRemoteTracesTest.java @@ -979,7 +979,7 @@ public class FetchRemoteTracesTest { SWTBotTreeItem currentNode = tree.getTreeItem(nodeNames[0]); for (int i = 1; i < nodeNames.length; i++) { String nodeName = nodeNames[i]; - bot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable(nodeName, currentNode)); + bot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable(nodeName, currentNode)); SWTBotTreeItem newNode = currentNode.getNode(nodeName); currentNode = newNode; } @@ -996,7 +996,7 @@ public class FetchRemoteTracesTest { SWTBotTreeItem treeNode = tree.getTreeItem("Tracing"); treeNode.select(); treeNode.expand(); - bot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable("Remote Profiles", treeNode)); + bot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable("Remote Profiles", treeNode)); treeNode = treeNode.getNode("Remote Profiles"); treeNode.select(); } diff --git a/tmf/org.eclipse.tracecompass.tmf.ui.swtbot.tests/shared/org/eclipse/tracecompass/tmf/ui/swtbot/tests/shared/ConditionHelpers.java b/tmf/org.eclipse.tracecompass.tmf.ui.swtbot.tests/shared/org/eclipse/tracecompass/tmf/ui/swtbot/tests/shared/ConditionHelpers.java index 4a481a00fe..394075c91e 100644 --- a/tmf/org.eclipse.tracecompass.tmf.ui.swtbot.tests/shared/org/eclipse/tracecompass/tmf/ui/swtbot/tests/shared/ConditionHelpers.java +++ b/tmf/org.eclipse.tracecompass.tmf.ui.swtbot.tests/shared/org/eclipse/tracecompass/tmf/ui/swtbot/tests/shared/ConditionHelpers.java @@ -33,6 +33,7 @@ import org.eclipse.swtbot.eclipse.finder.widgets.SWTBotEditor; import org.eclipse.swtbot.eclipse.finder.widgets.SWTBotView; import org.eclipse.swtbot.swt.finder.SWTBot; import org.eclipse.swtbot.swt.finder.finders.UIThreadRunnable; +import org.eclipse.swtbot.swt.finder.results.BoolResult; import org.eclipse.swtbot.swt.finder.results.Result; import org.eclipse.swtbot.swt.finder.utils.TableCollection; import org.eclipse.swtbot.swt.finder.waits.DefaultCondition; @@ -49,6 +50,9 @@ import org.eclipse.tracecompass.tmf.ui.editors.TmfEventsEditor; import org.eclipse.tracecompass.tmf.ui.viewers.xycharts.TmfXYChartViewer; import org.eclipse.tracecompass.tmf.ui.views.timegraph.AbstractTimeGraphView; import org.eclipse.ui.IEditorReference; +import org.eclipse.ui.IWorkbenchPage; +import org.eclipse.ui.IWorkbenchWindow; +import org.eclipse.ui.PlatformUI; import org.hamcrest.Matcher; import org.swtchart.Chart; @@ -86,9 +90,9 @@ public final class ConditionHelpers { * the name of the node * @param tree * the parent tree - * @return true or false, it should swallow all exceptions + * @return ICondition for verification */ - public static ICondition IsTreeNodeAvailable(final String name, final SWTBotTree tree) { + public static ICondition isTreeNodeAvailable(final String name, final SWTBotTree tree) { return new SWTBotTestCondition() { @Override public boolean test() throws Exception { @@ -120,7 +124,7 @@ public final class ConditionHelpers { * the name of the item * @param table * the parent table - * @return true or false, it should swallow all exceptions + * @return ICondition for verification */ public static ICondition isTableItemAvailable(final String name, final SWTBotTable table) { return new SWTBotTestCondition() { @@ -147,9 +151,9 @@ public final class ConditionHelpers { * the name of the node * @param treeItem * the treeItem - * @return true or false, it should swallow all exceptions + * @return ICondition for verification */ - public static ICondition IsTreeChildNodeAvailable(final String name, final SWTBotTreeItem treeItem) { + public static ICondition isTreeChildNodeAvailable(final String name, final SWTBotTreeItem treeItem) { return new SWTBotTestCondition() { @Override public boolean test() throws Exception { @@ -175,7 +179,7 @@ public final class ConditionHelpers { * length of the node after removal * @param treeItem * the treeItem - * @return true or false, it should swallow all exceptions + * @return ICondition for verification */ public static ICondition isTreeChildNodeRemoved(final int length, final SWTBotTreeItem treeItem) { return new SWTBotTestCondition() { @@ -226,7 +230,7 @@ public final class ConditionHelpers { * * @param wizard * the null - * @return false if either are null + * @return ICondition for verification */ public static ICondition isWizardReady(final Wizard wizard) { return new SWTBotTestCondition() { @@ -247,7 +251,7 @@ public final class ConditionHelpers { * wizard * @param page * the desired page - * @return true or false + * @return ICondition for verification */ public static ICondition isWizardOnPage(final Wizard wizard, final IWizardPage page) { return new SWTBotTestCondition() { @@ -267,34 +271,60 @@ public final class ConditionHelpers { } /** - * Wait for a view to close + * Condition to wait for a view to close * * @param view - * bot view for the view - * @return true if the view is closed, false if it's active. + * view bot for the view + * @return ICondition for verification */ - public static ICondition ViewIsClosed(final SWTBotView view) { + public static ICondition viewIsClosed(final SWTBotView view) { return new SWTBotTestCondition() { @Override public boolean test() throws Exception { - return (view != null) && (!view.isActive()); + return UIThreadRunnable.syncExec(new BoolResult() { + @Override + public Boolean run() { + if (!PlatformUI.isWorkbenchRunning()) { + return true; + } + IWorkbenchWindow workbenchWindow = PlatformUI.getWorkbench().getActiveWorkbenchWindow(); + if (workbenchWindow == null) { + return true; + } + IWorkbenchPage activePage = workbenchWindow.getActivePage(); + if (activePage == null) { + return true; + } + return !Arrays.asList(activePage.getViewReferences()).contains(view.getReference()); + } + }); + } + + @Override + public String getFailureMessage() { + return view.getTitle() + " view did not close"; } }; } /** - * Wait for a view to open + * Condition to wait for a view to become active * * @param view * bot view for the view - * @return true if the view is open, false otherwise + * @return ICondition for verification */ - public static ICondition viewIsOpened(final SWTBotView view) { + public static ICondition viewIsActive(final SWTBotView view) { return new SWTBotTestCondition() { @Override public boolean test() throws Exception { return (view != null) && (view.isActive()); } + + @Override + public String getFailureMessage() { + return view.getTitle() + " view did not become active"; + } }; } diff --git a/tmf/org.eclipse.tracecompass.tmf.ui.swtbot.tests/shared/org/eclipse/tracecompass/tmf/ui/swtbot/tests/shared/SWTBotUtils.java b/tmf/org.eclipse.tracecompass.tmf.ui.swtbot.tests/shared/org/eclipse/tracecompass/tmf/ui/swtbot/tests/shared/SWTBotUtils.java index 2ccdaf8c66..e315943257 100644 --- a/tmf/org.eclipse.tracecompass.tmf.ui.swtbot.tests/shared/org/eclipse/tracecompass/tmf/ui/swtbot/tests/shared/SWTBotUtils.java +++ b/tmf/org.eclipse.tracecompass.tmf.ui.swtbot.tests/shared/org/eclipse/tracecompass/tmf/ui/swtbot/tests/shared/SWTBotUtils.java @@ -366,7 +366,7 @@ public final class SWTBotUtils { for (SWTBotView view : openViews) { if (view.getTitle().equalsIgnoreCase(title)) { view.close(); - bot.waitUntil(ConditionHelpers.ViewIsClosed(view)); + bot.waitUntil(ConditionHelpers.viewIsClosed(view)); } } } @@ -383,7 +383,7 @@ public final class SWTBotUtils { public static void closeViewById(String viewId, SWTWorkbenchBot bot) { final SWTBotView view = bot.viewById(viewId); view.close(); - bot.waitUntil(ConditionHelpers.ViewIsClosed(view)); + bot.waitUntil(ConditionHelpers.viewIsClosed(view)); } /** @@ -612,7 +612,7 @@ public final class SWTBotUtils { SWTBot projectExplorerBot = projectExplorerView.bot(); final SWTBotTree tree = projectExplorerBot.tree(); - projectExplorerBot.waitUntil(ConditionHelpers.IsTreeNodeAvailable(projectName, tree)); + projectExplorerBot.waitUntil(ConditionHelpers.isTreeNodeAvailable(projectName, tree)); final SWTBotTreeItem treeItem = tree.getTreeItem(projectName); treeItem.expand(); @@ -939,7 +939,7 @@ public final class SWTBotUtils { return null; } - bot.waitUntil(ConditionHelpers.IsTreeNodeAvailable(nodeNames[0], tree)); + bot.waitUntil(ConditionHelpers.isTreeNodeAvailable(nodeNames[0], tree)); SWTBotTreeItem currentNode = tree.getTreeItem(nodeNames[0]); return getTreeItem(bot, currentNode, Arrays.copyOfRange(nodeNames, 1, nodeNames.length)); } @@ -968,13 +968,13 @@ public final class SWTBotUtils { String nodeName = nodeNames[i]; try { - bot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable(nodeName, currentNode)); + bot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable(nodeName, currentNode)); } catch (TimeoutException e) { // FIXME: Sometimes in a JFace TreeViewer, it expands to // nothing. Need to find out why. currentNode.collapse(); currentNode.expand(); - bot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable(nodeName, currentNode)); + bot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable(nodeName, currentNode)); } SWTBotTreeItem newNode = currentNode.getNode(nodeName); diff --git a/tmf/org.eclipse.tracecompass.tmf.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/ui/swtbot/tests/tracetype/preferences/TraceTypePreferencePageTest.java b/tmf/org.eclipse.tracecompass.tmf.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/ui/swtbot/tests/tracetype/preferences/TraceTypePreferencePageTest.java index 7aba1981f7..045e43b4ca 100644 --- a/tmf/org.eclipse.tracecompass.tmf.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/ui/swtbot/tests/tracetype/preferences/TraceTypePreferencePageTest.java +++ b/tmf/org.eclipse.tracecompass.tmf.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/ui/swtbot/tests/tracetype/preferences/TraceTypePreferencePageTest.java @@ -331,7 +331,7 @@ public class TraceTypePreferencePageTest { SWTBotTreeItem treeNode = tree.getTreeItem("Tracing"); treeNode.select(); treeNode.expand(); - bot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable("Trace Types", treeNode)); + bot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable("Trace Types", treeNode)); treeNode = treeNode.getNode("Trace Types"); treeNode.select(); return preferencesShell; diff --git a/tmf/org.eclipse.tracecompass.tmf.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/ui/swtbot/tests/viewers/events/FontEventEditorTest.java b/tmf/org.eclipse.tracecompass.tmf.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/ui/swtbot/tests/viewers/events/FontEventEditorTest.java index d0f047af84..4734756ce9 100644 --- a/tmf/org.eclipse.tracecompass.tmf.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/ui/swtbot/tests/viewers/events/FontEventEditorTest.java +++ b/tmf/org.eclipse.tracecompass.tmf.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/ui/swtbot/tests/viewers/events/FontEventEditorTest.java @@ -137,11 +137,11 @@ public class FontEventEditorTest { SWTBotTreeItem generalItem = bot.tree().getTreeItem("General"); generalItem.click(); generalItem.select(); - bot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable("Appearance", generalItem)); + bot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable("Appearance", generalItem)); SWTBotTreeItem appearanceNode = generalItem.getNode("Appearance"); appearanceNode.click(); appearanceNode.select(); - bot.waitUntil(ConditionHelpers.IsTreeChildNodeAvailable("Colors and Fonts", appearanceNode)); + bot.waitUntil(ConditionHelpers.isTreeChildNodeAvailable("Colors and Fonts", appearanceNode)); SWTBotTreeItem colorAndFontNode = appearanceNode.getNode("Colors and Fonts"); colorAndFontNode.click(); colorAndFontNode.select(); diff --git a/tmf/org.eclipse.tracecompass.tmf.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/ui/swtbot/tests/views/TimeGraphViewTest.java b/tmf/org.eclipse.tracecompass.tmf.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/ui/swtbot/tests/views/TimeGraphViewTest.java index 59a64b5958..940cb608c1 100644 --- a/tmf/org.eclipse.tracecompass.tmf.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/ui/swtbot/tests/views/TimeGraphViewTest.java +++ b/tmf/org.eclipse.tracecompass.tmf.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/ui/swtbot/tests/views/TimeGraphViewTest.java @@ -318,7 +318,7 @@ public class TimeGraphViewTest { TmfSignalManager.dispatchSignal(new TmfTraceClosedSignal(this, trace)); fBot.waitUntil(Conditions.shellCloses(legendShell)); fViewBot.close(); - fBot.waitUntil(ConditionHelpers.ViewIsClosed(fViewBot)); + fBot.waitUntil(ConditionHelpers.viewIsClosed(fViewBot)); fTrace.dispose(); } diff --git a/tmf/org.eclipse.tracecompass.tmf.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/ui/swtbot/tests/views/XYChartViewTest.java b/tmf/org.eclipse.tracecompass.tmf.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/ui/swtbot/tests/views/XYChartViewTest.java index 74bf07007f..f1cca8aed9 100644 --- a/tmf/org.eclipse.tracecompass.tmf.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/ui/swtbot/tests/views/XYChartViewTest.java +++ b/tmf/org.eclipse.tracecompass.tmf.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/ui/swtbot/tests/views/XYChartViewTest.java @@ -153,7 +153,7 @@ public class XYChartViewTest { assertNotNull(trace); UIThreadRunnable.syncExec(() -> TmfSignalManager.dispatchSignal(new TmfTraceClosedSignal(this, trace))); fViewBot.close(); - fBot.waitUntil(ConditionHelpers.ViewIsClosed(fViewBot)); + fBot.waitUntil(ConditionHelpers.viewIsClosed(fViewBot)); fTrace.dispose(); } |