Rm nullable annotations

Author: n1v0lg

x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/security/authz/permission/IndicesPermission.java

@@ -240,7 +240,7 @@ public boolean test(IndexAbstraction indexAbstraction) {
          * The resource must exist. Otherwise, use the {@link #test(String, IndexAbstraction, IndexComponentSelector)} method.
          * Returns {@code true} if access to the given resource is authorized or {@code false} otherwise.
          */
-        public boolean test(IndexAbstraction indexAbstraction, @Nullable IndexComponentSelector selector) {
+        public boolean test(IndexAbstraction indexAbstraction, IndexComponentSelector selector) {
             return test(indexAbstraction.getName(), indexAbstraction, selector);
         }
 
@@ -250,7 +250,7 @@ public boolean test(IndexAbstraction indexAbstraction, @Nullable IndexComponentS
          * if it doesn't.
          * Returns {@code true} if access to the given resource is authorized or {@code false} otherwise.
          */
-        public boolean test(String name, @Nullable IndexAbstraction indexAbstraction, @Nullable IndexComponentSelector selector) {
+        public boolean test(String name, @Nullable IndexAbstraction indexAbstraction, IndexComponentSelector selector) {
             return IndexComponentSelector.FAILURES.equals(selector)
                 ? isAuthorizedForFailuresAccess.test(name, indexAbstraction)
                 : isAuthorizedForDataAccess.test(name, indexAbstraction);

x-pack/plugin/security/src/test/java/org/elasticsearch/xpack/security/authz/accesscontrol/IndicesPermissionTests.java

@@ -974,10 +974,10 @@ public void testResourceAuthorizedPredicateForDatastreams() {
         );
         assertThat(predicate.test(dataStream), is(false));
         // test authorization for a missing resource with the datastream's name
-        assertThat(predicate.test(dataStream.getName(), null, null), is(true));
+        assertThat(predicate.test(dataStream.getName(), null, IndexComponentSelector.DATA), is(true));
         assertThat(predicate.test(backingIndex), is(false));
         // test authorization for a missing resource with the backing index's name
-        assertThat(predicate.test(backingIndex.getName(), null, null), is(true));
+        assertThat(predicate.test(backingIndex.getName(), null, IndexComponentSelector.DATA), is(true));
         assertThat(predicate.test(concreteIndex), is(true));
         assertThat(predicate.test(alias), is(true));
     }