Conversation
|
|
||
| class TestBooksCollector(unittest.TestCase): | ||
|
|
||
| def setUp(self): |
There was a problem hiding this comment.
Нужно исправить: объект класса BooksCollector() нужно создавать для каждого теста. Чтобы не дублировать код, можно вынести это в фикстуру
|
|
||
| # Тест на добавление новых книг | ||
| def test_add_new_book(self): | ||
| self.collector.add_new_book("1984") |
There was a problem hiding this comment.
Нужно исправить: ты указываешь объект, self для вызова не требуется. Так же он не требуется для вызова ассертов - это не методы объекта
| # Тест на добавление книги, превышающей максимальную длину | ||
| def test_add_new_book_too_long_name(self): | ||
| self.collector.add_new_book("A" * 41) | ||
| self.assertNotIn("A" * 41, self.collector.books_genre) |
There was a problem hiding this comment.
Можно улучшить: этот тест не исключает, что название обрезается
| def test_get_book_genre(self): | ||
| self.collector.add_new_book("The Shining") | ||
| self.collector.set_book_genre("The Shining", "Ужасы") | ||
| self.assertEqual(self.collector.get_book_genre("The Shining"), "Ужасы") |
There was a problem hiding this comment.
Нужно исправить: тест не отличается от предыдущего. Подумай, как иначе ты можешь убедится, что get_book_genre возвращает именно то значение, которое есть в словаре
|
|
||
| class TestBooksCollector(unittest.TestCase): | ||
|
|
||
| def setUp(self): |
There was a problem hiding this comment.
Нужно исправить: тут ничего не происходит, метод можно удалить
| collector = BooksCollector() | ||
| collector.add_new_book("The Shining") | ||
| collector.set_book_genre("The Shining", "Ужасы") | ||
| assert collector.get_book_genre("The Shining") == "Ужасы" |
There was a problem hiding this comment.
Нужно исправить: тест не отличается от проверки set метода. Чтобы методы не проверяли друг друга, сравни полученное с состоянием словаря
| assert collector.get_list_of_favorites_books() == [] | ||
|
|
||
|
|
||
| if __name__ == '__main__': |
| expected = {"War and Peace": "Classic"} | ||
| assert result == expected | ||
|
|
||
| # Дополнительный тест для метода get_list_of_favorites_books |
There was a problem hiding this comment.
Нужно исправить: обязательно должен быть тест на основной положительный сценарий. То есть проверка, что метод действительно возвращает список избранных книг
| @@ -0,0 +1,83 @@ | |||
| import unittest | |||
|
|
|||
| class TestBooksCollector(unittest.TestCase): | |||
There was a problem hiding this comment.
Нужно исправить: тестовый класс должен быть один. После ревью нужно внести изменения в существующий класс, а не добавлять новый
2 participants
No description provided.