Выполненное первое задания#704
Conversation
irisqul
left a comment
irisqul
left a comment
There was a problem hiding this comment.
✅ Сильные стороны:
- Код запускается без ошибок
- Обеспечено необходимое тестовое покрытие
| def test_bun_getters_return_init_values(name, price): | ||
| bun = Bun(name, price) | ||
|
|
||
| assert bun.get_name() == name |
There was a problem hiding this comment.
Нужно исправить: здесь и далее: это юнит тесты, на каждый метод должен быть отдельный тест
tests/test_bun.py
Outdated
| 'name, price', | ||
| [ | ||
| ('black bun', 100.0), | ||
| ('white bun', 200.5), |
There was a problem hiding this comment.
Можно лучше: для параметризации используют данные которые отличаются друг от друга, например принадлежат к разным классам эквивалентости (либо тестируют границы одного класса) и имеют одинаковый ОР. Твой список - это по сути один и тот же класс, нам не нужно проверять его много раз на самом деле. Можно было бы попробовать вводить латиницу, нелатиницу, с цифрами, с спецсимволами. Хотя у нас нет четких требований о валидации, мы все таки можем их предположить.
Можно лучше: надежнее тестировать параметр при остальных неизменных. Например когда тестируем цену, значит в параметризации должна меняться только цена, а имя быть всегда одинаковым. в других случаях - аналогично
tests/test_database.py
Outdated
|
|
||
| buns = db.available_buns() | ||
|
|
||
| assert len(buns) == 3 |
There was a problem hiding this comment.
Можно лучше: мы тестируем не атрибуты экземпляров класса или наполнение базы данных, а методы приложения. Необходимо добавить шаги в которых будут использованы методы и проверить результат. Базу данных тестировать юниттестами не получится
2 participants
No description provided.