RFR(s): 8176213: 78 sun/security/krb5/auto tests failing due to undeclared dependecies (original) (raw)

Sergei Kovalev sergei.kovalev at oracle.com
Tue Mar 7 09:50:20 UTC 2017

Hi Max,

Thank you for review.

07.03.17 02:15, Weijun Wang wrote:

Hi Sergei

Two questions: 1. Can we remove the @modules section in Renew.java so that it uses the one in TEST.properties? We can however TEST.properties contains jdk.security.jgss module dependency but Renew.java doesn't. If we remove module declaration from the last one we reduce the number of configuration where the test able to run. Should I do this? 2. I cannot see why jdk.crypto.cryptoki is necessary for PrincipalSystemPropTest.java. What is the error? Thank you for the question. It should be jdk.security.auth instead. BTW, can you tell me how to make the test fail? Maybe with some special jtreg options? e.g.:

jtreg -jdk:/usr/home/jdk9_latest -verbose:all -javaoptions:"--limit-modules java.base,java.security.jgss,jdk.security.auth" /usr/home/repos/jdk9-dev/jdk/test/sun/security/krb5/auto/principalProperty/PrincipalSystemPropTest.java

If you get outjdk.security.auth from the module list the test will start failing. >> Thanks > Max >> On 03/07/2017 12:24 AM, Sergei Kovalev wrote: >> Hello Team, >>>> Please review small fix for tests. >>>> BugID: https://bugs.openjdk.java.net/browse/JDK-8176213 >> Webrev: http://cr.openjdk.java.net/~skovalev/8176213/webrev.00 >>>> Issue: The tests failing due to undeclared dependencies on >> jdk.security.auth and jdk.security.jgss. >> Solution: add missed module declaration into TEST.properties file. Two >> of tests also required individual dependencies on jdk.httpserver and >> jdk.crypto.cryptoki therefore we creating the module declaration section >> in each individual test. >>

With best regards, Sergei

More information about the security-dev mailing list