[9] Review request: JDK-8129595: New DTLS tests need @modules

[Konstantin Shefov]

Alan,

Do the tests form "test/javax/net/ssl/DTLS" compile against jake? They 
have just the same dependencies as "test/javax/net/ssl/TLS" and the 
TEST.properties file just like one suggested in this fix.
If yes, it is not worth changing TEST.properties any more.

-Konstantin

On 06/24/2015 01:08 PM, Alan Bateman wrote:
> On 24/06/2015 11:01, Konstantin Shefov wrote:
>> Hello,
>>
>> Please review the test bug fix
>> https://bugs.openjdk.java.net/browse/JDK-8129595
>> Webrev is http://cr.openjdk.java.net/~kshefov/8129595/webrev.00/
> security-dev would be a better place to review this.
>
> The updates to TEST.properties look okay but I suspect the proposed 
> patch is incomplete - for example the tests that use SSLEngineTestCase 
> use sun.security.krb5 classes so they will need to be added to the 
> @modules line.
>
> -Alan