Skip to content

дз №14, задача 2#175

Open
AlenaStupina wants to merge 2 commits into
mainfrom
AlenaStupina
Open

дз №14, задача 2#175
AlenaStupina wants to merge 2 commits into
mainfrom
AlenaStupina

Conversation

@AlenaStupina
Copy link
Copy Markdown
Collaborator

Не понимаю, почему такое низкое тестовое покрытие...

@IlyaOrlov
Copy link
Copy Markdown
Owner

Не понимаю, почему такое низкое тестовое покрытие...

Тесты написаны корректно, а с расчетом покрытия что-то не так. Как Вы его вычисляли/запускали?
Если использовать pytest-cov - получается 77%.

@AlenaStupina
Copy link
Copy Markdown
Collaborator Author

AlenaStupina commented Aug 14, 2023

Не понимаю, почему такое низкое тестовое покрытие...

Тесты написаны корректно, а с расчетом покрытия что-то не так. Как Вы его вычисляли/запускали? Если использовать pytest-cov - получается 77%.

мы это вопрос обсудили уже, но продублирую тут , слайд 21:
Для получения аналогичного отчета с использованием unittest потребовалось бы выполнить следующий набор команд:
$ coverage erase
$ coverage run --source tasks test_tasks.py
$ coverage report -m
$ coverage html

А через pytest-cov тоже получилось 77%

@IlyaOrlov
Copy link
Copy Markdown
Owner

Не понимаю, почему такое низкое тестовое покрытие...

Тесты написаны корректно, а с расчетом покрытия что-то не так. Как Вы его вычисляли/запускали? Если использовать pytest-cov - получается 77%.

мы это вопрос обсудили уже, но продублирую тут , слайд 21: Для получения аналогичного отчета с использованием unittest потребовалось бы выполнить следующий набор команд: $ coverage erase $ coverage run --source tasks test_tasks.py $ coverage report -m $ coverage html

А через pytest-cov тоже получилось 77%

Этот вариант рассчитан на наличие тест раннера в коде:

if __name__ == "__main__":
    unittest.main()

Без этого тест фактически не запускается.

Ещё вариант вместо
coverage run --source task_2 test_task_2.py
запускать так:
coverage run --source task_2 -m unittest test_task_2.py

@AlenaStupina
Copy link
Copy Markdown
Collaborator Author

Этот вариант рассчитан на наличие тест раннера в коде:

if __name__ == "__main__":
    unittest.main()

Без этого тест фактически не запускается.

Ещё вариант вместо coverage run --source task_2 test_task_2.py запускать так: coverage run --source task_2 -m unittest test_task_2.py

Да, получилось тоже 77%

<p> Описание: {{ film.desc }} </p>
<p> Дата создания: {{ film.date_exist }} </p>
{% if film.pub_date %}
<p> Дата публикации отзыва: - {{ film.pub_date }} </p>
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

у меня тут код частично дублируется из шаблона films.html. Как расширить шаблон films.html?

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Сделать общий шаблон film.html для films.html и film_id.html, "наследующий" base.html.
А films.html и film_id.html "унаследовать" от film.html

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