해결된 질문
작성
·
167
0
학습 관련 질문을 최대한 상세히 남겨주세요!
고민 과정도 같이 나열해주셔도 좋습니다.
먼저 유사한 질문이 있었는지 검색해보세요.
인프런 서비스 운영 관련 문의는 1:1 문의하기를 이용해주세요.
안녕하세요. 강의를 따라가던 도중 테스트 코드에서 에러가 발생해 질문 드립니다.
섹션 3. 댓글 최대 2depth - CUD API 구현 강의를 들으며
void deleteShouldMarkDeletedIfHashChildren() 테스트에서 에러가 발생했습니다.
에러 내용은 PotentialStubbingProblem
에 관련된 내용으로
org.mockito.exceptions.misusing.PotentialStubbingProblem:
Strict stubbing argument mismatch. Please check:
- this invocation of 'countBy' method:
commentRepository.countBy(1L, 0L, 2L);
-> at shoon.board.comment.service.CommentService.hasChildren(CommentService.java:70)
- has following stubbing(s) with different arguments:
1. commentRepository.countBy(1L, 2L, 2L);
위와 같이 에러가 발생했습니다.
이 내용을 토대로 살펴보았을 때, 강의에서 제공된 코드가 아래와 같은데
@Test
@DisplayName("삭제할 댓글이 자식 있으면, 삭제 표시만 한다.")
void deleteShouldMarkDeletedIfHasChildren() {
// given
Long articleId = 1L;
Long commentId = 2L;
Comment comment = createComment(articleId, commentId);
given(commentRepository.findById(commentId))
.willReturn(Optional.of(comment));
given(commentRepository.countBy(articleId, commentId, 2L)).willReturn(2L);
// //when
commentService.delete(commentId);
//
// //then
verify(comment).delete();
}
이때 comment는 mock 객체로 articleId와 commentId만 가지고 있는 상황이며 getArticleId와 getCommentId만 mocking이 되어있는데, 실제 countBy 메서드가 서비스 객체에서
private boolean hasChildren(Comment comment) {
return commentRepository.countBy(comment.getArticleId(), comment.getParentCommentId(), 2L) == 2;
}
위와 같이 호출이 되는데, 이때 getParentCommentId()의 값이 존재하지 않는데 테스트가 정상적으로 작동할 수 있나요?
테스트 코드에서는 countyBy(articleId(1), commentId(2), 2)로 고정해두었는데, 실제 호출은 이와 달라지기 때문에 에러가 발생한다고 생각이 들었는데 맞을까요? 강사님 코드에서는 정상 작동하고 제 코드는 작동이 에러가 나서 이유를 정확히 모르겠네요 ㅜㅜ
혹시 제 코드가 필요하실 수도 있을까봐 구글 드라이브 링크로 제 코드 파일도 남기도록 하겠습니다.
https://drive.google.com/file/d/1tqV1PkvwpnaRqI9msxEj4X_t7iqg_CFr/view?usp=drive_link
답변 2
0
상훈님, 안녕하세요!
코드 확인해보았는데, CommentService에서 hasChildren 메소드가 잘못 작성되어 있네요.
private boolean hasChildren(Comment comment) {
return commentRepository.countBy(comment.getArticleId(), comment.getParentCommentId(), 2L) == 2;
}
두번째 파라미터 getParentCommentId가 아니라 getCommentId를 전달해주셔야 합니다!
파라미터로 전달받은 comment가 children을 가지고 있는지 확인하는 메소드이기 때문입니다.
0
안녕하세요, 인프런 AI 인턴입니다. 박상훈님.
질문해주신 내용은 Mockito를 사용한 테스트에서 PotentialStubbingProblem
오류와 관련이 있습니다. 이 문제는 주로 Mockito가 예상하지 못한 메서드 호출이나 다른 인수 값으로 호출될 때 발생합니다. 이 경우, countBy
메서드에서 발생한 문제로 보입니다.
인자 값 일치:
countBy
메서드의 인자들이 실제 테스트 실행 시점과 일치하지 않아 오류가 발생하는 것입니다. 테스트에서 countBy
가 정확히 어떤 인지로 호출될 지 예상하기 어려운 경우, 유연한 매칭을 위해 anyLong()
같은 매처를 사용할 수 있습니다.정확한 Stubbing:
hasChildren
메서드에서는 comment.getParentCommentId()
가 호출되고 있습니다. 이 메서드 호출의 결과를 설정하는 부분이 필요할 수 있습니다.매처 사용법:
countBy
메서드를 스텁할 수 있습니다:
given(commentRepository.countBy(anyLong(), anyLong(), eq(2L))).willReturn(2L);
위 방법들을 시도해 보시고 문제가 해결되지 않는다면, 코드 전체 흐름에서 Comment
객체 생성 과정 및 데이터를 검토하여 특정 부분에서 예상 못한 데이터 흐름이 발생하지 않는지 확인이 필요합니다.
아래는 관련된 문제 해결의 방향성을 찾는데 도움이 될 수 있는 인프런의 유사한 질문입니다:
- Mockito의 인자 매칭 문제 해결 방법 (게시글 제목)
저는 질문자님의 이해를 돕고자 지식공유자님의 기존 답변을 참고하여 유사한 답변 링크를 추천해드리고 있습니다. 현재 베타 기능으로 답변이 만족스럽지 않을 수 있는 점 양해 부탁드립니다. 🙏 추가적으로 궁금한 점이 있으시면, 이어서 질문해 주세요. 곧 지식공유자께서 답변해 주실 것입니다.
앗 몇 번을 다시 확인했는데 왜 이걸 못봤을까요? ㅜㅜ 감사합니다!