github
/
OpenID-Connect-Java-Spring-Server
mirror of https://github.com/mitreid-connect/OpenID-Connect-Java-Spring-Server
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Revised test methods for checking for offline_access scope. Previous tests were doing trivial checking.

pull/357/merge
William Kim 2013-06-18 09:37:44 -04:00 committed by Justin Richer
parent 8851f4d037
commit 5fb1ef3aca
1 changed files with 35 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

49
openid-connect-server/src/test/java/org/mitre/oauth2/service/impl/TestDefaultOAuth2ClientDetailsEntityService.java
View File

@ -17,9 +17,10 @@
package org.mitre.oauth2.service.impl;
import static org.junit.Assert.*;
import static org.junit.matchers.JUnitMatchers.*;
import static org.hamcrest.CoreMatchers.*;
import java.util.Set;
import org.junit.Before;
import org.junit.Test;
@ -121,11 +122,16 @@ public class TestDefaultOAuth2ClientDetailsEntityService {
Mockito.when(client.getId()).thenReturn(null);
Mockito.when(client.isAllowRefresh()).thenReturn(true);
Mockito.when(client.getScope()).thenReturn(Sets.newHashSet("offline_access"));
// scopes returned by client entities are Strings
@SuppressWarnings("unchecked")
Set<String> scopes = Mockito.mock(Set.class);
Mockito.when(client.getScope()).thenReturn(scopes);
service.saveNewClient(client);
assertThat(client.getScope(), hasItem("offline_access"));
Mockito.verify(scopes).add("offline_access");
}
/**
@ -138,11 +144,16 @@ public class TestDefaultOAuth2ClientDetailsEntityService {
Mockito.when(client.getId()).thenReturn(null);
Mockito.when(client.isAllowRefresh()).thenReturn(false);
Mockito.when(client.getScope()).thenReturn(Sets.newHashSet(""));
// scopes returned by client entities are Strings
@SuppressWarnings("unchecked")
Set<String> scopes = Mockito.mock(Set.class);
Mockito.when(client.getScope()).thenReturn(scopes);
service.saveNewClient(client);
assertThat(client.getScope(), not(hasItem("offline_access")));
Mockito.verify(scopes).remove("offline_access");
}
@Test
@ -151,7 +162,7 @@ public class TestDefaultOAuth2ClientDetailsEntityService {
// null id
try {
service.loadClientByClientId(null);
fail("Expected an IllegalArgumentException.");
fail("Null client id. Expected an IllegalArgumentException.");
} catch (IllegalArgumentException e) {
assertThat(e, is(notNullValue()));
}
@ -159,7 +170,7 @@ public class TestDefaultOAuth2ClientDetailsEntityService {
// empty id
try {
service.loadClientByClientId("");
fail("Expected an IllegalArgumentException.");
fail("Empty client id. Expected an IllegalArgumentException.");
} catch (IllegalArgumentException e) {
assertThat(e, is(notNullValue()));
}
@ -169,7 +180,7 @@ public class TestDefaultOAuth2ClientDetailsEntityService {
Mockito.when(clientRepository.getClientByClientId(clientId)).thenReturn(null);
try {
service.loadClientByClientId(clientId);
fail("Expected an InvalidClientException.");
fail("Client id not found. Expected an InvalidClientException.");
} catch (InvalidClientException e) {
assertThat(e, is(notNullValue()));
}
@ -259,11 +270,16 @@ public class TestDefaultOAuth2ClientDetailsEntityService {
ClientDetailsEntity newClient = Mockito.mock(ClientDetailsEntity.class);
Mockito.when(newClient.isAllowRefresh()).thenReturn(true);
Mockito.when(newClient.getScope()).thenReturn(Sets.newHashSet("offline_access"));
// scopes returned by client entities are Strings
@SuppressWarnings("unchecked")
Set<String> scopes = Mockito.mock(Set.class);
Mockito.when(newClient.getScope()).thenReturn(scopes);
service.updateClient(oldClient, newClient);
assertThat(newClient.getScope(), hasItem("offline_access"));
Mockito.verify(scopes).add("offline_access");
}
@Test
@ -273,10 +289,15 @@ public class TestDefaultOAuth2ClientDetailsEntityService {
ClientDetailsEntity newClient = Mockito.mock(ClientDetailsEntity.class);
Mockito.when(newClient.isAllowRefresh()).thenReturn(false);
Mockito.when(newClient.getScope()).thenReturn(Sets.newHashSet(""));
// scopes returned by client entities are Strings
@SuppressWarnings("unchecked")
Set<String> scopes = Mockito.mock(Set.class);
Mockito.when(newClient.getScope()).thenReturn(scopes);
service.updateClient(oldClient, newClient);
assertThat(newClient.getScope(), not(hasItem("offline_access")));
Mockito.verify(scopes).remove("offline_access");
}
}