RFR: 8343074: test/jdk/com/sun/net/httpserver/docs/test1/largefile.txt could be generated [v2]

Volkan Yazici vyazici at openjdk.org
Mon Feb 3 09:28:54 UTC 2025


On Sun, 2 Feb 2025 13:20:39 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

>> Volkan Yazici has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Remove `assertFileContentsEqual()`
>
> test/lib/jdk/test/lib/Asserts.java line 623:
> 
>> 621:      * @throws IOException on I/O failures
>> 622:      */
>> 623:     public static void assertFileContentsEqual(Path f1, Path f2) throws IOException {
> 
> Hello Volkan, is this new method needed? Can its call sites instead be replaced with `java.nio.file.Files.mismatch(...)` call?

I thought it reports more useful diagnostics compared to the earlier `file compare failed` message. Nevertheless, replaced it with `Files::mismatch` in 421d19d468d62bdb04aee458c72a338a5c053e73.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23401#discussion_r1939052590


More information about the core-libs-dev mailing list