Skip to content

Commit e1a5b3f

Browse files
cwperksdlin2028
authored andcommitted
Update test
Signed-off-by: Craig Perkins <cwperx@amazon.com>
1 parent e595452 commit e1a5b3f

File tree

2 files changed

+4
-16
lines changed

2 files changed

+4
-16
lines changed

src/test/java/org/opensearch/security/dlic/rest/api/AbstractApiActionValidationTest.java

+1-2
Original file line numberDiff line numberDiff line change
@@ -71,8 +71,7 @@ public void setup() {
7171
null,
7272
restApiAdminPrivilegesEvaluator,
7373
null,
74-
Settings.EMPTY,
75-
null
74+
Settings.EMPTY
7675
);
7776
}
7877

src/test/java/org/opensearch/security/dlic/rest/api/SecurityConfigApiActionValidationTest.java

+3-14
Original file line numberDiff line numberDiff line change
@@ -30,16 +30,7 @@ public void accessHandlerForDefaultSettings() {
3030
final var securityConfigApiAction = new SecurityConfigApiAction(
3131
clusterService,
3232
threadPool,
33-
new SecurityApiDependencies(
34-
null,
35-
configurationRepository,
36-
null,
37-
null,
38-
restApiAdminPrivilegesEvaluator,
39-
null,
40-
Settings.EMPTY,
41-
null
42-
)
33+
new SecurityApiDependencies(null, configurationRepository, null, null, restApiAdminPrivilegesEvaluator, null, Settings.EMPTY)
4334
);
4435
assertTrue(securityConfigApiAction.accessHandler(FakeRestRequest.builder().withMethod(RestRequest.Method.GET).build()));
4536
assertFalse(securityConfigApiAction.accessHandler(FakeRestRequest.builder().withMethod(RestRequest.Method.PUT).build()));
@@ -58,8 +49,7 @@ public void accessHandlerForUnsupportedSetting() {
5849
null,
5950
restApiAdminPrivilegesEvaluator,
6051
null,
61-
Settings.builder().put(SECURITY_UNSUPPORTED_RESTAPI_ALLOW_SECURITYCONFIG_MODIFICATION, true).build(),
62-
null
52+
Settings.builder().put(SECURITY_UNSUPPORTED_RESTAPI_ALLOW_SECURITYCONFIG_MODIFICATION, true).build()
6353
)
6454
);
6555
assertTrue(securityConfigApiAction.accessHandler(FakeRestRequest.builder().withMethod(RestRequest.Method.GET).build()));
@@ -80,8 +70,7 @@ public void accessHandlerForRestAdmin() {
8070
null,
8171
restApiAdminPrivilegesEvaluator,
8272
null,
83-
Settings.builder().put(SECURITY_RESTAPI_ADMIN_ENABLED, true).build(),
84-
null
73+
Settings.builder().put(SECURITY_RESTAPI_ADMIN_ENABLED, true).build()
8574
)
8675
);
8776
assertTrue(securityConfigApiAction.accessHandler(FakeRestRequest.builder().withMethod(RestRequest.Method.GET).build()));

0 commit comments

Comments
 (0)