Re: RFR: 8256244: java/lang/ProcessHandle/PermissionTest.java fails with TestNG 7.1

2020-11-12 Thread Igor Ignatyev 
On Thu, 12 Nov 2020 15:48:33 GMT, Roger Riggs  wrote:

> TestNG 7.1 changed/corrected the way that @BeforeGroups are selected at 
> runtime.
> The test was depending on @BeforeGroups to initialize common security policy 
> for a number of tests.
> The tests are modified to individually setup the needed security manager and 
> policy.

Marked as reviewed by iignatyev (Reviewer).

-

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

Re: RFR: 8256244: java/lang/ProcessHandle/PermissionTest.java fails with TestNG 7.1

2020-11-12 Thread Lance Andersen 
On Thu, 12 Nov 2020 15:48:33 GMT, Roger Riggs  wrote:

> TestNG 7.1 changed/corrected the way that @BeforeGroups are selected at 
> runtime.
> The test was depending on @BeforeGroups to initialize common security policy 
> for a number of tests.
> The tests are modified to individually setup the needed security manager and 
> policy.

Marked as reviewed by lancea (Reviewer).

-

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

RFR: 8256244: java/lang/ProcessHandle/PermissionTest.java fails with TestNG 7.1

2020-11-12 Thread Roger Riggs 
TestNG 7.1 changed/corrected the way that @BeforeGroups are selected at runtime.
The test was depending on @BeforeGroups to initialize common security policy 
for a number of tests.
The tests are modified to individually setup the needed security manager and 
policy.

-

Commit messages:
 - 8256244: java/lang/ProcessHandle/PermissionTest.java fails with TestNG 7.1

Changes: https://git.openjdk.java.net/jdk/pull/1190/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=1190&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8256244
  Stats: 19 lines in 1 file changed: 9 ins; 1 del; 9 mod
  Patch: https://git.openjdk.java.net/jdk/pull/1190.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/1190/head:pull/1190

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