Nits

n1v0lg · Sep 13, 2024 · 3bb23ea · 3bb23ea
1 parent e994721
commit 3bb23ea
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/...terTest/java/org/elasticsearch/xpack/security/support/SecurityIndexManagerIntegTests.java b/...terTest/java/org/elasticsearch/xpack/security/support/SecurityIndexManagerIntegTests.java
@@ -122,6 +122,7 @@ public void testWhenIndexAvailableForSearchIndexCompletesWithinTimeout() throws
 
         assertSecurityIndexActive();
 
+        // security index creation is complete and index is active; therefore whenIndexAvailableForSearch should report success
         future.actionGet();
     }
 

diff --git a/x-pack/plugin/security/src/test/java/org/elasticsearch/test/SecurityIntegTestCase.java b/x-pack/plugin/security/src/test/java/org/elasticsearch/test/SecurityIntegTestCase.java
@@ -391,7 +391,7 @@ public void assertSecurityIndexActive(TestCluster testCluster) throws Exception
                 ClusterState clusterState = client.admin().cluster().prepareState(TEST_REQUEST_TIMEOUT).setLocal(true).get().getState();
                 assertFalse(clusterState.blocks().hasGlobalBlock(GatewayService.STATE_NOT_RECOVERED_BLOCK));
                 Index securityIndex = resolveSecurityIndex(clusterState.metadata());
-                // TODO this is bug -- since we are not tripping assertions here, this will complete successfully even if the security
+                // TODO this is a bug -- since we are not tripping assertions here, this will complete successfully even if the security
                 // index does not exist
                 if (securityIndex != null) {
                     IndexRoutingTable indexRoutingTable = clusterState.routingTable().index(securityIndex);