aosandy / auto-testing Goto Github PK

Я в тамтам https://tt.me/riinj

Увеличивай проект! :)

Перед выполнением действий с элементами необходимо проверить, что они есть на странице
С помощью селенида это можно сделать так, например $(ELEM).shoudBe(visible.because("Нет элемента")).click()

---

Сейчас нигде не проверяется видимость элементов, с которыми мы взаимодействуем. Как проверить видимость элемента понятно, а вдруг страница вообще не прогрузилась?? Как мы это поймем? Предлагаю изучить паттерн LoadableComponent

---

Я бы не доверяла этим ребятам, а ты?

---

Нейминг неочевидный, что это за элементы - остается догадываться по id в локаторе

---

Сейчас все тесты в одном классе, возможно, из-за того, что у них общее предусловие? Я бы разделила тесты по логическим группам на классы (мы кстати придерживаемся стиля один тест = один класс, хотя сейчас тесты маленькие и в этом нет смысла, можно сразу привыкать к такому стилю, так как на самом деле тесты должны быть независимыми и чем дальше, тем сложнее они становятся)
Так вот, чтобы у тестов было это общее предусловие и общее завершение (переход по ссылке на одноклассники и завершение сессии после), стоит использовать базовый тестовый класс и наследовать тесты от него

---

Проверки стоит делать через Assertions, например, asserEquals()

---

У селенида есть метод $$ для поиска нескольких элементов, но всю работу с локаторами мы выносим в методы в пейджах, в тесте не должно быть локаторов и поиска элементов

---

Ожидаемые значения можно вынести в константы

---

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