Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Userdata update #11

Merged
merged 4 commits into from
Jul 31, 2023
Merged

Userdata update #11

merged 4 commits into from
Jul 31, 2023

Conversation

Temmmmmo
Copy link
Member

@Temmmmmo Temmmmmo commented Jul 30, 2023

Изменения

  1. Добавлена ошибка 404 при получении информации о не существующем в базе данных репозитория userdata пользователе.
  2. Теперь при успешном обновлении данных о пользователе возвращается другая модель данных.
  3. Переделаны старые тесты под данные изменения.

Реализации

  1. Ошибка 404 прокидывается при отсутствии пользователя с заданным при запросе id.
  2. При посылании post-запроса/user/{id} возвращается модель StatusResponseModel.

Детали

  1. Раньше ручка post-запроса /user/{id} своим результатом работы имела возвращаемое значение ручки get-запроса /user/{id}. Теперь просто возвращается успешно ли отработала ручка post-запроса или нет.
  2. Необходимость изменения тестов вызвана тем, что, изменена возвращаемая модель данных ручки post-запроса.
  3. Изменение результата работы ручки post-запроса было необходимо ввиду того, что при удалении информации о пользователе из базы данных, возвращалась ошибка 404. Ответ приходил с ошибкой, так как сразу после удаления в возврат функции уходил результат работы ручки get-запроса, в которой не удавалось найти удаленную ранее запись в базе.

Check-List

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

@Temmmmmo Temmmmmo self-assigned this Jul 30, 2023
@github-actions
Copy link

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

@github-actions
Copy link

github-actions bot commented Jul 30, 2023

Code Coverage

Coverage Report
FileStmtsMissCoverMissing
userdata_api
   __main__.py440%1–7
   exceptions.py9189%8
userdata_api/models
   base.py65592%25–28, 76
userdata_api/routes
   category.py31197%30
   exc_handlers.py14193%21
   param.py40490%34, 48, 85, 105
   source.py29197%32
userdata_api/schemas
   base.py8450%6–9
   user.py20765%18–24
userdata_api/schemas/types
   scope.py24675%29–31, 36, 39, 41
userdata_api/utils
   user.py62297%67, 71
TOTAL4633692% 

Summary

Tests Skipped Failures Errors Time
45 0 💤 0 ❌ 0 🔥 5.155s ⏱️

userdata_api/routes/user.py Outdated Show resolved Hide resolved
userdata_api/utils/user.py Outdated Show resolved Hide resolved
@grigoriev-semyon
Copy link
Member

@Temmmmmo ты забыл тесты поменять после последнего коммита

@Temmmmmo Temmmmmo merged commit 9bb4845 into main Jul 31, 2023
2 checks passed
@Temmmmmo Temmmmmo deleted the add-404-error branch July 31, 2023 19:15
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.

Добавить 404 ошибку на ручку <code>/get/user/{id}</code>
2 participants