Conversation
There was a problem hiding this comment.
⛔️Нужно исправить. Папку .idea не нужно было загружать в репозиторий. Эта папка должна быть добавлена в .gitignore.
|
|
||
| burger.addIngredient(ingredient); | ||
|
|
||
| assertEquals(1, burger.ingredients.size()); |
There was a problem hiding this comment.
⛔️Нужно исправить. Для юнит-тестов придерживаемся подхода: один тест, значит одна проверка. Если очень хочется несколько проверок -- тогда используем softAssertions. Поправь, пожалуйста, во всем коде
| } | ||
|
|
||
| @Test | ||
| public void addIngredient_addsToIngredientsList() { |
There was a problem hiding this comment.
⛔️Нужно исправить. Неверный нейминг. Не используем спецсимволы в названии методов
|
|
||
| String receipt = burger.getReceipt(); | ||
|
|
||
| assertTrue(receipt.contains(String.format("= %s %s =%n", expectedLowercase, "test ingredient"))); |
There was a problem hiding this comment.
⛔️Нужно исправить. Для юнит-тестов применим подход: один тест, значит одна проверка. В этом тесте две проверки (Mockito.verify, assertEquals), а должна быть одна. Исправь, пожалуйста, этот момент во всем коде.
There was a problem hiding this comment.
⛔️Нужно исправить. нужно составить только папку site
2 participants
выполнила задание 1, Подключила библиотеки: JaCoCo, Mockito, JUnit 4. покрыла тестами класс бургер