testomat.io управление авто тестамиПривет, хочу чтоб вы посмотрели на мой код в репе и сказали свое мнение. Сильно желательно больше логики/конструктивных замечаний и меньше от сердца.
ты назвал pytests - но используешь unittest?)
спасибо, что еще? может в коде чет не на месте или структура пейджобжект плохая или тесты не очивидные?
Напишите 5-6 нормальных теста. Один недотест - это не портфолио.
Сделайте 3-4 page objects.
Возможно после этого вы сами увидите что можно улучшить.
А если по делу - у вас сильно хромает семантика: wait который не ждёт, find который не ищет…
- Непорядок с названием (то ли pytest, то ли unittest)
- сплипы - это не айс (“sleep(1)”)
- Бардак в base.py:
- встречается и find, и get_elements для поиска элемента(ов).
- switch_to_window - из названия не угадаешь, что на самом деле делает метод
- compare_values - мне кажется ассертить в базовом классе - не лучший вариант
- base/wait.py - плохая практика использовать “sleep(2)”, см.п.2
- структура каталогов не очевидна. Почему в data у вас пейджи + лежит сам хромдрайвер?
Название локаторов не информативны, научитесь сразу писать понятные названия. Даже для одного теста “BTN” или “INPUT” не понятно про какой инпут или кнопку говорится.
спасибо, исправлю и учту ваши замечания асап, anything else?
Переходите на pytest )