Develop1 #1142
Develop1 #1142
Conversation
| private Bun bun; | ||
|
|
||
| @Mock | ||
| private Ingredient ingredient1; |
There was a problem hiding this comment.
⛔️Нужно исправить. При нейминге не рекомендуется использовать числа (Field2), их еще называют magicNumbers. Очень тяжело поддерживать код с magicNumbers.
|
|
||
| @Test | ||
| public void testSetBuns() { | ||
| Burger burger = new Burger(); |
There was a problem hiding this comment.
| Burger burger = new Burger(); | ||
| burger.addIngredient(ingredient1); | ||
|
|
||
| assertEquals(1, burger.ingredients.size()); |
There was a problem hiding this comment.
⛔️Нужно исправить. Для юнит-тестов придерживаемся подхода: один тест, значит одна проверка. Если очень хочется несколько проверок -- тогда используем softAssertions. Поправь, пожалуйста, во всем коде
| @RunWith(Parameterized.class) | ||
| public static class GetPriceParameterizedTest { | ||
|
|
||
| @Parameterized.Parameters |
There was a problem hiding this comment.
|
|
||
| String receipt = burger.getReceipt(); | ||
|
|
||
| assertTrue(receipt.contains("Price: 20,000000")); |
There was a problem hiding this comment.
⛔️Нужно исправить. Для юнит-тестов применим подход: один тест, значит одна проверка. В этом тесте две проверки (Mockito.verify, assertEquals), а должна быть одна. Исправь, пожалуйста, этот момент во всем коде.
- поменял magic numbers в названиях переменных; - азделил тесты с несколькими проверками на отдельные тесты - реализовл принцип 'один тест - одна проверка'
2 participants
No description provided.