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

안녕하세요 영한님 강의 정말 잘 듣고 있습니다. ㅎㅎ

@Transactional에 대해서 궁금한 점이 생겨서 질문드립니다.

@Test
@Transactional
public void 상품주문() throws Exception {
    //given
    Member member = createMember();

    Book book = createBook("시골 JPA", 10000, 10);

    int orderCount = 2;

    //when
    Long orderId = orderService.order(member.getId(), book.getId(), orderCount);

    //then
    Order getOrder = orderRepository.findOne(orderId);

    assertEquals("상품 주문시 상태는 ORDER", OrderStatus.ORDER, getOrder.getStatus());
    assertEquals("주문한 상품 종류 수가 정확해야 한다.", 1, getOrder.getOrderItems().size());
    assertEquals("주문 가격은 가격 * 수량이다.", 10000 * orderCount, getOrder.getTotalPrice());
    assertEquals("주문 수량만큼 재고가 줄어야 한다.", 8, book.getStockQuantity());
}

해당 코드는 강의에서 작성한 테스트 코드 입니다.

지만 저는 지금까지 @Transactional을 테스트 코드에 작성하지 않는 것이 좋다고 알고 있었습니다. 그 이유는 @Transactional을 깜빡하고 작성하지 않는 문제를 테스트 코드에서 잡아 낼 수 있고, 테스트 트랜잭션의 롤백 정책으로 인해 실제 로직과 다르게 작동하기 때문입니다.

아래는 제가 @Transactional을 제거하고 코드를 작성해 보았습니다.

@Test
public void 상품주문() throws Exception {
    //given
    Member member = createMember();

    Book book = createBook("시골 JPA", 10000, 10);

    int orderCount = 2;

    //when
    Long orderId = orderService.order(member.getId(), book.getId(), orderCount);

    //then
    Order getOrder = orderRepository.findOne(orderId);

    assertEquals("상품 주문시 상태는 ORDER", OrderStatus.ORDER, getOrder.getStatus());
    assertEquals("주문한 상품 종류 수가 정확해야 한다.", 1, getOrder.getOrderItems().size());
    assertEquals("주문 가격은 가격 * 수량이다.", 10000 * orderCount, getOrder.getTotalPrice());

    //Transactional이 없기 때문에 객체를 다시 불러와서 조회
    Item findBook = itemRepository.findOne(book.getId());
    assertEquals("주문 수량만큼 재고가 줄어야 한다.", 8, findBook.getStockQuantity());
}

private Book createBook(String name, int price, int stockQuantity) {
    Book book = new Book();
    book.setName(name);
    book.setPrice(price);
    book.setStockQuantity(stockQuantity);
    itemService.saveItem(book); // service에서 저장
    return book;
}

private Member createMember() {
    Member member = new Member();
    member.setName("회원1");
    member.setAddress(new Address("서울", "강가", "123-123"));
    memberService.join(member); // service에서 저장
    return member;
}

@Repository
public class OrderRepository {

    private final EntityManager em;

    public OrderRepository(EntityManager em) {
        this.em = em;
    }

    // fetch join으로 LazyLoding 문제 해결
    public Order findOne(Long id) {
        return em.createQuery(
                        "select o from Order o join fetch o.orderItems where o.id = :id",
                        Order.class
                )
                .setParameter("id", id)
                .getSingleResult();
    }

위와 같이 코드를 수정해서 @Transactional이 없어도 테스트를 통과하도록 변경하였습니다. 하지만 아래의 코드는 모든 객체를 테스트 작성자가 신경써야 하고 JPA가 관리하는 여러 가지 기능들을 사용하지 못하게 됩니다. 또한 rollback 기능을 사용하지 못해 테스트 작성자가 일일이 DB에 저장된 값을 지워줘야 합니다.

영한님은 이와 같은 상황에서는 어떤 선택이 좋다고 생각하는지 궁금합니다.