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

Reply via email to