RFR: 8343074: test/jdk/com/sun/net/httpserver/docs/test1/largefile.txt could be generated [v6]
Jaikiran Pai
jpai at openjdk.org
Fri Feb 7 15:10:13 UTC 2025
On Tue, 4 Feb 2025 14:26:23 GMT, Volkan Yazici <vyazici at openjdk.org> wrote:
>> Adds `test.lib.Utils::createTempFileOfSize` to generate `test/jdk/com/sun/net/httpserver/docs` contents at runtime. This directory contains `largefile.txt` of size 2.6MiB showing up as the 4th largest file tracked by git:
>>
>>
>> $ git ls-files | while read f; do echo -e $(stat -c %s "$f")"\t$f"; done >/tmp/trackedFileSizes.txt
>> $ sort -n /tmp/trackedFileSizes.txt | tail -n 4
>> 2730088 test/jdk/com/sun/net/httpserver/docs/test1/largefile.txt
>> 2798680 src/java.base/share/data/unicodedata/NormalizationTest.txt
>> 3574947 test/jdk/java/foreign/libTestDowncallStack.c
>> 7128495 test/jdk/java/foreign/libTestUpcallStack.c
>>
>>
>> **Other highlights:**
>>
>> - `jdk.httpclient.test.lib.common.TestUtil` is removed in favor of similar alternatives in `test.lib.Utils` and `test.lib.Asserts`
>> - `test.lib.Asserts::assertFileContentsEqual` is added
>
> Volkan Yazici has updated the pull request incrementally with one additional commit since the last revision:
>
> Add missing `assertFileContentsEqual` replacement to `FixedThreadPoolTest`
The updated changes look good to me.
-------------
Marked as reviewed by jpai (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/23401#pullrequestreview-2601992489
More information about the core-libs-dev
mailing list