테스트 코드 작성
테스트를 한다는 것은 원하는 "결과"가 나오는지 "검증" 하는 것이므로
검증할 사항을 위한 조건들을 생성하고 그 결과를 보면 되는 것이다
테스트 코드를 할 때 어렵게 느껴졌던 부분은 Mock 이다
이유는, mock 이 어디까지 반영이 되는 존재인지를 잘 파악하지 못해서이다
리팩토링
효율적인 Test를 위해서는 리팩토링을 피할 수 없다
단순 CRUD 를 담당하는 클래스와 검증이 필요한 (단순 CRUD가 아닌) 메서드를 분리하는 것이 중요하다
이렇게 Mock 이 필요없는 부분을 떼어놓은 후엔 POJO 테스트라고 불리는 것을 할 수 있게 된다
또한 Mock 이 필요한 경우라도 "단순" CRUD 는 굳이 테스트를 할 필요도 없다
리팩토링이 끝나고 POJO 테스트 코드 작성을 하려고 하는데
객체를 생성해야 하는 일이 여간 귀찮은 일이 아니다
더 간단히 생성하는 방법이 있을까 싶어서 구글링을 해봤지만
예시들이 모두 간단한 것들 뿐이었다
Dto 는 입력할 내용이 상대적으로 적기때문에
for 문으로 Dto 를 생성했고, user의 매개변수에 넣어주었다
consumerRequestDtoList = new ArrayList<>();
for (int i = 0; i < 3; i++) {
ConsumersSignupRequestDto consumerRequestDto = new ConsumersSignupRequestDto(
"email" + i + "@g.com", "password" + i, "User" + i,
"Nickname" + i, "Address" + i);
consumerRequestDtoList.add(consumerRequestDto);
}
그렇게 thanks 등록이 성공적으로 진행되는 것을 확인한 것 까지는 좋았으나,
thanks 를 한 번 더 누를 경우 thanks 등록이 취소 처리되는 경우를 상정하여 테스트 하였는데 실패했다
Review의 Id 부분이 nullPointException 이 뜬 것이다
아무리 service 메서드를 손보아도 exception이 발생했다
더 이상 손 볼 일이 없다고 판단되어 테스트 코드 추가 특강을 진행해주셨던 튜터님을 찾아갔다
어떤 상황인지 파악(전체 로직 흐름 중 어느부분을 테스트 하는 것인지 확인, 문제가 발생한 부분 확인)하실 때
등록만 할 때는 성공이 뜨는데, 취소를 진행하기 위해 같은 로직을 태웠을 때 실패가 뜨는(Exception이 발생하는) 상황임을 말씀드렸다
바로 디버그 모드를 진행했다
확인하니 Review id가 null로 뜨고 있었다
Dto를 통해서 Review를 생성하고, 그걸 받아서 thanks 등록을 진행하는데
생성자에 Review 의 id 가 생성되도록 짜여져 있지가 않은 것이 문제였다
처음 한 번 thanks 를 눌렀을 때
thanks 등록이 성공하는 이유는 getThanksList() 를 할 때 null 이어서 바로 등록하는 부분으로 넘어가기 때문이었고
두 번째 thanks 를 눌렀을 때
thanks 등록이 성공하지 못하는 이유는 id 가 없는 Review의 id 를 얻으려고 하므로 null 이 떠서 에러가 발생한 것이었다
따라서 Review의 생성자를 지우고 builder를 적용하기로 했다
생성자를 지운 대신 AllArgsConstructor도 설정해 주었다
그리고 테스트 코드 안에 id를 부여하여 Review 를 생성했고
thanks 취소 테스트를 무사히 완수했