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

[Christian Tornqvist]

Hi Leonid,

Change looks good.

Thanks,
Christian

> On May 11, 2016, at 12:36 PM, Leonid Mesnik <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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20160512/e97cc486/attachment-0001.html>