Re: [9] RFR 8134487：updated sun/security/ssl/StatusStapling/* to work with modules

[Tim Du]

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