Sprint4#26
Conversation
|
Для Ирины Рудниченко 4 спринт от Костроминой Анастасии |
| # затем, что тестируем add_two_books - добавление двух книг | ||
| def test_add_new_book_add_two_books(self): | ||
| # создаем экземпляр (объект) класса BooksCollector | ||
| collector = BooksCollector() |
There was a problem hiding this comment.
Отлично: в каждом тесте создается собственный экземпляр BooksCollector, но можно лучше - перенести создание экземпляра в фикстуры в отдельный файл conftest, а не повторять это предусловие в каждом тесте
|
|
||
| # проверяем, что добавилось именно две | ||
| # словарь books_rating, который нам возвращает метод get_books_rating, имеет длину 2 | ||
| assert len(collector.get_books_rating()) == 2 |
There was a problem hiding this comment.
в приложении нет такого метода, тест не работает
|
|
||
| # напиши свои тесты ниже | ||
| # чтобы тесты были независимыми в каждом из них создавай отдельный экземпляр класса BooksCollector() | ||
| import pytest |
There was a problem hiding this comment.
в сдаваемом пул реквесте должны войти все строки написанного кода.
также нужно прислать полное решение задания, без лишних строк из примера.
импорты размещаются в начале файла
| assert len(collector.get_books_genre()) == 1 | ||
|
|
||
| @pytest.mark.parametrize('name', [ | ||
| '', |
There was a problem hiding this comment.
Отлично: удачное применение параметризации, протестированы разные граничные значения для длины имени
| collector.add_book_in_favorites('1984') | ||
| collector.delete_book_from_favorites('1984') | ||
|
|
||
| assert '1984' not in collector.get_list_of_favorites_books() |
There was a problem hiding this comment.
Нужно исправить: отсутствуют тесты на проверку метода get_books_genre и get_book_genre. На каждый метод должен быть отдельный тест, вне зависимости, вызывается он в других тестах или нет
No description provided.