Добавлены юнит-тесты для приложения бургеров #664
Conversation
There was a problem hiding this comment.
Нужно исправить: из финального проекта нужно удалить все файлы и папки, которые не относятся к решению задачи
| bun = Bun(name, price) | ||
|
|
||
| assert bun.get_name() == name | ||
| assert bun.get_price() == price No newline at end of file |
There was a problem hiding this comment.
нужно исправить: в юнит тестировании мы хотим проверить методы по отдельности
|
|
||
| def test_add_ingredient(): | ||
| burger = Burger() | ||
| ingredient = Mock() |
There was a problem hiding this comment.
Отлично: благодаря мокам тесты стали быстрее и управляемее, теперь они меньше зависят от внешних данных
tests/test_database.py
Outdated
|
|
||
| def test_database_available_buns(): | ||
| database = Database() | ||
| buns = database.available_buns() |
There was a problem hiding this comment.
Можно лучше: мы тестируем не атрибуты экземпляров класса или наполнение базы данных, а методы приложения. Необходимо добавить шаги в которых будут использованы методы и проверить результат. Базу данных тестировать юниттестами не получится
| def test_ingredient_getters(ingredient_type, name, price): | ||
| ingredient = Ingredient(ingredient_type, name, price) | ||
|
|
||
| assert ingredient.get_type() == ingredient_type |
| BUN_DATA = [ | ||
| ("black bun", 100), | ||
| ("white bun", 200), | ||
| ("gluten free", 300), |
There was a problem hiding this comment.
Можно лучше: надежнее тестировать параметр при остальных неизменных. Например тут тестируем имя, значит в параметризации должна менять только имя, а цена быть всегда одинаковой. Также имеет смысл тестировать разные КЭ а не только латинские буквы с пробелом
2 participants
Добавлены юнит-тесты для классов Bun, Burger, Ingredient и Database
Использованы моки и параметризация
Покрытие pytest-cov: 88%