On Tue, 12 May 2026 19:11:23 GMT, Ashay Rane <[email protected]> wrote:
>> 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 two additional > commits since the last revision: > > - Add missing year to copyright notice. > - Move `powerShellPath()` to `jdk.test.lib.os.windows.FileUtils` > > This avoids having to duplicate the function in individual tests. test/lib/jdk/test/lib/os/windows/FileUtils.java line 30: > 28: public static String powerShellPath() { > 29: String systemRoot = System.getenv("SystemRoot"); > 30: String suffix = > "\\System32\\WindowsPowerShell\\v1.0\\powershell.exe"; Can we check if `systemRoot + suffix` exists before using it? ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/31127#discussion_r3233694439
