Fix 57 ERRORED tests; added tests for the import_comment route; mocked aiohttp requests; fix 52 FAILED tests in CI

[CaseAsLimbo]

Изменения

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

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

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

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

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

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

• (update) Был реализован мок aiohttp запросов(для выдачи ачивки в test_comment_create) через библиотеку pytest.mocker. От aioresponses решено отказаться, потому что текущая реализация (==0.7.9) не совместима с последними версиями aiohttp (==3.14.1)

• Исправлена причина падения 52 тестов в CI. Причина была особенностях реализации fastapi (==0.137.0+), где сигнатуры классов и функций передаваемых в параметрах ручки считывались как query-параметры. Так в моке юзера использовался MagicMock, чья сигнатура *args, **kwargs. Решено с использованием autospec=True в MagicMock для копирования сигнатуры мокнутого объекта. Подробнее в issue: issue.

---

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

• Баг из-за которого падало 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	7	84%	35–37, 48–50, 58
rating_api/models
base.py	64	4	94%	24–27
db.py	162	18	89%	115, 117, 119, 121, 132, 171, 189, 199, 213, 222–224, 258, 272–284
rating_api/routes
base.py	16	1	94%	41
comment.py	151	27	82%	77, 95, 192, 265–266, 268–269, 277–282, 291, 298–300, 330, 353, 393–404, 431
exc_handlers.py	32	3	91%	36, 43, 50
lecturer.py	109	9	92%	72–73, 83, 89–90, 230, 238, 260, 266
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	88	89%

Summary

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

[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.

[petrCher]

у тебя мок всегда возвращает наличие ачивки, игнорируя параметр achievement_id
надо корректно проверять обе ветки логики: когда ачивки нет — POST запрос отправляется, когда есть — не отправляется

[petrCher]

и еще, переименуй пр, пожалуйста, в более содержательное название

[petrCher]

https://github.com/profcomff/rating-api/actions/runs/28372071943/job/84052082838?pr=171
(см print report)
Там все еще достаточное количество ошибок, в основном 422 из-за неправильного формата данных. постарался достаточно глубоко поресерчить, очень вероятно причина в datetime в новой версии pydantic
Понял, почему у меня (и возможно у тебя) локально этого не возникает.
Надо сравнивать версии библиотек которые в CI здесь используются и у себя. Я, например, пользуюсь pytest без виртуального окружения, поэтому те библиотеки что у меня давно устарели, а в прошлых версиях регулярно были warnings про datetime. Но, возможно, помимо datetime есть еще что-то.
Так что, пожалуйста, проверь так ли и с теми же библиотеками запускается у тебя pytest и постарайся разобраться где и как поменять формат данных для корректной валидации под новые версии библиотек.

[petrCher]

для того, чтобы понять 422 из-за чего возникает, просто сделай отладку виде проверки if responce==422 то сделать принт

[petrCher]

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

[petrCher]

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

[petrCher]

#173
вот еще сделай git rebase чтобы не потерялись изменения (к ишью по ссылке приложен пр от Артема)
@CaseAsLimbo

[petrCher]

Супер, что глубоко разобрался в проблеме, очень здорово сделано, молодец!
Посмотри только, пожалуйста, два коммента, они скорее косметические, но для грамотной разработки очень важны. Там даже дело не только в том, что код станет чище и красивее, самое главное, что при изменении какой-то логики другой разраб сможет грамотно что-то исправить/переделать, сейчас он вероятно наделает еще больше ошибок)

[petrCher]

вот теперь красиво