SEC-449: Reamed template test class to match tested class. Added test method for case when no attribute value is found.

core/src/test/java/org/acegisecurity/ldap/LdapTemplateTests.java → core/src/test/java/org/acegisecurity/ldap/SpringSecurityLdapTemplateTests.java

@@ -28,7 +28,7 @@ import javax.naming.directory.DirContext;
  * @author Luke Taylor
  * @version $Id$
  */
-public class LdapTemplateTests extends AbstractLdapIntegrationTests {
+public class SpringSecurityLdapTemplateTests extends AbstractLdapIntegrationTests {
     //~ Instance fields ================================================================================================
 
     private SpringSecurityLdapTemplate template;
@@ -75,7 +75,7 @@ public class LdapTemplateTests extends AbstractLdapIntegrationTests {
         } catch (UncategorizedLdapException expected) {}
     }
 
-    public void testSearchForSingleAttributeValues() {
+    public void testRoleSearchReturnsCorrectNumberOfRoles() {
         String param = "uid=ben,ou=people,dc=acegisecurity,dc=org";
 
         Set values = template.searchForSingleAttributeValues("ou=groups", "(member={0})", new String[] {param}, "ou");
@@ -85,4 +85,12 @@ public class LdapTemplateTests extends AbstractLdapIntegrationTests {
         assertTrue(values.contains("manager"));
         assertTrue(values.contains("submanager"));
     }
+
+    public void testRoleSearchForMissingAttributeFailsGracefully() {
+        String param = "uid=ben,ou=people,dc=acegisecurity,dc=org";
+
+        Set values = template.searchForSingleAttributeValues("ou=groups", "(member={0})", new String[] {param}, "mail");
+
+        assertEquals(0, values.size());
+    }
 }