Can I please get a review of this test-only change which updates the `test/jdk/tools/jar/JarExtractTest.java` test to use the jtreg scratch directory for its temporary files?
In addition to that change, the `testExtractNoDestDirWithPFlag` is a bit different and that one intentionally expects to write out to a directory that starts with a leading slash in its path. We use `/tmp` directory in that test intentionally. In this PR, that test method now cleans up the `/tmp/` directory after the test is done. I've looked through the discussions in the PR for JDK-8173970, where this test was introduced and I've verified that this change doesn't impact any original expectations of this test. I've also verified that the test continues to pass with this change on all platforms in our CI. --------- - [x] I confirm that I make this contribution in accordance with the [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). ------------- Commit messages: - Cleanup the directory created by the test under "/tmp" directory - use scratch directory for the test's JAR file Changes: https://git.openjdk.org/jdk/pull/30847/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=30847&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8381840 Stats: 36 lines in 1 file changed: 27 ins; 1 del; 8 mod Patch: https://git.openjdk.org/jdk/pull/30847.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/30847/head:pull/30847 PR: https://git.openjdk.org/jdk/pull/30847
