강의

멘토링

로드맵

Inflearn brand logo image

인프런 커뮤니티 질문&답변

박상훈님의 프로필 이미지
박상훈

작성한 질문수

스프링부트로 직접 만들면서 배우는 대규모 시스템 설계 - 게시판

댓글 테스트 작성 시 에러 발생

해결된 질문

작성

·

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 메서드에서 발생한 문제로 보입니다.

문제 해결 방향

  1. 인자 값 일치:

    • countBy 메서드의 인자들이 실제 테스트 실행 시점과 일치하지 않아 오류가 발생하는 것입니다. 테스트에서 countBy 가 정확히 어떤 인지로 호출될 지 예상하기 어려운 경우, 유연한 매칭을 위해 anyLong() 같은 매처를 사용할 수 있습니다.
  2. 정확한 Stubbing:

    • Mockito를 사용할 때는 보통 정확한 인자로 메서드 호출 기대 값을 설정합니다. 메서드 호출 시 얻는 객체들이 정확히 일치하는지 확인하세요. hasChildren 메서드에서는 comment.getParentCommentId()가 호출되고 있습니다. 이 메서드 호출의 결과를 설정하는 부분이 필요할 수 있습니다.
  3. 매처 사용법:

    • 아래처럼 매처를 사용하여 countBy 메서드를 스텁할 수 있습니다:
      
      given(commentRepository.countBy(anyLong(), anyLong(), eq(2L))).willReturn(2L);
      

위 방법들을 시도해 보시고 문제가 해결되지 않는다면, 코드 전체 흐름에서 Comment 객체 생성 과정 및 데이터를 검토하여 특정 부분에서 예상 못한 데이터 흐름이 발생하지 않는지 확인이 필요합니다.

아래는 관련된 문제 해결의 방향성을 찾는데 도움이 될 수 있는 인프런의 유사한 질문입니다:
- Mockito의 인자 매칭 문제 해결 방법 (게시글 제목)

저는 질문자님의 이해를 돕고자 지식공유자님의 기존 답변을 참고하여 유사한 답변 링크를 추천해드리고 있습니다. 현재 베타 기능으로 답변이 만족스럽지 않을 수 있는 점 양해 부탁드립니다. 🙏 추가적으로 궁금한 점이 있으시면, 이어서 질문해 주세요. 곧 지식공유자께서 답변해 주실 것입니다.

박상훈님의 프로필 이미지
박상훈

작성한 질문수

질문하기