Mercurial Mercurial > jdk-sandbox / changeset
summary | shortlog | changelog | graph | tags | bookmarks | branches | files | changeset | raw | help
Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
http-client-branch: review comment: minor cleanup in security test http-client-branch
authorchegar
Fri, 06 Apr 2018 13:41:14 +0100
branchhttp-client-branch
changeset 56388 701dd37aa97a
parent 56387 c08eb1e2dc38
child 56389 0ba90c4f1e3f
http-client-branch: review comment: minor cleanup in security test
test/jdk/java/net/httpclient/SplitResponseSSL.java file | annotate | diff | comparison | revisions
test/jdk/java/net/httpclient/security/Security.java file | annotate | diff | comparison | revisions 
--- a/test/jdk/java/net/httpclient/SplitResponseSSL.java	Fri Apr 06 13:21:08 2018 +0100
+++ b/test/jdk/java/net/httpclient/SplitResponseSSL.java	Fri Apr 06 13:41:14 2018 +0100
@@ -27,7 +27,10 @@
  * @library /lib/testlibrary
  * @build jdk.testlibrary.SimpleSSLContext
  * @build MockServer SplitResponse
- * @run main/othervm -Djdk.internal.httpclient.debug=true -Djdk.httpclient.HttpClient.log=all SplitResponseSSL SSL
+ * @run main/othervm
+ *      -Djdk.internal.httpclient.debug=true
+ *      -Djdk.httpclient.HttpClient.log=all
+ *      SplitResponseSSL SSL
  */
 public class SplitResponseSSL {
     public static void main(String[] args) throws Exception {
--- a/test/jdk/java/net/httpclient/security/Security.java	Fri Apr 06 13:21:08 2018 +0100
+++ b/test/jdk/java/net/httpclient/security/Security.java	Fri Apr 06 13:41:14 2018 +0100
@@ -144,7 +144,7 @@
         movefile("ProxyServer$Connection.class");
         movefile("ProxyServer$1.class");
 
-        URL url = subdir.toURL();
+        URL url = subdir.toURI().toURL();
         System.out.println("URL for class loader = " + url);
         URLClassLoader urlc = new URLClassLoader(new URL[] {url});
         proxyClass = Class.forName("ProxyServer", true, urlc);
@@ -256,7 +256,7 @@
                 directProxyTest(proxyPort, true);
             }),
             // (12) policy has permission for both destination URL and proxy
-            test(false, () -> { //Policy 11
+            test(false, () -> { //Policy 12 ( 11 & 12 are the same )
                 directProxyTest(proxyPort, false);
             }),
             // (13) async version of test 0
jdk-sandbox