Дополнен pom зависимостями и плагинами, добавлены тесты по классу Burger#1252
Дополнен pom зависимостями и плагинами, добавлены тесты по классу Burger#1252yanchilenkoaa-cmd wants to merge 7 commits intoyandex-praktikum:mainfrom
Conversation
|
для проверки |
|
проверка |
src/test/java/BurgerTest.java
Outdated
| } | ||
|
|
||
| // Данные для параметризованных тестов по цене | ||
| @Parameterized.Parameters |
There was a problem hiding this comment.
| burger.addIngredient(ingredient2); | ||
|
|
||
| String receipt = burger.getReceipt(); | ||
| assertTrue(receipt.contains("Ketchup")); |
There was a problem hiding this comment.
⛔️Нужно исправить. Для юнит-тестов придерживаемся подхода: один тест, значит одна проверка. Если очень хочется несколько проверок -- тогда используем softAssertions. Поправь, пожалуйста, во всем коде
src/test/java/BurgerTest.java
Outdated
|
|
||
| @Test | ||
| public void testRemoveAndMoveIngredient() { | ||
| Ingredient ingredient1 = mock(Ingredient.class); |
There was a problem hiding this comment.
⛔️Нужно исправить. При нейминге не рекомендуется использовать числа (Field2), их еще называют magicNumbers. Очень тяжело поддерживать код с magicNumbers.
src/test/java/BurgerTest.java
Outdated
| // Тест: установка булочек | ||
| @Test | ||
| public void testSetBuns() { | ||
| Bun mockBun = mock(Bun.class); |
There was a problem hiding this comment.
⛔️Нужно исправить. Тесты , где параметризация не нужна, необходимо вынести в отдельный класс
src/test/java/BurgerTest.java
Outdated
| assertSame(mockBun, burger.bun); | ||
| } | ||
|
|
||
| // Тест: добавление ингредиентов |
| </executions> | ||
| </plugin> | ||
| </plugins> | ||
| </build> |
There was a problem hiding this comment.
⛔️Нужно исправить. Нужно приложить отчет в тестировании в пулл реквест
|
Внесены исправления, класс с тестами разбит на 2 класса, приложен отчет |
src/test/java/BurgerTest.java
Outdated
| when(mockBun.getName()).thenReturn("Bun"); | ||
| burger.setBuns(mockBun); | ||
|
|
||
| Ingredient ingredient1 = mock(Ingredient.class); |
There was a problem hiding this comment.
⛔️Нужно исправить. При нейминге не рекомендуется использовать числа (Field2), их еще называют magicNumbers. Очень тяжело поддерживать код с magicNumbers.
src/test/java/BurgerTest.java
Outdated
| public void testAddIngredientIsPresent() { | ||
| Ingredient mockIngredient = mock(Ingredient.class); | ||
| burger.addIngredient(mockIngredient); | ||
| assertTrue(burger.ingredients.contains(mockIngredient)); |
There was a problem hiding this comment.
⛔️Нужно исправить. Строку рецепта проверяем целиком, чтобы не пропустить ошибки форматирования
There was a problem hiding this comment.
⛔️Нужно исправить. Нужно приложить папку jacoco
2 participants
Дополнен pom зависимостями и плагинами, добавлены тесты по классу Burger