Skip to content

Sprint_4#5

Open
oreshnitel2-spec wants to merge 4 commits intomainfrom
develop
Open

Sprint_4#5
oreshnitel2-spec wants to merge 4 commits intomainfrom
develop

Conversation

@oreshnitel2-spec
Copy link
Copy Markdown
Owner

Добавлены тесты для класса BooksCollector и README с описанием тестов

# напиши свои тесты ниже
# чтобы тесты были независимыми в каждом из них создавай отдельный экземпляр класса BooksCollector() No newline at end of file
# чтобы тесты были независимыми в каждом из них создавай отдельный экземпляр класса BooksCollector()
@pytest.mark.parametrize('book_name',['А' * 41,'Б' * 50])
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Необходимо исправить: не хватает теста на проверку границы в 40 символов

def test_add_book_in_favorites_existing_book_added(self):
collector = BooksCollector()
collector.add_new_book('Властелин колец')
collector.add_book_in_favorites('Властелин колец')
Copy link
Copy Markdown

@DolgoarshinnykhLG DolgoarshinnykhLG Jan 26, 2026

Choose a reason for hiding this comment

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

Необходимо исправить: не хватает теста на добавление в избранное книги, которая не была добавлена в коллекцию

@pytest.mark.parametrize('book_name',['А' * 41,'Б' * 50])
def test_add_new_book_name_longer_than40_not_added(self, book_name):
collector = BooksCollector()
collector.add_new_book(book_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.

Необходимо исправить: не хватает теста на повторное добавление книги

…обавление книги не создаёт дубль
…книги длиной 40 символов успешно добавляется
… - нельзя добавить в избранное книгу, отсутствующую в коллекции
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