diff --git a/elasticsearch/x-pack/security/src/test/java/org/elasticsearch/xpack/security/authz/indicesresolver/DefaultIndicesResolverTests.java b/elasticsearch/x-pack/security/src/test/java/org/elasticsearch/xpack/security/authz/indicesresolver/DefaultIndicesResolverTests.java index 85687d32998..ae66de55286 100644 --- a/elasticsearch/x-pack/security/src/test/java/org/elasticsearch/xpack/security/authz/indicesresolver/DefaultIndicesResolverTests.java +++ b/elasticsearch/x-pack/security/src/test/java/org/elasticsearch/xpack/security/authz/indicesresolver/DefaultIndicesResolverTests.java @@ -22,7 +22,6 @@ import org.elasticsearch.action.search.SearchRequest; import org.elasticsearch.action.support.IndicesOptions; import org.elasticsearch.client.Requests; import org.elasticsearch.cluster.ClusterState; -import org.elasticsearch.cluster.metadata.AliasAction; import org.elasticsearch.cluster.metadata.AliasMetaData; import org.elasticsearch.cluster.metadata.IndexMetaData; import org.elasticsearch.cluster.metadata.IndexNameExpressionResolver; @@ -382,8 +381,8 @@ public class DefaultIndicesResolverTests extends ESTestCase { public void testResolveIndicesAliasesRequestDeleteActions() { IndicesAliasesRequest request = new IndicesAliasesRequest(); - request.addAliasAction(AliasAction.newRemoveAliasAction("foo", "foofoobar")); - request.addAliasAction(AliasAction.newRemoveAliasAction("foofoo", "barbaz")); + request.addAliasAction(AliasActions.remove().index("foo").alias("foofoobar")); + request.addAliasAction(AliasActions.remove().index("foofoo").alias("barbaz")); Set indices = defaultIndicesResolver.resolve(user, IndicesAliasesAction.NAME, request, metaData); //the union of all indices and aliases gets returned String[] expectedIndices = new String[]{"foo", "foofoobar", "foofoo", "barbaz"}; @@ -397,8 +396,8 @@ public class DefaultIndicesResolverTests extends ESTestCase { public void testResolveIndicesAliasesRequestDeleteActionsMissingIndex() { IndicesAliasesRequest request = new IndicesAliasesRequest(); - request.addAliasAction(AliasAction.newRemoveAliasAction("foo", "foofoobar")); - request.addAliasAction(AliasAction.newRemoveAliasAction("missing_index", "missing_alias")); + request.addAliasAction(AliasActions.remove().index("foo").alias("foofoobar")); + request.addAliasAction(AliasActions.remove().index("missing_index").alias("missing_alias")); Set indices = defaultIndicesResolver.resolve(user, IndicesAliasesAction.NAME, request, metaData); //the union of all indices and aliases gets returned, doesn't matter is some of them don't exist String[] expectedIndices = new String[]{"foo", "foofoobar", "missing_index", "missing_alias"}; @@ -412,8 +411,8 @@ public class DefaultIndicesResolverTests extends ESTestCase { public void testResolveWildcardsIndicesAliasesRequestDeleteActions() { IndicesAliasesRequest request = new IndicesAliasesRequest(); - request.addAliasAction(AliasAction.newRemoveAliasAction("foo*", "foofoobar")); - request.addAliasAction(AliasAction.newRemoveAliasAction("bar*", "barbaz")); + request.addAliasAction(AliasActions.remove().index("foo*").alias("foofoobar")); + request.addAliasAction(AliasActions.remove().index("bar*").alias("barbaz")); Set indices = defaultIndicesResolver.resolve(user, IndicesAliasesAction.NAME, request, metaData); //union of all resolved indices and aliases gets returned, based on what user is authorized for String[] expectedIndices = new String[]{"foofoobar", "foofoo", "bar", "barbaz"}; @@ -428,8 +427,8 @@ public class DefaultIndicesResolverTests extends ESTestCase { public void testResolveAliasesWildcardsIndicesAliasesRequestDeleteActions() { IndicesAliasesRequest request = new IndicesAliasesRequest(); - request.addAliasAction(AliasAction.newRemoveAliasAction("*", "foo*")); - request.addAliasAction(AliasAction.newRemoveAliasAction("*bar", "foo*")); + request.addAliasAction(AliasActions.remove().index("*").alias("foo*")); + request.addAliasAction(AliasActions.remove().index("*bar").alias("foo*")); Set indices = defaultIndicesResolver.resolve(user, IndicesAliasesAction.NAME, request, metaData); //union of all resolved indices and aliases gets returned, based on what user is authorized for //note that the index side will end up containing matching aliases too, which is fine, as es core would do @@ -446,8 +445,8 @@ public class DefaultIndicesResolverTests extends ESTestCase { public void testResolveAllAliasesWildcardsIndicesAliasesRequestDeleteActions() { IndicesAliasesRequest request = new IndicesAliasesRequest(); - request.addAliasAction(AliasAction.newRemoveAliasAction("*", "_all")); - request.addAliasAction(new IndicesAliasesRequest.AliasActions(AliasAction.Type.REMOVE, "_all", new String[]{"_all", "explicit"})); + request.addAliasAction(AliasActions.remove().index("*").alias("_all")); + request.addAliasAction(AliasActions.remove().index("_all").aliases("_all", "explicit")); Set indices = defaultIndicesResolver.resolve(user, IndicesAliasesAction.NAME, request, metaData); //union of all resolved indices and aliases gets returned, based on what user is authorized for //note that the index side will end up containing matching aliases too, which is fine, as es core would do @@ -464,9 +463,9 @@ public class DefaultIndicesResolverTests extends ESTestCase { public void testResolveAliasesWildcardsIndicesAliasesRequestDeleteActionsNoAuthorizedIndices() { IndicesAliasesRequest request = new IndicesAliasesRequest(); - request.addAliasAction(AliasAction.newRemoveAliasAction("foo*", "foo*")); + request.addAliasAction(AliasActions.remove().index("foo*").alias("foo*")); //no authorized aliases match bar*, hence this action fails and makes the whole request fail - request.addAliasAction(AliasAction.newRemoveAliasAction("*bar", "bar*")); + request.addAliasAction(AliasActions.remove().index("*bar").alias("bar*")); try { defaultIndicesResolver.resolve(user, IndicesAliasesAction.NAME, request, metaData); fail("Expected IndexNotFoundException"); @@ -477,8 +476,8 @@ public class DefaultIndicesResolverTests extends ESTestCase { public void testResolveWildcardsIndicesAliasesRequestAddAndDeleteActions() { IndicesAliasesRequest request = new IndicesAliasesRequest(); - request.addAliasAction(AliasAction.newRemoveAliasAction("foo*", "foofoobar")); - request.addAliasAction(AliasAction.newAddAliasAction("bar*", "foofoobar")); + request.addAliasAction(AliasActions.remove().index("foo*").alias("foofoobar")); + request.addAliasAction(AliasActions.add().index("bar*").alias("foofoobar")); Set indices = defaultIndicesResolver.resolve(user, IndicesAliasesAction.NAME, request, metaData); //union of all resolved indices and aliases gets returned, based on what user is authorized for String[] expectedIndices = new String[]{"foofoobar", "foofoo", "bar"};