3 asked

1

안녕하세요, 좋은 질문 감사드립니다!! 🙏 먼저 테스트 코드를 작성해보시고 유추해보시는게 정말 좋네요! 👍

제가 확인한 내용을 말씀드려보면, 현재 loanHistory 불러오고 확인하는 시점의 문제입니다!

@Test
@DisplayName("책 반납 성공")
fun returnTest(){
    //given
    val user = User("phd",31)
    userRepository.save(user)

    val book = Book("testBook")
    bookRepository.save(book)

    val loanHistory = UserLoanHistory(user,"testBook",false) <--- (1)
    userLoanHistoryRepository.save(loanHistory) 

    //when
    assertThat(loanHistory.isReturn).isFalse() <--- (2)
    bookService.returnBook(BookReturnRequest(user.name, "testBook"))
        
    //then
    assertThat(loanHistory.isReturn).isTrue() <--- (3)
}

(1)번 코드를 보시면, loanHistory 는 저희가 만든 객체입니다! DB에서 데이터를 가져오거나 하지 않았고, 그냥 객체를 만든거죠! 때문에 (2)번은 당연히 성공하게 됩니다!

자 그리고 이제 bookService.returnBook() 이 호출되고요~~~ 문제의 (3)번 코드로 가게 되는데요!

여기서도 loanHistory 는 bookService.returnBook() 으로 인해 업데이트된 데이터가 반영되지 않은 저희가 (1)번에서 만든 객체입니다. 때문에 여전히 isReturn이 false인거!

여기서 변경을 최소화하시면서 테스트를 성공시키고 싶으시다면, DB에서 데이터를 가져와주기만 하면 됩니다!

@Test
@DisplayName("책 반납 성공")
fun returnTest(){
    //given
    val user = User("phd",31)
    userRepository.save(user)

    val book = Book("testBook")
    bookRepository.save(book)

    val loanHistory = UserLoanHistory(user,"testBook",false)
    userLoanHistoryRepository.save(loanHistory) 

    //when
    assertThat(loanHistory.isReturn).isFalse()
    bookService.returnBook(BookReturnRequest(user.name, "testBook"))
        
    //then - 변경된 부분!!!!
    val changedHistory = userLoanHistoryRepository.findAll()[0]
    assertThat(changedHistory.isReturn).isTrue()
}

이렇게 되면 DB에 있는 데이터를 실제로 가져와 확인하기 때문에 테스트가 통과하시게 될겁니다!!! 👍

감사합니다!!! 또 질문 있으시면 편하게 올려주세요~!! 🙏🙏