inflearn logo
강의

강의

N
챌린지

챌린지

멘토링

멘토링

N
클립

클립

로드맵

로드맵

지식공유

실전! 코틀린과 스프링 부트로 도서관리 애플리케이션 개발하기 (Java 프로젝트 리팩토링)

26강. 유저 대출 현황 보여주기 - 테스트 코드 개발

테스트코드 질문드립니다.

해결된 질문

510

랫서

작성한 질문수 1

1

안녕하세요, 강사님.
강사님 덕분에 항상 코틀린 공부 재미있게 하고 있습니다!

getUserLoanHistoriesTest2 테스트 코드 관련해서 질문드립니다.
제가 강의를 보기 전에 혼자 작성해보았던 코드는 아래와 같은데요.

// given
val savedUser = userRepository.save(User("A", null))
savedUser.loanBook(Book.fixture("book1"))
savedUser.loanBook(Book.fixture("book2"))
savedUser.loanBook(Book.fixture("book3"))
savedUser.returnBook("book3")
userRepository.save(savedUser)

테스트는 통과가 되는데, 위와 같이 해도 테스트 코드로서 의미가 있는지, 그런면에서 userLoanHistoryRepository.saveAll()을 사용하는 것이 더 적절한 건지 궁금합니다.

java spring kotlin spring-boot 리팩토링

답변 1

1

최태현

안녕하세요 랫서님! 질문 올려주셔서 감사드립니다!! 🙏 강의 보기 전 테스트 코드를 직접 작성해보시니 너무 좋네요~!! 😊

결론부터 말씀드리자면 충분히 의미 있는 테스트 코드가 맞습니다!!! 👍

 

제가 강의해서 saveAll() 을 사용했던 이유는, 테스트 코드 역시 저희가 유지보수해야할 대상이기 때문에, 코드의 중복을 조금 더 제거하기 위해서 였습니다!! save() 보다는 saveAll() 의 코드가 (아주 아주 약간이지만) 조금 더 간결하고, 테스트 메소드가 많아지면, saveAll() 보다는 아래와 같은 private 함수가 조금 더 간결해질거에요!!

@Test
fun test1() {
  // given
  // 여기서는 책 2권이 필요했다.
  saveBooks("책1", "책2")
}

@Test
fun test2() {
  // given
  // 여기서는 3권이 필요했다.
  saveBooks("책1", "책2", "책3")
}

// 느낌만 말씀드려보면, 대략 이런 느낌입니다!!!!!
private fun saveBooks(vararg bookNames: String) {
  for (bookName in bookNames) {
    bookRepository.save(Book.fixture(bookName))
  }
}

포인트는 "테스트 코드 역시 어떻게 하면 프로덕션 코드처럼 깔끔하게 잘 관리할 수 있을까" 를 고민하는 것입니다!! 🙏

 

혹시나 또 궁금한 점이 있으시면 편하게 질문 부탁드려요!!

감사합니다~!!! 🙇🙇

1

랫서

답변 감사합니다~!
이해가 잘 되었습니다.
좀 더 코드의 중복과 간결성을 생각하면서 짜봐야겠네요😀

안녕하세요 혹시 프론트 코드 제공받을 수 있을까요?

0

69

2

실행이 안되네요

0

67

2

프론트 영역 보는법

0

52

2

companion object

0

79

2

Custom 레프직토리 형식

0

60

2

Querydsl 도입

0

68

2

fetch join DISTINCT 중복제거

1

84

2

표준 예외와 커스텀 예외 사용 전략 질문

0

92

3

이 질문이 왜 없는지 이해가 안 되지만 문제 인식 및 해결 방법 남깁니다.

1

177

2

테스트를 위한 코드

1

105

2

프로젝트 실행 에러

0

153

2

PDF 문서에 오타가 있어서 알려드립니다.

1

102

1

enum질문

1

86

1

테스트 후 AfterEach 함수에서 나오는 쿼리

0

136

2

테스트 fixture

1

211

2

./gradlew test 실행시 인식할수 없다고 뜹니다.

0

141

1

test 코드 실행시 경고가 발생합니다.

0

141

1

13강 User Kotlin 변환중

2

170

3

'추가 - 코프링과 플러그인' 강의 7:46 allopen 관련 질문

2

174

1

-

0

141

2

4:28 build.gradle 수정 시 kotlin-reflect관련 implementation 추가 해야할까요?

0

314

3

junit import 불가

0

276

3

테스트 코드와 관련하여 질문이 있습니다.

1

219

1

hibernate가 select를 두번 하는 이유

0

240

1