RFR 8153545: sun/security/pkcs11/Provider/Login.sh fails on Linux

[Amanda Jiang]

Hi Max,

Thanks for reviewing, you are right, "sm" should be the correct argument.
I updated the test argument and PKCS11Test, please check the new webrev:
http://cr.openjdk.java.net/~amjiang/8153545/webrev.02/

Thanks,
Amanda

On 16/4/8 上午12:28, Wang Weijun wrote:
> Hi Amanda
>
> I am not familiar with the tests, but it looks like the 1st args argument should be "sm" [1] to enable the security manager, not "true".
>
> If PKCS11Test always uses args as command and optional argument(s), maybe you should throw an exception if the command is unknown.
>
> Thanks
> Max
>
> [1] http://hg.openjdk.java.net/jdk9/dev/jdk/file/cac52c9da800/test/sun/security/pkcs11/PKCS11Test.java#l184
>
>
>> On Apr 8, 2016, at 1:09 PM, Amanda Jiang <amanda.jiang at oracle.com> wrote:
>>
>> Hi All,
>>
>> Please help to review following changeset which fixes test bug for sun/security/pkcs11/Provider/Login.sh and another similar failed test.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8153545
>> Changest: http://cr.openjdk.java.net/~amjiang/8153545/webrev.01/
>>
>> Thanks,
>> Amanda