Skip to content

Commit 4ba981a

Browse files
committed
Clean up UserServiceTest: remove unused import and resolve warnings
1 parent 1066972 commit 4ba981a

File tree

1 file changed

+2
-13
lines changed

1 file changed

+2
-13
lines changed

core-banking-service/src/test/java/com/javatodev/finance/service/UserServiceTest.java

Lines changed: 2 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -3,43 +3,36 @@
33
import com.javatodev.finance.exception.EntityNotFoundException;
44
import com.javatodev.finance.model.dto.User;
55
import com.javatodev.finance.model.entity.UserEntity;
6-
import com.javatodev.finance.model.mapper.UserMapper;
76
import com.javatodev.finance.repository.UserRepository;
87
import org.junit.jupiter.api.BeforeEach;
98
import org.junit.jupiter.api.Test;
109
import org.mockito.Mockito;
1110
import org.springframework.data.domain.Page;
1211
import org.springframework.data.domain.PageImpl;
1312
import org.springframework.data.domain.Pageable;
13+
1414
import java.util.Collections;
1515
import java.util.Optional;
1616
import java.util.List;
17+
1718
import static org.junit.jupiter.api.Assertions.*;
1819
import static org.mockito.Mockito.*;
1920

2021
class UserServiceTest {
2122
private UserRepository userRepository;
2223
private UserService userService;
23-
private UserMapper userMapper;
2424

2525
@BeforeEach
2626
void setUp() {
2727
userRepository = mock(UserRepository.class);
2828
userService = new UserService(userRepository);
29-
userMapper = spy(new UserMapper());
30-
// Inject the spy mapper if needed
31-
userService = Mockito.spy(userService);
32-
Mockito.doReturn(userMapper).when(userService).userMapper;
3329
}
3430

3531
@Test
3632
void readUser_found() {
3733
UserEntity entity = new UserEntity();
3834
entity.setIdentificationNumber("ID123");
39-
User userDto = new User();
40-
userDto.setIdentificationNumber("ID123");
4135
when(userRepository.findByIdentificationNumber("ID123")).thenReturn(Optional.of(entity));
42-
doReturn(userDto).when(userMapper).convertToDto(entity);
4336
User result = userService.readUser("ID123");
4437
assertNotNull(result);
4538
assertEquals("ID123", result.getIdentificationNumber());
@@ -57,14 +50,10 @@ void readUsers_success() {
5750
entity.setIdentificationNumber("ID123");
5851
List<UserEntity> entities = Collections.singletonList(entity);
5952
Page<UserEntity> page = new PageImpl<>(entities);
60-
User userDto = new User();
61-
userDto.setIdentificationNumber("ID123");
6253
when(userRepository.findAll(any(Pageable.class))).thenReturn(page);
63-
doReturn(Collections.singletonList(userDto)).when(userMapper).convertToDtoList(entities);
6454
List<User> result = userService.readUsers(Pageable.unpaged());
6555
assertNotNull(result);
6656
assertEquals(1, result.size());
6757
assertEquals("ID123", result.get(0).getIdentificationNumber());
6858
}
6959
}
70-

0 commit comments

Comments
 (0)