diff --git a/uw-spring-security-core/src/main/java/edu/wisc/uwss/uds/UdsPersonUserDetailsImpl.java b/uw-spring-security-core/src/main/java/edu/wisc/uwss/uds/UdsPersonUserDetailsImpl.java index 2356f9f685670f1b22411e7ffc3a3d3f250c0d6a..cc5772f9e9d5a6dc1ba5812a800f82fe8e05c010 100644 --- a/uw-spring-security-core/src/main/java/edu/wisc/uwss/uds/UdsPersonUserDetailsImpl.java +++ b/uw-spring-security-core/src/main/java/edu/wisc/uwss/uds/UdsPersonUserDetailsImpl.java @@ -46,38 +46,38 @@ public class UdsPersonUserDetailsImpl implements UWUserDetails { /** * {@inheritDoc} * - * Always returns false, as users from this source can't log in. + * Always returns true. */ @Override public boolean isAccountNonExpired() { - return false; + return true; } /** * {@inheritDoc} * - * Always returns false, as users from this source can't log in. + * Always returns true. */ @Override public boolean isAccountNonLocked() { - return false; + return true; } /** * {@inheritDoc} * - * Always returns false, as users from this source can't log in. + * Always returns true. */ @Override public boolean isCredentialsNonExpired() { - return false; + return true; } /** * {@inheritDoc} * - * Always returns false, as users from this source can't log in. + * Always returns true. */ @Override public boolean isEnabled() { - return false; + return true; } /** * {@inheritDoc} @@ -212,6 +212,14 @@ public class UdsPersonUserDetailsImpl implements UWUserDetails { public String getIsisEmplid() { return null; } + + @Override + public String toString() { + return "UdsPersonUserDetailsImpl{" + + "person=" + person + + '}'; + } + /** * {@inheritDoc} */ diff --git a/uw-spring-security-core/src/test/java/edu/wisc/uwss/uds/UdsPersonUserDetailsImplTest.java b/uw-spring-security-core/src/test/java/edu/wisc/uwss/uds/UdsPersonUserDetailsImplTest.java index 56eda990296164740dab5bb39ddd66d52d179ee1..036dfa9c10845c76f6386bf871921710db53d068 100644 --- a/uw-spring-security-core/src/test/java/edu/wisc/uwss/uds/UdsPersonUserDetailsImplTest.java +++ b/uw-spring-security-core/src/test/java/edu/wisc/uwss/uds/UdsPersonUserDetailsImplTest.java @@ -48,10 +48,7 @@ public class UdsPersonUserDetailsImplTest { assertEquals(UdsPersonUserDetailsImpl.USER_DETAILS_SOURCE, userDetails.getSource()); assertEquals(Collections.emptyList(), userDetails.getUddsMembership()); assertNull(userDetails.getUsername()); - assertFalse(userDetails.isAccountNonExpired()); - assertFalse(userDetails.isAccountNonLocked()); - assertFalse(userDetails.isCredentialsNonExpired()); - assertFalse(userDetails.isEnabled()); + assertNull(userDetails.getFirstName()); assertNull(userDetails.getLastName()); }