Hi,
I have one more update, with a couple of suggested changes to simplify
the execute() calls:
* execute() takes a vararg, so explicit String[] creation can be omitted
(mostly).
* args common to every execute() call are consolidated into a List.
(The resulting arg reordering should not affect test execution.)
http://cr.openjdk.java.net/~bchristi/8177328/webrev.02/
Thanks,
-Brent
On 5/10/17 9:42 AM, Brent Christian wrote:
On 5/9/17 4:39 PM, Brent Christian wrote:
On 5/9/17 4:27 PM, Mandy Chung wrote:
Webrev:
http://cr.openjdk.java.net/~bchristi/8177328/webrev.01/
Can it restore to the default timeout?
Yes. Between the additional @runs and no longer testing automated
modules, I believe that should be fine.
Webrev updated in place, with the timeout value removed.
-Brent