RFR(S) : 8039953 : [TESTBUG] Timeout java/lang/invoke/MethodHandles/CatchExceptionTest.java

[Paul Sandoz]

Hi Igor,

This looks ok. 

I like how you have factored out things into TimeLimitedRunner. Do you plan in a future patch to update lambda form test code that uses similar functionality?

Is the adjustment "timeout *= 0.9" necessary? does reduction by 10% make a difference?

Paul.

On Nov 29, 2014, at 5:36 PM, Igor Ignatyev <igor.ignatyev at oracle.com> wrote:

> http://cr.openjdk.java.net/~iignatyev/8039953/webrev.00
> 98 lines changed: 93 ins; 3 del; 2 mod;
> 
> Hi all,
> 
> Please review the patch:
> 
> Problem:
> on some configurations, java/lang/invoke/MethodHandles/CatchExceptionTest.java can timeout before all test cases are run
> 
> Fix:
> interrupt test execution if it's not enough time to continue
> 
> bug : https://bugs.openjdk.java.net/browse/JDK-8039953
> changes in testlibrary : https://bugs.openjdk.java.net/browse/JDK-8066191
> testing: locally
> 
> -- 
> Igor