http-client-branch: more no body tests http-client-branch
authorchegar
Sat, 25 Nov 2017 21:37:14 +0000
branchhttp-client-branch
changeset 55876 42cf279e4a89
parent 55875 39e9e9883c4e
child 55877 8448ee02f6aa
http-client-branch: more no body tests
test/jdk/java/net/httpclient/NoBody.java
test/jdk/java/net/httpclient/http2/NoBody.java
--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
+++ b/test/jdk/java/net/httpclient/NoBody.java	Sat Nov 25 21:37:14 2017 +0000
@@ -0,0 +1,349 @@
+/*
+ * Copyright (c) 2015, 2017, Oracle and/or its affiliates. All rights reserved.
+ * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
+ *
+ * This code is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License version 2 only, as
+ * published by the Free Software Foundation.
+ *
+ * This code is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+ * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
+ * version 2 for more details (a copy is included in the LICENSE file that
+ * accompanied this code).
+ *
+ * You should have received a copy of the GNU General Public License version
+ * 2 along with this work; if not, write to the Free Software Foundation,
+ * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
+ *
+ * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
+ * or visit www.oracle.com if you need additional information or have any
+ * questions.
+ */
+
+/*
+ * @test
+ * @bug 8161157
+ * @summary Test response body handlers/subscribers when there is no body
+ * @library /lib/testlibrary http2/server
+ * @build jdk.testlibrary.SimpleSSLContext
+ * @modules java.base/sun.net.www.http
+ *          jdk.incubator.httpclient/jdk.incubator.http.internal.common
+ *          jdk.incubator.httpclient/jdk.incubator.http.internal.frame
+ *          jdk.incubator.httpclient/jdk.incubator.http.internal.hpack
+ * @run testng/othervm NoBody
+ */
+
+import java.io.IOException;
+import java.io.InputStream;
+import java.net.InetSocketAddress;
+import java.net.URI;
+import java.nio.file.Files;
+import java.nio.file.Path;
+import java.nio.file.Paths;
+import java.util.Optional;
+import java.util.concurrent.Executor;
+import java.util.concurrent.Executors;
+import java.util.function.Consumer;
+import com.sun.net.httpserver.HttpExchange;
+import com.sun.net.httpserver.HttpHandler;
+import com.sun.net.httpserver.HttpServer;
+import com.sun.net.httpserver.HttpsConfigurator;
+import com.sun.net.httpserver.HttpsServer;
+import jdk.incubator.http.HttpClient;
+import jdk.incubator.http.HttpRequest;
+import jdk.incubator.http.HttpResponse;
+import jdk.incubator.http.HttpResponse.BodyHandler;
+import javax.net.ssl.SSLContext;
+import jdk.testlibrary.SimpleSSLContext;
+import org.testng.annotations.AfterTest;
+import org.testng.annotations.BeforeTest;
+import org.testng.annotations.DataProvider;
+import org.testng.annotations.Test;
+import static java.lang.System.out;
+import static java.nio.charset.StandardCharsets.UTF_8;
+import static jdk.incubator.http.HttpRequest.BodyPublisher.fromString;
+import static jdk.incubator.http.HttpResponse.BodyHandler.asByteArray;
+import static jdk.incubator.http.HttpResponse.BodyHandler.asByteArrayConsumer;
+import static jdk.incubator.http.HttpResponse.BodyHandler.asFile;
+import static jdk.incubator.http.HttpResponse.BodyHandler.asInputStream;
+import static jdk.incubator.http.HttpResponse.BodyHandler.asString;
+import static jdk.incubator.http.HttpResponse.BodyHandler.buffering;
+import static jdk.incubator.http.HttpResponse.BodyHandler.discard;
+import static org.testng.Assert.assertEquals;
+import static org.testng.Assert.assertTrue;
+import static org.testng.Assert.fail;
+
+public class NoBody {
+
+    SSLContext sslContext;
+    HttpServer httpTestServer;         // HTTP/1.1    [ 4 servers ]
+    HttpsServer httpsTestServer;       // HTTPS/1.1
+    Http2TestServer http2TestServer;   // HTTP/2 ( h2c )
+    Http2TestServer https2TestServer;  // HTTP/2 ( h2  )
+    String httpURI_fixed;
+    String httpURI_chunk;
+    String httpsURI_fixed;
+    String httpsURI_chunk;
+    String http2URI_fixed;
+    String http2URI_chunk;
+    String https2URI_fixed;
+    String https2URI_chunk;
+
+    static final String SIMPLE_STRING = "Hello world. Goodbye world";
+    static final int ITERATION_COUNT = 10;
+    // a shared executor helps reduce the amount of threads created by the test
+    static final Executor executor = Executors.newCachedThreadPool();
+
+    @DataProvider(name = "variants")
+    public Object[][] variants() {
+        return new Object[][]{
+                { httpURI_fixed,    false },
+                { httpURI_chunk,    false },
+                { httpsURI_fixed,   false },
+                { httpsURI_chunk,   false },
+                { http2URI_fixed,   false },
+                { http2URI_chunk,   false },
+                { https2URI_fixed,  false,},
+                { https2URI_chunk,  false },
+
+                { httpURI_fixed,    true },
+                { httpURI_chunk,    true },
+                { httpsURI_fixed,   true },
+                { httpsURI_chunk,   true },
+                { http2URI_fixed,   true },
+                { http2URI_chunk,   true },
+                { https2URI_fixed,  true,},
+                { https2URI_chunk,  true },
+        };
+    }
+
+    HttpClient newHttpClient() {
+        return HttpClient.newBuilder()
+                         .executor(executor)
+                         .sslContext(sslContext)
+                         .build();
+    }
+
+    @Test(dataProvider = "variants")
+    public void testAsString(String uri, boolean sameClient) throws Exception {
+        HttpClient client = null;
+        for (int i=0; i< ITERATION_COUNT; i++) {
+            if (!sameClient || client == null)
+                client = newHttpClient();
+
+            HttpRequest req = HttpRequest.newBuilder(URI.create(uri))
+                                         .PUT(fromString(SIMPLE_STRING))
+                                         .build();
+            BodyHandler<String> handler = i % 2 == 0 ? asString() : asString(UTF_8);
+            HttpResponse<String> response = client.send(req, handler);
+            String body = response.body();
+            assertEquals(body, "");
+        }
+    }
+
+    @Test(dataProvider = "variants")
+    public void testAsFile(String uri, boolean sameClient) throws Exception {
+        HttpClient client = null;
+        for (int i=0; i< ITERATION_COUNT; i++) {
+            if (!sameClient || client == null)
+                client = newHttpClient();
+
+            HttpRequest req = HttpRequest.newBuilder(URI.create(uri))
+                                         .PUT(fromString(SIMPLE_STRING))
+                                         .build();
+            Path p = Paths.get("NoBody_testAsFile.txt");
+            HttpResponse<Path> response = client.send(req, asFile(p));
+            Path bodyPath = response.body();
+            assertTrue(Files.exists(bodyPath));
+            assertEquals(Files.size(bodyPath), 0);
+        }
+    }
+
+    @Test(dataProvider = "variants")
+    public void testAsByteArray(String uri, boolean sameClient) throws Exception {
+        HttpClient client = null;
+        for (int i=0; i< ITERATION_COUNT; i++) {
+            if (!sameClient || client == null)
+                client = newHttpClient();
+
+            HttpRequest req = HttpRequest.newBuilder(URI.create(uri))
+                                         .PUT(fromString(SIMPLE_STRING))
+                                         .build();
+            HttpResponse<byte[]> response = client.send(req, asByteArray());
+            byte[] body = response.body();
+            assertEquals(body.length, 0);
+        }
+    }
+
+    volatile boolean consumerHasBeenCalled;
+    @Test(dataProvider = "variants")
+    public void testAsByteArrayConsumer(String uri, boolean sameClient) throws Exception {
+        HttpClient client = null;
+        for (int i=0; i< ITERATION_COUNT; i++) {
+            if (!sameClient || client == null)
+                client = newHttpClient();
+
+            HttpRequest req = HttpRequest.newBuilder(URI.create(uri))
+                                         .PUT(fromString(SIMPLE_STRING))
+                                         .build();
+            Consumer<Optional<byte[]>>  consumer = oba -> {
+                consumerHasBeenCalled = true;
+                oba.ifPresent(ba -> fail("Unexpected non-empty optional:" + ba));
+            };
+            consumerHasBeenCalled = false;
+            client.send(req, asByteArrayConsumer(consumer));
+            assertTrue(consumerHasBeenCalled);
+        }
+    }
+
+    @Test(dataProvider = "variants")
+    public void testAsInputStream(String uri, boolean sameClient) throws Exception {
+        HttpClient client = null;
+        for (int i=0; i< ITERATION_COUNT; i++) {
+            if (!sameClient || client == null)
+                client = newHttpClient();
+
+            HttpRequest req = HttpRequest.newBuilder(URI.create(uri))
+                                         .PUT(fromString(SIMPLE_STRING))
+                                         .build();
+            HttpResponse<InputStream> response = client.send(req, asInputStream());
+            byte[] body = response.body().readAllBytes();
+            assertEquals(body.length, 0);
+        }
+    }
+
+    @Test(dataProvider = "variants")
+    public void testBuffering(String uri, boolean sameClient) throws Exception {
+        HttpClient client = null;
+        for (int i=0; i< ITERATION_COUNT; i++) {
+            if (!sameClient || client == null)
+                client = newHttpClient();
+
+            HttpRequest req = HttpRequest.newBuilder(URI.create(uri))
+                                         .PUT(fromString(SIMPLE_STRING))
+                                         .build();
+            HttpResponse<byte[]> response = client.send(req, buffering(asByteArray(), 1024));
+            byte[] body = response.body();
+            assertEquals(body.length, 0);
+        }
+    }
+
+    @Test(dataProvider = "variants")
+    public void testDiscard(String uri, boolean sameClient) throws Exception {
+        HttpClient client = null;
+        for (int i=0; i< ITERATION_COUNT; i++) {
+            if (!sameClient || client == null)
+                client = newHttpClient();
+
+            HttpRequest req = HttpRequest.newBuilder(URI.create(uri))
+                                         .PUT(fromString(SIMPLE_STRING))
+                                         .build();
+            Object obj = new Object();
+            HttpResponse<Object> response = client.send(req, discard(obj));
+            assertEquals(response.body(), obj);
+        }
+    }
+
+
+    @BeforeTest
+    public void setup() throws Exception {
+        sslContext = new SimpleSSLContext().get();
+        if (sslContext == null)
+            throw new AssertionError("Unexpected null sslContext");
+
+        // HTTP/1.1
+        HttpHandler h1_fixedLengthNoBodyHandler = new HTTP1_FixedLengthNoBodyHandler();
+        HttpHandler h1_chunkNoBodyHandler = new HTTP1_ChunkedNoBodyHandler();
+        InetSocketAddress sa = new InetSocketAddress("localhost", 0);
+        httpTestServer = HttpServer.create(sa, 0);
+        httpTestServer.createContext("/http1/noBodyFixed", h1_fixedLengthNoBodyHandler);
+        httpTestServer.createContext("/http1/noBodyChunk", h1_chunkNoBodyHandler);
+        httpURI_fixed = "http://127.0.0.1:" + httpTestServer.getAddress().getPort() + "/http1/noBodyFixed";
+        httpURI_chunk = "http://127.0.0.1:" + httpTestServer.getAddress().getPort() + "/http1/noBodyChunk";
+
+        httpsTestServer = HttpsServer.create(sa, 0);
+        httpsTestServer.setHttpsConfigurator(new HttpsConfigurator(sslContext));
+        httpsTestServer.createContext("/https1/noBodyFixed", h1_fixedLengthNoBodyHandler);
+        httpsTestServer.createContext("/https1/noBodyChunk", h1_chunkNoBodyHandler);
+        httpsURI_fixed = "https://127.0.0.1:" + httpsTestServer.getAddress().getPort() + "/https1/noBodyFixed";
+        httpsURI_chunk = "https://127.0.0.1:" + httpsTestServer.getAddress().getPort() + "/https1/noBodyChunk";
+
+        // HTTP/2
+        Http2Handler h2_fixedLengthNoBodyHandler = new HTTP2_FixedLengthNoBodyHandler();
+        Http2Handler h2_chunkedNoBodyHandler = new HTTP2_ChunkedNoBodyHandler();
+
+        http2TestServer = new Http2TestServer("127.0.0.1", false, 0);
+        http2TestServer.addHandler(h2_fixedLengthNoBodyHandler, "/http2/noBodyFixed");
+        http2TestServer.addHandler(h2_chunkedNoBodyHandler, "/http2/noBodyChunk");
+        int port = http2TestServer.getAddress().getPort();
+        http2URI_fixed = "http://127.0.0.1:" + port + "/http2/noBodyFixed";
+        http2URI_chunk = "http://127.0.0.1:" + port + "/http2/noBodyChunk";
+
+        https2TestServer = new Http2TestServer("127.0.0.1", true, 0);
+        https2TestServer.addHandler(h2_fixedLengthNoBodyHandler, "/https2/noBodyFixed");
+        https2TestServer.addHandler(h2_chunkedNoBodyHandler, "/https2/noBodyChunk");
+        port = https2TestServer.getAddress().getPort();
+        https2URI_fixed = "https://127.0.0.1:" + port + "/https2/noBodyFixed";
+        https2URI_chunk = "https://127.0.0.1:" + port + "/https2/noBodyChunk";
+
+        httpTestServer.start();
+        httpsTestServer.start();
+        http2TestServer.start();
+        https2TestServer.start();
+    }
+
+    @AfterTest
+    public void teardown() throws Exception {
+        httpTestServer.stop(0);
+        httpsTestServer.stop(0);
+        http2TestServer.stop();
+        https2TestServer.stop();
+    }
+
+    static class HTTP1_FixedLengthNoBodyHandler implements HttpHandler {
+        @Override
+        public void handle(HttpExchange t) throws IOException {
+            //out.println("NoBodyHandler received request to " + t.getRequestURI());
+            try (InputStream is = t.getRequestBody()) {
+                is.readAllBytes();
+            }
+            t.sendResponseHeaders(200, -1); // no body
+        }
+    }
+
+    static class HTTP1_ChunkedNoBodyHandler implements HttpHandler {
+        @Override
+        public void handle(HttpExchange t) throws IOException {
+            //out.println("NoBodyHandler received request to " + t.getRequestURI());
+            try (InputStream is = t.getRequestBody()) {
+                is.readAllBytes();
+            }
+            t.sendResponseHeaders(200, 0); // chunked
+            t.getResponseBody().close();  // write nothing
+        }
+    }
+
+    static class HTTP2_FixedLengthNoBodyHandler implements Http2Handler {
+        @Override
+        public void handle(Http2TestExchange t) throws IOException {
+            //out.println("NoBodyHandler received request to " + t.getRequestURI());
+            try (InputStream is = t.getRequestBody()) {
+                is.readAllBytes();
+            }
+            t.sendResponseHeaders(200, 0);
+        }
+    }
+
+    static class HTTP2_ChunkedNoBodyHandler implements Http2Handler {
+        @Override
+        public void handle(Http2TestExchange t) throws IOException {
+            //out.println("NoBodyHandler received request to " + t.getRequestURI());
+            try (InputStream is = t.getRequestBody()) {
+                is.readAllBytes();
+            }
+            t.sendResponseHeaders(200, -1);
+            t.getResponseBody().close();  // write nothing
+        }
+    }
+}
--- a/test/jdk/java/net/httpclient/http2/NoBody.java	Sat Nov 25 21:36:35 2017 +0000
+++ /dev/null	Thu Jan 01 00:00:00 1970 +0000
@@ -1,92 +0,0 @@
-/*
- * Copyright (c) 2015, 2017, Oracle and/or its affiliates. All rights reserved.
- * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
- *
- * This code is free software; you can redistribute it and/or modify it
- * under the terms of the GNU General Public License version 2 only, as
- * published by the Free Software Foundation.
- *
- * This code is distributed in the hope that it will be useful, but WITHOUT
- * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
- * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
- * version 2 for more details (a copy is included in the LICENSE file that
- * accompanied this code).
- *
- * You should have received a copy of the GNU General Public License version
- * 2 along with this work; if not, write to the Free Software Foundation,
- * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
- *
- * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
- * or visit www.oracle.com if you need additional information or have any
- * questions.
- */
-
-/*
- * @test
- * @bug 8161157
- * @library /lib/testlibrary server
- * @build jdk.testlibrary.SimpleSSLContext
- * @modules java.base/sun.net.www.http
- *          jdk.incubator.httpclient/jdk.incubator.http.internal.common
- *          jdk.incubator.httpclient/jdk.incubator.http.internal.frame
- *          jdk.incubator.httpclient/jdk.incubator.http.internal.hpack
- * @run testng/othervm -Djdk.httpclient.HttpClient.log=ssl,frames,errors NoBody
- */
-
-import java.net.URI;
-import jdk.incubator.http.HttpClient;
-import jdk.incubator.http.HttpRequest;
-import jdk.incubator.http.HttpResponse;
-import javax.net.ssl.SSLContext;
-import java.util.concurrent.Executors;
-import java.util.concurrent.ExecutorService;
-import jdk.testlibrary.SimpleSSLContext;
-import static jdk.incubator.http.HttpClient.Version.HTTP_2;
-import static jdk.incubator.http.HttpRequest.BodyPublisher.fromString;
-import static jdk.incubator.http.HttpResponse.BodyHandler.asString;
-
-import org.testng.annotations.Test;
-
-public class NoBody {
-
-    static final String SIMPLE_STRING = "Hello world. Goodbye world";
-
-    @Test(timeOut=60000)
-    public void test() throws Exception {
-        SSLContext sslContext = (new SimpleSSLContext()).get();
-        ExecutorService exec = Executors.newCachedThreadPool();
-        HttpClient client = HttpClient.newBuilder()
-                                      .executor(exec)
-                                      .sslContext(sslContext)
-                                      .version(HTTP_2)
-                                      .build();
-
-        Http2TestServer httpsServer = null;
-        try {
-            httpsServer = new Http2TestServer(true,
-                                              0,
-                                              exec,
-                                              sslContext);
-            httpsServer.addHandler(new NoBodyHandler(), "/");
-
-            int httpsPort = httpsServer.getAddress().getPort();
-            String httpsURIString = "https://127.0.0.1:" + httpsPort + "/bar/";
-
-            httpsServer.start();
-            URI uri = URI.create(httpsURIString);
-            System.err.println("Request to " + uri);
-
-            HttpRequest req = HttpRequest.newBuilder(uri)
-                                         .PUT(fromString(SIMPLE_STRING))
-                                         .build();
-            HttpResponse<String> response = client.send(req, asString());
-            String body = response.body();
-            if (!body.equals(""))
-                throw new RuntimeException("expected empty body");
-            System.err.println("DONE");
-        } finally {
-            if (httpsServer != null )  { httpsServer.stop(); }
-            exec.shutdownNow();
-        }
-    }
-}