Добавил тесты на бургер, покрытие JoCoCo 100% #1146
Conversation
| private Bun bun; | ||
|
|
||
| @Mock | ||
| private Ingredient ingredient1; |
There was a problem hiding this comment.
⛔️Нужно исправить. При нейминге не рекомендуется использовать числа (Field2), их еще называют magicNumbers. Очень тяжело поддерживать код с magicNumbers.
| burger = new Burger(); | ||
| } | ||
|
|
||
| @Parameterized.Parameters |
There was a problem hiding this comment.
| public void testSetBuns() { | ||
| when(bun.getName()).thenReturn(bunName); | ||
| when(bun.getPrice()).thenReturn(bunPrice); | ||
|
|
There was a problem hiding this comment.
⛔️Нужно исправить. Тесты, где не нужна параметризация, нужно вынести в отдельный класс. Мы не должны их запускать несколько раз
|
|
||
| burger.addIngredient(ingredient1); | ||
|
|
||
| assertEquals(1, burger.ingredients.size()); |
There was a problem hiding this comment.
⛔️Нужно исправить. Для юнит-тестов придерживаемся подхода: один тест, значит одна проверка. Если очень хочется несколько проверок -- тогда используем softAssertions. Поправь, пожалуйста, во всем коде
…tests into BurgerTest and BurgerSimpleTest, update .gitignore
2 participants
Подключил библиотеки Jococo, Junit4, Mockito. Провел тестирование класса Burger, по результатам отчета Jococo составило 100%