RFR: 8304014: Convert test/jdk/java/util/zip/ZipFile/CorruptedZipFiles.java to junit [v5]
Eirik Bjorsnos
duke at openjdk.org
Fri Mar 31 22:37:24 UTC 2023
On Fri, 31 Mar 2023 18:24:26 GMT, Martin Buchholz <martin at openjdk.org> wrote:
>> Eirik Bjorsnos has updated the pull request incrementally with four additional commits since the last revision:
>>
>> - Document the structure of the 'good' ZIP
>> - Use the "Validate that.." comment style
>> - Document the ByteBuffer with a comment
>> - Use ByteBuffer when manipulating multi-byte fields
>
> test/jdk/java/util/zip/ZipFile/CorruptedZipFiles.java line 246:
>
>> 244: /*
>> 245: * Validate that a ZipException is thrown if the last CEN header is
>> 246: * not immediatly followed by the start of the 'End of central directory' record
>
> typo: immediatly
Thanks, fixed.
> test/jdk/java/util/zip/ZipFile/CorruptedZipFiles.java line 300:
>
>> 298:
>> 299: /*
>> 300: * Validate that a ZipException is thrown when a LOC header has an unexpected signature
>
> I would leave off "Validate that"
Thanks, updated all test comments to strip this shared prefix.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/12563#discussion_r1154844736
PR Review Comment: https://git.openjdk.org/jdk/pull/12563#discussion_r1154844995
More information about the core-libs-dev
mailing list