Removed previous constructor from DefaultUserInforUserDetailsService.java and put in getter/setter for UserInfoRepository as a replacement.
parent
de77f2c75a
commit
1b601abd6f
|
@ -40,22 +40,13 @@ import org.springframework.stereotype.Service;
|
||||||
public class DefaultUserInfoUserDetailsService implements UserDetailsService {
|
public class DefaultUserInfoUserDetailsService implements UserDetailsService {
|
||||||
|
|
||||||
@Autowired
|
@Autowired
|
||||||
UserInfoRepository repository;
|
private UserInfoRepository repository;
|
||||||
|
|
||||||
public static final GrantedAuthority ROLE_USER = new SimpleGrantedAuthority("ROLE_USER");
|
public static final GrantedAuthority ROLE_USER = new SimpleGrantedAuthority("ROLE_USER");
|
||||||
public static final GrantedAuthority ROLE_ADMIN = new SimpleGrantedAuthority("ROLE_ADMIN");
|
public static final GrantedAuthority ROLE_ADMIN = new SimpleGrantedAuthority("ROLE_ADMIN");
|
||||||
|
|
||||||
private List<String> admins = new ArrayList<String>();
|
private List<String> admins = new ArrayList<String>();
|
||||||
|
|
||||||
/**
|
|
||||||
* Parameterized constructor for use in test harnesses.
|
|
||||||
*
|
|
||||||
* @param repository the UserInfoRepository to set
|
|
||||||
*/
|
|
||||||
public DefaultUserInfoUserDetailsService(UserInfoRepository repository) {
|
|
||||||
this.repository = repository;
|
|
||||||
}
|
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public UserDetails loadUserByUsername(String username) throws UsernameNotFoundException {
|
public UserDetails loadUserByUsername(String username) throws UsernameNotFoundException {
|
||||||
UserInfo userInfo = repository.getByUsername(username);
|
UserInfo userInfo = repository.getByUsername(username);
|
||||||
|
@ -94,4 +85,12 @@ public class DefaultUserInfoUserDetailsService implements UserDetailsService {
|
||||||
this.admins = admins;
|
this.admins = admins;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
public UserInfoRepository getUserInfoRepository() {
|
||||||
|
return repository;
|
||||||
|
}
|
||||||
|
|
||||||
|
public void setUserInfoRepository(UserInfoRepository repository) {
|
||||||
|
this.repository = repository;
|
||||||
|
}
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
|
@ -37,7 +37,8 @@ public class TestDefaultUserInfoUserDetailsService {
|
||||||
@Before
|
@Before
|
||||||
public void prepare() {
|
public void prepare() {
|
||||||
userInfoRepository = Mockito.mock(UserInfoRepository.class);
|
userInfoRepository = Mockito.mock(UserInfoRepository.class);
|
||||||
service = new DefaultUserInfoUserDetailsService(userInfoRepository);
|
service = new DefaultUserInfoUserDetailsService();
|
||||||
|
service.setUserInfoRepository(userInfoRepository);
|
||||||
service.setAdmins(Lists.newArrayList(adminUsername));
|
service.setAdmins(Lists.newArrayList(adminUsername));
|
||||||
|
|
||||||
userInfoAdmin = new DefaultUserInfo();
|
userInfoAdmin = new DefaultUserInfo();
|
||||||
|
|
Loading…
Reference in New Issue