[8u20] Request for approval/Review for CR 8041931: test/sun/net/www/http/HttpClient/B8025710.java fails in 8 Update

[Seán Coffey]

Looks good to me Chris.

Approved for jdk8u-dev.

regards,
Sean.

On 25/04/14 18:22, Chris Hegarty wrote:
> Hi 8u maintainers,
>
> Stupidly I mixed up testing results, and consequently pushed a new test to 8u-dev [1] that has a minor issue, that causes it to fail on all platforms, all of the time. Root cause, the keystore used by the test has moved location in JDK 9, and the test should be updated to refer to the keystore location in JDK 8. Clearly this is not an issue in JDK 9.
>
> Since the change is trivial, it should be possible to do the approval and review here?
>
> diff --git a/test/sun/net/www/http/HttpClient/B8025710.java b/test/sun/net/www/http/HttpClient/B8025710.java
> --- a/test/sun/net/www/http/HttpClient/B8025710.java
> +++ b/test/sun/net/www/http/HttpClient/B8025710.java
> @@ -43,7 +43,7 @@
>       private final static AtomicBoolean connectInServer = new AtomicBoolean();
>       private static final String keystorefile =
>               System.getProperty("test.src", "./")
> - + "/../../../../../javax/net/ssl/etc/keystore";
> + + "/../../../../../sun/security/ssl/etc/keystore";
>       private static final String passphrase = "passphrase";
>    
>       public static void main(String[] args) throws Exception {
>
> -Chris.
>
> [1] http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/47a5f3d6aeac