RFR: JDK-8151196 Several tests fail due to test library not found
Staffan Larsen
staffan.larsen at oracle.com
Wed Mar 9 11:31:56 UTC 2016
Christian, Bengt: Thanks for the reviews. Pushing now.
> On 9 mars 2016, at 11:29, Christian Tornqvist <christian.tornqvist at oracle.com> wrote:
>
> Hi Staffan,
>
> Looks good, thanks for fixing this.
>
> Thanks,
> Christian
>
> -----Original Message-----
> From: serviceability-dev
> [mailto:serviceability-dev-bounces at openjdk.java.net] On Behalf Of Staffan
> Larsen
> Sent: Wednesday, March 9, 2016 5:03 AM
> To: Bengt Rutisson <bengt.rutisson at oracle.com>
> Cc: serviceability-dev at openjdk.java.net; hotspot-gc-dev at openjdk.java.net
> Subject: Re: RFR: JDK-8151196 Several tests fail due to test library not
> found
>
> Good catch. Incremental change:
>
> diff --git a/test/gc/g1/plab/TestPLABResize.java
> b/test/gc/g1/plab/TestPLABResize.java
> --- a/test/gc/g1/plab/TestPLABResize.java
> +++ b/test/gc/g1/plab/TestPLABResize.java
> @@ -27,7 +27,7 @@
> * @summary Test for PLAB resizing
> * @requires vm.gc=="G1" | vm.gc=="null"
> * @requires vm.opt.FlightRecorder != true
> - * @library /testlibrary /../../test/lib /
> + * @library /testlibrary /test/lib /
> * @modules java.management
> * @build ClassFileInstaller
> * sun.hotspot.WhiteBox
>
>> On 9 mars 2016, at 10:54, Bengt Rutisson <bengt.rutisson at oracle.com>
> wrote:
>>
>>
>> Hi Staffan,
>>
>> Changes look good.
>>
>> However it looks like this test has the same issue, right?
>>
> http://hg.openjdk.java.net/jdk9/hs-rt/hotspot/file/59d3a69564dc/test/gc/g1
> /plab/TestPLABResize.java
>>
>> It is currently ignored, but we should probably fix it too...
>>
>> Thanks,
>> Bengt
>>
>> On 2016-03-09 10:47, Staffan Larsen wrote:
>>> Please review this small fix to some tests with the wrong paths in
> them. Tested locally with jtreg 4.1 b12 and b13.
>>>
>>> Thanks,
>>> /Staffan
>>>
>>>
>>> $ hg diff
>>> diff --git a/test/gc/g1/plab/TestPLABPromotion.java
> b/test/gc/g1/plab/TestPLABPromotion.java
>>> --- a/test/gc/g1/plab/TestPLABPromotion.java
>>> +++ b/test/gc/g1/plab/TestPLABPromotion.java
>>> @@ -27,7 +27,7 @@
>>> * @summary Test PLAB promotion
>>> * @requires vm.gc=="G1" | vm.gc=="null"
>>> * @requires vm.opt.FlightRecorder != true
>>> - * @library /testlibrary /../../test/lib /
>>> + * @library /testlibrary /test/lib /
>>> * @modules java.management
>>> * @build ClassFileInstaller
>>> * sun.hotspot.WhiteBox
>>> diff --git a/test/serviceability/dcmd/gc/HeapDumpAllTest.java
> b/test/serviceability/dcmd/gc/HeapDumpAllTest.java
>>> --- a/test/serviceability/dcmd/gc/HeapDumpAllTest.java
>>> +++ b/test/serviceability/dcmd/gc/HeapDumpAllTest.java
>>> @@ -35,7 +35,7 @@
>>> * @build jdk.test.lib.hprof.*
>>> * @build jdk.test.lib.hprof.model.*
>>> * @build jdk.test.lib.hprof.parser.*
>>> - * @build jdk.test.lib.hprof.utils.*
>>> + * @build jdk.test.lib.hprof.util.*
>>> * @build HeapDumpTest
>>> * @run testng HeapDumpAllTest
>>> */
>>> diff --git a/test/serviceability/dcmd/gc/HeapDumpTest.java
> b/test/serviceability/dcmd/gc/HeapDumpTest.java
>>> --- a/test/serviceability/dcmd/gc/HeapDumpTest.java
>>> +++ b/test/serviceability/dcmd/gc/HeapDumpTest.java
>>> @@ -51,7 +51,7 @@
>>> * @build jdk.test.lib.hprof.*
>>> * @build jdk.test.lib.hprof.model.*
>>> * @build jdk.test.lib.hprof.parser.*
>>> - * @build jdk.test.lib.hprof.utils.*
>>> + * @build jdk.test.lib.hprof.util.*
>>> * @run testng HeapDumpTest
>>> */
>>> public class HeapDumpTest {
>>
>
>
More information about the serviceability-dev
mailing list