Conversation
There was a problem hiding this comment.
Необходимо исправить: эта директория лишняя в проекте. Необходимо убрать её из ветки и добавить в .gitignore в корне проекта
There was a problem hiding this comment.
Необходимо исправить: эта директория лишняя в проекте. Необходимо убрать её из ветки и добавить в .gitignore в корне проекта
3_task/pages/base_page.py
Outdated
There was a problem hiding this comment.
Необходимо исправить здесь и далее: не хватает аннотации allure для шага - allure.step
3_task/pages/feed_page.py
Outdated
There was a problem hiding this comment.
Необходимо исправить здесь и далее: все вызовы к driver/WebDriverWait должны быть описаны в методах BasePage, а затем вызываться через эти методы в *Page-классах через наследование
3_task/tests/test_feed.py
Outdated
There was a problem hiding this comment.
Нужно исправить здесь и далее: Для корректного запуска тестов необходимо код с шагами теста поместить в тестовый метод (нейминг начинается с test_), а метод - в тестовый класс (нейминг начинается с Test). Необходимо привести к такому формату все модули внутри пакета tests
3_task/tests/test_feed.py
Outdated
There was a problem hiding this comment.
Необходимо исправить здесь и далее: тут явно не хватает шагов с оформлением заказа
3_task/tests/test_feed.py
Outdated
There was a problem hiding this comment.
Необходимо исправить здесь и далее: тут явно не хватает шагов с оформлением заказа
3_task/tests/test_main_functional.py
Outdated
There was a problem hiding this comment.
Необходимо исправить: этим тестам не хватает шагов и проверок
3_task/conftest.py
Outdated
There was a problem hiding this comment.
Нужно исправить: фикстуры не занимаются выполнением примитивной логики, они выполняют сложную логику предусловий\постусловий и вычислений. Эти объекты можно сразу создавать в тестах или фикстурах
3_task/tests/test_feed.py
Outdated
There was a problem hiding this comment.
Необходимо исправить: оформление заказа - это ключевой шаг теста, он должен быть выполнен на стороне UI
3_task/tests/test_feed.py
Outdated
There was a problem hiding this comment.
Необходимо исправить: оформление заказа - это ключевой шаг теста, он должен быть выполнен на стороне UI. Шагов с api в теле теста быть не должно
2 participants
No description provided.