RFR: 8224112: [TESTBUG] java/util/Base64/TestEncodingDecodingLength.java should be skipped with insufficient memory

[Jie Fu]

Hi all,

Please review the updated version: 
http://cr.openjdk.java.net/~jiefu/8224112/webrev.01/

If the memory isn't enough, the test won't run (filtered out by the 
"requires" tag) instead of being skipped in the previous patch.
It fixes the false sense of passing problem pointed out by Aleksey.
Could you please review it and give me some advice?

Thanks a lot.
Best regards,
Jie

On 2019/5/17 下午10:55, Jie Fu wrote:
> Okay.
>
> Thank you Aleksey.
>
> On 2019年05月17日 22:30, Aleksey Shipilev wrote:
>> On 5/17/19 4:25 PM, Jie Fu wrote:
>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8224112
>>> Webrev: http://cr.openjdk.java.net/~jiefu/8224112/webrev.00/
>>>
>>> java/util/Base64/TestEncodingDecodingLength.java may fail due to 
>>> insufficient memory when our test
>>> machine is busy.
>>> It might be better to skip it when the available memory is 
>>> insufficient.
>>> Could you please review it and give me some advice?
>> I am against special casing like that: it may give you the false 
>> sense of passing the test when it
>> is actually broken. In other words, having the infrastructure-induced 
>> test failure is better than
>> having the test that allows bugs to creep in. Get more memory on a 
>> test machine, and maybe lobby for
>> jtreg to unhoard some [1].
>>
>> Thanks,
>> -Aleksey
>>
>> [1] https://bugs.openjdk.java.net/browse/CODETOOLS-7902454
>>
>
>