> The powershell binary is required to run these tests but in the GitHub
> workflow that runs tests (specifically, in test.yml), we reset the PATH
> environment variable on Windows, causing the GitHub workflow to fail if
> you run these tests.  The tests pass locally since local test runs don't
> modify PATH.
> 
> To make these tests pass in GitHub workflows, this patch updates the
> tests to refer to PowerShell using the canonical path on Windows.  See
> https://github.com/raneashay/openjdk-jdk/actions/runs/25680763981/job/75401590988
> for a sample workflow run of the tier2 part1 tests, which include the
> CngCipher and DupAlias tests.
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

Ashay Rane has updated the pull request incrementally with one additional 
commit since the last revision:

  Address PR comments
  
  1. Replaced `PowerShellPath()` with `powerShellPath()` to maintain
     consistency with the rest of the functions in FileUtils.java.
  
  2. Updated copyright year to 2026.

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/31127/files
  - new: https://git.openjdk.org/jdk/pull/31127/files/d1583211..3a5f0bc7

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=31127&range=03
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=31127&range=02-03

  Stats: 6 lines in 5 files changed: 0 ins; 0 del; 6 mod
  Patch: https://git.openjdk.org/jdk/pull/31127.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/31127/head:pull/31127

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

Reply via email to