Skip to content

Add unit tests for Burger class - #931

Open
ggeorgi106 wants to merge 2 commits into
Yandex-Practicum:mainfrom
ggeorgi106:develop1
Open

ggeorgi106 wants to merge 2 commits into
Yandex-Practicum:mainfrom
ggeorgi106:develop1

Conversation

@ggeorgi106

@ggeorgi106 ggeorgi106 commented Oct 6, 2026 •

Copy link
Copy Markdown

Задание 1. Юнит-тесты

  1. Добавлены юнит-тесты для класса Burger
  2. Проверены методы set_buns, add_ingredient, remove_ingredient, move_ingredient, get_price и get_receipt
  3. Для зависимостей использованы моки
  4. Для remove_ingredient использована параметризация
  5. Все тесты успешно проходят
  6. Покрытие класса burger.py — 100%

@irisqul irisqul left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ Сильные стороны:

  • Код запускается без ошибок
  • Тестовая архитектура соответствует изученным практикам написания юниттестов
  • Обеспечено необходимое тестовое покрытие

Comment thread tests/test_burger.py Outdated
burger = Burger()

assert burger.bun is None
assert burger.ingredients == []

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