Add some debug output to TestRefreshTextTrace
authorMarc-Andre Laperle <marc-andre.laperle@ericsson.com>
Sat, 9 Apr 2016 02:58:08 +0000 (22:58 -0400)
committerMarc-Andre Laperle <marc-andre.laperle@ericsson.com>
Mon, 11 Apr 2016 18:27:21 +0000 (14:27 -0400)
This test has been failing intermittently but my suspicion is that
it fails when the server is under heavy load and the trace fails to
index in 20 seconds. To verify this, this change adds some output
as the wait condition is evaluated. This test didn't already log the
SWTBot output so this means that even with this change, it's not too
verbose.

Change-Id: Ic6da7b19c45faa5345b2118c902101db6e18276b
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/70329
Reviewed-by: Hudson CI
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
tmf/org.eclipse.tracecompass.tmf.ui.swtbot.tests/src/org/eclipse/tracecompass/tmf/ui/swtbot/tests/viewers/events/TestRefreshTextTrace.java

index 522072fda3024bd32c897672b85868431e4b5b2a..cfea536781389d84530d502ebdd9e80fa8fca179 100644 (file)
@@ -135,12 +135,16 @@ public class TestRefreshTextTrace {
 
         @Override
         public boolean test() throws Exception {
-            return fTrace.getNbEvents() == fNbEvents;
+            if (fTrace.getNbEvents() != fNbEvents) {
+                System.out.println("Waiting for expected " + fNbEvents + " events. Current: " + fTrace.getNbEvents());
+                return false;
+            }
+            return true;
         }
 
         @Override
         public String getFailureMessage() {
-            return fTrace.getName() + " did not contain the expected number of " + fNbEvents + " events";
+            return fTrace.getName() + " did not contain the expected number of " + fNbEvents + " events. Current: " + fTrace.getNbEvents();
         }
     }
 
This page took 0.027389 seconds and 5 git commands to generate.