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

[Stanislav Smirnov]

Hi Dima,

changes look good, I will only suggest using lambda’s in couple of iterations
+            for (String s : tr.testOutput) {
+                System.out.println(s);
+            }
tr.testOutput.forEach(System.out::println)

         for (String x : tr.testOutput) {
             alist.add(x.trim());
         }
tr.testOutput.stream.map(String::trim).forEach(aList:add)


Best regards,
Stanislav Smirnov





> On 21 Dec 2016, at 17:16, Dmitry Fazunenenko <dmitry.fazunenko at oracle.com> wrote:
> 
> 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)----------
>