인프런 커뮤니티 질문&답변

 @DisplayName("상품 번호 리스트로 상품목록을 조회하기")
 @Test
 void findAllByProductNumberIn(){
     //given
     Product product1 = createProduct("001", 1000);
     Product product2 = createProduct("002", 3000);
     Product product3 = createProduct("003", 5000);
     productRepository.saveAll(List.of(product1, product2, product3));

     //when
     List<Product> findProducts = productRepository.findAllByProductNumberIn(List.of("001", "003"));

     //then
     assertThat(findProducts).hasSize(2)
             .extracting("productNumber","price")
             .containsExactlyInAnyOrder(
                    tuple("001",1000),
                    tuple("003",5000)
             );
 }

1.

데이터 계층은 유효한 상품 번호 리스트만 넘어왔다는 전제로만 테스트를 해도 충분한가요?

유효하지 않은 상품번호는 검증 로직에서 걸러졌다고 생각하고

테스트를 한다고 생각하면 될까요?

  @DisplayName("미등록된 상품 번호 리스트로 상품목록을 조회하기")
    @Test
    void findAllByProductNumberIn(){
        //given
        Product product1 = createProduct("001", 1000);
        Product product2 = createProduct("002", 3000);
        Product product3 = createProduct("003", 5000);
        productRepository.saveAll(List.of(product1, product2, product3));

        //when
        List<Product> findProducts = productRepository.findAllByProductNumberIn(List.of("004", "005"));

        //then
        assertThat(findProducts).hasSize(0);
    }

2.

JpaRepository가 제공하는 기본 save,findAll 등도

개발자가 원하는 데이터가 올바르게 저장 되었는지, 조회가 되었는지 테스트도

실무에서는 작성하시나요 ?

아니면 이미 만들어진 코드이기 때문에 불필요한 테스트 코드라 생각해서

넘어가는지 궁금합니다.

    @DisplayName("주문 생성시 상품 리스트에서 주문의 총 금액을 계산한다.")
    @Test
    void calculrateTotalPrice(){
        //given
        List<Product> products = List.of(
                createProduct("001", 1000),
                createProduct("002", 2000)
        );
        //when
        Order order = Order.create(products, LocalDateTime.now());

        //then
        assertThat(order.getTotalPrice()).isEqualTo(3000);
    }

    private Product createProduct(String productNumber, int price) {
        
        return Product.builder()
                .type(HANDMADE)
                .productNumber(productNumber)
                .price(price)
                .sellingStatus(SELLING)
                .name("메뉴 이름")
                .build();
    }

    @DisplayName("주문 생성시 주문 상태는 INIT 이다.")
    @Test
    void init(){
        //given
        List<Product> products = List.of(
                createProduct("001", 1000),
                createProduct("002", 2000)
        );
        //when
        Order order = Order.create(products, LocalDateTime.now());

        //then
        assertThat(order.getOrderStatus()).isEqualByComparingTo(OrderStatus.INIT);
    }

    @DisplayName("주문 생성시 등록시간을 기록한다.")
    @Test
    void registeredDataTime(){
        //given
        LocalDateTime registeredDateTime = LocalDateTime.now();
        List<Product> products = List.of(
                createProduct("001", 1000),
                createProduct("002", 2000)
        );
        //when
       
        Order order = Order.create(products, registeredDateTime);

        //then
        assertThat(order.getRegisteredDateTime()).isEqualTo(registeredDateTime);
    }

Order.create 테스트 코드를 작성할때엔

각각 필드 초기화를 단위테스트를 진행했는데,

    @DisplayName("주문번호 리스트를 받아 주문을 생성한다.")
    @Test
    void createOrder(){
        
        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));
        
        OrderCreateRequest request = OrderCreateRequest.builder()
                .productNumbers(List.of("001", "002"))
                .build();

        OrderResponse orderResponse = orderService.createOrder(request, registeredDateTime);
        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)
                );
    }

createOrder 테스트 코드는 같이 검사를 했습니다.

3.

각 초기화를 해주는 정적 매서드를 테스트 코드로 작성을 했는데

Order.create 테스트 코드는 필드마다 분리를 해서 테스트 코드를 작성하고,

createOrder 테스트 코드는 같이 검사를 했습니다.

또 강의에는 없지만 ProductResponse.of 정적 메서드로도 초기화를 했는데

따로 분리해서 테스트 코드를 작성하지 않고

OrderServiceTest의 createOrder()에서 같이 테스트 코드에 포함되었습니다.

테스트 코드를 분리하는 경우와 같이 검사하는 경우를 나누는 기준이 있을까요?