RFR:8153584:New jtreg test to verify PathSearchingVirutalMachine.bootClassPath() behaviour
Alexander Kulyakhtin
alexander.kulyakhtin at oracle.com
Wed Apr 6 10:13:27 UTC 2016
I've moved the test to the jdk repo.
Webrev: http://cr.openjdk.java.net/~akulyakh/8153584_02/
Thank you very much for your help.
Best regards,
Alexander
----- Original Message -----
From: serguei.spitsyn at oracle.com
To: alexander.kulyakhtin at oracle.com, Alan.Bateman at oracle.com
Cc: serviceability-dev at openjdk.java.net
Sent: Wednesday, April 6, 2016 1:00:47 PM GMT +03:00 Iraq
Subject: Re: RFR:8153584:New jtreg test to verify PathSearchingVirutalMachine.bootClassPath() behaviour
As Dmitry noted:
jdk/test/com/sun/jdi
Sorry, we did not point to it in the internal review.
Thanks,
Serguei
On 4/6/16 02:56, Alexander Kulyakhtin wrote:
> Alan,
>
> What would be, the most suitable location for the test? I'm going to make sure this new test does not duplicate any existing one and, if not, will change the patch accordingly then.
>
> Best regards,
> Alexander
>
>
> ----- Original Message -----
> From: Alan.Bateman at oracle.com
> To: alexander.kulyakhtin at oracle.com, serviceability-dev at openjdk.java.net
> Sent: Wednesday, April 6, 2016 12:51:44 PM GMT +03:00 Iraq
> Subject: Re: RFR:8153584:New jtreg test to verify PathSearchingVirutalMachine.bootClassPath() behaviour
>
> On 06/04/2016 10:46, Alexander Kulyakhtin wrote:
>> Hi,
>>
>> Could you, please, review, the following test-only changes (adding a new test).
>>
>> CR: https://bugs.openjdk.java.net/browse/JDK-8153584 "New jtreg test to verify PathSearchingVirutalMachine.bootClassPath() behaviour"
>> Webrev: http://cr.openjdk.java.net/~akulyakh/8153584/
>>
>> The new test verifies that in the JDK9 PathSearchingVirutalMachine.bootClassPath() always returns an empty string list, regardless of the, now obsolete, sun.boot.class.path property.
>> In the JDK 8 bootClassPath() used to be verified by a closed test (therefore the CR is closed). We are changing the test and moving it to the open as it verifies the open, public API.
>>
> The tests for JDI are in the jdk repo, would it be saner to put
> additional tests there so that they aren't split between two repos?
>
> -Alan
More information about the serviceability-dev
mailing list