RFR(XS) : 8028482 : [TESTBUG] tests that use JMX should be in need_compact3 test group

[David Holmes]

On 21/01/2014 7:41 PM, Igor Ignatyev wrote:
> Hi David,
>
> Whitebox API itself doesn't use JMX, but CompilerWhiteBoxTest,
> super-class of these tests, uses it.

I see. That is unfortunate as it constrains the test environment for 
users of the Whitebox API even if they don't need the JMX part directly. 
Whomever owns this should see if they can rectify this some how.

Thanks,
David

> Igor
>
> On 01/21/2014 07:01 AM, David Holmes wrote:
>> Hi Igor,
>>
>> On 20/01/2014 10:27 PM, Igor Ignatyev wrote:
>>> Hi all,
>>>
>>> Please review patch:
>>>
>>> Problem:
>>> tests 'compiler/tiered/NonTieredLevelsTest.java' and
>>> 'TieredLevelsTest.java' use JMX, but not in 'need_compact3' test group
>>
>> I don't see any direct JMX use in those tests. Is it coming in via the
>> Whitebox API? That would be unfortunate.
>>
>> Thanks,
>> David
>>
>>
>>> Fix:
>>> tests are added into 'need_compact3'
>>>
>>> webrev: http://cr.openjdk.java.net/~iignatyev/8028482/webrev.00/
>>> jbs: https://bugs.openjdk.java.net/browse/JDK-8028482