Reviewers: fabbott,

Description:
Description:
=============
RPC tests often timeout because of slow server responses while the
server is busy compiling other GWT modules.

Fix:
===
I added an RpcTestBase class that has a static RPC_TIMEOUT value.  We
can increase or decrease it if needed.

I used 15 seconds because some tests transfer large amounts of data and
were using 12 second delays.  There are a few tests that transfer BMPs,
and they have much higher timeouts.

Also, I decided to create the RpcTestBase class instead of just putting
RPC_TIMEOUT in RpcSuite because RpcSuite isn't on the source path for a
GWT compile.  RpcTestBase#delayTestFinishForRpc() is named that way so
it shows up in the Eclipse auto complete when users type
delayTestFinish.

I also moved the delayTestFinishForRpc() directly above the RPC call.
Currently, some tests create the service between delayTestFinish and the
RPC call, which introduces a GWT.create() into the mix.


Testing:
=========
All tests passed.

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

Affected files:
   user/test/com/google/gwt/user/client/rpc/CollectionsTest.java
   user/test/com/google/gwt/user/client/rpc/CustomFieldSerializerTest.java
   user/test/com/google/gwt/user/client/rpc/EnumsTest.java
   user/test/com/google/gwt/user/client/rpc/InheritanceTest.java
   user/test/com/google/gwt/user/client/rpc/ObjectGraphTest.java
   user/test/com/google/gwt/user/client/rpc/RemoteServiceServletTest.java
   user/test/com/google/gwt/user/client/rpc/RpcTestBase.java
    
user/test/com/google/gwt/user/client/rpc/RunTimeSerializationErrorsTest.java
   user/test/com/google/gwt/user/client/rpc/ValueTypesTest.java



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

Reply via email to