Re: [JDK 8]Code review request 8022228, Intermittent test failures in sun/security/ssl/javax/net/ssl/NewAPIs

[Weijun Wang]

SessionTimeOutTests.java:

411             local.initCause(remote);
What if local already has a cause? Will this overwrite it?

427                 exception.addSuppressed(startException);

Is it possible startException is null?

Same for the other test.

Also, if you have to do all these checks in every SSL test, can they be 
added into a library?

Thanks
Max

On 8/22/13 9:18 AM, Xuelei Fan wrote:
Hi Weijun,

Please review this regression test bug fix for JDK 8.

Webrev: http://cr.openjdk.java.net/~xuelei/8022228/webrev.00/

The timeout issue of SessionTimeOutTests.java is still unknown.  This
fix is just try to use new SSL testing template so as to expose more
information.

The issue of SessionCacheSizeTests is that the client side may kick off
a connection before server is ready to listen on a particular port.

Thanks,
Xuelei