inflearn logo
강의

Course

Instructor

Learning Large-Scale System Design by Building Directly with Spring Boot - Bulletin Board

CommentServiceTest의 테스트 로직 질문

Resolved

50

bebe

11 asked

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()의 판정 로직상 0L1L 은 동일한 결과를 반환하므로 테스트 결과에는 영향을 주지 않는 것 같습니다.)

카운트 쿼리의 동작 부분을 제가 잘못 이해한 것인지 질문드립니다.

java mysql spring-boot kafka redis

Answer 2

0

kuke

bebe님, 안녕하세요!

 

저도 다시 살펴보니 위 가정에서는 굳이 1로 반환할 필요가 없네요.

테스트 결과에 영향은 없지만 commentId=2의 자식은 아무도 없으므로 0으로 반환하는게 이치 상 맞고 이해도 수월할 것 같습니다.

결론은 bebe님이 이해하신 부분이 맞습니다! 혼란을 드려 죄송합니다.

0

communityai8509

안녕하세요, 인프런 AI 인턴입니다. bebe님.

질문해 주신 내용은 countBy 메서드를 사용하는 테스트 코드 내의 인자 값과 관련이 있습니다. 이 부분은 JUnit과 Mockito를 사용한 단위 테스트에서 목 객체의 동작을 설정하는 부분과 연관이 있습니다.

테스트 동작

given(commentRepository.countBy(articleId, commentId, 2L)).willReturn(1L);
위 코드는 테스트에서 특정 상황을 모의하기 위해 commentRepository.countBy 메서드가 반환할 값을 명시적으로 설정하는 부분입니다. 이 경우, countBy 메서드가 호출될 때 어떤 값을 반환할지 결정하는 것이며, 실제 데이터베이스의 반환 값과는 관계없이 테스트를 위한 시나리오를 구성하는 용도로 사용됩니다.

의도된 동작

  • 테스트 환경 설정: 실제 비즈니스 로직에서 Comment 엔티티가 특정 조건을 만족하도록 하기 위해 임의의 값을 반환하도록 설정할 수 있습니다. 테스트 환경에서 hasChildren() 메서드의 판정을 위한 데이터 시뮬레이션이 주요 목적이 되기도 합니다.

  • 비즈니스 로직에 맞춘 시나리오 설정: 만약 hasChildren 메서드가 1을 반환하는 것이 비즈니스 로직 시나리오에 맞는 것으로 설계되어 있다면, 그렇게 설정해야 하기도 합니다.

해결 방법

테스트와 실제 구현이 어떻게 연계되어 있는지를 이해하기 위해 비즈니스 로직의 흐름에 맞추어 countBy 메서드 호출의 결과를 역할에 맞게 적절히 설정하는 것이 중요합니다.

다음은 관련된 유사 질문 및 추가적인 배경지식을 제공할 수 있는 인프런의 게시물입니다:

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

consumer에서 에러가 발생할 경우 데이터 유실 문의

0

38

2

게시글 테스트 데이터 삽입

0

43

2

정렬, 필터, 검색 등의 조건이 붙을 경우 최적화할 수 있는 방법이 무엇이 있을까요?

0

95

2

좋아요 기능 정합성 보장 방법

0

90

2

좋아요 동시성처리 최적의 선택?

1

115

2

프론트엔드 msa 환경 api 주소 통합? 과 서버끼리 통신 방식에대해

0

85

2

안녕하세요 무한스크롤 강의듣다가 질문이 있습니다.

0

54

1

조회수 조회 로직 질문

1

64

2

비로그인 유저도 어뷰징 방지 정책

1

63

2

무한 스크롤 방식에서 페이지 번호 방식 쿼리의 문제점 의문

1

69

2

path 쿼리 관련 질문드립니다!

0

55

2

antigravity 에디터를 쓰신다면 종료해주세요

1

122

0

프로젝트 구조

0

83

2

article_like_count api test

0

70

2

이벤트 페이로드 객체의 생성 방식이 팩토리 메서드 패턴이 아닌 빌더 패턴인 이유가 궁금합니다!.

0

101

2

[33. 좋아요 수 구현] 에서 테스트 하는 화면 동시성 문제

0

88

2

findByPath에서 articleId로도 검색을 해야 할 것 같아요.

0

87

3

jpa ddl-auto none을 하는 이유와 join 방법

0

92

2

팩터리 메소드와 response 객체 사용 이유가 궁금합니다!

0

85

2

커서 기반 페이지네이션 과 무한 스크롤링

0

97

2

게시글 생성 로직에서 오류 발생시 redis 게시글 수 되돌리기

0

95

2

멀티 모듈이 아닌 MSA 환경에서 common

0

146

2

2Depth 강의 도중 궁금한 점 있어요!!

0

73

2

샤딩에 대해서 궁금점있습니다.

0

112

2