Conversation
|
|
||
|
|
||
| class TestMainPage: | ||
| @allure.description('Проверяем, что при клике на вопрос ответ правильный') |
There was a problem hiding this comment.
Нужно исправить: здесь и далее: для читаемого отображения отчетов над каждым тестом необходимо ставить декоратор title.
| ) | ||
| def test_questions_and_answers(self, driver, num, result): | ||
| main_page = MainPage(driver) | ||
| driver.get('https://qa-scooter.praktikum-services.ru/') |
There was a problem hiding this comment.
Можно лучше: здесь и далее: урлы стоит хранить в отдельном модуле и конкатенировать с базовым, так будет проще поддерживать несколько стендов
| def test_questions_and_answers(self, driver, num, result): | ||
| main_page = MainPage(driver) | ||
| driver.get('https://qa-scooter.praktikum-services.ru/') | ||
| main_page.click_to_element(MainPageLocators.COOKIE_LOCATOR) |
There was a problem hiding this comment.
Нужно исправить: здесь и далее: в классическом POM тест не должен знать о внутренней реализации, в том числе и о используемых локаторах (за исключением передачи локаторов в рамках параметризации). Подобная реализация должна быть сокрыта в рамках методов пейджи.
| handles = driver.window_handles | ||
| driver.switch_to.window(handles[1]) | ||
| WebDriverWait(driver, 5).until(expected_conditions.visibility_of_element_located(HeaderLocators. |
There was a problem hiding this comment.
Нужно исправить: в тестах и методах пейджей не должно быть обращений к драйверу и использований WebdriverWait. Все подобные реализации должны находиться в рамках базового класса пейджей.
| order_page = OrderPage(driver) | ||
| driver.get('https://qa-scooter.praktikum-services.ru/') | ||
| main_page.click_to_element(MainPageLocators.UPPER_ORDER_BUTTON) | ||
| assert order_page.get_text_from_element(OrderPageLocators.ORDER_PAGE_HEADER_LOCATOR) == 'Для кого самокат' |
There was a problem hiding this comment.
Можно лучше: здесь и далее: тестовые данные стоит хранить отдельно в data модуле, так поддерживать и переиспользовать будет проще
1 participant
No description provided.