qa_python_4 #2
qa_python_4 #2
Conversation
tests.py
Outdated
| collector.add_new_book('Карлсон') | ||
| assert len(collector.get_books_genre()) == 5 | ||
|
|
||
| def test_set_book_genge(self, collector): |
There was a problem hiding this comment.
Нужно исправить: зачем вызывать фикстуру, если она не используется, а создается новый объект?
There was a problem hiding this comment.
по идеии да, поправил, но тут просто добавил два фильма к уже имеющимся.
| collector.set_book_genre('Гордость и предубеждение и зомби', 'Детективы') | ||
| assert collector.get_book_genre('Гордость и предубеждение и зомби') == 'Детективы' | ||
|
|
||
| def test_get_book_genre(self, collector): |
There was a problem hiding this comment.
Нужно исправить: уже проверено в test_set_book_genge
| from main import BooksCollector | ||
|
|
||
| @pytest.fixture | ||
| def collector(): |
There was a problem hiding this comment.
Можно лучше: Фикстура реально используется в одном тесте, может без нее лучше обойтись?
There was a problem hiding this comment.
решил использвать ее для тренировки по написанию кода.
tests.py
Outdated
| collector.set_book_genre('Крошка - коврошка', 'Мультфильмы') | ||
| collector.add_new_book('Что делать, если ваш кот хочет вас убить') | ||
| collector.set_book_genre('Что делать, если ваш кот хочет вас убить', 'Фантастика') | ||
| assert collector.get_books_for_children('Мультфильмы') == ['Крошка - коврошка'] |
There was a problem hiding this comment.
Нужно исправить: метод get_books_for_children не принимает ничего
tests.py
Outdated
| collector = BooksCollector() | ||
| collector.add_new_book('Гордость и предубеждение и зомби') | ||
| collector.set_book_genre('Гордость и предубеждение и зомби', 'Детективы') | ||
| assert collector.set_book_genre('Гордость и предубеждение и зомби') == 'Детективы' |
There was a problem hiding this comment.
Нужно исправить: set_book_genre не возвращает ничего, так писать нельзя. Прошлый коммент был для того, чтобы один из тестов удалить, т.к. тут проверяется и set_book_genre и get_book_genre
There was a problem hiding this comment.
я если честкно не понимаю тут , что не так . я удаляю этот тест
There was a problem hiding this comment.
Тест проверял, что collector.set_book_genre('Гордость и предубеждение и зомби') вернет 'Детективы'. Но этот метод ничего не возвращает, он просто присваивает жанр книге. Иными словами этот assert выглядит как None == 'Детективы'
закончил написание кода по проверкам файла main.py