Repository navigation
Conversation
irisqul
left a comment
irisqul
left a comment
There was a problem hiding this comment.
✅ Сильные стороны:
- Код запускается без ошибок
- Тестовая архитектура соответствует изученным практикам написания юниттестов
There was a problem hiding this comment.
Нужно исправить: из финального проекта нужно удалить все файлы и папки, которые не относятся к решению задачи
| def test_get_name_returns_bun_name(name, price): | ||
| bun = Bun(name, price) | ||
|
|
||
| assert bun.get_name() == name |
There was a problem hiding this comment.
Нужно исправить: не тестировался метод get_price
| from unittest.mock import Mock | ||
| from burger import Burger | ||
|
|
||
|
|
There was a problem hiding this comment.
Нужно испрваить: Тестовые методы внутри одного тестового модуля объединяют в тестовые классы.
Один тестовый класс (соответственно один тестовый модуль) на несколько методов, проверяющих схожую функциональность
| "(==== black bun ====)\n" | ||
| "\n" | ||
| "Price: 250" | ||
| ) No newline at end of file |
There was a problem hiding this comment.
Нужно исправить: не все методы протестированы
| def test_get_type_returns_ingredient_type(ingredient_type): | ||
| ingredient = Ingredient(ingredient_type, "ingredient", 100) | ||
|
|
||
| assert ingredient.get_type() == ingredient_type No newline at end of file |
There was a problem hiding this comment.
Нужно исправить: не все методы протестированы
|
|
||
|
|
||
| class TestBurger: | ||
| @pytest.fixture |
There was a problem hiding this comment.
Нужно исправить: фикстуры нужно расположить в файле conftest.py
|
|
||
|
|
||
| class TestIngredient: | ||
| @pytest.fixture |
There was a problem hiding this comment.
Нужно исправить: фикстуры нужно расположить в файле conftest.py
| def ingredient(self): | ||
| return Ingredient(INGREDIENT_TYPE_SAUCE, "hot sauce", 50) | ||
|
|
||
| def test_get_price_returns_ingredient_price(self, ingredient): |
There was a problem hiding this comment.
Нужно исправить: не стоит хранить одно и то же тестовое значение отдельно - если что-то было создано в фикстуре, то тут стоит вызвать это значение, а не дублировать его. Иначе тесты станут неподдерживаемыми - легко изменить что то в фикстуре (например) и не заметить что и в тесте тоже нужно поменять. Так как сейчас эти данные не связаны, но должны быть. Удачное решение - создать файл с тестовыми данными и обращаться к нему и в фикстуре и в тестах
No description provided.