Test/dev 72 add tests for post comment import#172
Conversation
…дготовленных SQL-запросов и override_dbsession_in_route, мок для переопределения сессии ручки на тестовую сессию
…стройкой cascade, что дать алхимии понять в каком порядке их удалять, вне зависимости от связанных других таблиц
…turerUserComments
…ACK в фикстуру logging_sql_req_before_execute
Coverage Report
Summary
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| .pyre/ | ||
|
|
||
| # uv pocket manager | ||
| uv.lock |
| lecturers[-1].is_deleted = True | ||
| for lecturer in lecturers: | ||
| dbsession.add(lecturer) | ||
| dbsession.add_all(lecturers) |
| "user_scopes": [{"id": 0, "name": "string", "comment": "string"}], | ||
| "indirect_groups": [{"id": 0, "name": "string", "parent_id": 0}], | ||
| "groups": [{"id": 0, "name": "string", "parent_id": 0}], | ||
| def authlib_user(): |
There was a problem hiding this comment.
здесь аналогично прошлому пр исправить(по комментам прошлого)
p.s. см коммент на этот счет в отдельном комментарии в этом пр
| def test_create_comment(client, dbsession, lecturers, body, lecturer_n, response_status): | ||
| params = {"lecturer_id": lecturers[lecturer_n].id} | ||
| post_response = client.post(url, json=body, params=params) | ||
|
|
| avatar_link: Mapped[str] = mapped_column(String, nullable=True, comment="Ссылка на аву препода") | ||
| timetable_id: Mapped[int] | ||
| comments: Mapped[list[Comment]] = relationship("Comment", back_populates="lecturer") | ||
| lecturer_user_comments: Mapped[list[LecturerUserComment]] = relationship( |
| status.HTTP_422_UNPROCESSABLE_CONTENT, | ||
| ), | ||
| ], | ||
| ) |
There was a problem hiding this comment.
тут точно больше тестов не надо? на неудачные кейсы? если нет, то все ок, сделал правильно
| import logging | ||
|
|
||
| import pytest | ||
| from auth_lib.fastapi import UnionAuth |
|
Смотри, сейчас у тебя два пр, дело в том, что ты поменял код в обоих пр приблизительно в одном и том же месте. В таком случае рекомендуется только в первом пр добавлять новый код, во втором код только решающий проблему только конкретного таска, предыдущий код надо подтягивать с помощью git rebase после успешного мерджа первого пр. |
|
Также посмотри, пожалуйста, действительно ли ты закрыл все неудачные рейсы на тест импорта? |
2 participants
Изменения
Check-List
blackиisortдля Back-End илиPrettierдля Front-End?