RFR: 8329662: Add a test to verify the behaviour of the default HEAD() method on HttpRequest.Builder
Jaikiran Pai
jpai at openjdk.org
Fri Apr 5 13:33:03 UTC 2024
On Thu, 4 Apr 2024 11:50:32 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:
> Can I please get a review of this test-only change which updates an existing test case to verify the behaviour of `HttpRequest.Builder.HEAD()` method?
>
> As noted in https://bugs.openjdk.org/browse/JDK-8329662, this test now verifies the default implementation specified for `HttpRequest.Builder.HEAD()`. The existing tests were only verifing the behaviour of the `HEAD()` method implementation in the class of the instance returned from `HttpRequest.newBuilder()`.
>
> I have run this test in our CI with these changes and the test continues to pass.
Thank you Daniel for the review.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/18622#issuecomment-2039810460
More information about the net-dev
mailing list