[tomcat] 03/03: MacOS seems to need longer for the test to complete

[markt]

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

commit 4839d6997992f2615eb36b371c88745a43ac2ad7
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Tue May 9 20:41:53 2023 +0100

    MacOS seems to need longer for the test to complete
---
 test/org/apache/catalina/filters/TestRateLimitFilter.java | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/test/org/apache/catalina/filters/TestRateLimitFilter.java 
b/test/org/apache/catalina/filters/TestRateLimitFilter.java
index bf6c397972..b47e67ed60 100644
--- a/test/org/apache/catalina/filters/TestRateLimitFilter.java
+++ b/test/org/apache/catalina/filters/TestRateLimitFilter.java
@@ -71,7 +71,14 @@ public class TestRateLimitFilter extends TomcatBaseTest {
         TestClient tc3 = new TestClient(rateLimitFilter, filterChain, 
"10.20.20.20", 200, 20);
         TestClient tc4 = new TestClient(rateLimitFilter, filterChain, 
"10.20.20.40", 200, 40);
 
-        Thread.sleep(5000);
+        // Sleep for up to 10s for clients to complete
+        int count = 0;
+        while (count < 100 && (tc1.results[24] == 0 || tc2.results[49] == 0 || 
tc3.results[allowedRequests - 1] == 0 ||
+                tc3.results[allowedRequests] == 0 || 
tc3.results[allowedRequests - 1] == 0 ||
+                tc4.results[allowedRequests] == 0)) {
+            Thread.sleep(100);
+            count++;
+        }
 
         Assert.assertEquals(200, tc1.results[24]); // only 25 requests made, 
all allowed
 


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