Reviewers: kjin,

Description:
Fix flaxy MessageTransport test.
Review by: k...@google.com

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

Affected files:
M dev/core/test/com/google/gwt/dev/shell/remoteui/MessageTransportTest.java


Index: dev/core/test/com/google/gwt/dev/shell/remoteui/MessageTransportTest.java
===================================================================
--- dev/core/test/com/google/gwt/dev/shell/remoteui/MessageTransportTest.java (revision 7743) +++ dev/core/test/com/google/gwt/dev/shell/remoteui/MessageTransportTest.java (working copy)
@@ -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