Conversation
| @@ -0,0 +1,100 @@ | |||
| <?xml version="1.0" encoding="UTF-8"?> | |||
| <project xmlns="http://maven.apache.org/POM/4.0.0" | |||
There was a problem hiding this comment.
Можно улучшить: хорошей практикой является добавление README.md с описанием проекта, которое включает название, технологии в проекте (например, Java 11, JUnit 4.13.2, maven 3.9.0 и т.д.), настройка (если необходима) и как запускать (mvn clean test). Как оформлять README.md можно ознакомиться в статье https://dzen.ru/a/YbzudflBPE1IDJF-
|
|
||
|
|
||
| @Test | ||
| public void courierCanBeCreated() { |
There was a problem hiding this comment.
Необходимо исправить: При реализации тестов используй аннотацию @DisplayName("Авторизация существующего пользователя"). В скобках указывай название теста. Это сделает отчет Allure информативнее, что позволит сократить время локализации дефекта.
| assertEquals("Not valid status code", HTTP_CREATED, status); | ||
| assertTrue("Not valid response. Courier wasn't created", ok); | ||
|
|
||
| courierClient.deleteCourierInTheEnd(testCourier.getLogin(), testCourier.getPassword()); |
There was a problem hiding this comment.
Необходимо исправить: Проверки должны быть в конце тестовых методов. Между проверками и после проверок не должно быть дополнительных действий. Иначе в случае фейла первой проверки, последующие действия(шаги и методы) не будут выполнены, это может аффектить другие тесты, что увеличит время локализации дефекта. Если нужны дополнительные действия после проверок, такие как удаление тестовых данных - используй дополнительный метод с аннотацией @After.
There was a problem hiding this comment.
Вынесла удаление в after.
| private Order order; | ||
| OrderClient orderClient; | ||
|
|
||
| @Parameterized.Parameters |
There was a problem hiding this comment.
Можно улучшить: В параметризованных тестах для аннотации @Parameterized.Parameters лучше использовать аргумент name: @Parameterized.Parameters(name = "Стоимость булочки. Тестовые данные: {0} {1}"), где {0}, {1} - индексы параметров. С аргументом name наименование теста отобразится в отчете. А использование индексов параметров повысит информативность проверки.
There was a problem hiding this comment.
Это я тоже попробовала, но там значение параметра - массив из строк, так что и в отчете он отображается неверно.
2 participants
No description provided.