RFR 8171441: tools/launcher/VersionCheck.java doesn't report names of tools which failed checks

Dmitry Fazunenenko dmitry.fazunenko at oracle.com
Wed Dec 21 14:16:16 UTC 2016


Hello,

I'm looking for reviews of a relatively simple test change:
     http://cr.openjdk.java.net/~dfazunen/8171441/webrev.00/
     https://bugs.openjdk.java.net/browse/JDK-8171441

The purpose of the change is to improve diagnostic.

Thanks,
Dima

PS: After the fix the failures will be reported as:

----------System.err:(13/956)----------
java.lang.AssertionError: VersionCheck failed: testJVersionStrings: 
[java]; testToolVersion: [jar];
     at VersionCheck.main(VersionCheck.java:295)
     at 
java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native 
Method)
     at 
java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
     at 
java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
     at java.base/java.lang.reflect.Method.invoke(Method.java:538)
     at 
com.sun.javatest.regtest.agent.MainWrapper$MainThread.run(MainWrapper.java:110)
     at java.base/java.lang.Thread.run(Thread.java:844)

JavaTest Message: Test threw exception: java.lang.AssertionError: 
VersionCheck failed: testJVersionStrings: [java]; testToolVersion: [jar];
JavaTest Message: shutting down test

STATUS:Failed.`main' threw exception: java.lang.AssertionError: 
VersionCheck failed: testJVersionStrings: [java]; testToolVersion: [jar];

PPS: The output of passed test now looks like:

=== testJVersionStrings ===
Testing servertool
Testing jstat
Testing jmod
Testing jjs
Testing jimage
Testing jlink
Testing jrunscript
Testing jdeprscan
Testing jconsole
Testing rmiregistry
Testing keytool
Testing schemagen
Testing javac
Testing jar
Testing jhsdb
Testing jcmd
Testing jstack
Testing wsgen
Testing jshell
Testing serialver
Testing jmap
Testing javap
Testing jps
Testing jstatd
Testing javadoc
Testing tnameserv
Testing jdb
Testing jinfo
Testing jdeps
Testing xjc
Testing rmid
Testing jarsigner
Testing idlj
Testing rmic
Testing appletviewer
Testing pack200
Testing javah
Testing policytool
Testing orbd
testJVersionStrings passed
=== testInternalStrings ===
testInternalStrings passed
=== testToolVersion ===
Testing java
#> 
/net/jse-st01.ru.oracle.com/export4/java/re/jdk/9/promoted/ea/149/binaries/solaris-x64/bin/java 
-version
java version "9-ea"
Java(TM) SE Runtime Environment (build 9-ea+149)
Java HotSpot(TM) 64-Bit Server VM (build 9-ea+149, mixed mode)
#> echo $?
0
Testing javac
#> 
/net/jse-st01.ru.oracle.com/export4/java/re/jdk/9/promoted/ea/149/binaries/solaris-x64/bin/javac 
-version
javac 9-ea
#> echo $?
0
Testing jhsdb
#> 
/net/jse-st01.ru.oracle.com/export4/java/re/jdk/9/promoted/ea/149/binaries/solaris-x64/bin/jhsdb 
-version
     clhsdb           command line debugger
     debugd           debug server
     hsdb             ui debugger
     jstack --help    to get more information
     jmap   --help    to get more information
     jinfo  --help    to get more information
     jsnap  --help    to get more information
#> echo $?
0
Testing jshell
#> 
/net/jse-st01.ru.oracle.com/export4/java/re/jdk/9/promoted/ea/149/binaries/solaris-x64/bin/jshell 
-version
jshell 9-ea
#> echo $?
0
Testing javap
#> 
/net/jse-st01.ru.oracle.com/export4/java/re/jdk/9/promoted/ea/149/binaries/solaris-x64/bin/javap 
-version
9-ea
#> echo $?
0
Testing jdb
#> 
/net/jse-st01.ru.oracle.com/export4/java/re/jdk/9/promoted/ea/149/binaries/solaris-x64/bin/jdb 
-version
This is jdb version 9.0 (Java SE version 9-ea)
#> echo $?
0
Testing idlj
#> 
/net/jse-st01.ru.oracle.com/export4/java/re/jdk/9/promoted/ea/149/binaries/solaris-x64/bin/idlj 
-version
IDL-to-Java compiler (portable), version "3.2"
#> echo $?
0
Testing javah
#> 
/net/jse-st01.ru.oracle.com/export4/java/re/jdk/9/promoted/ea/149/binaries/solaris-x64/bin/javah 
-version

Warning: The javah tool is planned to be removed in the next major
JDK release. The tool has been superseded by the '-h' option added
to javac in JDK 8. Users are recommended to migrate to using the
javac '-h' option; see the javac man page for more information.

javah version "9-ea"
#> echo $?
0
testToolVersion passed
=== testInternalStrings ===
testDebugVersion passed
All Version string comparisons: PASS
----------System.err:(1/15)----------



More information about the core-libs-dev mailing list