Часть 1, unit-тесты и отчет о 100% покрытии JaCoCo #1
Conversation
.idea/compiler.xml
Outdated
There was a problem hiding this comment.
⛔️Нужно исправить. Папку .idea не нужно было загружать в репозиторий. Эта папка должна быть добавлена в .gitignore.
pom.xml
Outdated
| </dependency> | ||
|
|
||
| <!-- Зависимость Gson для работы с JSON --> | ||
| <dependency> |
There was a problem hiding this comment.
⛔️Нужно исправить. данном задание с Gson не работаем
src/test/java/praktikum/BunTest.java
Outdated
| import junitparams.Parameters; | ||
|
|
||
| @RunWith(JUnitParamsRunner.class) | ||
| public class BunTest { |
There was a problem hiding this comment.
|
|
||
| private Burger burger; | ||
| private Bun bunMock; | ||
| private Ingredient ingredientMock1; |
There was a problem hiding this comment.
⛔️Нужно исправить. При нейминге не рекомендуется использовать числа (Field2), их еще называют magicNumbers. Очень тяжело поддерживать код с magicNumbers.
| public void testAddIngredient() { | ||
| burger.setBuns(bunMock); | ||
| burger.addIngredient(ingredientMock1); | ||
| assertEquals(1, burger.ingredients.size()); |
There was a problem hiding this comment.
⛔️Нужно исправить. Для юнит-тестов придерживаемся подхода: один тест, значит одна проверка. Если очень хочется несколько проверок -- тогда используем softAssertions. Поправь, пожалуйста, во всем коде
| private Burger burger; | ||
| private Bun bunMock; | ||
| private Ingredient ingredientMock1; | ||
| private Ingredient ingredientMock2; |
There was a problem hiding this comment.
⛔️Нужно исправить. По заданию для бургер тестов нужно реализовать параметризированные тесты. Какие именно, решай сама
| burger.addIngredient(fillingIngredient); | ||
|
|
||
| burger.moveIngredient(0, 1); | ||
| assertEquals(fillingIngredient, burger.ingredients.get(0)); |
There was a problem hiding this comment.
⛔️Нужно исправить. Для юнит-тестов придерживаемся подхода: один тест, значит одна проверка. Если очень хочется несколько проверок -- тогда используем softAssertions. Поправь, пожалуйста, во всем коде
| public void testReceiptContainsBunName() { | ||
| burger.setBuns(bunMock); | ||
| String receipt = burger.getReceipt(); | ||
| assertTrue(receipt.contains(bunMock.getName())); |
There was a problem hiding this comment.
⛔️Нужно исправить. Необходимо проверять строку рецепта целиком. Иначе, можно пропустить ошибки форматирования текста
There was a problem hiding this comment.
⛔️Нужно исправить. Нужно оставить только папку jacoco. Остальные файлы удалить из пулл реквеста
No description provided.