Sprint_5 done - #1
Conversation
| import locators | ||
|
|
||
|
|
||
| def test_move_on_toppings_by_click(chrome_driver): |
There was a problem hiding this comment.
Нужно исправить: Тестовые методы внутри одного тестового модуля объединяют в тестовые классы. Один тестовый класс (соответственно один тестовый модуль) на несколько методов, проверяющих схожую функциональность
| from data import LOGIN_URL, PROFILE_URL | ||
|
|
||
|
|
||
| def test_login_button_sign_in_to_account_on_home_page(chrome_driver): |
There was a problem hiding this comment.
Нужно исправить: Тестовые методы внутри одного тестового модуля объединяют в тестовые классы. Один тестовый класс (соответственно один тестовый модуль) на несколько методов, проверяющих схожую функциональность
|
|
||
|
|
||
| def test_move_on_toppings_by_click(chrome_driver): | ||
| chrome_driver.find_element(By.XPATH, locators.TOPPINGS).click() |
There was a problem hiding this comment.
способ поиска (XPATH, CSS) ужно тоже зашить в локаторы
| ).click() | ||
| wait(chrome_driver, 5).until(EC.url_to_be(LOGIN_URL)) | ||
|
|
||
| assert chrome_driver.current_url == LOGIN_URL |
There was a problem hiding this comment.
Нужно оставить взаимодействие с элементом-маркером успеха только в ассерте, сейчас если есть баг то тест упадет не на ассерте, а на предыдущем шаге. Если нужно подождать появления элемента это можно сделать прямо внутри ассерта
| chrome_driver.find_element(By.XPATH, locators.PERSONAL_CABINET).click() | ||
| wait(chrome_driver, 5).until(EC.url_to_be(LOGIN_URL)) | ||
|
|
||
| assert chrome_driver.current_url == LOGIN_URL |
| ).click() | ||
| wait(chrome_driver, 5).until(EC.url_to_be(LOGIN_URL)) | ||
|
|
||
| assert chrome_driver.current_url == LOGIN_URL |
There was a problem hiding this comment.
Нужно оставить взаимодействие с элементом-маркером успеха только в ассерте, сейчас если есть баг то тест упадет не на ассерте, а на предыдущем шаге. Если нужно подождать появления элемента это можно сделать прямо внутри ассерта
| from data import LOGIN_URL, MAIN_URL | ||
|
|
||
|
|
||
| def test_open_personal_cabinet_on_click_personal_cabinet(chrome_driver): |
There was a problem hiding this comment.
Нужно исправить: Тестовые методы внутри одного тестового модуля объединяют в тестовые классы. Один тестовый класс (соответственно один тестовый модуль) на несколько методов, проверяющих схожую функциональность
| BUNS = "//div[.='Булки']" | ||
|
|
||
| # Заголовок раздела «Булки» | ||
| TEXT_BUNS = "//h2[.='Булки']" |
There was a problem hiding this comment.
Отлично: удачно подобраны селекторы, локаторы надежные и не зависят от верстки всей страницы
|
Исправил там где "Нужно исправить" и убрал комментарии. |
| @@ -0,0 +1,52 @@ | |||
| PERSONAL_CABINET = "//a[.='Личный Кабинет']" | |||
There was a problem hiding this comment.
не исправлено: метод поиска нужно записать внутрь локатора. То есть локатор это селектор плюс метод поиска.
например BUTTON_LOGIN_ACCOUNT = (By.XPATH, "//button[contains(text(), 'Войти в аккаунт')]")
| ).click() | ||
| wait(chrome_driver, 5).until(EC.url_to_be(LOGIN_URL)) | ||
|
|
||
| assert chrome_driver.current_url == LOGIN_URL |
There was a problem hiding this comment.
Нужно оставить взаимодействие с элементом-маркером успеха только в ассерте, сейчас если есть баг то тест упадет не на ассерте, а на предыдущем шаге. Если нужно подождать появления элемента это можно сделать прямо внутри ассерта
|
Исправил замечания |
2 participants
No description provided.