RFR 9: 8130296 [TESTBUG] java/lang/ProcessHandle/OnExitTest - Unaccounted for children

[Roger Riggs]

Hi Joe,

Webrev updated in place.

The jdk.testlibrary.Utils.adjustTimeout(timeout) returns the timeout 
adjusted
by the jtreg -timeoutFactor argument.

Thanks, Roger


On 7/8/2015 4:50 PM, joe darcy wrote:
> Hi Roger,
>
> I think the test would be more robust if the hard-coded time out 
> window was a function of the time factor the test was running under. 
> Maybe
>
>     10 seconds * timeout
>
> IIRC, the timeout factor is available as an environment variable (or 
> similar) in the running tests. I checked the jtreg docs quickly and 
> didn't find it, but I believe the jdk regression tests have already 
> had a pass to do this kind of update so there should be some guiding 
> examples in the sources.
>
> Thanks,
>
> -Joe
>
> On 7/8/2015 1:16 PM, Roger Riggs wrote:
>> Ping and note this change is simplified after the fix for 8085981.
>>
>> Please review this change to a ProcessHandle test to address 
>> intermittent failures.
>> Unexpected subprocesses has been observed. The test succeeds if all 
>> of the
>> processes under test are accounted for; others are ignored.
>>
>> Webrev:
>>    http://cr.openjdk.java.net/~rriggs/webrev-exit-8130296/
>>
>> Issue:
>>    https://bugs.openjdk.java.net/browse/JDK-8130296
>>
>> Thanks, Roger
>>
>>
>