Fix Ice/maxConnection test failures (#3135)

zeroc-ice · Nov 12, 2024 · c2f5cd1 · c2f5cd1
1 parent d71ced7
commit c2f5cd1
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 2 deletions.
diff --git a/cpp/test/Ice/maxConnections/AllTests.cpp b/cpp/test/Ice/maxConnections/AllTests.cpp
@@ -112,6 +112,11 @@ testCreateConnectionsWithMaxAndRecovery(TestIntfPrx p, int max, function<void()>
     {
         postCloseDelay();
     }
+    else
+    {
+        // We need to wait a tiny bit to let the server remove the connection from its incoming connection factory.
+        this_thread::sleep_for(10ms);
+    }
 
     // Try again
     try

diff --git a/csharp/test/Ice/maxConnections/AllTests.cs b/csharp/test/Ice/maxConnections/AllTests.cs
@@ -136,8 +136,7 @@ private static async Task testCreateConnectionsWithMaxAndRecovery(
         }
         else
         {
-            // We need to wait a tiny bit to let the server remove the connection from its incoming connection
-            // factory.
+            // We need to wait a tiny bit to let the server remove the connection from its incoming connection factory.
             await Task.Delay(TimeSpan.FromMilliseconds(10));
         }
 

diff --git a/java/test/src/main/java/test/Ice/maxConnections/AllTests.java b/java/test/src/main/java/test/Ice/maxConnections/AllTests.java
@@ -134,6 +134,14 @@ private static void testCreateConnectionsWithMaxAndRecovery(
 
         if (postCloseDelay != null) {
             postCloseDelay.run();
+        } else {
+            // We need to wait a tiny bit to let the server remove the connection from its incoming
+            // connection factory.
+            try {
+                Thread.sleep(10);
+            } catch (InterruptedException ex) {
+                // ignore
+            }
         }
 
         // Try again