RFR 8048621: Implement basic keystore tests

zaiyao liu zaiyao.liu at oracle.com
Wed Sep 3 02:57:15 UTC 2014


Hi Xuelei,

Thanks for review, please review the update: 
http://cr.openjdk.java.net/~tyan/kevin/JDK-8048621/webrev02/

Thanks

Kevin
于 2014/9/1 17:42, Xuelei Fan 写道:
> Providers.java
> ---------------
> May be not necessary to define Providers.SUN_JCE,
> Providers.SUNPKCS11_SOLARIS. As add unnecessary dependencies among
> tests, and slow down the test performance.
>
> TestKeyStoreBasic.java
> ----------------------
> Better to throw exception, rather than return false so that the cause of
> failure is exposed in the exception stacks.
>
> TestKeyStoreEntry.java
> ----------------------
> See above.
>
> A question: in which case PKCS11 keystore is disabled?
>
> Xuelei
>
>
> On 9/1/2014 3:14 PM, zaiyao liu wrote:
>> Hi Xuelei,
>>
>> Can you help to review those tests?
>>
>> Thanks
>>
>> Kevin
>> 于 2014/8/28 9:45, zaiyao liu 写道:
>>> Hi All,
>>>
>>> Please review the code change,the purpose of this fix is implement
>>> basic tests for keystore types supported by JDK.
>>>
>>> Webrev: http://cr.openjdk.java.net/~tyan/kevin/JDK-8048621/webrev01/
>>> Bug:     https://bugs.openjdk.java.net/browse/JDK-8048621
>>>
>>> Thanks
>>>
>>> Kevin




More information about the security-dev mailing list