Reviewers: fabbott,

Description:
DisclosurePanelTest#testCoreFunctionality has a 50ms window for a Timer
to fire before the test completes, which is sometimes too small.

Fix:
===
Increasing timeout to 5000ms.

Testing:
========
Verified the test passes, although it only failed rarely before.

Please review this at http://gwt-code-reviews.appspot.com/103802

Affected files:
   user/test/com/google/gwt/user/client/ui/DisclosurePanelTest.java


Index: user/test/com/google/gwt/user/client/ui/DisclosurePanelTest.java
===================================================================
--- user/test/com/google/gwt/user/client/ui/DisclosurePanelTest.java     
(revision 6956)
+++ user/test/com/google/gwt/user/client/ui/DisclosurePanelTest.java     
(working copy)
@@ -48,7 +48,7 @@

      panel.setOpen(true);

-    delayTestFinish(500);
+    delayTestFinish(5000);
      // Allow the animation time to finish
      Timer t = new Timer() {
        @Override


-- 
http://groups.google.com/group/Google-Web-Toolkit-Contributors

Reply via email to