tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ma...@apache.org
Subject [tomcat] branch 7.0.x updated: Fix potential test failure / hang if timing isn't as expected
Date Mon, 25 Nov 2019 13:17:01 GMT
This is an automated email from the ASF dual-hosted git repository.

markt pushed a commit to branch 7.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git


The following commit(s) were added to refs/heads/7.0.x by this push:
     new 609d102  Fix potential test failure / hang if timing isn't as expected
609d102 is described below

commit 609d10230ad1cb67c9745764ccee98450df89880
Author: Mark Thomas <markt@apache.org>
AuthorDate: Mon Nov 25 11:36:33 2019 +0000

    Fix potential test failure / hang if timing isn't as expected
---
 test/org/apache/catalina/core/TestAsyncContextStateChanges.java | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/test/org/apache/catalina/core/TestAsyncContextStateChanges.java b/test/org/apache/catalina/core/TestAsyncContextStateChanges.java
index 9c7e723..3e5cb46 100644
--- a/test/org/apache/catalina/core/TestAsyncContextStateChanges.java
+++ b/test/org/apache/catalina/core/TestAsyncContextStateChanges.java
@@ -85,6 +85,7 @@ public class TestAsyncContextStateChanges extends TomcatBaseTest {
     private ServletRequest servletRequest = null;
     private AsyncContext asyncContext = null;
     private AtomicBoolean failed = new AtomicBoolean();
+    private CountDownLatch servletLatch;
     private CountDownLatch threadLatch;
     private CountDownLatch closeLatch;
     private CountDownLatch endLatch;
@@ -98,6 +99,7 @@ public class TestAsyncContextStateChanges extends TomcatBaseTest {
 
         // Initialise tracking fields
         failed.set(true);
+        servletLatch = new CountDownLatch(1);
         threadLatch = new CountDownLatch(1);
         closeLatch = new CountDownLatch(1);
         endLatch = new CountDownLatch(1);
@@ -123,6 +125,9 @@ public class TestAsyncContextStateChanges extends TomcatBaseTest {
         client.connect();
         client.sendRequest();
 
+        // Wait for Servlet to start processing request
+        servletLatch.await();
+
         if (asyncEnd.isError()) {
             client.disconnect();
             closeLatch.countDown();
@@ -155,6 +160,8 @@ public class TestAsyncContextStateChanges extends TomcatBaseTest {
         @Override
         protected void doGet(HttpServletRequest req, HttpServletResponse resp)
                 throws ServletException, IOException {
+            servletLatch.countDown();
+
             if (dispatch) {
                 return;
             }


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org


Mime
View raw message