RFR: 8377675: java.net.http tests should not depend on ../../../com/sun/net/httpserver test classes [v5]
Daniel Fuchs
dfuchs at openjdk.org
Fri Feb 13 13:20:06 UTC 2026
On Fri, 13 Feb 2026 10:37:12 GMT, Daniel Jeliński <djelinski at openjdk.org> wrote:
>> Daniel Fuchs 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 14 additional commits since the last revision:
>>
>> - Merge branch 'master' into sanitize-echo-handlers-8377675
>> - Review feedback: removed HttpTestExchange.chunkedRsp(long)
>> - Update test/jdk/java/net/httpclient/lib/jdk/httpclient/test/lib/common/HttpServerAdapters.java
>>
>> Co-authored-by: Daniel Jelinski <djelinski1 at gmail.com>
>> - Update test/jdk/java/net/httpclient/lib/jdk/httpclient/test/lib/common/HttpServerAdapters.java
>>
>> Co-authored-by: Daniel Jelinski <djelinski1 at gmail.com>
>> - Update test/jdk/java/net/httpclient/lib/jdk/httpclient/test/lib/common/HttpServerAdapters.java
>>
>> Co-authored-by: Daniel Jelinski <djelinski1 at gmail.com>
>> - Update test/jdk/java/net/httpclient/lib/jdk/httpclient/test/lib/common/HttpServerAdapters.java
>>
>> Co-authored-by: Daniel Jelinski <djelinski1 at gmail.com>
>> - Update test/jdk/java/net/httpclient/lib/jdk/httpclient/test/lib/common/HttpServerAdapters.java
>>
>> Co-authored-by: Daniel Jelinski <djelinski1 at gmail.com>
>> - Update test/jdk/java/net/httpclient/lib/jdk/httpclient/test/lib/common/HttpServerAdapters.java
>>
>> Co-authored-by: Daniel Jelinski <djelinski1 at gmail.com>
>> - Update test/jdk/java/net/httpclient/lib/jdk/httpclient/test/lib/common/HttpServerAdapters.java
>>
>> Co-authored-by: Daniel Jelinski <djelinski1 at gmail.com>
>> - Update test/jdk/java/net/httpclient/lib/jdk/httpclient/test/lib/common/HttpServerAdapters.java
>>
>> Co-authored-by: Daniel Jelinski <djelinski1 at gmail.com>
>> - ... and 4 more: https://git.openjdk.org/jdk/compare/4d3b9844...71f9ea91
>
> test/jdk/java/net/httpclient/lib/jdk/httpclient/test/lib/common/HttpServerAdapters.java line 281:
>
>> 279: * This method always return {@link #RSPBODY_CHUNKED}
>> 280: */
>> 281: public static long chunkedRsp(long bytes) {
>
> This method has limited utility; chunked response is usually used when the content length is not known. Inline?
Removed
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/29702#discussion_r2804173733
More information about the net-dev
mailing list