On 13/04/2012 17:59, Kurchi Hazra wrote:
Hi,
This test was failing on Windows since it was using the HttpServer
in test/sun/net/www/httptest. The HttpServer implementation
there is buggy and does not close the connection properly, resulting
in the test hanging. We therefore write our own server, which sends back
10 bytes less than what the client expects, and see if the client
raises an IOException.
Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7152856
Webrev: http://cr.openjdk.java.net/~khazra/7152856/webrev.00
Thanks,
Kurchi
Thanks for sorting out this test. A couple of comments:
- I don't think the @run is right as samevm or agentvm is specified to
jtreg rather than on specific tests (it is possible to add /othervm to
force a test to run in its own VM).
- "Server" might be better than XServer (as X server normally means a
X11 server).
- XServer.srv should be final.
- It looks like the server socket is closed when the test terminates.
Also to ensure that the accepted connection is closed I would suggest
that run be changed to try (Socket s = srv.accept()) { ... }.
Otherwise I think it's okay.
-Alan.
