RFR 8176296: Test sun/security/krb5/auto/Basic.java faling after adding module declaration into TEST.properties.

[Valerie Peng]

Looks fine.
Thanks,
Valerie

On 3/14/2017 1:24 AM, Amy Lu wrote:
> Looks good.Thank you Max!
>
> (I'm not a reviewer)
>
> Thanks,
> Amy
>
> On 3/14/17 3:37 PM, Weijun Wang wrote:
>> How can you launch jtreg like this? :-)
>>
>> Anyway, I've updated the webrev at
>>
>>    http://cr.openjdk.java.net/~weijun/8176296/webrev.01
>>
>> The test now creates the Java process itself so you won't be able to 
>> manipulate it from outside.
>>
>> Thanks
>> Max
>>
>> On 03/14/2017 12:02 PM, Amy Lu wrote:
>>> I looked at the new test: BasicLimited.java
>>> This test will fail if "jdk.security.jgss" is specified from jtreg
>>> javaoptions:
>>> jtreg  ... -javaoptions:"--limit-modules
>>> java.security.jgss,jdk.security.auth,jdk.security.jgss"
>>> jdk9/test/sun/security/krb5/auto/BasicLimited.java
>>>
>>> Thanks,
>>> Amy
>>>
>>> On 3/14/17 11:34 AM, Weijun Wang wrote:
>>>> Ping again.
>>>>
>>>> On 03/07/2017 10:10 PM, Weijun Wang wrote:
>>>>> Please review this changeset at
>>>>>
>>>>>    http://cr.openjdk.java.net/~weijun/8176296/webrev.00/
>>>>>
>>>>> It looks with jdk.security.jgss added into @modules in 
>>>>> TEST.properties
>>>>> (JDK-8176213), it cannot be excluded using --limit-modules.
>>>>>
>>>>> Therefore I split Basic.java into 2 and the new test has its own
>>>>> @modules tag.
>>>>>
>>>>> In case you don't remember this test, it was about the Oracle 
>>>>> JGSS-API
>>>>> extension. If the jdk.security.jgss module is available, the 
>>>>> GSSContext
>>>>> implementation is ExtendedGSSContextImpl inside this module that 
>>>>> can do
>>>>> more than GSSContextImpl defined in java.security.jgss (which will be
>>>>> returned when jdk.security.jgss is not available).
>>>>>
>>>>> Thanks
>>>>> Max
>>>
>