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

Zdenek Zambersky zzambers at openjdk.org
Tue Mar 19 16:48:43 UTC 2024


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)
    at java.lang.Thread.run(Thread.java:750)

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

Commit messages:
 - Backport ffb3771816910d2eddba367812e57a9c5459ff8c

Changes: https://git.openjdk.org/jdk8u-dev/pull/469/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=469&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8265019
  Stats: 11 lines in 1 file changed: 8 ins; 2 del; 1 mod
  Patch: https://git.openjdk.org/jdk8u-dev/pull/469.diff
  Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/469/head:pull/469

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


More information about the jdk8u-dev mailing list