👩‍💻 Про код-ревью автотестов

Код автотестов должен обязательно проходить код-ревью, а то проект превратится в тыкву рискует стать нестабильным, нечитаемым и проблемным.

🔹Первоначальное ревью
Первым ревьюером является сам автор. Написав код, необходимо тщательно его проверить. Полезно составить чек-лист, учитывающий особенности проекта. Проходя по чек-листу можно детально проверить различные моменты, дабы ничего не упустить. Составление типовых ошибок на ревью также будет хорошей практикой 📝
Можно ещё попросить 📱 AI выполнить ревью кода, чтобы получить стороннее мнение на раннем этапе.

🔹Ревью коллегами
На первом этапе часть моментов можно улучшить, но не всегда, поэтому далее код-ревью проводят коллеги. Некоторые компании внедряет многоуровневую систему ревью, чтобы как можно больше людей посмотрело, тем самым минимизируя количество проблем в будущем.
Со стороны ревьюера важно видеть картину целиком, а не только несвязанные кусочки кода 🌐
В Merge Request (MR) обязательно должно быть описание, прикреплённая задача, вспомогательные лейблы и другая ценная информация, которая даёт контекста происходящего. Изучение всего материала – маст хэв, иначе вы просто можете не понять что-то важное.
Код обязательно должен проходить через несколько Quality Gate, такие как статические анализаторы кода, юнит-тесты и другие проверки, запуски самих автотестов и т.д.
Благодаря этим проверкам процесс ревью упрощается, оставляя сложную работу ревьюерам. Максимально автоматизируйте то, что можно и нужно автоматизировать 😉

В процессе ревью автотестов важно знать сценарий, который покрывается автотестом. Нужно проверить полноту покрытия, логику работы теста, все ли шаги и проверки правильно описаны.
Стабильность автотеста можно проверить запусками, но не всегда это возможно. Здесь я бы посоветовал обращать внимание на ожидания элементов перед действиями. Желательно видеть UI и нужно знать, как этот функционал выглядит вживую и отрабатывает. И про стабильные локаторы не забывать 😁
Ещё должны быть правила, по которым изменения считаются блокирующими или нет для влития. Моменты, которые не блокируют, можно выносить в отдельные задачи.

🔹Ревью комментария
Немаловажную роль играет сам комментарий, так что нужно самостоятельно проревьюить комментарий к ревью 😃
Никогда не критикуйте автора, пишите только про сам код.
Подача комментария должна быть доброжелательной и конструктивной. Рекомендуется подкреплять комментарий ссылкой на доку или другие ресурсы.

🔹Как развиваться в ревью?
Смотреть другие MR-ы, как и на что обращают внимание другие ревьюеры.
Зафиксировать часто встречающиеся моменты в доке, чтобы другие знали, на что обращать внимание, попробуйте их отлавливать через линтеры.
Делайте побольше качественного ревью, в том числе в других MR, если вы видите какие-то моменты.

Что в итоге:

✅Самопроверка, чек-листы, AI
✅Коллективное ревью
✅Конструктивные комментарии
✅Постоянное развитие

#автотесты@qa_relia #ревью@qa_relia