Adjusted integ tests for when security is enabled. (#1528)

* Adjusted integ tests for when security is enabled. Signed-off-by: AWSHurneyt <[email protected]> * Adjusted integ tests for when security is enabled. Signed-off-by: AWSHurneyt <[email protected]> --------- Signed-off-by: AWSHurneyt <[email protected]> (cherry picked from commit e7e7725) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-project · Apr 30, 2024 · e603cc5 · e603cc5
1 parent 51a0f2a
commit e603cc5
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/alerting/src/test/kotlin/org/opensearch/alerting/transport/GetRemoteIndexesActionIT.kt b/alerting/src/test/kotlin/org/opensearch/alerting/transport/GetRemoteIndexesActionIT.kt
@@ -101,7 +101,9 @@ class GetRemoteIndexesActionIT : AlertingRestTestCase() {
 
             assertNotNull(clusterDetails[ClusterIndexes.INDEXES_FIELD])
             val indexes = clusterDetails[ClusterIndexes.INDEXES_FIELD] as Map<String, Map<String, Any>>
-            assertEquals(expectedNames.size, indexes.keys.size)
+
+            // Skipping this assert when security is enabled as it doesn't consider the existence of system indexes.
+            if (!securityEnabled()) assertEquals(expectedNames.size, indexes.keys.size)
 
             // Validate index-level response details
             expectedNames.forEach { indexName ->
@@ -149,7 +151,9 @@ class GetRemoteIndexesActionIT : AlertingRestTestCase() {
 
             assertNotNull(clusterDetails[ClusterIndexes.INDEXES_FIELD])
             val indexes = clusterDetails[ClusterIndexes.INDEXES_FIELD] as Map<String, Map<String, Any>>
-            assertEquals(expectedNames.size, indexes.keys.size)
+
+            // Skipping this assert when security is enabled as it doesn't consider the existence of system indexes.
+            if (!securityEnabled()) assertEquals(expectedNames.size, indexes.keys.size)
 
             // Validate index-level response details
             expectedNames.forEach { indexName ->
@@ -197,7 +201,8 @@ class GetRemoteIndexesActionIT : AlertingRestTestCase() {
 
             assertNotNull(clusterDetails[ClusterIndexes.INDEXES_FIELD])
             val indexes = clusterDetails[ClusterIndexes.INDEXES_FIELD] as Map<String, Map<String, Any>>
-            assertEquals(expectedNames.size, indexes.keys.size)
+            // Skipping this assert when security is enabled as it doesn't consider the existence of system indexes.
+            if (!securityEnabled()) assertEquals(expectedNames.size, indexes.keys.size)
 
             // Validate index-level response details
             expectedNames.forEach { indexName ->