1
2
3
4
5
6
7
8
9

class OrderValidator:
    def validate(self, order) -> bool:
        return self._check_items(order) and self._check_total(order)

    def _check_items(self, order) -> bool:
        return len(order.lines) > 0

    def _check_total(self, order) -> bool:
        return order.total() > 0

1
2
3
4
5

# 안티패턴: private 메서드를 이름 맹글링을 뚫고 직접 호출
def test_check_items_directly():
    validator = OrderValidator()
    order = make_order(lines=[])
    assert validator._check_items(order) is False

1
2
3
4
5

# 개선: 공개 API(validate)를 통해서만 검증
def test_validate_rejects_empty_order():
    validator = OrderValidator()
    order = make_order(lines=[])
    assert validator.validate(order) is False

1
2
3
4
5
6

# 안티패턴: 테스트가 계산 공식을 그대로 복제함
def test_calculate_late_fee_leaks_logic():
    days_overdue = 10
    daily_rate = 500
    expected = min(days_overdue * daily_rate, 20000)  # 프로덕션 코드와 동일한 공식
    assert calculate_late_fee(days_overdue) == expected

1
2
3
4

# 개선: 계산 공식이 아니라 구체적인 숫자로 기대값을 명시
def test_calculate_late_fee_explicit_expected_value():
    assert calculate_late_fee(10) == 5000
    assert calculate_late_fee(100) == 20000  # 상한선에 걸림을 명시적으로 확인

1
2
3
4
5
6

from datetime import datetime


class Membership:
    def is_expired(self, expires_at: datetime) -> bool:
        return datetime.now() > expires_at  # 시스템 시간에 직접 의존

1
2
3
4
5

# 안티패턴: 테스트 실행 시점에 따라 결과가 달라짐(재현 불가능)
def test_is_expired_flaky():
    membership = Membership()
    almost_now = datetime.now()
    assert membership.is_expired(almost_now) is False  # 실행 속도에 따라 결과가 바뀔 수 있음

 1
 2
 3
 4
 5
 6
 7
 8
 9
10
11

class Membership:
    def is_expired(self, expires_at: datetime, now: datetime) -> bool:
        return now > expires_at  # 시간을 외부에서 주입받음


def test_is_expired_deterministic():
    membership = Membership()
    expires_at = datetime(2026, 1, 1)

    assert membership.is_expired(expires_at, now=datetime(2026, 1, 2)) is True
    assert membership.is_expired(expires_at, now=datetime(2025, 12, 31)) is False

 1
 2
 3
 4
 5
 6
 7
 8
 9
10
11

# 안티패턴: 모든 테스트가 이 하나의 거대한 픽스처에 의존함
@pytest.fixture
def everything():
    user = make_user(role="admin", verified=True, subscription="premium")
    order = make_order(user=user, lines=[("apple", 2, 1000)], status="CONFIRMED")
    payment = make_payment(order=order, method="card", status="AUTHORIZED")
    return {"user": user, "order": order, "payment": payment}


def test_order_total(everything):
    assert everything["order"].total() == 2000

1
2
3
4

# 개선: 이 테스트에 필요한 만큼만 준비
def test_order_total_focused():
    order = make_order(lines=[("apple", 2, 1000)])
    assert order.total() == 2000

1
2
3
4
5

class NotificationService:
    def send(self, message: str, is_test_mode: bool = False) -> None:
        if is_test_mode:
            return  # 테스트에서만 실제 발송을 건너뛰기 위한 플래그
        self._client.send_sms(message)

 1
 2
 3
 4
 5
 6
 7
 8
 9
10

class NotificationService:
    def send(self, message: str) -> None:
        self._client.send_sms(message)  # 테스트 분기 없음


def test_notification_service_calls_client():
    mock_client = Mock(spec=SmsClient)
    service = NotificationService(mock_client)
    service.send("hello")
    mock_client.send_sms.assert_called_once_with("hello")