Conversation
| assert len(collector.get_books_genre()) == 2 | ||
|
|
||
| def test_add_new_book_add_two__equal_books_failed(self): | ||
| collector = BooksCollector() |
There was a problem hiding this comment.
Можно улучшить: общее для всех тестов предусловие можно вынести в фикстуру
| collector = BooksCollector() | ||
| collector.add_new_book(name) | ||
| collector.set_book_genre(name, genre) | ||
| assert collector.get_book_genre(name) == genre |
There was a problem hiding this comment.
Нужно исправить:
тест аналогичен set_book_genre. Попробуй установить значение жанра иначе, или иначе получить его для сравнения. Используй доступ к словарю
| books_for_children = collector.get_books_for_children() | ||
| assert len(books_for_children) != books_with_specific_genre | ||
|
|
||
| def test_get_list_of_favorites_books_success(self): |
There was a problem hiding this comment.
Нужно исправить: тест аналогичен тесту добавления в избранное. Измени один из тестов: либо там для проверки не используй get_list_of_favorites_books(), либо здесь проверяй вид списка
| collector = BooksCollector() | ||
| collector.add_new_book('Джейн Эйр') | ||
| collector.add_new_book('Джейн Эйр') | ||
| assert len(collector.get_books_genre()) == 1 |
There was a problem hiding this comment.
Нужно исправить: для get_books_genre() нужен отдельный тест позитивного сценария. Тесты не должны дублировать друг друга. В идеале - чтобы в каждом тесте использовался только один проверяемый метод, все остальное можно получить прямым доступом к словарям и спискам
2 participants
Добрый день!
В ветке develop представлены мои юнит-тесты