На code review автотестов проверяют смысл сценария, assertions, данные, изоляцию, ожидания, cleanup, читаемость и устойчивость, а не только стиль кода.
Как проходит код-ревью твоих изменений в автотестах?
На code review автотестов проверяют смысл сценария, assertions, данные, изоляцию, ожидания, cleanup, читаемость и устойчивость, а не только стиль кода.
Короткий ответ
Что ответить на собеседовании
Подробный разбор
Ответ с пояснениями
На code review автотестов проверяют смысл сценария, assertions, данные, изоляцию, ожидания, cleanup, читаемость и устойчивость, а не только стиль кода.
Расскажите, кто смотрит изменения и какие автоматические проверки запускаются до merge. Хороший review задаёт вопросы: может ли тест пройти при дефекте, зависит ли от порядка, что произойдёт при падении посередине, нет ли секретов в отчёте. Если затронута общая fixture, оценивают влияние на остальные тесты. Замечания обсуждают по сути, затем автор исправляет код и получает повторную проверку. При срочности сокращать можно объём, но не скрывать непросмотренные риски.