RFR(XXS): 8156777: [TESTBUG] test/testlibrary_tests/SimpleClassFileLoadHookTest.java requires non minimal VM

[Leonid Mesnik]

Chris, Misha, George

Thank you for review.

Leonid

On 12.05.2016 14:21, Christian Tornqvist wrote:
> Hi Leonid,
>
> Change looks good.
>
> Thanks,
> Christian
>
> On May 11, 2016, at 12:36 PM, Leonid Mesnik <Leonid.Mesnik at oracle.com 
> <mailto:Leonid.Mesnik at oracle.com>> wrote:
>
>> Hi
>>
>> Could you please review this extra small fix which add @requires to 
>> the single test. This test is not compatible with minimal VM even it 
>> doesn't require any specific modules outside compact2. So I've just 
>> added corresponding @requires.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8156777
>>
>> Tested by manual jtreg execution with -minimal/-client/default vm 
>> options.
>>
>> Fix inlined:
>> --- a/test/testlibrary_tests/SimpleClassFileLoadHookTest.java	Wed May 11 02:32:14 2016 -0400
>> +++ b/test/testlibrary_tests/SimpleClassFileLoadHookTest.java	Wed May 11 19:24:34 2016 +0300
>> @@ -24,7 +24,7 @@
>>   /*
>>    * @test
>>    * @library /testlibrary
>> - *
>> + * @requires vm.flavor != "minimal"
>>    * @run main/othervm/native -agentlib:SimpleClassFileLoadHook=Foo,XXX,YYY
>>    *      SimpleClassFileLoadHookTest
>>    */
>>
>> Leonid