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

안녕하세요.

강의를 들은 후 테스트 코드 작성을 연습하고 있습니다.

엔티티에서 Soft delete를 위한 필드(isDeleted)가 정의되어 있습니다.

Soft delete가 된 데이터는 조회할 필요가 없기 때문에 엔티티에 @SQLRestriction("is_deleted = false")를 정의했습니다.

테스트 코드에서 해당 어노테이션 때문에 tearDown() 메서드 동작시 문제가 발생했습니다.

.deleteAllInBatch()가 실행될 때, where 조건이 포함되기 때문에 데이터가 삭제되지 않습니다.

상품 목록 조회에 대한 테스트 코드를 작성할 때, soft delete 된 데이터는 빠지고 정상적인 데이터만 조회되는지 보려고 isDeleted = true 값을 준 테스트 데이터도 생성해서 테스트 코드를 작성했습니다. 해당 경우에는 어떤식으로 테스트 코드를 작성해야 할까요?

1. isDeleted = true 테스트 데이터를 만들지 않는다.

2. @SQLRestriction("is_deleted = false") 를 사용하지 않고, 쿼리 조회 시 IsDeletedTrue 조건을 주도록 한다.

위 방법이 아닌 Best Practice가 있는지 궁금합니다.

아래는 테스트 코드 예시 입니다.

• @SQLRestriction("is_deleted = false") 적용된 엔티티 예시입니다.

@Entity
@SQLRestriction("is_deleted = false")
@Getter
@NoArgsConstructor(access = AccessLevel.PROTECTED)
public class Product extends BaseDateTimeEntity {

    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    @Column(name = "product_id")
    private Long id;

    @Column(length = 50)
    private String name;

    private Boolean isDeleted;

    @OneToMany(mappedBy = "product", cascade = CascadeType.ALL)
    private List<ProductOption> productOptions = new ArrayList<>();

   // ... 중략
}

@Entity
@SQLRestriction("is_deleted = false")
@Getter
@NoArgsConstructor(access = AccessLevel.PROTECTED)
public class ProductOption extends BaseDateTimeEntity {

    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    @Column(name = "product_option_id")
    private Long id;

    @ManyToOne(fetch = FetchType.LAZY)
    @JoinColumn(name = "product_id")
    private Product product;

    @Column(length = 30)
    private String name;

    private Boolean isDeleted;

    // ... 중략
}

• isDeleted = true 테스트 데이터를 포함한 테스트 코드 예시입니다.

class ProductServiceTest extends ServiceTestSupport {

    @Autowired private ProductRepository productRepository;

    @AfterEach
    void tearDown() {
        productOptionRepository.deleteAllInBatch();
        productRepository.deleteAllInBatch();
    }

    @Test
    @DisplayName("상품 목록을 조회한다.")
    void getSellingProducts() throws Exception {
        // given
        ProductOption productOption1 = createProductOption("옵션1", false);
        ProductOption productOption2 = createProductOption("옵션2", false);
        ProductOption productOption3 = createProductOption("옵션3", false);
        ProductOption productOption4 = createProductOption("옵션4", true);  // isDeleted = true
        ProductOption productOption5 = createProductOption("옵션5", false);
        ProductOption productOption6 = createProductOption("옵션6", true);  // isDeleted = true

        Product product1 = createProduct("상품1", false, List.of(productOption1, productOption2));
        Product product2 = createProduct("상품2", false, List.of(productOption3, productOption4));
        Product product3 = createProduct("상품3", true, List.of(productOption5, productOption6)); // isDeleted = true
        productRepository.saveAll(List.of(product1, product2, product3));

        // ... 중략

        // when
        List<product> products = productService.getSellingProducts();

       // then
       // ... 중략
    }

    // ... 중략
}

• deleteAllInBatch() 메서드가 실행된 후 로그 입니다.

Hibernate: 
    delete      
    from
        product_option      
    where
        (
            product_option.is_deleted = false         
        )

Hibernate: 
    delete 
    from
        product 
    where
        (
            product.is_deleted = false
        )

org.springframework.dao.DataIntegrityViolationException: JDBC exception executing SQL [delete from product where (product.is_deleted = false)] [Referential integrity constraint violation: "FKN4HMM6EX1VGN60C6UIQTE400F: PUBLIC.PRODUCT_OPTION FOREIGN KEY(PRODUCT_ID) REFERENCES PUBLIC.PRODUCT(PRODUCT_ID) (CAST(3 AS BIGINT))"; SQL statement:
delete from product where (product.is_deleted = false) [23503-224]] [n/a]; SQL [n/a]; constraint ["FKN4HMM6EX1VGN60C6UIQTE400F: PUBLIC.PRODUCT_OPTION FOREIGN KEY(PRODUCT_ID) REFERENCES PUBLIC.PRODUCT(PRODUCT_ID) (CAST(3 AS BIGINT))"; SQL statement:
delete from product where (product.is_deleted = false) [23503-224]]
 

감사합니다.