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

Alan Bateman alanb at openjdk.org
Thu Feb 19 07:37: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.

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

> 80:             throws IOException {
> 81:         int remaining = maxLen >= 0 ? maxLen : Integer.MAX_VALUE;
> 82:         Path dst = Utils.createTempFile("modules-" + label, "");

Just curious why this is changed.

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

> 98:         } catch (IOException e) {
> 99:             Files.deleteIfExists(dst);
> 100:             throw e;

Good, this allows it to be saved in the event of a test failure too.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/29783#discussion_r2826103532
PR Review Comment: https://git.openjdk.org/jdk/pull/29783#discussion_r2826106775


More information about the core-libs-dev mailing list