libre.sh/keycloak-scim
4
0
Fork 0
Code Issues 15 Pull requests Releases 1 Activity Actions

[KEYCLOAK-4902] - Only set effect if result exists and removing ignore from tests

Browse source
This commit is contained in:
Pedro Igor 2018-08-20 18:14:16 -03:00
parent 457c6a9d80
commit 3c2339ba33
2 changed files with 10 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
server-spi-private/src/main/java/org/keycloak/authorization/policy/evaluation/Result.java
View file

@ -86,7 +86,16 @@ public class Result {
}
public PolicyResult policy(Policy policy, Effect effect) {
return associatedPolicies.computeIfAbsent(policy.getId(), id -> new PolicyResult(policy, effect));
PolicyResult policyResult = associatedPolicies.get(policy.getId());
if (policyResult == null) {
policyResult = new PolicyResult(policy, effect);
associatedPolicies.put(policy.getId(), policyResult);
} else {
policyResult.setEffect(effect);
}
return policyResult;
}
public Policy getPolicy() {

2
testsuite/integration-arquillian/tests/base/src/test/java/org/keycloak/testsuite/adapter/example/authorization/AbstractPhotozExampleAdapterTest.java
View file

@ -369,7 +369,6 @@ public abstract class AbstractPhotozExampleAdapterTest extends AbstractExampleAd
assertThat(getResourcesOfUser("alice"), is(empty()));
}
@Ignore
@Test
public void testClientRoleRepresentingUserConsent() throws Exception {
ContainerAssume.assumeNotAuthServerUndertow();
@ -402,7 +401,6 @@ public abstract class AbstractPhotozExampleAdapterTest extends AbstractExampleAd
}
@Test
@Ignore
public void testClientRoleNotRequired() throws Exception {
loginToClientPage("alice", "alice");