[UNDERTOW-2400] At AsyncResponseWriterOnPostServlet for servlet tests…

…, configure a bigger timeout in the async context, to prevent ci failures caused by slowness in ResponseWriterTestCase.testAsyncRespondBeforeRead Also, clear the exception field in the superclass ResponseWriterOnPostServlet after reading it, to prevent a non null exception from being read twice, which could cause false test failures Signed-off-by: Flavia Rainone <[email protected]>
undertow-io · Jun 13, 2024 · 786f839 · 786f839
1 parent a41521a
commit 786f839
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/.../test/java/io/undertow/servlet/test/response/writer/AsyncResponseWriterOnPostServlet.java b/.../test/java/io/undertow/servlet/test/response/writer/AsyncResponseWriterOnPostServlet.java
@@ -39,6 +39,7 @@ protected void doPost(final HttpServletRequest req, final HttpServletResponse re
             throw new IllegalArgumentException("not a test " + test);
         }
         final AsyncContext asyncContext = req.startAsync();
+        asyncContext.setTimeout(50000);
         new Thread(()->{
             try {
                 HttpServletResponse response = (HttpServletResponse) asyncContext.getResponse();

diff --git a/...t/src/test/java/io/undertow/servlet/test/response/writer/ResponseWriterOnPostServlet.java b/...t/src/test/java/io/undertow/servlet/test/response/writer/ResponseWriterOnPostServlet.java
@@ -58,6 +58,8 @@ protected void doPost(final HttpServletRequest req, final HttpServletResponse re
     }
 
     public static Throwable getExceptionIfAny() {
-        return exception;
+        Throwable result = exception;
+        exception = null;
+        return result;
     }
 }