Sprint_4 by AnnaShamraeva · Pull Request #6 · AnnaShamraeva/Tests_python

AnnaShamraeva · 2025-10-28T13:27:05Z

No description provided.

irisqul · 2025-10-29T07:15:09Z

tests.py

-    def test_add_new_book_add_two_books(self):
-        # создаем экземпляр (объект) класса BooksCollector
-        collector = BooksCollector()
+    @pytest.fixture # создаем функцию для создания тестовых объектов в начале тестов после класса 


Нужно исправить: перенести фикстуру в файл conftest

irisqul · 2025-10-29T07:16:11Z

tests.py

+    @pytest.mark.parametrize('genre', ['Фантастика', 'Ужасы', 'Детективы', 'Мультфильмы', 'Комедии'])
+# с помощью @pytest.mark.parametrize тест будет выполнен для каждого жанра из списка 
+    def test_init_all_genre_exist(self, collector, genre):
+        assert genre in collector.genre # проверяем жанр на наличие в списке genre


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

tests.py

+
+    def test_get_book_genre_return_genre_set(self, collector):   
+        assert collector.get_books_genre('Новая книга ужасов') == 'Ужасы'
+        # здесь и далее замена создания объекта (экземпляра класса) на использование фикстуры collector через параметр


irisqul · 2025-10-29T07:20:20Z

tests.py

+    @pytest.mark.parametrize('invalid_genre', ['Комиксы', ' ', '666', None])
+    def test_init_all_genre_is_invalid(self, collector, invalid_genre):
+        collector.set_book_genre('Книга', invalid_genre)
+        assert collector.get_books_genre('Книга') != invalid_genre


есть конкретное ожидание чему жанр должен быть равен в данном тесте, нужно сделать проверку более точной

Предыдущая проверка направлена на проверку ожидаемого жанра из списка

нет, предыдущая проверка не проверят этого. В случае попытки присвоить невалидный жанр, жанр не должен назначится, т.е. будет например равен None или пустым

irisqul · 2025-10-31T04:13:38Z

tests.py

+# с помощью @pytest.mark.parametrize тест будет выполнен для каждого жанра из списка 
+    def test_init_all_genre_exist(self, genre):
+        collector = BooksCollector() # создаем объект класса BooksCollector
+        genre_list=collector.get_book_genre() # вызываем метод, возвращающий список жанров


во-первых сейчас ошибка и не верно вызывается метод
TypeError: BooksCollector.get_book_genre() missing 1 required positional argument: 'name'

Во-вторых мы не должны тестировать атрибуты экземпляра класса - список жанров закодирован внутри приложения, его не нужно проверять, он может меняться.Нам нужно проверять методы приложения - добавление книг и что ожидаем после, назначение жанров книгам и т.п.

irisqul · 2025-10-31T04:14:31Z

tests.py

+    @pytest.mark.parametrize('invalid_genre', ['Комиксы', ' ', '666', None])
+    def test_init_all_genre_is_invalid(self, invalid_genre):
+        collector = BooksCollector() # создаем объект класса BooksCollector
+        collector.set_book_genre('Книга', invalid_genre)


нужно исправить: прежде чем назначить жанр нужно добавить книгу в приложение.

irisqul · 2025-10-31T04:15:36Z

tests.py

+    @pytest.mark.parametrize('invalid_genre', ['Комиксы', ' ', '666', None])
+    def test_init_all_genre_is_invalid(self, collector, invalid_genre):
+        collector.set_book_genre('Книга', invalid_genre)
+        assert collector.get_books_genre('Книга') != invalid_genre


нет, предыдущая проверка не проверят этого. В случае попытки присвоить невалидный жанр, жанр не должен назначится, т.е. будет например равен None или пустым

irisqul · 2025-10-31T04:17:43Z

tests.py

+
+    def test_get_books_with_specific_genre_get_book_by_genre(self):    
+        collector = BooksCollector() # создаем объект класса BooksCollector
+        assert len(collector.get_books_with_specific_genre('Ужасы')) == 1


нужно исправить: в этом и следующем тесте - почему ожидаем что есть книги с этими жанрами? их не добавляли.

irisqul · 2025-10-31T04:18:30Z

tests.py

+
+    def test_get_books_for_children_get_book_without_age_rating_possible(self):
+        collector = BooksCollector() # создаем объект класса BooksCollector
+        assert len(collector.get_books_for_children()) == 1


нужно исправить: тут нужно сначала добавить книги и назначить им детские жанры, потом добавить книги с недетскими жанрами и проверить что список верный

tests.py

+# 7.1 Проверка добавления книги в Избранное
+
+    def test_add_book_in_favorites_add_new_book_possible(self, collector):
+        collector.add_book_in_favorites('Новая книга ужасов')


tests.py

+
+    def test_add_book_in_favorites_add_new_book_possible(self, collector):
+        collector.add_book_in_favorites('Новая книга ужасов')
+        assert len(collector.get_list_of_favorites_books()) == 1


irisqul · 2025-10-31T04:19:41Z

tests.py

+
+    def test_add_book_in_favorites_add_one_book_two_times_not_possible(self):
+        collector = BooksCollector() # создаем объект класса BooksCollector
+        collector.add_book_in_favorites('Новая книга ужасов')


в этом тесте не хватает шагов

tests.py

-        collector.add_new_book('Гордость и предубеждение и зомби')
-        collector.add_new_book('Что делать, если ваш кот хочет вас убить')
+    def test_delete_book_from_favorites_delete_book_possible(self):
+        collector = BooksCollector() # создаем объект класса BooksCollector


tests.py

-    # напиши свои тесты ниже
-    # чтобы тесты были независимыми в каждом из них создавай отдельный экземпляр класса BooksCollector()
+    def test_get_list_of_favorites_books_get_list_of_favorites_books_successful(self):
+        collector = BooksCollector() # создаем объект класса BooksCollector


irisqul · 2025-10-31T04:21:49Z

tests.py

+# 2 Проверка жанра книги  
+# 2.1 Проверка валидного жанра книги
+
+    @pytest.mark.parametrize('genre', ['Фантастика', 'Ужасы', 'Детективы', 'Мультфильмы', 'Комедии'])


Нужно исправить: В данном задании для параметризации лучше всего подходят граничные значения в классе длины имени - отдельно невалидные значения и отдельным тестом - ГЗ для валидных

irisqul · 2025-11-04T08:24:55Z

tests.py

+# 1.1 Проверка добавления одной новой книги
+    def test_add_new_book_plus_one_book(self): # в имени функции test_ название тестируемого метода указание что тестируем
+# self показывает, что этот метод является методом класса
+        collector = BooksCollector() # создаем объект класса BooksCollector


оптимально было бы сделать фикстуру для создания экземпляра класса и использовать во всех тестах

AnnaShamraeva added 4 commits October 26, 2025 12:45

Commit on main branch

4d96936

Add tests

2c23a64

Add tests

98fc773

Fix issues based on review comments

253c84c

irisqul reviewed Oct 29, 2025

View reviewed changes

AnnaShamraeva added 3 commits October 30, 2025 14:00

Apply changes from second review

8901698

Ignore __pycache__ directories

68cc2f1

Apply changes from second review

efc3fc6

irisqul reviewed Oct 31, 2025

View reviewed changes

AnnaShamraeva added 2 commits November 1, 2025 11:56

6 failed, 7 passed, 1 error in 0.11s

52214ce

Run 16 tests and confirm all pass

7936315

irisqul reviewed Nov 4, 2025

View reviewed changes

Some files was hide

1497b1a

Conversation

AnnaShamraeva commented Oct 28, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

This comment was marked as outdated.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Comments