On Thu, 4 Apr 2024 11:50:32 GMT, Jaikiran Pai <[email protected]> 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. This pull request has now been integrated. Changeset: 4a11db8b Author: Jaikiran Pai <[email protected]> URL: https://git.openjdk.org/jdk/commit/4a11db8b606f2b10f48f0b45335b661fe3095fc4 Stats: 86 lines in 1 file changed: 85 ins; 0 del; 1 mod 8329662: Add a test to verify the behaviour of the default HEAD() method on HttpRequest.Builder Reviewed-by: dfuchs ------------- PR: https://git.openjdk.org/jdk/pull/18622
