From e97f5b8e3b2d0a686bee33191fe4d35c0f867e83 Mon Sep 17 00:00:00 2001 From: "opensearch-trigger-bot[bot]" <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com> Date: Tue, 18 Jun 2024 19:15:53 -0700 Subject: [PATCH] fix integ test failures (#1580) (#1581) * fix integ test failures in jenkins infra * fix integ test failures in jenkins infra --------- (cherry picked from commit 8801a635cc8736cb9214b3ae9f6a55be65fb57cf) Signed-off-by: Riya Saxena Signed-off-by: github-actions[bot] Co-authored-by: github-actions[bot] --- .../alerting/transport/GetRemoteIndexesActionIT.kt | 8 -------- 1 file changed, 8 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 index 0b9282d10..83e24332e 100644 --- a/alerting/src/test/kotlin/org/opensearch/alerting/transport/GetRemoteIndexesActionIT.kt +++ b/alerting/src/test/kotlin/org/opensearch/alerting/transport/GetRemoteIndexesActionIT.kt @@ -102,9 +102,6 @@ class GetRemoteIndexesActionIT : AlertingRestTestCase() { assertNotNull(clusterDetails[ClusterIndexes.INDEXES_FIELD]) val indexes = clusterDetails[ClusterIndexes.INDEXES_FIELD] as Map> - // 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 -> assertNotNull(indexes[indexName]) @@ -152,9 +149,6 @@ class GetRemoteIndexesActionIT : AlertingRestTestCase() { assertNotNull(clusterDetails[ClusterIndexes.INDEXES_FIELD]) val indexes = clusterDetails[ClusterIndexes.INDEXES_FIELD] as Map> - // 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 -> assertNotNull(indexes[indexName]) @@ -201,8 +195,6 @@ class GetRemoteIndexesActionIT : AlertingRestTestCase() { assertNotNull(clusterDetails[ClusterIndexes.INDEXES_FIELD]) val indexes = clusterDetails[ClusterIndexes.INDEXES_FIELD] as Map> - // 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 ->