diff --git a/core/src/main/java/org/springframework/security/authorization/AuthorityAuthorizationManager.java b/core/src/main/java/org/springframework/security/authorization/AuthorityAuthorizationManager.java index f266fa3663c..db4cf42e48d 100644 --- a/core/src/main/java/org/springframework/security/authorization/AuthorityAuthorizationManager.java +++ b/core/src/main/java/org/springframework/security/authorization/AuthorityAuthorizationManager.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2022 the original author or authors. + * Copyright 2002-2023 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -16,8 +16,6 @@ package org.springframework.security.authorization; -import java.util.Arrays; -import java.util.HashSet; import java.util.Set; import java.util.function.Supplier; @@ -43,7 +41,7 @@ public final class AuthorityAuthorizationManager implements AuthorizationMana private final Set authorities; private AuthorityAuthorizationManager(String... authorities) { - this.authorities = new HashSet<>(Arrays.asList(authorities)); + this.authorities = Set.of(authorities); } /** diff --git a/core/src/test/java/org/springframework/security/authorization/AuthoritiesAuthorizationManagerTests.java b/core/src/test/java/org/springframework/security/authorization/AuthoritiesAuthorizationManagerTests.java index 92359a64b86..79b4540c961 100644 --- a/core/src/test/java/org/springframework/security/authorization/AuthoritiesAuthorizationManagerTests.java +++ b/core/src/test/java/org/springframework/security/authorization/AuthoritiesAuthorizationManagerTests.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2022 the original author or authors. + * Copyright 2002-2023 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -74,14 +74,14 @@ void checkWhenUserHasNotAnyAuthorityThenDeniedDecision() { } @Test - void hasRoleWhenRoleHierarchySetThenGreaterRoleTakesPrecedence() { + void checkWhenRoleHierarchySetThenGreaterRoleTakesPrecedence() { AuthoritiesAuthorizationManager manager = new AuthoritiesAuthorizationManager(); RoleHierarchyImpl roleHierarchy = new RoleHierarchyImpl(); roleHierarchy.setHierarchy("ROLE_ADMIN > ROLE_USER"); manager.setRoleHierarchy(roleHierarchy); Supplier authentication = () -> new TestingAuthenticationToken("user", "password", "ROLE_ADMIN"); - assertThat(manager.check(authentication, Collections.singleton("ROLE_ADMIN")).isGranted()).isTrue(); + assertThat(manager.check(authentication, Collections.singleton("ROLE_USER")).isGranted()).isTrue(); } }