inflearn logo
강의

강의

N
챌린지

챌린지

멘토링

멘토링

N
클립

클립

로드맵

로드맵

지식공유

Practical Testing: 실용적인 테스트 가이드

Test Fixture 클렌징

이 경우, @BeforeEach에 fixture를 구성해도 될까요?

983

재영

작성한 질문수 55

0

지금 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를 분리괜찮다고 생각하시나요?

spring tdd jpa mockito 소프트웨어-테스트 junit5

답변 2

3

박우빈

안녕하세요, 재영님! :)
좋은 질문 주셨네요 ㅎㅎ

이 부분은 정답이 있는 것도 아니고 확실히 사람마다 기준이 다를 수 있기 때문에, 제 기준을 말씀드려 보겠습니다.
결론부터 얘기하자면 저는 같은 상황에서 Product 생성도 지금처럼 각 테스트에서 진행할 것 같아요.

그렇게 생각하는 이유는, 이미 말씀해주신 부분처럼 주문을 하기 위해서 결국 Product의 productNumber를 알아야 하고, 이 정보를 가지고 Request 를 생성하여 요청해야 하기 때문입니다.
Order 생성에 Product가 조금이나마 직접적인 영향을 주고 있다고 생각한 것이죠.
(다시 말하지만 이는 사람마다 생각이 다를 수 있습니다 ㅎㅎ)

그래서 저는 보통 @BeforeEach에서 하는 작업을 분리하려고 할 때, 기본적으로는 보수적으로 접근하는 편이고, 각 테스트에서 전혀 몰라도 되는 준비 과정만 setUp() 과정으로 분리하려고 하는 편입니다.

예를 들면 다음과 같은 내용이에요.

  • 해당 서비스 로직의 요청/응답에도 드러나지 않지만, 테스트가 정상적으로 동작하기 위해서 이미 생성되어 있어야 하는 fixture들

    • ex. 매장에 주문을 등록하는 테스트를 하려고 한다.

      • setUp() - 가정 : 하나의 매장을 해당 테스트 클래스 전체에서 공유해도 문제가 없고, 매장의 상태 변화에 관심을 가질 필요도 없을 때 (매장의 상태가 변하지 않을 때)

        • 1. 매장을 생성한다.

        • 2. 매장을 운영 상태로 만들기 위한 여러 설정값을 세팅한다. (운영시간 등)

      • 각 테스트

        • 3. 상품을 생성한다.

        • 4. 주문을 등록한다.

  • 비즈니스 로직과 관련은 없지만 통합 테스트를 위한 서버를 띄우기 위해 해야 하는 작업들 (그런게 있다면 말이죠)

가독성에 관련된 부분이라, 저도 작성할 때마다 늘 고민을 하는 부분이기는 해요.
제 기준이 부디 도움이 되셨기를 바랍니다.

감사합니다. :)

0

재영

setUp() 과정으로 분리해야 하는 부분에 대해 적당한 예시가 떠오르지 않아서 고민이 많았는데, 시원하게 해결된 것 같습니다!

답변 정말 감사합니다 ㅎ

0

인프런 AI 인턴

안녕하세요, 인프런 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