RFR: 8314094: java/lang/ProcessHandle/InfoTest.java fails on Windows when run as user with Administrator privileges [v2]

Christoph Langer clanger at openjdk.org
Fri Sep 1 08:32:20 UTC 2023


> On Windows, the test java/lang/ProcessHandle/InfoTest.java can fail when run as user that is member of the Administrators group. In that case new files are not owned by the user but instead by BUILTIN\ADMINISTRATORS. This breaks the assumptions of the test's whoami check. My suggestion is to cater for this case and don't fail the test but write a warning message to stdout that a whoami check is not correctly possible.

Christoph Langer has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:

 - Use System.getProperty(user.name) for the Windows Adminstrators case
 - Merge branch 'master' into JDK-8314094
 - JDK-8314094

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/15222/files
  - new: https://git.openjdk.org/jdk/pull/15222/files/4e5cbf82..db6d3d14

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=15222&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=15222&range=00-01

  Stats: 27267 lines in 894 files changed: 17633 ins; 3604 del; 6030 mod
  Patch: https://git.openjdk.org/jdk/pull/15222.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/15222/head:pull/15222

PR: https://git.openjdk.org/jdk/pull/15222


More information about the core-libs-dev mailing list