Какое покрытие кода тестами (test coverage) считается достаточным?

Question

Универсального "достаточного" покрытия не существует, но общепринятые рекомендации: 70-80% для большинства проектов, 90%+ для критически важных систем. Однако важно помнить, что высокое покрытие не гарантирует качество тестов. 100% покрытие редко достижимо и экономически нецелесообразно. Гораздо важнее тестировать ключевые бизнес-логики и сложные алгоритмы, чем гнаться за процентами. Качество тестов важнее их количества.

YeaHub · Accepted Answer

Покрытие кода тестами — это метрика, показывающая процент кода, который выполняется при запуске тестов.1. Рекомендуемые уровни покрытия70-80%: Хороший уровень для большинства коммерческих проектов80-90%: Высокий уровень для проектов со строгими требованиями к качеству90%+: Для критически важных систем (медицина, авиация, финансы)<70%: Требует улучшения, возможны серьезные пробелы в тестировании2. Что важнее процентов покрытияКачество тестов: Лучше 70% meaningful тестов, чем 95% бесполезныхТестирование сложной логики: Критически важные алгоритмы должны быть протестированы thoroughlyEdge cases: Тестирование граничных условий и исключительных ситуацийИнтеграционные тесты: Проверка взаимодействия между компонентами3. Практический пример# Код для тестирования
def calculate_discount(price, user_type, is_premium=False):
    if price <= 0:
        raise ValueError("Price must be positive")
    
    if user_type == "vip":
        discount = 0.2
    elif user_type == "regular" and is_premium:
        discount = 0.1
    else:
        discount = 0.0
    
    return price * (1 - discount)

# Плохой тест - только happy path
def test_discount_basic():
    assert calculate_discount(100, "vip") == 80

# Хорошие тесты - покрывают разные сценарии
def test_discount_comprehensive():
    # Happy path
    assert calculate_discount(100, "vip") == 80
    assert calculate_discount(100, "regular", True) == 90
    
    # Edge cases
    try:
        calculate_discount(0, "vip")
        assert False, "Should raise exception"
    except ValueError:
        pass
    
    # Default case
    assert calculate_discount(100, "regular") == 100Вывод: Стремитесь к разумному балансу между покрытием и качеством тестов. Используйте покрытие как инструмент для выявления непротестированных областей, а не как самоцель. Фокус на тестировании бизнес-логики и сложных алгоритмов принесет больше пользы, чем погоня за 100% покрытием.