/hg/icedtea-web: AbsolutePathsAndQueryStrings reproducer adapted...

[jvanek at icedtea.classpath.org]

changeset a286e1424097 in /hg/icedtea-web
details: http://icedtea.classpath.org/hg/icedtea-web?cmd=changeset;node=a286e1424097
author: Jiri Vanek <jvanek at redhat.com>
date: Mon Nov 21 11:31:22 2016 +0100

	AbsolutePathsAndQueryStrings reproducer adapted to port in cache path

	* tests/reproducers/simple/AbsolutePathsAndQueryStrings/testcases/AbsolutePathsAndQueryStrings.java: (testCaching) exemplar urls now uses server's port instead of random and hardcoded 1234


diffstat:

 ChangeLog                                                                                         |   6 ++++++
 tests/reproducers/simple/AbsolutePathsAndQueryStrings/testcases/AbsolutePathsAndQueryStrings.java |  10 +++++-----
 2 files changed, 11 insertions(+), 5 deletions(-)

diffs (40 lines):

diff -r 6869500e766c -r a286e1424097 ChangeLog
--- a/ChangeLog	Fri Nov 18 12:53:20 2016 +0100
+++ b/ChangeLog	Mon Nov 21 11:31:22 2016 +0100
@@ -1,3 +1,9 @@
+2016-11-21  Jiri Vanek <jvanek at redhat.com>
+
+	AbsolutePathsAndQueryStrings reproducer adapted to port in cache path
+	* tests/reproducers/simple/AbsolutePathsAndQueryStrings/testcases/AbsolutePathsAndQueryStrings.java:
+	(testCaching) exemplar urls now uses server's port instead of random and hardcoded 1234
+
 2016-11-18  Jiri Vanek <jvanek at redhat.com>
 
 	* netx/net/sourceforge/jnlp/Launcher.java: (createApplet) call to setContextClassLoaderForAllThreads
diff -r 6869500e766c -r a286e1424097 tests/reproducers/simple/AbsolutePathsAndQueryStrings/testcases/AbsolutePathsAndQueryStrings.java
--- a/tests/reproducers/simple/AbsolutePathsAndQueryStrings/testcases/AbsolutePathsAndQueryStrings.java	Fri Nov 18 12:53:20 2016 +0100
+++ b/tests/reproducers/simple/AbsolutePathsAndQueryStrings/testcases/AbsolutePathsAndQueryStrings.java	Mon Nov 21 11:31:22 2016 +0100
@@ -82,10 +82,10 @@
         testAbsolutePathAndQueryStringWebstart();
         /* Test that caching ignores path parameters and double-slash issue from absolute codebase paths
          */
-        URL plainLocation = new URL("http://localhost:1234/StripHttpPathParams.jar");
-        URL paramLocation = new URL("http://localhost:1234/StripHttpPathParams.jar?i=abcd");
-        URL absoluteLocation = new URL("http://localhost:1234//StripHttpPathParams.jar");
-        URL absoluteParamLocation = new URL("http://localhost:1234//StripHttpPathParams.jar?i=abcd");
+        URL plainLocation = new URL("http://localhost:" + server.getPort() + "/StripHttpPathParams.jar");
+        URL paramLocation = new URL("http://localhost:" + server.getPort() + "/StripHttpPathParams.jar?i=abcd");
+        URL absoluteLocation = new URL("http://localhost:" + server.getPort() + "//StripHttpPathParams.jar");
+        URL absoluteParamLocation = new URL("http://localhost:" + server.getPort() + "//StripHttpPathParams.jar?i=abcd");
 
         DeploymentConfiguration config = JNLPRuntime.getConfiguration();
         config.load();
@@ -118,7 +118,7 @@
                 }
             }
         }
-        Assert.assertTrue(hasCachedCopy>=4);
+        Assert.assertTrue("At least 4 items should be in cach. Are not. Are: " + hasCachedCopy, hasCachedCopy >= 4);
     }
 
 }