Revision: 7744
Author: [email protected]
Date: Thu Mar 18 08:02:42 2010
Log: Fix flaxy MessageTransport test.
Review at http://gwt-code-reviews.appspot.com/234801

Review by: [email protected]
http://code.google.com/p/google-web-toolkit/source/detail?r=7744

Modified:
/trunk/dev/core/test/com/google/gwt/dev/shell/remoteui/MessageTransportTest.java

=======================================
--- /trunk/dev/core/test/com/google/gwt/dev/shell/remoteui/MessageTransportTest.java Thu Nov 12 13:26:49 2009 +++ /trunk/dev/core/test/com/google/gwt/dev/shell/remoteui/MessageTransportTest.java Thu Mar 18 08:02:42 2010
@@ -123,9 +123,21 @@
requestMessageBuilder.setServiceType(Message.Request.ServiceType.DEV_MODE);
     Message.Request request = requestMessageBuilder.build();

-    // Close the server's input stream; that will close the client's output
+    // Close the server's socket; that will close the client's output
     // stream
-    network.getServerSocket().getInputStream().close();
+    network.getServerSocket().close();
+
+    int sleepCycles = 0;
+    while (!network.getServerSocket().isClosed() && sleepCycles < 8) {
+      // Wait until the stream is closed before attempting to execute the
+      // request.
+      Thread.sleep(250);
+      sleepCycles++;
+    }
+
+    assertTrue(
+ "Unable to shut down server's input stream; cannot proceed with the test.",
+        network.getServerSocket().isClosed());

     Future<Response> responseFuture = null;
     responseFuture = messageTransport.executeRequestAsync(request);

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

Reply via email to