Conversation
|
плохое название для ветки 🥲 |
|
мне кажется в названии файла |
| .get_notification_saved() | ||
| ) | ||
|
|
||
| assert apply_result == "Saved" |
There was a problem hiding this comment.
Это логически странная проверка
Ты ищешь элемент, у которого есть текст'Saved' ( "//span[text()='Saved']")
А потом проверяешь, что у него текст = 'Saved'
Ну естественно у него будет этот текст ! - Ты же по нему и искал!)
Тогда уже лучше проверить, что элемент у которого есть текст 'Saved' видим.
Но мне кажется, что лучше найти элемент по id = 'notification-bar' -так ты покажешь , что это именно всплывающее окно подтверждения.
А потом проверить, что у этого окна текст = 'Saved'
| ) | ||
|
|
||
| assert apply_result == "Saved" | ||
| assert "configure" in browser.current_url, "Apply применяется, редиректа нет" |
There was a problem hiding this comment.
после запятой мы же пишем ошибку, если ассерт провалился, что произошло при неудаче.
"Apply применяется, редиректа нет" -а эта фраза, вроде, просто поясняет успех
There was a problem hiding this comment.
спасибо, учел, и изменил ассерты. видимо пропустил на уроказ__))
| ) | ||
|
|
||
| assert "configure" not in browser.current_url, "Save button redirected to the main page" | ||
|
|
There was a problem hiding this comment.
аналогично, "Save button redirected to the main page"
|
|
Не согласна насчет названия файла для теста. У нас юзерстори по конфигурации и есть файл test_folder_configuration. Логичнее добавить тесты туда |
5 participants
No description provided.