Conversation
src/test/java/tests/BurgerTest.java
Outdated
| private Bun bun; | ||
|
|
||
| @Mock | ||
| private Ingredient ingredient1; |
There was a problem hiding this comment.
⛔️Нужно исправить. При нейминге не рекомендуется использовать числа (Field2), их еще называют magicNumbers. Очень тяжело поддерживать код с magicNumbers.
| public void addIngredientTest() { | ||
| burger.addIngredient(ingredient1); | ||
|
|
||
| assertEquals(1, burger.ingredients.size()); |
There was a problem hiding this comment.
⛔️Нужно исправить. Для юнит-тестов придерживаемся подхода: один тест, значит одна проверка. Если очень хочется несколько проверок -- тогда используем softAssertions. Поправь, пожалуйста, во всем коде
src/test/java/tests/BurgerTest.java
Outdated
|
|
||
| String receipt = burger.getReceipt(); | ||
|
|
||
| assertTrue(receipt.contains("black bun")); |
There was a problem hiding this comment.
⛔️Нужно исправить. Нужно проверять рецепт одной строкой, чтобы не пропустить ошибки форматирования
|
|
||
| assertEquals(320f, actualPrice, 0.001f); | ||
| } | ||
|
|
There was a problem hiding this comment.
⛔️Нужно исправить. Должен быть хотя бы один параметризированный тест
| </execution> | ||
| </executions> | ||
| </plugin> | ||
| </plugins> |
There was a problem hiding this comment.
⛔️Нужно исправить. Нужно приложить отчет
There was a problem hiding this comment.
⛔️Нужно исправить. Нужно приложить все файлы из папки
2 participants
Сделан BurgerTest и класс Burger покрыт тестами