[ATDD] 테스트 격리, 버그 수정 최소화, 테스트코드의 중복 제거

테스트 격리 문제를 항상 주의하자

문제 상황

예약 취소 테스트가 통과했다. 그런데 바로 다음 테스트가 실패했다.

Scenario: 취소 가능한 예약을 취소한다     → 성공 (예약 1001: CONFIRMED → CANCELLED)
Scenario: 예약 상태를 CHECKED_IN으로 변경  → 실패 (예약 1001: 이미 CANCELLED 상태)

 

테스트 케이스 값이 변경이 되어 위와 같은 문제가 생겼다 

변경하는 시나리오에 대해 전용 ID 사용 

DB 리셋 대신, 시나리오 유형에 따라 다른 ID를 사용했다.

public class ReservationTestHelper {
    // 변경하는 시나리오용 - 각각 전용 ID
    public static final int CANCEL_TARGET_ID = 1005;
    public static final int CHECKIN_TARGET_ID = 1006;
    public static final int CHECKOUT_TARGET_ID = 1007;

    // 조회/실패 시나리오용 - 공유 가능
    public static final int CANCELLED_ID = 1003;    // 이미 취소된 상태
    public static final int CHECKED_OUT_ID = 1004;  // 이미 체크아웃된 상태
}

 

혹은 API 호출해서 테스트마다 다른 데이터를 생성해도 되고, 아니면 데이터를 초기화하는 방법도 있다  

---

버그인가, 기능인가?

문제 상황

테스트를 작성하다 보니 의문이 생겼다.

• 당일 숙박을 허용해야 하나?
• 체크아웃 날짜에 새 예약이 가능해야 하나?

이게 버그인지, 의도된 동작인지 모르겠다.

해결 방향

정의된 스펙을 우선 검증한다.

모호한 부분까지 전부 테스트하려 하지 않는다. 스펙 문서에 명시된 기능부터 검증하고, 모호한 부분은 우선순위를 낮춘다.

---

테스트 통과를 위한 버그 수정은 최소한으로 진행한다

"딱 그 부분만 고치려다가 다른 부분까지 함께 고치다 보면 수정 범위가 너무 넓어져서 롤백하는 경우도 있다"

 

따라서, 리팩터링 욕구를 참고 일단 최소한의 부분만 고친다.

---

테스트 코드도 코드이므로 유지보수를 생각한다 

테스트 코드도 중복이 쌓이면 유지보수가 어렵다. 재사용되는 로직은 Fixture로 묶었다.

public class ReservationTestHelper {
    // 테스트용 고정 ID
    public static final int CANCEL_TARGET_ID = 1005;
    public static final int CANCELLED_ID = 1003;

    // API 호출 메서드
    public Response updateStatus(int id, String status) { ... }

    // 검증 유틸리티
    public String getReservationStatus(int id) { ... }
}

 

테스트 코드가 깔끔해지고, 변경 시 한 곳만 수정하면 된다.