You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2021/07/09 16:37:54 UTC

[tomcat] branch 8.5.x updated: Possible fix for intermittent unit test failure

This is an automated email from the ASF dual-hosted git repository.

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


The following commit(s) were added to refs/heads/8.5.x by this push:
     new 47653e6  Possible fix for intermittent unit test failure
47653e6 is described below

commit 47653e68fb97c4322b94d6e5e67bfb3d600e2dd5
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Fri Jul 9 17:36:32 2021 +0100

    Possible fix for intermittent unit test failure
---
 test/org/apache/tomcat/websocket/server/TestSlowClient.java | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/test/org/apache/tomcat/websocket/server/TestSlowClient.java b/test/org/apache/tomcat/websocket/server/TestSlowClient.java
index 0978dbc..084d13a 100644
--- a/test/org/apache/tomcat/websocket/server/TestSlowClient.java
+++ b/test/org/apache/tomcat/websocket/server/TestSlowClient.java
@@ -74,6 +74,11 @@ public class TestSlowClient extends WebSocketBaseTest {
         // BZ 64848 (non-container thread variant)
         // Confirm there are no waiting processors
         AbstractProtocol<?> protocol = (AbstractProtocol<?>) tomcat.getConnector().getProtocolHandler();
+        count = 0;
+        while (protocol.getWaitingProcessorCount() > 0 && count < 200) {
+            Thread.sleep(100);
+            count++;
+        }
         Assert.assertEquals(0, protocol.getWaitingProcessorCount());
     }
 

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