Conversation
| } | ||
|
|
||
| @Test | ||
| public void testGetReceipt_ReturnsCorrectReceipt() { |
There was a problem hiding this comment.
⛔️Нужно исправить. Неверный нейминг. Не используем спецсимволы в названии методов
|
|
||
| @Test | ||
| public void testGetReceipt_ReturnsCorrectReceipt() { | ||
| Bun bun = new Bun(bunName, bunPrice); |
There was a problem hiding this comment.
⛔️Нужно исправить. Для всех зависимостей используем моки
| Ingredient ingredient = Mockito.mock(Ingredient.class); | ||
| Burger burger = new Burger(); | ||
| burger.addIngredient(ingredient); | ||
| assertEquals(1, burger.ingredients.size()); |
There was a problem hiding this comment.
⛔️Нужно исправить. Для юнит-тестов придерживаемся подхода: один тест, значит одна проверка. Если очень хочется несколько проверок -- тогда используем softAssertions. Поправь, пожалуйста, во всем коде
|
|
||
| @Test | ||
| public void testRemoveIngredient_RemovesIngredient() { | ||
| Ingredient ingredient1 = Mockito.mock(Ingredient.class); |
There was a problem hiding this comment.
⛔️Нужно исправить. При нейминге не рекомендуется использовать числа (Field2), их еще называют magicNumbers. Очень тяжело поддерживать код с magicNumbers.
| burger.setBuns(bun); | ||
| burger.addIngredient(ingredient1); | ||
| burger.addIngredient(ingredient2); | ||
| assertEquals(325.0f, burger.getPrice(), 0.001); |
There was a problem hiding this comment.
⛔️Нужно исправить. Для юнит-тестов применим подход: один тест, значит одна проверка. В этом тесте две проверки (Mockito.verify, assertEquals), а должна быть одна. Исправь, пожалуйста, этот момент во всем коде.
2 participants
Добавлены unit-тесты для класса Burger + настройка покрытия JaCoCo. Использованы JUnit 4, Mockito, параметризованные тесты. Подключён JaCoCo для анализа покрытия, результат:
Покрытие тестами класса Burger: 100%