RFR: 8378118: Test tools/jimage/JImageBadFileTest.java failed on Windows

Roger Riggs rriggs at openjdk.org
Mon Feb 23 15:03:13 UTC 2026


On Wed, 18 Feb 2026 10:42:01 GMT, David Beaumont <duke at openjdk.org> wrote:

> Removed the manual file tidyup on the expectation that jtreg will sort that all out.
> Switched to using Utils.createTempFile() while I was there.

Good to clean up the CI.

test/jdk/tools/jimage/JImageBadFileTest.java line 145:

> 143:         // Self test that the file copying isn't itself corrupting anything.
> 144:         Path tempJimage = writeModifiedJimage("good_file", -1, b -> {});
> 145:         jimage("info", tempJimage.toString()).assertSuccess();

I know its a bit late to complain now, but "jimage" is not a very nor a very descriptive name of a central utility method.

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

Marked as reviewed by rriggs (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/29783#pullrequestreview-3841592785
PR Review Comment: https://git.openjdk.org/jdk/pull/29783#discussion_r2841339788


More information about the core-libs-dev mailing list