RFR(M): 8054892: Improve compiler's CLI tests error reporting
christian.thalinger at oracle.com
Mon Dec 1 22:49:25 UTC 2014
I didn’t verify all new messages but this looks very useful. The only thing I could complain about is:
+ "JVMStartup should have exit value '%d'.%n%s",
JVMStartup probably refers to some internal method. A more general message might be better.
In any case, looks good to me.
> On Nov 27, 2014, at 3:13 AM, Evgeniya Stepanova <evgeniya.stepanova at oracle.com> wrote:
> Could you please review fix for 8054892?
> Problem: CLI tests do not show what exactly went wrong when test failed. They show an error behavior is not as expected.
> Solution: Added explanation which behavior is expected and why it is so.
> Base class /testlibrary/com/oracle/java/testlibrary/cli/CommandLineOptionTest.java changed to use error string, submitted by tests classes.
> -exitErrorMessage message to be shown if exit code of JVM process is not as expected.
> -wrongWarningMessage message to be shown if warning messages in output are not as expected.
> -optionErrorString to be shown if option value is not as expected.
> Updated tests from test/compiler/rtm/cli/, test/compiler/rtm/cli/ and test/compiler/intrinsics/sha/cli folders.
> bug: https://bugs.openjdk.java.net/browse/JDK-8054892 <https://bugs.openjdk.java.net/browse/JDK-8054892>
> webrev: http://cr.openjdk.java.net/~eistepan/8054892/webrev.00/ <http://cr.openjdk.java.net/~eistepan/8054892/webrev.00/>
> Evgeniya Stepanova
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the hotspot-compiler-dev