Skip to content

Sprint4#26

Open
llorichT wants to merge 4 commits into
yandex-praktikum:developfrom
llorichT:develop
Open

Sprint4#26
llorichT wants to merge 4 commits into
yandex-praktikum:developfrom
llorichT:develop

Conversation

@llorichT
Copy link
Copy Markdown
Collaborator

No description provided.

@llorichT
Copy link
Copy Markdown
Collaborator Author

Для Ирины Рудниченко 4 спринт от Костроминой Анастасии

Comment thread tests.py Outdated
# затем, что тестируем add_two_books - добавление двух книг
def test_add_new_book_add_two_books(self):
# создаем экземпляр (объект) класса BooksCollector
collector = BooksCollector()
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Отлично: в каждом тесте создается собственный экземпляр BooksCollector, но можно лучше - перенести создание экземпляра в фикстуры в отдельный файл conftest, а не повторять это предусловие в каждом тесте

Comment thread tests.py Outdated

# проверяем, что добавилось именно две
# словарь books_rating, который нам возвращает метод get_books_rating, имеет длину 2
assert len(collector.get_books_rating()) == 2
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

в приложении нет такого метода, тест не работает

Comment thread tests.py

# напиши свои тесты ниже
# чтобы тесты были независимыми в каждом из них создавай отдельный экземпляр класса BooksCollector()
import pytest
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

в сдаваемом пул реквесте должны войти все строки написанного кода.

также нужно прислать полное решение задания, без лишних строк из примера.

импорты размещаются в начале файла

Comment thread tests.py
assert len(collector.get_books_genre()) == 1

@pytest.mark.parametrize('name', [
'',
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Отлично: удачное применение параметризации, протестированы разные граничные значения для длины имени

Comment thread tests.py
collector.add_book_in_favorites('1984')
collector.delete_book_from_favorites('1984')

assert '1984' not in collector.get_list_of_favorites_books()
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Нужно исправить: отсутствуют тесты на проверку метода get_books_genre и get_book_genre. На каждый метод должен быть отдельный тест, вне зависимости, вызывается он в других тестах или нет

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants