Conversation
| private final By orderConfirmationField = By.xpath(".//button[contains(@class, 'Button_Middle') and text() = 'Заказать']"); | ||
| private final By yesButton = By.xpath(".//button[contains(@class, 'Button_Middle') and text() = 'Да']"); | ||
| private final By orderCreatedField = By.className("Order_Modal__YZ-d3"); | ||
| private final String URL = "https://qa-scooter.praktikum-services.ru/"; |
There was a problem hiding this comment.
Нужно исправить: константу лучше вынести в отдельный класс, Сейчас у тебя один урл в двух классах
There was a problem hiding this comment.
URL оставила в 1м классе
| @@ -0,0 +1,13 @@ | |||
| package $org.example; | |||
|
|
|||
There was a problem hiding this comment.
Нужно исправить: не должно быть лишнего кода
There was a problem hiding this comment.
папка resources не должна быть проиндексирована. Все еще вижу ее в проекте
There was a problem hiding this comment.
удалила папку resources
| import junit.framework.TestCase; | ||
| import junit.framework.TestSuite; | ||
|
|
||
| /** |
There was a problem hiding this comment.
Нужно исправить: не должно быть закомментированного кода
|
|
||
| @Before | ||
|
|
||
| public void start() { |
There was a problem hiding this comment.
Нужно исправить: у тебя уже есть класс для предусловия - запуска драйвера. Он будет одинаковым для любых тестов, не дублируй код
There was a problem hiding this comment.
оставила один класс Base
There was a problem hiding this comment.
класс не используется, но все еще есть в коде
| }; | ||
| } | ||
| @Test | ||
| public void buttonOrderTest() { |
There was a problem hiding this comment.
Нужно исправить: нельзя сворачивать весь тест в один метод. Тест должен быть читаем: в нем должны быть названы шаги и указано ожидание
сейчас тесты ничего не проверяют
There was a problem hiding this comment.
Тест разбила .
Шаги заполнения формы объединены в один метод, а кнопки Заказать и Далее вынесены в отдельные методы в Test
| @@ -0,0 +1,36 @@ | |||
| <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" | |||
There was a problem hiding this comment.
Нужно исправить: должен быть только один Pom файл
There was a problem hiding this comment.
Не исправлено, в проекте все еще 2 файла
| @@ -0,0 +1,9 @@ | |||
| <archetype> | |||
There was a problem hiding this comment.
Нужно исправить: не должно быть файлов с настройками в проекте
There was a problem hiding this comment.
отправила в .gitignore
в readme ссылка на скрин запуска тестов
| <name>Archetype - Sprint_4</name> | ||
| <url>http://maven.apache.org</url> | ||
|
|
||
|
|
There was a problem hiding this comment.
Нужно исправить: отсутствует настройка на java 11
There was a problem hiding this comment.
добавлена зависимость в pom.xml
| } | ||
| // методы аренды | ||
|
|
||
| public void fillingOutRentalFields(String data, String dropdownOption, String ScooterColor, String comment) { |
There was a problem hiding this comment.
Нужно исправить: dropdownOption не используется в методе
| orderStepsPage.rentSteps(data, dropdownOption, scooterColor,comment); | ||
| orderStepsPage.getOrderConfirmation(); | ||
| orderStepsPage.clickYesButton(); | ||
| orderStepsPage.getOrderCreatedField(); |
There was a problem hiding this comment.
Нужно исправить: тест все еще ничего не проверяет, отсутствует ассерт
There was a problem hiding this comment.
добавила проверку ассертом
| public void buttonOrderTest2 () { | ||
| mainQuestionPage.openPage(); | ||
| mainQuestionPage.clickCookieButton(); | ||
| orderStepsPage.clickOrderButton2(); |
There was a problem hiding this comment.
Нужно исправить: тест ничего не проверяет
There was a problem hiding this comment.
добавила проверку ассертом перехода на страницу "Для кого самокат"
|
|
||
| } | ||
| @Test | ||
| public void buttonOrderTest2 () { |
There was a problem hiding this comment.
Можно улучшить: для этого теста не нужна параметризация, избыточно проходить его дважды.
2 participants
В классах Base добавлены 2 браузера, один из них закомментирован, поэтому в импорте браузеры могут быть не активны.