[jdk8u-dev] Withdrawn: 8265019: Update tests for additional TestNG test permissions

[duke]

On Tue, 19 Mar 2024 16:44:08 GMT, Zdenek Zambersky <zzambers at openjdk.org> wrote:

> Backport fixes 2 tests failures, when newer jtreg is used (6+1). Affects only test code. It excludes changes to `test/jdk/java/lang/ProcessHandle/PermissionTest.java` as jdk8 does not have this file (ProcessHandle is jdk9+). Otherwise, when dealt with different paths, changeset applied cleanly (except for copyright line, which was  done manually).
> 
> **Fixed tests:**
> 
> javax/sql/testng/test/rowset/spi/SyncFactoryPermissionsTests.java
> java/sql/testng/test/sql/DriverManagerPermissionsTests.java
> 
> 
> **Exception:**
> 
> java.security.AccessControlException: access denied ("java.util.PropertyPermission" "testng.thread.affinity" "read")
>     at java.security.AccessControlContext.checkPermission(AccessControlContext.java:472)
>     at java.security.AccessController.checkPermission(AccessController.java:886)
>     at java.lang.SecurityManager.checkPermission(SecurityManager.java:549)
>     at java.lang.SecurityManager.checkPropertyAccess(SecurityManager.java:1294)
>     at java.lang.System.getProperty(System.java:769)
>     at org.testng.internal.RuntimeBehavior.enforceThreadAffinity(RuntimeBehavior.java:106)
>     at org.testng.internal.TestMethodWorker.run(TestMethodWorker.java:111)
>     at java.util.ArrayList.forEach(ArrayList.java:1259)
>     at org.testng.TestRunner.privateRun(TestRunner.java:764)
>     at org.testng.TestRunner.run(TestRunner.java:585)
>     at org.testng.SuiteRunner.runTest(SuiteRunner.java:384)
>     at org.testng.SuiteRunner.runSequentially(SuiteRunner.java:378)
>     at org.testng.SuiteRunner.privateRun(SuiteRunner.java:337)
>     at org.testng.SuiteRunner.run(SuiteRunner.java:286)
>     at org.testng.SuiteRunnerWorker.runSuite(SuiteRunnerWorker.java:53)
>     at org.testng.SuiteRunnerWorker.run(SuiteRunnerWorker.java:96)
>     at org.testng.TestNG.runSuitesSequentially(TestNG.java:1218)
>     at org.testng.TestNG.runSuitesLocally(TestNG.java:1140)
>     at org.testng.TestNG.runSuites(TestNG.java:1069)
>     at org.testng.TestNG.run(TestNG.java:1037)
>     at com.sun.javatest.regtest.agent.TestNGRunner.main(TestNGRunner.java:94)
>     at com.sun.javatest.regtest.agent.TestNGRunner.main(TestNGRunner.java:54)
>     at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
>     at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
>     at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
>     at java.lang.reflect.Method.invoke(Method.java:498)
>     at com.sun.javatest.regtest.agent.MainWrapper$MainThread.run(MainWrapper.java:127)
> ...

This pull request has been closed without being integrated.

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

PR: https://git.openjdk.org/jdk8u-dev/pull/469