RFR: 8272600: (test) Use native "sleep" in Basic.java [v4]

[Roger Riggs]

On Wed, 15 Sep 2021 21:47:39 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Roger Riggs has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Revert to using BasicSleep on Windows
>>   Added diagnostic output for a test that sometimes fails on Linux when using /bin/sleep.
>>   Addressed review comments.
>
> test/jdk/java/lang/ProcessBuilder/Basic.java line 2452:
> 
>> 2450: 
>> 2451:             if (p.waitFor(10, TimeUnit.MILLISECONDS)) {
>> 2452:                 System.out.println("WaitFor didn't wait long enough: " + (System.nanoTime() - start));
> 
> Either the condition or the message seems wrong here. If waitFor returns true then the process has exited and we obviously did wait long enough.

This code is diagnostic.  
After switching to native sleep, I had intermittent failures claiming it did not sleep long enough.
I was unable to find a specific cause for those failures.
Many of the tests fail to check if the sleep processes terminate prematurely and if the executable is not found, it never launched.

-------------

PR: https://git.openjdk.java.net/jdk/pull/5239