RFR: 7022325: TEST_BUG: test/java/util/zip/ZipFile/ReadLongZipFileName.java leaks files if it fails [v2]
Lance Andersen
lancea at openjdk.org
Sat Jun 1 20:12:01 UTC 2024
On Sat, 1 Jun 2024 05:18:17 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:
>> Can I please get a review of this test-only change which updates a couple of places in the test to use `try-with-resource`?
>>
>> As noted in https://bugs.openjdk.org/browse/JDK-7022325 this change should prevent leaking of resources in case there's any failure in the test. The test continues to pass with this change.
>
> Jaikiran Pai has updated the pull request incrementally with one additional commit since the last revision:
>
> convert the test to junit
thank you Jai, changes look good
-------------
Marked as reviewed by lancea (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/19492#pullrequestreview-2092169070
More information about the core-libs-dev
mailing list