Skip to content

Commit 63ae895

Browse files
sliceofapplepienvazquez
authored andcommitted
fix cachePeriodField illegal arg
1 parent 8e04b15 commit 63ae895

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

plugins/acl/dynamic-role-based/src/test/java/org/apache/cloudstack/acl/DynamicRoleBasedAPIAccessCheckerTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -232,7 +232,7 @@ public void testCheckAccessAccountUsesCachedPermissions() throws Exception {
232232
// Enable caching by setting a positive cachePeriod
233233
Field cachePeriodField = DynamicRoleBasedAPIAccessChecker.class.getDeclaredField("cachePeriod");
234234
cachePeriodField.setAccessible(true);
235-
cachePeriodField.set(apiAccessCheckerSpy, 1L);
235+
cachePeriodField.set(apiAccessCheckerSpy, 1);
236236

237237
final String allowedApiName = "someAllowedApi";
238238
final RolePermission permission = new RolePermissionVO(1L, allowedApiName, Permission.ALLOW, null);
@@ -299,7 +299,7 @@ public void testGetApisAllowedToAccountUsesCachedPermissions() {
299299
// Ensure caching is enabled by setting a positive cachePeriod
300300
Field cachePeriodField = DynamicRoleBasedAPIAccessChecker.class.getDeclaredField("cachePeriod");
301301
cachePeriodField.setAccessible(true);
302-
cachePeriodField.set(apiAccessCheckerSpy, 1L);
302+
cachePeriodField.set(apiAccessCheckerSpy, 1);
303303

304304
final RolePermission permission = new RolePermissionVO(1L, "api1", Permission.ALLOW, null);
305305
Mockito.when(roleServiceMock.findAllPermissionsBy(Mockito.anyLong())).thenReturn(Collections.singletonList(permission));

0 commit comments

Comments
 (0)