Repository navigation
Conversation
irisqul
left a comment
irisqul
left a comment
There was a problem hiding this comment.
✅ Сильные стороны:
- Код запускается без ошибок
- Тестовая архитектура соответствует изученным практикам написания юниттестов
- Обеспечено необходимое тестовое покрытие
|
|
||
| from praktikum.burger import Burger | ||
|
|
||
| def create_bun(name="Булочка", price=100): |
There was a problem hiding this comment.
Нужно исправить: в файлах с тестами должны быть только тесты, все остальное нужно вынести во внешние файлы
| ingredient.get_price.return_value = price | ||
| return ingredient | ||
|
|
||
| def test_burger_init(): |
There was a problem hiding this comment.
Нужно исправить: мы тестируем не атрибуты экземпляров класса или наполнение базы данных, а методы приложения. Необходимо добавить шаги в которых будут использованы методы и проверить результат.
| ingredient.get_name.return_value = name | ||
| ingredient.get_price.return_value = price | ||
| return ingredient | ||
|
|
There was a problem hiding this comment.
Тестовые методы внутри одного тестового модуля объединяют в тестовые классы.
Один тестовый класс (соответственно один тестовый модуль) на несколько методов, проверяющих схожую функциональность
There was a problem hiding this comment.
Нужно исправить: в директории tests хранятся только тесты и conftest, все остальное должно быть в корне
No description provided.