📌 8주차 11장 의존관계 주입(DI)를 통한 테스트하기 쉬운 코드 만들기

✅ Controller, Service에서 생성해서 사용하던 의존관계를 생성자 주입으로 변경

기존 Service 코드

기존 QuestionService는 다음과 같이 dao 객체를 사용하고 있다.

위 형태는 dao객체가 Service에 강하게 결합한 형태이며 강한 결합으로 인해 DB를 제외한 순서 서비스 코드를 테스트하기 쉽지 않다.

DI 적용(생성자 주입) Service

QuestionService 객체를 생성할때 필요한 dao에 대한 의존성을 주입하도록 변경할 수 있다.

DI 적용 Service 테스트

@ExtendWith(MockitoExtension.class)
class QuestionServiceTest {

    @InjectMocks
    QuestionService questionService;

    @Mock
    QuestionDao questionDao;

    @Mock
    AnswerDao answerDao;

    @Test
    @DisplayName("존재하지 않는 사용자 질문 삭제시 예외 발생 테스트")
    void deleteNotExistQuestion_fail_withException() {
        // given
        given(answerDao.findAllByQuestionId(1L)).willReturn(Collections.emptyList());
        given(questionDao.findById(1L)).willReturn(null);

        // then
        Assertions.assertThatThrownBy(
                        () -> questionService.deleteQuestion(1L, new User("test", "1234", "name", "[email protected]")))
                .isInstanceOf(CannotDeleteQuestionException.class)
                .hasMessageContaining("존재하지 않는 질문글은 삭제할 수 없습니다.");
    }

    @Test
    @DisplayName("타인이 작성한 질문 삭제시 예외 발생 테스트")
    void deleteNotSameUserQuestion_fail_withException() {
        // given
        given(answerDao.findAllByQuestionId(1L)).willReturn(Collections.emptyList());
        given(questionDao.findById(1L)).willReturn(new Question(1L, "differentWriter", "test", "test", new Date(), 0));

        // then
        Assertions.assertThatThrownBy(
                        () -> questionService.deleteQuestion(1L, new User("test", "1234", "name", "[email protected]")))
                .isInstanceOf(CannotDeleteQuestionException.class)
                .hasMessageContaining("다른 사용자의 글은 삭제할 수 없습니다.");
    }

    @Test
    @DisplayName("다른 사용자의 질문이 달려있는 질문 삭제시 예외 발생 테스트")
    void deleteQuestionWithOtherUserAnswer_fail_withException() {
        // given
        given(answerDao.findAllByQuestionId(1L)).willReturn(List.of(new Answer(1L, "differentWriter", "test", new Date(), 1L)));
        given(questionDao.findById(1L)).willReturn(new Question(1L, "test", "test", "test", new Date(), 0));

        // then
        Assertions.assertThatThrownBy(
                        () -> questionService.deleteQuestion(1L, new User("test", "1234", "name", "[email protected]")))
                .isInstanceOf(CannotDeleteQuestionException.class)
                .hasMessageContaining("질문에 다른 사용자의 답변이 달려있으므로 삭제할 수 없습니다.");
    }

    @Test
    @DisplayName("답변이 없는 사용자의 질문 삭제시 정상 동작 테스트")
    void deleteQuestionWithNoAnswer_success() {
        // given
        given(answerDao.findAllByQuestionId(1L)).willReturn(Collections.emptyList());
        given(questionDao.findById(1L)).willReturn(new Question(1L, "test", "test", "test", new Date(), 0));

        // then
        Assertions.assertThatNoException().isThrownBy(
                () -> questionService.deleteQuestion(1L, new User("test", "1234", "name", "[email protected]")));
    }

    @Test
    @DisplayName("자신이 답변을 단 질문 삭제시 정상 동작 테스트")
    void deleteQuestionWithAnswers_success() {
        // given
        given(answerDao.findAllByQuestionId(1L)).willReturn(List.of(new Answer(1L, "test", "test", new Date(), 1L)));
        given(questionDao.findById(1L)).willReturn(new Question(1L, "test", "test", "test", new Date(), 0));

        // then
        Assertions.assertThatNoException().isThrownBy(
                () -> questionService.deleteQuestion(1L, new User("test", "1234", "name", "[email protected]")));

    }

}

DI를 적용한 QuestionService에서 Mockito를 이용하여 실제 DB와 연결하지 않고, 순수 서비스 코드만 격리해 테스트할 수 있다.

❗️ 이슈: QuestionServiceTest 실패: Mocito의 UnnecessaryStubbingException

컨트롤러, 매핑 코드 리팩토링

기존 레거시 컨트롤러, New MVC 컨트롤러 및 Legacy Handler Mapping 코드들은 생성자 의존성 주입이 적용되지 않은 상태이므로 이 또한 생성자로 주입받을 수 있게 수정한다.

• LegacyHandlerMapping, LegacyController