On Tue, 13 Apr 2021 18:01:49 GMT, Lance Andersen <lan...@openjdk.org> wrote:

> Hi all,
> 
> Please review the following patch which adds additional permissions needed 
> for when JTREG upgrades to a newer version of TestNG.
> 
> Best,
> Lance

test/jdk/java/lang/ProcessHandle/PermissionTest.java line 215:

> 213:         permissions.add(new PropertyPermission("testng.report.xml.name", 
> "read"));
> 214:         permissions.add(new ReflectPermission("suppressAccessChecks"));
> 215:         permissions.add(new PropertyPermission("testng.timezone", 
> "read"));

might be better to group the testng.* properties so they aren't mixed up with 
the other runtime permissions.

-------------

PR: https://git.openjdk.java.net/jdk/pull/3471

Reply via email to