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.spit...@oracle.com To: alexander.kulyakh...@oracle.com, alan.bate...@oracle.com Cc: serviceability-dev@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.bate...@oracle.com > To: alexander.kulyakh...@oracle.com, serviceability-dev@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
