Conversation
tests.py
Outdated
| 'Ехала машина полный бак бензина ехала ехала и в гараж заехала', | ||
| 'По поля по полям синий трактор едет к нам']) | ||
| def test_add_new_book_title_greater_than40_unsuccessfull(self, name): | ||
| collector = BooksCollector() |
There was a problem hiding this comment.
Можно улучшить: общее для всех тестов предусловие можно вынести в фикстуру
There was a problem hiding this comment.
Создание объекта BooksCollector вынесено в фикстуру
tests.py
Outdated
| collector.add_new_book(name) | ||
| collector.set_book_genre(name, genre) | ||
|
|
||
| assert len(collector.get_books_genre()) == 1 |
There was a problem hiding this comment.
Нужно исправить здесь и далее: лишняя проверка. Добавление в словарь проверяется для метода add_new_book в тесте test_add_new_book_title_less_than40_successfull
tests.py
Outdated
| assert len(collector.get_books_genre()) == 1 | ||
| assert collector.get_book_genre(name) == '' | ||
|
|
||
| def test_get_book_genre_not_from_list_empty_string(self): |
There was a problem hiding this comment.
Нужно исправить: для get_book_genre должен быть отдельный тест позитивного сценария. Тесты не должны дублировать друг друга. В идеале - чтобы в каждом тесте использовался только один проверяемый метод, все остальное можно получить прямым доступом к словарям и спискам
There was a problem hiding this comment.
Добавлены тесты для позитивных сценариев функций get...()
tests.py
Outdated
| name = 'Гордость и предубеждение' | ||
| genre = 'Детективы' | ||
| collector.add_new_book(name) | ||
| collector.set_book_genre(name, genre) |
There was a problem hiding this comment.
можно улучшить: для добавления в избранное не обязательно устанавливать жанр
There was a problem hiding this comment.
Убрана установка жанра в тестах с добавлением в избранное
2 participants
Добавлены тесты к приложению BooksCollector