Skip to content

my first commit#1271

Open
lidiagermogenova wants to merge 2 commits intoyandex-praktikum:mainfrom
lidiagermogenova:develop1
Open

my first commit#1271
lidiagermogenova wants to merge 2 commits intoyandex-praktikum:mainfrom
lidiagermogenova:develop1

Conversation

@lidiagermogenova
Copy link

Добавление юнит-тестов для класса Burger

Что сделано в BurgerTest. Написаны базовые тесты для проверки методов работы с ингридиентами:

  • testSetBuns(): проверка установки булки
  • testAddIngredient(): проверка добавления ингридиента
  • testRemoveIngredient(): проверка удаления ингридиента
  • testMoveIngredient(): проверка перемещения ингридиента

Что сделано в BurgerReceiptParameterizedTest. Параметризованные тесты для проверки формирования чека:

  • Проверка, что название выводится дважды
  • Проверка, что тип ингридиента выводится в нижнем регистре
  • Проверка форматирования цены через %f

Что сделано в BurgerPriceParameterizedTest. Параметризованные тесты для проверки расчета цены:

  • 6 тестовых сценариев с разными комбинациями
  • Проверка корректности суммирования цен

Использованы программы: JUnit 4, Mockito, Parameterixed tests, JaCoCo

Достигнуто покрытие кода
Снимок экрана 2026-03-04 в 09 44 42
100%

Assert.assertEquals(ingredient, burger.ingredients.get(0));
}

@Test

Choose a reason for hiding this comment

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

⛔️Нужно исправить. Нужно приложить отчет

Copy link
Author

Choose a reason for hiding this comment

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

Снимок экрана 2026-03-09 в 01 39 13

Copy link
Author

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