Skip to content

Commit 1ca4f8d

Browse files
author
muriloalvesdev
committed
improving user update to validate in test if it really updates his data
1 parent aaff088 commit 1ca4f8d

2 files changed

Lines changed: 7 additions & 6 deletions

File tree

src/main/java/br/com/developers/login/service/UserService.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ public interface UserService {
1010

1111
AccessToken authenticateUser(LoginDTO loginDto);
1212

13-
void updateUser(RegisterDTO registerData);
13+
User updateUser(RegisterDTO registerData);
1414

1515
void delete(LoginDTO loginDTO);
1616
}

src/main/java/br/com/developers/login/service/impl/UserServiceImpl.java

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -66,13 +66,14 @@ private Set<Role> getAllRoles(RegisterDTO registerData) {
6666
}
6767

6868

69-
public void updateUser(RegisterDTO registerData) {
69+
public User updateUser(RegisterDTO registerData) {
7070
User user = this.userRepository.findByEmail(registerData.getEmail().toLowerCase()).orElseThrow(
7171
() -> new UserNotFoundException(String.format(USER_NOT_FOUND, registerData.getEmail())));
72-
user.setFirstName(registerData.getName());
73-
user.setLastName(registerData.getLastName());
74-
user.setPassword(registerData.getPassword());
75-
user.setRoles(getAllRoles(registerData));
72+
user.setFirstName(registerData.getName());
73+
user.setLastName(registerData.getLastName());
74+
user.setPassword(registerData.getPassword());
75+
user.setRoles(getAllRoles(registerData));
76+
return this.userRepository.save(user);
7677
}
7778

7879
public void delete(LoginDTO loginDTO) {

0 commit comments

Comments
 (0)