You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ignite.apache.org by pt...@apache.org on 2023/10/04 08:22:53 UTC

[ignite-3] branch main updated: IGNITE-20510 Fix testConnectionMetrics flakiness (#2659)

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 <pt...@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