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

ptupitsyn pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/ignite-3.git


The following commit(s) were added to refs/heads/main by this push:
     new 75901c4412 IGNITE-20510 Fix testConnectionMetrics flakiness (#2659)
75901c4412 is described below

commit 75901c4412d411319b130cfac6ca9324ca6b20bc
Author: Pavel Tupitsyn <ptupit...@apache.org>
AuthorDate: Wed Oct 4 11:22:47 2023 +0300

    IGNITE-20510 Fix testConnectionMetrics flakiness (#2659)
---
 .../src/test/java/org/apache/ignite/client/ClientMetricsTest.java    | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git 
a/modules/client/src/test/java/org/apache/ignite/client/ClientMetricsTest.java 
b/modules/client/src/test/java/org/apache/ignite/client/ClientMetricsTest.java
index 06fc4e5ef7..185f303c9c 100644
--- 
a/modules/client/src/test/java/org/apache/ignite/client/ClientMetricsTest.java
+++ 
b/modules/client/src/test/java/org/apache/ignite/client/ClientMetricsTest.java
@@ -82,8 +82,11 @@ public class ClientMetricsTest extends 
BaseIgniteAbstractTest {
                 IgniteTestUtils.waitForCondition(() -> 
metrics.connectionsActive() == 0, 1000),
                 () -> "connectionsActive: " + metrics.connectionsActive());
 
+        assertTrue(
+                IgniteTestUtils.waitForCondition(() -> 
metrics.connectionsLost() == (gracefulDisconnect ? 0 : 1), 1000),
+                () -> "connectionsLost: " + metrics.connectionsLost());
+
         assertEquals(1, metrics.connectionsEstablished());
-        assertEquals(gracefulDisconnect ? 0 : 1, metrics.connectionsLost());
     }
 
     @Test

Reply via email to