add tests for BooksCollector#25
add tests for BooksCollector#25kronosss1703 wants to merge 2 commits intoyandex-praktikum:developfrom
Conversation
tests.py
Outdated
| assert len(collector.get_books_genre()) == expected_count | ||
|
|
||
| def test_set_book_genre_correct_genre_success(self): | ||
| if len(name) <= 40 and len(name) > 0: |
There was a problem hiding this comment.
Нужно исправить: условий в тестах быть не может, тест всегда однозначен. Если это не вписывается в параметризацию, значит ее быть не должно и это просто несколько отдельных тестовых методов
| assert len(collector.get_books_genre()) == 1 | ||
|
|
||
| def test_set_book_genre_incorrect_genre_unsuccess(self): | ||
| def test_new_book_has_no_genre(self): |
There was a problem hiding this comment.
Отлично: в каждом тесте создается собственный экземпляр BooksCollector, но можно лучше - перенести создание экземпляра в фикстуры в отдельный файл conftest, а не повторять это предусловие в каждом тесте
tests.py
Outdated
| collector.add_new_book(name) | ||
| collector.set_book_genre(name, genre) | ||
| books_for_children = collector.get_books_for_children() | ||
| if genre in collector.genre_age_rating: |
There was a problem hiding this comment.
Нужно исправить: условий в тестах быть не может, тест всегда однозначен. Если это не вписывается в параметризацию, значит ее быть не должно и это просто несколько отдельных тестовых методов
| collector.add_book_in_favorites('Богомолов') | ||
| collector.delete_book_from_favorites('Несуществующая книга') | ||
| assert len(collector.get_list_of_favorites_books()) == 1 | ||
| assert 'Богомолов' in collector.get_list_of_favorites_books() No newline at end of file |
There was a problem hiding this comment.
Нужно исправить: отсутствуют тесты на проверку методов get_books_genre. На каждый метод должен быть отдельный тест, вне зависимости, вызывается он в других тестах или нет
2 participants
No description provided.