On Fri, 17 Jan 2025 15:38:16 GMT, Daniel Fuchs <dfu...@openjdk.org> wrote:
> The jdk/java/net/httpclient/offline/FixedResponseHttpClient.java test class
> makes unreasonable assumptions on BodyPublisher implementation. There's no
> guarantee that all buffers will be available until the subscriber is
> completed.
> The fix is to way for completion before getting the consumed buffers.
test/jdk/java/net/httpclient/offline/FixedResponseHttpClient.java line 258:
> 256: public final CompletableFuture<List<ByteBuffer>> getBuffers() {
> 257: return consumed;
> 258: }
Is this method really needed? Above you can replace
`subscriber.getBuffers().join()` with `subscriber.consumed.join()` too.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23176#discussion_r1920404423
