Conversation
| long_name = ('A' * 41) | ||
| collector.add_new_book(long_name) | ||
| assert long_name not in collector.get_books_genre() | ||
| # assert len(long_name) == 40 |
There was a problem hiding this comment.
Нужно исправить: решение не должно содержать закомментированный код
|
|
||
| children_books = collector.get_books_for_children() | ||
|
|
||
| assert 'name' not in children_books |
There was a problem hiding this comment.
Нужно исправить: здесь должно обращение к переменной name , а не использование строки 'name'
| class TestBooksCollector: | ||
|
|
||
| def test_add_new_book_add_two_books(self): | ||
| collector = BooksCollector() |
There was a problem hiding this comment.
Можно лучше: создание объекта BooksCollector можно вынести в фикстуры
| def test_add_book_in_favorites_with_genre(self): | ||
| collector = BooksCollector() | ||
| collector.add_new_book('Рыцари Сорока Островов') | ||
| collector.set_book_genre('Рыцари Сорока Островов', 'Фантастика') |
There was a problem hiding this comment.
Можно лучше: здесь и далее: для тестов проверки favorites - не обязательно устанавливать жанр книги
| not_at_list_genre = collector.get_book_genre(book_name) | ||
|
|
||
| assert book_name not in not_at_list_genre |
There was a problem hiding this comment.
Отсутствует коммент: метод .get_book_genre возвращает жандр книги, в данном случае он вернет пустую строку.
Проверка что название книги не содержится в пустой строке не может считаться валидной для проверки установки жанра не из списка
| collector.add_new_book('Рыцари Сорока Островов') | ||
| collector.set_book_genre('Рыцари Сорока Островов', 'Фантастика') | ||
| book_collection = collector.get_books_genre() | ||
| assert 'Рыцари Сорока Островов' in book_collection |
There was a problem hiding this comment.
Отсутствует коммент: Нужно исправить: здесь и далее: в юнит тестах assert ставится только на проверяемой функциональности, методы добавления книги и получения книг проверяются в рамках других юнит тестов
| collector = BooksCollector() | ||
| collector.add_new_book('Звездный лабиринт:Черновик') | ||
| collector.set_book_genre('Звездный лабиринт:Черновик', 'Фантастика') | ||
| book_collection = collector.get_books_genre() |
There was a problem hiding this comment.
Отсутствует коммент: Нужно исправить: переменная создается но нигде не используется
2 participants
No description provided.