RFR: 8304014: Convert test/jdk/java/util/zip/ZipFile/CorruptedZipFiles.java to testNG [v2]

Eirik Bjorsnos duke at openjdk.org
Wed Mar 29 14:04:49 UTC 2023


On Mon, 27 Feb 2023 21:13:47 GMT, Lance Andersen <lancea at openjdk.org> wrote:

>> Eirik Bjorsnos has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains ten additional commits since the last revision:
>> 
>>  - Replace the u8, u16, u32 methods with using a little-endian ByteBuffer. Collapse the checkZipException and checkZipExceptionInGetInputStream into one method by always consuming the InputStream. Add a block comment for the assertZipException method.
>>  - Use block comments instead of javadoc comments
>>  - Merge branch 'master' into corrupted-zip-files-ng
>>  - Merge branch 'master' into corrupted-zip-files-ng
>>  - Give the @BeforeMethod and @AfterMethod more meaningful names
>>  - Improve comments and method names to help future maintainers understand what these tests verify.
>>  - Merge branch 'master' into corrupted-zip-files-ng
>>  - Trim whitespace and fix some spelling
>>  - Convert test CorruptedZipFiles to testNG and delete ZipFile constants copies which are now obsolete.
>
> Hi Eirik,
> 
> Thank you for  your suggested changes to this test.
> 
> I think if we are going to re-work this test, we should go further including improving the comments for future maintainers
> 
> I made a quick pass and some initial thoughts are below
> 
> Best
> Lance

> @LanceAndersen Do you have any opinion on junit/testNG for tests like this?

Here's a junit version for consideration:

https://github.com/eirbjo/jdk/blob/corrupted-zip-files-ng-junit/test/jdk/java/util/zip/ZipFile/CorruptedZipFiles.java

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

PR Comment: https://git.openjdk.org/jdk/pull/12563#issuecomment-1488678652


More information about the core-libs-dev mailing list