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

[mikhailo]

Fix looks good to me.

Thank you,
Misha

On 05/11/2016 09:36 AM, Leonid Mesnik wrote:
> Hi
> <https://bugs.openjdk.java.net/browse/JDK-8156777>
> 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