Revision: 8476
Author: amitman...@google.com
Date: Wed Aug 4 05:46:33 2010
Log: Made the error messages thrown in case of GWTTestCase timeouts more
informative, as requested in
http://code.google.com/p/google-web-toolkit/issues/detail?id=4099
Patch by: amitmanjhi
Review by: mmendez
http://code.google.com/p/google-web-toolkit/source/detail?r=8476
Modified:
/trunk/user/src/com/google/gwt/junit/JUnitShell.java
=======================================
--- /trunk/user/src/com/google/gwt/junit/JUnitShell.java Fri Jul 30
07:51:15 2010
+++ /trunk/user/src/com/google/gwt/junit/JUnitShell.java Wed Aug 4
05:46:33 2010
@@ -1015,7 +1015,8 @@
+ testBatchingMethodTimeoutMillis
+ "ms.\n We have no results from:\n"
+ messageQueue.getWorkingClients(currentTestInfo)
- + "Actual time elapsed: " + elapsed + " seconds.\n");
+ + "Actual time elapsed: " + elapsed + " seconds.\n"
+ + "Try increasing this timeout using
the '-testMethodTimeout minutes' option\n");
}
} else if (testBeginTimeout < currentTimeMillis) {
double elapsed = (currentTimeMillis - testBeginTime) / 1000.0;
@@ -1023,7 +1024,9 @@
"The browser did not contact the server within "
+ baseTestBeginTimeoutMillis + "ms.\n"
+ messageQueue.getUnretrievedClients(currentTestInfo)
- + "\n Actual time elapsed: " + elapsed + " seconds.\n");
+ + "\n Actual time elapsed: " + elapsed + " seconds.\n"
+ + "Try increasing this timeout using the '-testBeginTimeout
minutes' option\n"
+ + "The default value of minutes is 1, i.e., the server waits
1 minute or 60 seconds.\n");
}
// Check that we haven't lost communication with a remote host.
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors