RFR(s): 8176213: 78 sun/security/krb5/auto tests failing due to undeclared dependecies
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