On 2/17/2016 11:40 PM, Alan Bateman wrote:
On 17/02/2016 09:57, Tim Du wrote:
Sure. I am aslo updated java.base/sun/security/testlibrary to test/java/security/testlibrary/ , so that keep the testlibrary files could be reused by other tests. Re-generate the latest webrev is http://cr.openjdk.java.net/~tidu/8134487/webrev.03/ , Thanks.
Are there other tests that uses this infrastructure that need to be updated too? I'm seeing test failures with other Stapling tests and it looks like they depend on CertificateBuilder and SimplerOSCPServer. Would it be possible to run the jdk_security tests group to make sure that everything is caught?
I grep the files which are using CertificateBuilder and SimpleOCSPServer from jake/jdk/test, the result as below:
- javax/net/ssl/Stapling/HttpsUrlConnClient.java
- javax/net/ssl/Stapling/SSLEngineWithStapling.java
- javax/net/ssl/Stapling/SSLSocketWithStapling.java
Above three tests are not using the internal and non-public class , They could pass now.No need to update.

- sun/security/ssl/StatusStapling/StatusResponseManagerTests.java
This one is in ProblemList.jake now, it could be fixed by this code review.

So I think everything about CertificateBuilder and SimpleOCSPServer is caught.Thank you.

Regards
Tim

-Alan

Reply via email to