This is an automated email from the ASF dual-hosted git repository.
markt pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/main by this push:
new 1f0d5cb018 Additional debug logging for intermittent test failure
1f0d5cb018 is described below
commit 1f0d5cb01892442cb68f067a74093c988ecbc48e
Author: Mark Thomas <[email protected]>
AuthorDate: Tue Aug 8 17:39:45 2023 +0100
Additional debug logging for intermittent test failure
---
test/org/apache/catalina/core/TestAsyncContextImpl.java | 11 ++++++++++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/test/org/apache/catalina/core/TestAsyncContextImpl.java
b/test/org/apache/catalina/core/TestAsyncContextImpl.java
index 79c22f3b7e..8158b4bca7 100644
--- a/test/org/apache/catalina/core/TestAsyncContextImpl.java
+++ b/test/org/apache/catalina/core/TestAsyncContextImpl.java
@@ -123,7 +123,12 @@ public class TestAsyncContextImpl extends TomcatBaseTest {
counter++;
}
- Assert.assertEquals("1false2true3true4true5false",
servlet.getResult());
+ String result = servlet.getResult();
+ Assert.assertTrue(result.startsWith("1false2true3true4true5"));
+
+ result = result.substring(22);
+ // Should be "false" (or possibly "") at this point. Must not be
"true".
+ Assert.assertNotEquals("true", servlet.getResult());
// Check the access log
alv.validateAccessLog(1, 200, Bug49528Servlet.THREAD_SLEEP_TIME,
@@ -303,6 +308,10 @@ public class TestAsyncContextImpl extends TomcatBaseTest {
result.append(req.isAsyncStarted());
} catch (NullPointerException npe) {
result.append("false");
+ } catch (Throwable t) {
+ // Additional debugging for intermittent test
failure
+ result.append(t.getClass().getName());
+ t.printStackTrace();
}
done = true;
} catch (InterruptedException | IOException e) {
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
