[GitHub] [ignite] oleg-ostanin commented on a change in pull request #7671: IGNITE-12891 Add userAttributes map to all GridClient messages

[GitBox <gi...@apache.org>]

oleg-ostanin commented on a change in pull request #7671: IGNITE-12891 Add userAttributes map to all GridClient messages
URL: https://github.com/apache/ignite/pull/7671#discussion_r409334647
 
 

 ##########
 File path: modules/core/src/test/java/org/apache/ignite/internal/processors/security/client/AdditionalSecurityCheckTest.java
 ##########
 @@ -250,6 +252,29 @@ public void testClientInfoIgniteClientFail() throws Exception {
         }
     }
 
+    /**
+     *
+     */
+    @Test
+    public void testClientInfoGridClientNotFail() throws Exception {
+        Ignite ignite = startGrids(2);
+
+        assertEquals(2, ignite.cluster().topologyVersion());
+
+        ignite.cluster().state(ClusterState.ACTIVE);
+
+        try (GridClient client = GridClientFactory.start(getGridClientConfiguration())) {
+            assertTrue(client.connected());
+
+            GridClientClusterState state = client.state();
+
+            // Close a coordinator to force the client to send a CLUSTER_CURRENT_STATE message to the other node.
 
 Review comment:
   added this to the comment as you suggested

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services