RFR 8033271: Manual security tests have @ignore rather than @run main/manual
Rajan Halade
rajan.halade at oracle.com
Mon Mar 17 20:17:17 UTC 2014
Thanks again! Updated review with corrections -
http://cr.openjdk.java.net/~wetmore/8033271/webrev.03/
- Rajan
On 3/14/2014 18:36, Xuelei Fan wrote:
> Minimal comments:
>
> test/sun/security/smartcardio/TestAll.java
> ==========================================
> Looks like there is no actual update.
>
>
> test/sun/security/smartcardio/*
> ===============================
> - 32 //This test requires special hardware.
> + 32 // This test requires special hardware.
>
> Looks nicer if there is leading space.
>
>
> test/sun/security/smartcardio/TestConnectAgain.java
> ===================================================
> - 29 * @run main/manual TestTransmit
> + 29 * @run main/manual TestConnectAgain
>
> Maybe a typo here. Would you please make the update?
>
>
> test/sun/security/ssl/X509TrustManagerImpl/ClientServer.java
> ===================================================
> - 35 * JSSE supports algorithm constraints with CR 6916074,
> - 36 * need to update this test case in JDK 7 soon
> +
> + 35 * JSSE supports algorithm constraints with CR 6916074, need to
> + 36 * update this test case in JDK 7 soon
>
> Would you mind add a blank line and join the two line accordingly?
>
> Otherwise, looks fine to me.
>
> Thanks,
> Xuelei
>
>
More information about the security-dev
mailing list