> On May 9, 2017, at 4:23 PM, Brent Christian <brent.christ...@oracle.com> > > I've removed the test case for automatic modules, and added a @run action for > each policy file + system classloader configuration. I also removed the code > to compile test sources, using @build instead. > > I also made some (hopefully) clarifying changes to the code. > > The String[] argument to execute() is now separated into args/status/message, > instead of being lumped into one String[], and then having to be split() out > of a single String. Where execute() is called, it's easier to read the args > in order (versus assembled using String.format()). > > I also simplified the code for expected status/output, and made some path > variable names more descriptive. > > Webrev: > http://cr.openjdk.java.net/~bchristi/8177328/webrev.01/
Thanks for the clean up. Does each @run action need 4 mins timeout? Can it restore to the default timeout? Mandy