Skip to content

Conversation

@petrCher
Copy link
Member

@petrCher petrCher commented Aug 23, 2025

Изменения

Исправил ошибку, при которой вместо error conflict 409 вылетал internal server error 500

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

Изначально в raise AlreadyExists передавали Lecturer.timetable_id, из-за чего требовался атрибут __ name __ у этого объекта.

Check-List

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

@github-actions
Copy link

Code Coverage

Coverage Report
FileStmtsMissCoverMissing
rating_api
   __main__.py440%1–7
   exceptions.py43784%35–37, 48–50, 58
rating_api/models
   base.py64494%24–27
   db.py126398%99, 152, 170
rating_api/routes
   base.py16194%41
   comment.py1233076%64, 82, 124–125, 146–155, 220–221, 223–224, 232–237, 264, 277, 304–315, 346
   exc_handlers.py32391%36, 43, 50
   lecturer.py93496%188, 196, 214, 220
rating_api/schemas
   base.py12467%6–9
   models.py127398%164, 166, 176
TOTAL6826391% 

Summary

Tests Skipped Failures Errors Time
72 0 💤 0 ❌ 0 🔥 24.438s ⏱️

@Zimovchik Zimovchik merged commit 957b053 into main Aug 23, 2025
2 checks passed
@Zimovchik Zimovchik deleted the patch-issue branch August 23, 2025 16:19
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.

[bug] PATCH/lecturer/{id} ошибка 500 при попытке изменить timetable_id на занятый

3 participants