Conversation
src/test/java/BurgerPriceTest.java
Outdated
| this.expectedPrice = expectedPrice; | ||
| } | ||
|
|
||
| @Parameterized.Parameters |
There was a problem hiding this comment.
src/test/java/BurgerReceiptTest.java
Outdated
| public void getReceiptTest() { | ||
| String receipt = burger.getReceipt(); | ||
|
|
||
| assertTrue(receipt.contains("white bun")); |
There was a problem hiding this comment.
⛔️Нужно исправить. Строку рецепта проверяем целиком, чтобы не пропустить ошибки форматирования
src/test/java/BurgerTest.java
Outdated
| private Burger burger; | ||
|
|
||
| @Mock | ||
| private Ingredient ingredient1; |
There was a problem hiding this comment.
⛔️Нужно исправить. При нейминге не рекомендуется использовать числа (Field2), их еще называют magicNumbers. Очень тяжело поддерживать код с magicNumbers.
| public void addIngredientTest() { | ||
| burger.addIngredient(ingredient1); | ||
|
|
||
| assertEquals(1, burger.ingredients.size()); |
There was a problem hiding this comment.
⛔️Нужно исправить. Для юнит-тестов придерживаемся подхода: один тест, значит одна проверка. Если очень хочется несколько проверок -- тогда используем softAssertions. Поправь, пожалуйста, во всем коде
2 participants
No description provided.