📌 [MarketBridge] 03. 개발일지 - IDENTITY 전략에 따른 테스트 실패

📖 IDENTITY 전략에 따른 테스트 실패

@Transactional
@SpringBootTest
@ActiveProfiles("test")
class ProductListResponseDtoTest {

    @Autowired ProductRepository productRepository;

    @Test
    @DisplayName("주문 상세가 주어졌을 때 ProductListResponseDto 변환")
    public void ofWithOrderDetail() {
        Product product = Product.builder()
                .price(1000L)
                .name("빵빵이")
                .productNo("123")
                .build();
        productRepository.save(product);

        OrderDetail orderDetail = OrderDetail.builder()
                .product(product)
                .price(product.getPrice())
                .quantity(2L)
                .build();

        // when
        ProductListResponseDto result = ProductListResponseDto.of(orderDetail);

        // then
        assertThat(result).extracting("productId", "productNo", "name", "price", "quantity")
                .contains(1L, "123", "빵빵이", 1000L, 2L);
    }


    @Test
    @DisplayName("상품과 수량이 주어졌을 때 ProductListResponseDto 변환")
    public void ofWithProductAndQuantity() {
        Long quantity = 2L;

        Product product = Product.builder()
                .price(1000L)
                .name("빵빵이")
                .productNo("123")
                .build();
        productRepository.save(product);

        // when
        ProductListResponseDto result = ProductListResponseDto.of(product, quantity);

        // then
        assertThat(result).extracting("productId", "productNo", "name", "price", "quantity")
                .contains(1L, "123", "빵빵이", 1000L, 2L);

    }

@Transactional를 통해 각 테스트가 롤백되어 productId가 모두 1L 될 것으로 예상했다.

기대와 달리 실패를 하였다. 첫번째 테스트(ofWithOrderDetail)의 productId의 값이 2L 되었다. 뭐야.. 왜..?

처음에는 롤백이 안됐나? 생각했지만

Hibernate: 
    insert 
    into
        product
        (category_id, created_at, deleted_at, discount_rate, is_own, is_subs, name, price, product_no, stock, thumb_img, updated_at, product_id) 
    values
        (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, default)
2024-01-29T22:28:07.219+09:00  INFO 3636 --- [           main] p6spy                                    : #1706534887219 | took 4ms | statement | connection 3| url jdbc:h2:mem:~/Marketbridge
insert into product (category_id,created_at,deleted_at,discount_rate,is_own,is_subs,name,price,product_no,stock,thumb_img,updated_at,product_id) values (?,?,?,?,?,?,?,?,?,?,?,?,default)
insert into product (category_id,created_at,deleted_at,discount_rate,is_own,is_subs,name,price,product_no,stock,thumb_img,updated_at,product_id) values (NULL,NULL,NULL,NULL,NULL,NULL,'빵빵이',1000,'123',NULL,NULL,NULL,default);
2024-01-29T22:28:07.299+09:00  INFO 3636 --- [           main] p6spy                                    : #1706534887299 | took 0ms | rollback | connection 3| url jdbc:h2:mem:~/Marketbridge

;
Hibernate: 
    insert 
    into
        product
        (category_id, created_at, deleted_at, discount_rate, is_own, is_subs, name, price, product_no, stock, thumb_img, updated_at, product_id) 
    values
        (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, default)
2024-01-29T22:28:07.309+09:00  INFO 3636 --- [           main] p6spy                                    : #1706534887309 | took 0ms | statement | connection 4| url jdbc:h2:mem:~/Marketbridge
insert into product (category_id,created_at,deleted_at,discount_rate,is_own,is_subs,name,price,product_no,stock,thumb_img,updated_at,product_id) values (?,?,?,?,?,?,?,?,?,?,?,?,default)
insert into product (category_id,created_at,deleted_at,discount_rate,is_own,is_subs,name,price,product_no,stock,thumb_img,updated_at,product_id) values (NULL,NULL,NULL,NULL,NULL,NULL,'빵빵이',1000,'123',NULL,NULL,NULL,default);
2024-01-29T22:28:07.317+09:00  INFO 3636 --- [           main] p6spy                                    : #1706534887317 | took 0ms | rollback | connection 4| url jdbc:h2:mem:~/Marketbridge

;

insert문을 보면 rollback이 된 것을 볼 수 있다.

이리 저리 삽질하던 중 엔티티의 ID가 Identity전략으로 만들어지고 있었는데 이것이 문제가 되고 있었다...

✏️ IDENTITY 전략

잠시 IDENTITY 대해 알아보자.

IDENTITY 전략은 분명한 단점을 가지고 있다. 데이터베이스에 값을 저장하고 나서야 기본 키 값을 구할 수 있다.
이것이 무슨 말이냐? 영속성 컨텍스트에 1차 캐시(영속화)를 할 때 엔티티는 Id를 무~조건 가지고 있어야 하는데 데이터베이스에서 값을 제공해주기 때문에 단순히 키를 얻는 과정인데도 불구하고 네트워크를 한 번 타야하는 불상사가 일어난다.

정리하면 데이터베이스에 가서 Id를 받아온 후 영속성 컨텍스트에 캐시가 된다.

만약 이게 싫다면? SEQUENCE 전략을 한 번 알아보는 것을 추천한다!
최적화 부분에서 이득을 얻을 수 있을 것이다.

❗IDENTITY 전략 주의점

IDENTITY 전략에 대해 대충 알아보았는데 도대체 이게 테스트에서 어떤 문제를 일으킨걸까?

IDENTITY 전략을 사용할 때 주의할 점이 존재한다.
AUTO INCREMENT 는 Transaction 을 사용해도 rollback 되지 않는다.
이게 무슨말이냐면, Insert 쿼리를 실패해도 Table 의 AUTO INCREMENT 컬럼의 값은 1이 증가된다.

위에 테스트를 다시 예로 들어보면 IDENTITY 전략은 rollback이 되지 않기 때문에 두번째로 진행된 테스트의 productId가 2가 된 것이다.

그렇다면 어떻게 해결해야 할까?
AUTO INCREMENT 값을 신뢰해서는 안된다. 만약 AUTO INCREMENT 값을 중요하게 사용한다면, innodb_autoinc_lock_mode을 잘 알아야한다.

innodb_autoinc_lock_mode에 대해서는 따로 알아보지 않겠다. 필요하다면 검색해서 보는 것을 추천하겠당

🕶️ 결론

지금 프로젝트는 my-sql을 사용하고 있기 때문에 IDENTITY전략을 바꿀 수 없을것으로 생각된다.
그렇다고 innodb_autoinc_lock_mode를 공부해서 적용하기엔 db무뇌한으로서는 스프린트를 해야하는 지금 시기에 어려워보인다. (아무것도 모르기에 시간 예측이 안되기 때문,,,ㅠㅠ)

지금 단순한 테스트를 하는데 Id를 위해서 스프링을 띄우는 것과 db를 공부하는 것 두개 다 부담스럽기 때문에 Reflection을 통해 스프링 없이 단위테스트를 진행 할 예정이다. Id만 넣어주면 되니깐..! 이 방법이 지금 상황에서 좋은 판단이라고 생각된다.

✍️ 코드 수정

class ProductListResponseDtoTest {


    @Test
    @DisplayName("주문 상세가 주어졌을 때 ProductListResponseDto 변환")
    public void ofWithOrderDetail() {
        Product product = Product.builder()
                .price(1000L)
                .name("빵빵이")
                .productNo("123")
                .build();
        ReflectionTestUtils.setField(product, "id", 1L, Long.class);
        OrderDetail orderDetail = OrderDetail.builder()
                .product(product)
                .price(product.getPrice())
                .quantity(2L)
                .build();

        // when
        ProductListResponseDto result = ProductListResponseDto.of(orderDetail);

        // then
        assertThat(result).extracting("productId", "productNo", "name", "price", "quantity")
                .contains(1L, "123", "빵빵이", 1000L, 2L);
    }


    @Test
    @DisplayName("상품과 수량이 주어졌을 때 ProductListResponseDto 변환")
    public void ofWithProductAndQuantity() {
        Long quantity = 2L;

        Product product = Product.builder()
                .price(1000L)
                .name("빵빵이")
                .productNo("123")
                .build();
        ReflectionTestUtils.setField(product, "id", 1L, Long.class);

        // when
        ProductListResponseDto result = ProductListResponseDto.of(product, quantity);

        // then
        assertThat(result).extracting("productId", "productNo", "name", "price", "quantity")
                .contains(1L, "123", "빵빵이", 1000L, 2L);

    }

}

ReflectionTestUtils를 통해 값을 셋팅해주었다.
스프링 없이 단위테스트로 진행하기 때문에 훨씬 빠른 속도를 보인다!!