인프런 커뮤니티 질문&답변
작성한 질문수
CommentServiceTest의 테스트 로직 질문
해결된 질문
작성
·
26
0
@Test
@DisplayName("하위 댓글이 삭제되고, 삭제되지 않은 부모면, 하위 댓글만 삭제한다.")
void deleteShouldDeleteChildrenOnlyIfNotDeletedParent() {
// given
Long articleId = 1L;
Long commentId = 2L;
Long parentCommentId = 1L;
Comment comment = createComment(articleId, commentId, parentCommentId);
given(comment.isRoot()).willReturn(false);
Comment parentComment = mock(Comment.class);
given(parentComment.getDeleted()).willReturn(false);
given(commentRepository.findById(commentId))
.willReturn(Optional.of(comment));
given(commentRepository.countBy(articleId, commentId, 2L)).willReturn(1L); // 이 부분
given(commentRepository.findById(parentCommentId))
.willReturn(Optional.of(parentComment));
// ...위 테스트에서
given(commentRepository.countBy(articleId, commentId, 2L)).willReturn(1L);
이 부분 때문에 질문을 드렸습니다.
commentRepository.countBy() 에서 commentId = 2L을 부모로 갖는 comment는 아무도 없으므로, 0L을 반환하는 것이 옳다고 이해했는데, 강의에선 1L을 반환하셨습니다!
(물론 서비스 계층의 메서드에서 hasChildren()의 판정 로직상 0L과 1L 은 동일한 결과를 반환하므로 테스트 결과에는 영향을 주지 않는 것 같습니다.)
카운트 쿼리의 동작 부분을 제가 잘못 이해한 것인지 질문드립니다.
답변 2
0
bebe님, 안녕하세요!
저도 다시 살펴보니 위 가정에서는 굳이 1로 반환할 필요가 없네요.
테스트 결과에 영향은 없지만 commentId=2의 자식은 아무도 없으므로 0으로 반환하는게 이치 상 맞고 이해도 수월할 것 같습니다.
결론은 bebe님이 이해하신 부분이 맞습니다! 혼란을 드려 죄송합니다.
0
안녕하세요, 인프런 AI 인턴입니다. bebe님.
질문해 주신 내용은 countBy 메서드를 사용하는 테스트 코드 내의 인자 값과 관련이 있습니다. 이 부분은 JUnit과 Mockito를 사용한 단위 테스트에서 목 객체의 동작을 설정하는 부분과 연관이 있습니다.
테스트 동작
given(commentRepository.countBy(articleId, commentId, 2L)).willReturn(1L);
위 코드는 테스트에서 특정 상황을 모의하기 위해 commentRepository.countBy 메서드가 반환할 값을 명시적으로 설정하는 부분입니다. 이 경우, countBy 메서드가 호출될 때 어떤 값을 반환할지 결정하는 것이며, 실제 데이터베이스의 반환 값과는 관계없이 테스트를 위한 시나리오를 구성하는 용도로 사용됩니다.
의도된 동작
테스트 환경 설정: 실제 비즈니스 로직에서 Comment 엔티티가 특정 조건을 만족하도록 하기 위해 임의의 값을 반환하도록 설정할 수 있습니다. 테스트 환경에서
hasChildren()메서드의 판정을 위한 데이터 시뮬레이션이 주요 목적이 되기도 합니다.비즈니스 로직에 맞춘 시나리오 설정: 만약 hasChildren 메서드가 1을 반환하는 것이 비즈니스 로직 시나리오에 맞는 것으로 설계되어 있다면, 그렇게 설정해야 하기도 합니다.
해결 방법
테스트와 실제 구현이 어떻게 연계되어 있는지를 이해하기 위해 비즈니스 로직의 흐름에 맞추어 countBy 메서드 호출의 결과를 역할에 맞게 적절히 설정하는 것이 중요합니다.
다음은 관련된 유사 질문 및 추가적인 배경지식을 제공할 수 있는 인프런의 게시물입니다:
저는 질문자님의 이해를 돕고자 지식공유자님의 기존 답변을 참고하여 유사한 답변 링크를 추천해드리고 있습니다. 현재 베타 기능으로 답변이 만족스럽지 않을 수 있는 점 양해 부탁드립니다. 🙏 추가적으로 궁금한 점이 있으시면, 이어서 질문해 주세요. 곧 지식공유자께서 답변해 주실 것입니다.