Skip to content

Develop1#1261

Open
auporotylis wants to merge 5 commits intoyandex-praktikum:mainfrom
auporotylis:develop1
Open

Develop1#1261
auporotylis wants to merge 5 commits intoyandex-praktikum:mainfrom
auporotylis:develop1

Conversation

@auporotylis
Copy link
Copy Markdown

Выполнено юнит-тестирование класса Burger.

Выполнены тесты:
Проверка установки булки, добавления и удаления ингредиентов.
Проверка перемещения ингредиентов с использованием параметризованных тестов.
Проверка расчета цены и генерации рецепта, включая сценарий без ингредиентов.

Используемые библиотеки:
JUnit 4 (запуск тестов),
Mockito (создание моков булок и ингредиентов для изоляции тестов)
JaCoCo (генерация отчета о покрытии кода тестами)

Тесты обеспечивают покрытие всех методов класса Burger.

Comment thread src/test/java/praktikum/BurgerTest.java Outdated

private int indexOld;
private int indexNew;
private int indexAfterMove0;
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⛔️Нужно исправить. При нейминге не рекомендуется использовать числа (Field2), их еще называют magicNumbers. Очень тяжело поддерживать код с magicNumbers.

Comment thread src/test/java/praktikum/BurgerTest.java Outdated

}

@Parameterized.Parameters
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️Можно улучшить. В параметризованных тестах для аннотации Parameterized.Parameters лучше использовать аргумент name: @Parameterized.Parameters(name = "Тестовые данные: {0} {1}"), где {0}, {1} - индексы параметров. Это повысит информативность проверки


@Test
@Description("Проверка удаления ингредиентов из бургера")
public void removeIngredientShouldDeleteIngredientFromBurgerTest() {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⛔️Нужно исправить. Тесты , где параметризация не нужна, необходимо вынести в отдельный класс

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants