RFR(S) : 8242471 : remove "temporarily" restrictions of nsk/jdi/Argument/value/value004

Chris Plummer chris.plummer at oracle.com
Thu Apr 9 22:15:09 UTC 2020


Hi Igor,

   89             log.display("Connector's transport is: " + 
c.transport().name());

Would be nice if you printed each transport's name (even the skipped 
ones). That way when reading the log after getting SkippedException you 
can see which transports were attempted.

You removed the following:

   50  * ================================================
   51  * WARNING:
   52  *         Temporarily the test is prepared only for
   53  *         Sparc.Solaris.dt_socket-transport of RawCommandLineLaunch
   54  *         connector
   55  * ================================================

Isn't this still somewhat true? It's still requires dt_socket, but no 
longer solaris-sparc. This is part of the reason I asked above to print 
which transports are attempted. I'm curious what the skipped transport 
is on Windows.

thanks,

Chris

On 4/9/20 2:43 PM, Igor Ignatyev wrote:
> http://cr.openjdk.java.net/~iignatyev/8242471/webrev.00/
>> 38 lines changed: 8 ins; 23 del; 7 mod;
> Hi all,
>
> could you please review this small patch for nsk/jdi/Argument/value/value004 test?
> from JBS:
>> nsk/jdi/Argument/value/value004 test has restrictions to be run only on solaris-sparc and w/ dt_socket transport, solaris-sparc restriction seems to be not valid any more. the test also should be updated to use skipped exception if there are no suitable connectors available.
>
> JBS: https://bugs.openjdk.java.net/browse/JDK-8242471
> testing: the changed test on {linux,windows,macosx}-x64 and solaris-sparcv9, w/ the test being skipped on windows-x64
> webrev: http://cr.openjdk.java.net/~iignatyev/8242471/webrev.00/
>
> Thanks,
> -- Igor




More information about the serviceability-dev mailing list