Fix/issue170

[CaseAsLimbo]

Изменения

• Проблема падения 57 тестов с ERROR(Исправить падающие тесты #170)

  • Был исправлен баг из-за которого после падения не анониманого комментария с ERROR падало 57 тестов

  • Была исправлена причина падения теста test_create_comment с тестовым набором включающим не анонимные комментарии

  • Добавлена проверка полей добавляемых комментариев на соответствие userdata

• Написание тестов для ручки import_comments(Написать тесты для ручки post comment/import)

  • Добавлены тесты для ручки post comment/import

• Был реализован мок aiohttp запросов(для выдачи ачивки в test_comment_create) через библиотеку aioresponses

---

Детали реализации

• Баг из-за которого падало 57 тестов с ERROR связан с не очевидным поведением sqlalchemy, где на основе того, были ли подготовлены на вставку (INSERT) в текущей не завершенной транзакции одновременно записи в таблицах LecturerUserComment и Comment. Если тест падал с TypeError и успевала добавиться только одна из этих таблиц, алхимия в tear-down фикстуры lecturers в итоговый SQL запрос не добавляла строку на удаление записи LecturerUserComment - сразу после этого происходило удаление лектора из таблицы Lecturer, что приводило к конфликту ведь не удаленный LecturerUserComment по-прежнему ссылался на него.
  Решение: (update) В фикстуре lecturers добавлен dbsession.flush() после добавления в транзакцию каждого элемента на удаление.
• Тест test_create_comment с не анонимаными комментариями падал, из-за того, что для user-а передавалась не корректная userdata. Были добавлены мок фикстуры authlib_user, authlib_mock и user_mock для подмены данных user-а возвращаемых UnionAuth

Check-List

• Вы проверили свой код перед отправкой запроса?
• Вы написали тесты к реализованным функциям?
• Вы не забыли применить форматирование black и isort для Back-End или Prettier для Front-End?

[github-actions]

Coverage Report

File	Stmts	Miss	Cover	Missing
rating_api
__main__.py	4	4	0%	1–7
exceptions.py	43	11	74%	24, 35–37, 48–50, 58, 70, 78, 86
rating_api/models
base.py	64	32	50%	35–38, 51–59, 63–84, 89–94
db.py	162	30	81%	115, 117, 119, 121, 132, 167, 171, 175–177, 181–183, 187–189, 199, 213, 222–224, 229, 250, 254, 258, 272–284
rating_api/routes
base.py	16	1	94%	41
comment.py	151	120	21%	59–155, 167–176, 190–197, 246–304, 327–332, 350–370, 393–404, 429–453
exc_handlers.py	32	7	78%	29, 36, 43, 50, 64, 71, 78
lecturer.py	109	46	58%	41–48, 61–93, 228–244, 258–269
rating_api/schemas
base.py	12	4	67%	6–9
models.py	155	3	98%	189, 191, 201
rating_api/utils
mark.py	8	8	0%	1–20
TOTAL	790	266	66%

Summary

Tests	Skipped	Failures	Errors	Time
94	0 💤	52 ❌	0 🔥	15.941s ⏱️

[petrCher]

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

[github-actions]

💩 Code linting failed, use black and isort to fix it.

[github-actions]

💩 Code linting failed, use black and isort to fix it.

[github-actions]

💩 Code linting failed, use black and isort to fix it.