RFR J8050281: New permission tests for JEP 140

[Sean Mullan]

On 08/06/2014 06:25 PM, Amanda Jiang wrote:
> Sorry, wrong webrev link, it should be:
> http://cr.openjdk.java.net/~tyan/amandaj/8050281/webrev.01/

Mostly looks good, just a few comments:

* NullPerms

I suggest renaming this test to LimitedDoPrivilegedWithNullPerms

[27-28] Suggest rewording as: "Test that NullPointerException is thrown 
if any element of perms parameter is null"

- you should add a couple of tests with PrivilegedActionException as the 
first parameter (1 for doPrivileged and 1 for doPrivilegedWithCombiner).

* LimitedDoPrivilegedWithThread

[27] typo: s/trhead/thread/

--Sean

>
> Thanks,
> Amanda
>
> On 8/6/14 3:24 PM, Amanda Jiang wrote:
>> Updated the original codes for better handling interrupted exception.
>> webrev: http://cr.openjdk.java.net/~tyan/amandaj/JDK-8031661/webrev.01/
>>
>> Thanks,
>> Amanda
>>
>> On 7/30/14 4:50 PM, Amanda Jiang wrote:
>>> Hi All,
>>>
>>> Could you please review following 2 new regression tests to be added
>>> for JEP140 (Limied doPrivileged).
>>> New tests are added to perform permission tests for limited
>>> doprivileged with multiple threads , and basic API testing.
>>>
>>> JDK Issue: https://bugs.openjdk.java.net/browse/JDK-8050281
>>> webrev: http://cr.openjdk.java.net/~rhalade/8050281/webrev.00/
>>>
>>> Thanks,
>>> Amanda
>>
>