From 33e35df54400d82857f1d0f1361b4689c8756cb8 Mon Sep 17 00:00:00 2001
From: Simone Bordet <simone.bordet@gmail.com>
Date: Fri, 20 Oct 2023 16:36:04 +0200
Subject: [PATCH] Removed TODO in ClientServerTest, adding a more explicative
 comment.

Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
---
 .../java/org/eclipse/jetty/http3/tests/ClientServerTest.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/jetty-http3/http3-tests/src/test/java/org/eclipse/jetty/http3/tests/ClientServerTest.java b/jetty-http3/http3-tests/src/test/java/org/eclipse/jetty/http3/tests/ClientServerTest.java
index d17576398cb..44504ddc337 100644
--- a/jetty-http3/http3-tests/src/test/java/org/eclipse/jetty/http3/tests/ClientServerTest.java
+++ b/jetty-http3/http3-tests/src/test/java/org/eclipse/jetty/http3/tests/ClientServerTest.java
@@ -519,10 +519,10 @@ public class ClientServerTest extends AbstractClientServerTest
                     @Override
                     public void onDataAvailable(Stream.Server stream)
                     {
-                        // TODO: we should not be needing this!!!
+                        // Calling readData() triggers the read+parse
+                        // of the trailer, and returns no data.
                         Stream.Data data = stream.readData();
                         assertNull(data);
-                        stream.demand();
                     }
 
                     @Override