Update test

Signed-off-by: Craig Perkins <[email protected]>
dlin2028 · Dec 18, 2023 · 89deeb6 · 89deeb6
1 parent a4c264a
commit 89deeb6
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 16 deletions.
diff --git a/src/test/java/org/opensearch/security/dlic/rest/api/AbstractApiActionValidationTest.java b/src/test/java/org/opensearch/security/dlic/rest/api/AbstractApiActionValidationTest.java
@@ -71,8 +71,7 @@ public void setup() {
             null,
             restApiAdminPrivilegesEvaluator,
             null,
-            Settings.EMPTY,
-            null
+            Settings.EMPTY
         );
     }
 

diff --git a/...est/java/org/opensearch/security/dlic/rest/api/SecurityConfigApiActionValidationTest.java b/...est/java/org/opensearch/security/dlic/rest/api/SecurityConfigApiActionValidationTest.java
@@ -30,16 +30,7 @@ public void accessHandlerForDefaultSettings() {
         final var securityConfigApiAction = new SecurityConfigApiAction(
             clusterService,
             threadPool,
-            new SecurityApiDependencies(
-                null,
-                configurationRepository,
-                null,
-                null,
-                restApiAdminPrivilegesEvaluator,
-                null,
-                Settings.EMPTY,
-                null
-            )
+            new SecurityApiDependencies(null, configurationRepository, null, null, restApiAdminPrivilegesEvaluator, null, Settings.EMPTY)
         );
         assertTrue(securityConfigApiAction.accessHandler(FakeRestRequest.builder().withMethod(RestRequest.Method.GET).build()));
         assertFalse(securityConfigApiAction.accessHandler(FakeRestRequest.builder().withMethod(RestRequest.Method.PUT).build()));
@@ -58,8 +49,7 @@ public void accessHandlerForUnsupportedSetting() {
                 null,
                 restApiAdminPrivilegesEvaluator,
                 null,
-                Settings.builder().put(SECURITY_UNSUPPORTED_RESTAPI_ALLOW_SECURITYCONFIG_MODIFICATION, true).build(),
-                null
+                Settings.builder().put(SECURITY_UNSUPPORTED_RESTAPI_ALLOW_SECURITYCONFIG_MODIFICATION, true).build()
             )
         );
         assertTrue(securityConfigApiAction.accessHandler(FakeRestRequest.builder().withMethod(RestRequest.Method.GET).build()));
@@ -80,8 +70,7 @@ public void accessHandlerForRestAdmin() {
                         null,
                         restApiAdminPrivilegesEvaluator,
                         null,
-                        Settings.builder().put(SECURITY_RESTAPI_ADMIN_ENABLED, true).build(),
-                        null
+                        Settings.builder().put(SECURITY_RESTAPI_ADMIN_ENABLED, true).build()
                 )
         );
         assertTrue(securityConfigApiAction.accessHandler(FakeRestRequest.builder().withMethod(RestRequest.Method.GET).build()));
-Original file line number
+Diff line change
@@ Expand Up / @@ -71,8 +71,7 @@ public void setup() { @@
                 null,
                 restApiAdminPrivilegesEvaluator,
                 null,
-                Settings.EMPTY,
-                null
+                Settings.EMPTY
             );
         }
@@ Expand Down @@