Conversation
|
|
||
| # напиши свои тесты ниже | ||
| # чтобы тесты были независимыми в каждом из них создавай отдельный экземпляр класса BooksCollector() No newline at end of file | ||
|
|
There was a problem hiding this comment.
В тесте есть ошибка. Если ты оставляешь этот тест, ее нужно исправить
| # чтобы тесты были независимыми в каждом из них создавай отдельный экземпляр класса BooksCollector() No newline at end of file | ||
|
|
||
| @pytest.fixture | ||
| def collector_with_books(self): |
There was a problem hiding this comment.
Нужно исправить: фикстуры нужны для хранения логики предусловий, а не создания и возвращения данных. Нужные для конкретного еста данные ты можешь задать прямо в нем, напрямую обращаясь к словарю или списку
| assert collector.get_book_genre('Книга 1') == 'Фантастика' | ||
|
|
||
|
|
||
| def test_get_book_genre_existing_book(self, collector_with_books): |
There was a problem hiding this comment.
Нужно исправить: тест аналогичен тесту set_book_genre. Попробуй установить значение жанра иначе, или иначе получить его для сравнения. Используй доступ к словарю
| collector.set_book_genre('Книга 2', 'Комедии') | ||
| collector.set_book_genre('Книга 3', 'Детективы') | ||
| result = collector.get_books_for_children() | ||
| assert 'Книга 1' in result |
There was a problem hiding this comment.
можно улучшить: состояние списка можно проверить одним ассертом
| assert 'Книга 3' not in favorites | ||
|
|
||
|
|
||
| def test_get_list_of_favorites_books_returns_list_with_two_books(self, collector_with_books): |
There was a problem hiding this comment.
Нужно исправить: по сути тест не отличается от добавление в избранное. Проверь методы независимо
| collector = collector_with_books | ||
| collector.set_book_genre('Книга 2', 'Комедии') | ||
| assert collector.get_book_genre('Книга 2') == 'Комедии' | ||
|
|
There was a problem hiding this comment.
Нужно исправить: по названию ты проверяешь get_book_genre, но этот метод отсутствует в тесте. Тест подойдет для set метода, названия должны соответствовать сути
| def test_get_books_genre_returns_dictionary_sum_3(self, collector_with_books): | ||
| collector = collector_with_books | ||
|
|
||
| books = ['Книга 1', 'Книга 2', 'Книга 3'] |
There was a problem hiding this comment.
можно улучшить: не обязательно каждый раз добавлять книги в словарь через add, значение словаря можно установить напрямую
2 participants
add 9 tests for BooksCollector