From d83395e90cd3e38e81c98f1b723159a9c43f6de0 Mon Sep 17 00:00:00 2001 From: Derek Ho Date: Tue, 11 Jun 2024 11:58:22 -0400 Subject: [PATCH] Fix unit test Signed-off-by: Derek Ho --- .../org/opensearch/security/DoNotFailOnForbiddenTests.java | 1 - .../security/privileges/PrivilegesEvaluatorUnitTest.java | 4 ++-- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/src/integrationTest/java/org/opensearch/security/DoNotFailOnForbiddenTests.java b/src/integrationTest/java/org/opensearch/security/DoNotFailOnForbiddenTests.java index e710f6b5b3..e5a63252e3 100644 --- a/src/integrationTest/java/org/opensearch/security/DoNotFailOnForbiddenTests.java +++ b/src/integrationTest/java/org/opensearch/security/DoNotFailOnForbiddenTests.java @@ -448,7 +448,6 @@ public void shouldPerformCatIndices_positive() throws IOException { public void shouldPerformCatAliases_positive() throws IOException { try (RestHighLevelClient restHighLevelClient = cluster.getRestHighLevelClient(LIMITED_USER)) { Request getAliasesRequest = new Request("GET", "/_cat/aliases"); - // High level client doesn't support _cat/_indices API Response getAliasesResponse = restHighLevelClient.getLowLevelClient().performRequest(getAliasesRequest); List aliases = new BufferedReader(new InputStreamReader(getAliasesResponse.getEntity().getContent())).lines() .collect(Collectors.toList()); diff --git a/src/test/java/org/opensearch/security/privileges/PrivilegesEvaluatorUnitTest.java b/src/test/java/org/opensearch/security/privileges/PrivilegesEvaluatorUnitTest.java index 03ea9fc264..37686ca3c4 100644 --- a/src/test/java/org/opensearch/security/privileges/PrivilegesEvaluatorUnitTest.java +++ b/src/test/java/org/opensearch/security/privileges/PrivilegesEvaluatorUnitTest.java @@ -41,13 +41,13 @@ public class PrivilegesEvaluatorUnitTest { "indices:data/read/search/template", "indices:data/read/tv", "indices:monitor/settings/get", - "indices:monitor/stats" + "indices:monitor/stats", + "indices:admin/aliases/get" ); private static final List disallowedDnfof = ImmutableList.of( "indices:admin/aliases", "indices:admin/aliases/exists", - "indices:admin/aliases/get", "indices:admin/analyze", "indices:admin/cache/clear", "indices:admin/close",