이 경우, @BeforeEach에 fixture를 구성해도 될까요?
983
작성한 질문수 55
지금 OrderServiceTest 클래스의 createOrder(), createOrderWithStock(), ... 메서드들은 주문을 생성하기 위해서 given절에서 모두 동일하게 product1, 2, 3을 먼저 생성해주는 fixture가 있습니다.
@Test
void createOrder() {
// given
LocalDateTime registeredDateTime = LocalDateTime.now();
Product product1 = createProduct(HANDMADE, "001", 1000);
Product product2 = createProduct(HANDMADE, "002", 3000);
Product product3 = createProduct(HANDMADE, "003", 5000);
productRepository.saveAll(List.of(product1, product2, product3));
OrderCreateServiceRequest request = OrderCreateServiceRequest.builder()
.productNumbers(List.of("001", "002"))
.build();
// when
OrderResponse orderResponse = orderService.createOrder(request, registeredDateTime);
// then
assertThat(orderResponse.getId()).isNotNull();
assertThat(orderResponse)
.extracting("registeredDateTime", "totalPrice")
.contains(registeredDateTime, 4000);
assertThat(orderResponse.getProducts()).hasSize(2)
.extracting("productNumber", "price")
.containsExactlyInAnyOrder(
tuple("001", 1000),
tuple("002", 3000)
);
}product를 생성해주는 부분을 @BeforeEach로 빼내려고 하는데, 적절하다고 판단되시는지 궁금합니다.
OrderServiceTest의 모든 메서드가 동일한 product를 생성하고는 있지만,
각 테스트 메서드에서 "001"이라는 상품이 있는지는 @BeforeEach 메서드까지 스크롤을 왔다갔다하면서 확인해야될거라 생각합니다.
(만약 없는 상품을 주문한다면 given절에서 예외가 발생하는 것이기 때문에, 이 부분은 당연히 있는 상품을 주문한다는 것을 보장하고 테스트 코드를 작성해야 할까요?)
order에 관한 테스트이기 때문에, product에 대한 부분은 분리해도 될거라 생각하지만, 우빈님의 말씀대로 스크롤을 왔다갔다하면서 @BeforeEach의 메서드를 수시로 보는 것은 안좋다고 느껴져서..
product 생성에 대한 fixture를 분리괜찮다고 생각하시나요?
답변 2
3
안녕하세요, 재영님! :)
좋은 질문 주셨네요 ㅎㅎ
이 부분은 정답이 있는 것도 아니고 확실히 사람마다 기준이 다를 수 있기 때문에, 제 기준을 말씀드려 보겠습니다.
결론부터 얘기하자면 저는 같은 상황에서 Product 생성도 지금처럼 각 테스트에서 진행할 것 같아요.
그렇게 생각하는 이유는, 이미 말씀해주신 부분처럼 주문을 하기 위해서 결국 Product의 productNumber를 알아야 하고, 이 정보를 가지고 Request 를 생성하여 요청해야 하기 때문입니다.
Order 생성에 Product가 조금이나마 직접적인 영향을 주고 있다고 생각한 것이죠.
(다시 말하지만 이는 사람마다 생각이 다를 수 있습니다 ㅎㅎ)
그래서 저는 보통 @BeforeEach에서 하는 작업을 분리하려고 할 때, 기본적으로는 보수적으로 접근하는 편이고, 각 테스트에서 전혀 몰라도 되는 준비 과정만 setUp() 과정으로 분리하려고 하는 편입니다.
예를 들면 다음과 같은 내용이에요.
해당 서비스 로직의 요청/응답에도 드러나지 않지만, 테스트가 정상적으로 동작하기 위해서 이미 생성되어 있어야 하는 fixture들
ex. 매장에 주문을 등록하는 테스트를 하려고 한다.
setUp()- 가정 : 하나의 매장을 해당 테스트 클래스 전체에서 공유해도 문제가 없고, 매장의 상태 변화에 관심을 가질 필요도 없을 때 (매장의 상태가 변하지 않을 때)1. 매장을 생성한다.
2. 매장을 운영 상태로 만들기 위한 여러 설정값을 세팅한다. (운영시간 등)
각 테스트3. 상품을 생성한다.
4. 주문을 등록한다.
비즈니스 로직과 관련은 없지만 통합 테스트를 위한 서버를 띄우기 위해 해야 하는 작업들 (그런게 있다면 말이죠)
가독성에 관련된 부분이라, 저도 작성할 때마다 늘 고민을 하는 부분이기는 해요.
제 기준이 부디 도움이 되셨기를 바랍니다.
감사합니다. :)
0
안녕하세요, 인프런 AI 인턴이에요. @BeforeEach는 각각의 테스트 메서드가 실행되기 전에 공통적으로 실행되는 메서드입니다. 따라서, fixture를 구성하는 작업을 @BeforeEach에 작성하면 모든 테스트 메서드에서 동일한 fixture를 사용할 수 있어요.
현재 OrderServiceTest 클래스의 createOrder(), createOrderWithStock() 메서드에서 product1, 2, 3을 먼저 생성하고 있어요. 이 경우에 @BeforeEach에 fixture를 구성한다면 각각의 테스트 메서드가 실행될 때마다 product1, 2, 3을 생성하는 코드를 작성하지 않아도 되겠죠.
그러나, 주의해야 할 점이 있습니다. @BeforeEach에서 fixture를 구성하면 fixture의 상태가 각각의 테스트 메서드에 영향을 줄 수 있기 때문에, 테스트 메서드가 독립적인 상태를 유지해야 하는 경우에는 조심해야해요. 만약 테스트 메서드 간에 fixture의 상태에 의존성이 있는 경우에는 @BeforeEach 대신에 각각의 테스트 메서드 안에서 fixture를 생성하고 사용하는 것이 더 좋을 수도 있어요.
결론적으로, @BeforeEach에 fixture를 구성해도 될지는 테스트 메서드의 독립성과 fixture의 상태에 따라 다를 수 있어요. 적절한 상황에 맞게 사용하시면 될 것 같아요. 그럼 좋은 결과 있으시길 바래요!
private 상수 테스트 관련 질문
0
83
1
void는 어떻게 테스트하나요..? void로 애초에 코딩하면 안되나요??
0
126
2
커버리지는 어떻게 활용하시는지 궁금합니다.
0
161
2
테스트 문서화 질문입니다
0
105
2
단위테스트 질문이 있습니다
0
96
2
컨트롤러는 모킹을 한 이유가 궁금합니다.
0
103
2
ERD 가장자리에 있는 도메인 테스트 질문
0
89
2
DTO 검증 필드에 대한 테스트 코드 작성은 어디까지?
0
135
2
OrderCreateRequest DTO에 대해서 궁금한점
0
102
2
고전파의 테스트 대역 사용 대상, 공유 의존성
0
156
2
계층 관련 질문이 있습니다.
0
139
3
'코틀린'에서는 빌더를 따로 쓰지 않는데, 이 때는 어떻게 test fixture를 만드시는지 궁금합니다
1
124
2
혹시 update 로직은 어떻게 테스트하나요? (@Setter?)
0
135
2
단위테스트와 통합테스트의 경계가 궁금합니다.
0
229
2
Service+Repository 통합테스트 관련 질문입니다.
0
150
2
OrderControllerDocsTest 작성 해봤는데요. 날짜 형식이 이상하게 나와요
0
185
2
test 용 .yml
0
90
2
throws Exception
0
80
2
카페키오스크 클래스 문의 ,,
0
89
2
Rest docs 문서용 테스트코드를 따로 작성해야 되나요?
0
174
2
테스트 코드에서 필요한 생성자
0
138
1
tearDown 순서
0
116
2
@Builder 생성자 private
0
136
2
@DisplayName gradle / intellJ
0
93
2





