Skip to content

Sprint_4 - #9

Open
MiG2026 wants to merge 14 commits into
mainfrom
develop
Open

MiG2026 wants to merge 14 commits into
mainfrom
develop

Conversation

@MiG2026

@MiG2026 MiG2026 commented Aug 9, 2026

Copy link
Copy Markdown
Owner

No description provided.

Comment thread tests.py
def test_add_book_in_favorites(self):
collector = BooksCollector()
collector.add_new_book('12 стульев')
collector.set_book_genre('12 стульев', 'Комедии')

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() No newline at end of file
# тест невозможности повторного добавления книги в "Избранное" (№11)
def test_add_book_in_favorites_dubl(self):

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Можно улучшить: названия тестов стоит делать более информативными

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