[Kotlin] 테스트하기 어려운 부분을 분리하기

AS-IS

정해진 시간 외에는 주문을 생성할 수 없다.(10:00~22:00)

위 요구사항을 만족하는 비즈니스 로직의 코드를 다음과 같이 작성할 수 있다.

object Constants {
    val SHOP_OPEN_TIME = LocalTime.of(10, 0)
    val SHOP_CLOSE_TIME = LocalTime.of(22, 0)
}

fun createOrder(beverages: List<Beverage>): Order {
    val currentDateTime = LocalDateTime.now()
    val currentTime = currentDateTime.toLocalTime()

    if (currentTime.isBefore(Constants.SHOP_OPEN_TIME) || currentTime.isAfter(Constants.SHOP_CLOSE_TIME)) {
        throw IllegalArgumentException("주문 시간이 아닙니다. 관리자에게 문의하세요.")
    }

    return Order(currentDateTime, beverages)
}

문제 상황

위 로직을 테스트한다고 하면, 아래처럼 작성할 수 있을 것이다.

@Test
fun createOrder() {
        val cafeKiosk = CafeKiosk()
        val americano = Americano()

        cafeKiosk.add(americano)

        val order = cafeKiosk.createOrder()
        order.beverages.shouldHaveSize(1)
        order.beverages[0].name.shouldBe("아메리카노")
    }

그러나 위 테스트는 항상 성공하는 테스트일 수 없다.

currentDateTime을 LocalDateTime.now()로 생성하기 때문에, 테스트하는 시각에 따라 성공, 실패로 결과가 나눠지기 때문이다.

상수인 SHOP_OPEN_TIME을 LocalTime.of()로 바꿀 경우, 테스트 결과가 달라진다.

 

그래서 문제가 되는 createOrder 메서드를 변경해서,  LocalDateTime.now()를 외부 파라미터로 빼면, 다음과 같이 바꿀 수 있다.

fun createOrder(currentDateTime: LocalDateTime): Order {
    val currentTime = currentDateTime.toLocalTime()

    if (currentTime.isBefore(SHOP_OPEN_TIME) || currentTime.isAfter(SHOP_CLOSE_TIME)) {
        throw IllegalArgumentException("주문 시간이 아닙니다. 관리자에게 문의하세요.")
    }

    return Order(currentDateTime, beverages)
}

TO-BE

새로이 작성한 테스트의 경우, 외부 파라미터로 currentDateTime을 받도록 수정했다.

@Test
fun `createOrderOutsideOpenTime`() {
    val cafeKiosk = CafeKiosk()
    val americano = Americano()

    cafeKiosk.add(americano)

    val exception = shouldThrow<IllegalArgumentException> {
        cafeKiosk.createOrder(LocalDateTime.of(2023, 1, 17, 9, 59))
    }
    exception.message shouldBe "주문 시간이 아닙니다. 관리자에게 문의하세요."
}

외부에서 파라미터로 받도록 변경함으로써, 항상 성공하는 테스트로 만들 수 있었다.

그러나 당연스럽게도, 위는 테스트용으로 생성한 메서드이기 때문에, 검증하는데 프로덕션용 코드를 사용하지 않아도 괜찮은지에 대한 고민이 생긴다.

 

그렇지만 위 테스트는 주문을 생성하는 것을 검증하는 테스트이며, 시간에 대한 테스트가 아니다.

즉, 테스트 하고자 하는 것이 주문 생성이기 때문에 테스트하기 어려운 영역을 구분하고, 분리해서 구현해야 한다.