Skip to content

Latest commit

 

History

History
35 lines (21 loc) · 3.07 KB

REVIEW.md

File metadata and controls

35 lines (21 loc) · 3.07 KB

Как ревьюить e2e-тесты

Проверяйте тесты на отказоустойчивость

Следите, чтобы в случае прерывания теста он не ломался. Например, есть такой тест для интернет-магазина:

  1. Логинимся.
  2. Добавляем товар в корзину.
  3. Проверяем, что в хедере теперь есть надпись «В корзине 1 товар».
  4. Удаляем товар из корзины.

Что не так с этим тестом? Если прервать его после второго шага, он начнёт падать при последующих прогонах, так как на момент добавления в корзине уже будет товар. Чтобы предотвратить это, нужно удалять товар из корзины до добавления товара, а не после.

Проверяйте скриншоты

При ревью нужно открывать скриншоты и проверять две вещи:

  • Сами изменения правильные.
  • Скриншоты соответствуют своим названиям.

Скриншот, как ревьюить скриншоты

Игнорируйте ненадёжные css-селекторы

В начале написания автотестов, скорее всего, в проекте не будут расставлены data-testid, поэтому будут использоваться обычные css-селекторы. Например, .Submenu-left_3GtUZ > div > span. Они могут быть ненадёжными, так как могут изменяться.

Об этом не стоит беспокоиться, так как после написания тестов создаётся задача на разработчиков на добавление data-testid. После её выполнения обычные селекторы заменяются на селекторы по data-testid.

Помогайте QA

Есть три основные вещи, с которыми QA нужна помощь:

  1. Написание хелперов. Например, вы хотите тестировать попадание событий в аналитику. Напишите функцию, при помощи которой можно проверить, что событие было отправлено, передав его имя.
  2. Git. Расскажите QA про гит-флоу у вас на проекте и помогите с трудностями и решением мёрж-конфликтов.
  3. Селекторы и data-testid. Иногда бывает довольно сложно подобрать селектор для элемента. Помогите с этим QA или добавьте data-testid.