Skip to content

Fix delete user rout#206

Merged
Temmmmmo merged 7 commits intomainfrom
Fix-delete-user-rout
Sep 21, 2024
Merged

Fix delete user rout#206
Temmmmmo merged 7 commits intomainfrom
Fix-delete-user-rout

Conversation

@DaymasS
Copy link
Copy Markdown
Contributor

@DaymasS DaymasS commented Sep 5, 2024

Изменения

При удалении юзера юзер ... удаляется.

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

Помимо удаления юзера добавлено удаление соединений юзера с его группами (soft-delete)

Check-List

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

@DaymasS DaymasS requested a review from Temmmmmo September 5, 2024 18:11
@DaymasS DaymasS self-assigned this Sep 5, 2024
@DaymasS DaymasS linked an issue Sep 5, 2024 that may be closed by this pull request
@DaymasS DaymasS force-pushed the Fix-delete-user-rout branch from db5de14 to e23a6b6 Compare September 11, 2024 15:24
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Sep 12, 2024

Code Coverage

Coverage Report
FileStmtsMissCoverMissing
auth_backend
   __main__.py330%1–5
   base.py12467%6–9
   exceptions.py39587%32, 47–49, 54
auth_backend/auth_method
   base.py63494%52, 54, 61, 143
   method_mixins.py32294%27, 50
   oauth.py531670%37, 43, 48–53, 57–68, 84–88
   outer.py1122776%27, 34, 41, 48, 99, 105, 114, 168–173, 186–192, 204–210
   userdata_mixin.py19574%32, 37–40
auth_backend/auth_plugins
   airflow.py331652%26–37, 42–65
   coder.py321650%25–34, 39–52
   email.py2491494%91, 231, 309, 311, 316, 327–333, 380, 421, 472, 482
   github.py1046339%57–116, 127–171, 178, 183, 189–201
   google.py1126443%65–124, 135–163, 170, 176–180, 184–189, 193–202
   keycloak.py1046240%53–115, 122–172, 179, 184, 190–195
   lkmsu.py1379034%59–111, 126–163, 170, 175, 181–195, 199–236, 240–249
   mailu.py341847%25–34, 39–54
   postgres.py432053%29–35, 40–46, 51–63
   telegram.py1116145%56–90, 102–120, 127, 133, 143–163, 167–181
   vk.py1288236%71–123, 134–172, 179, 184, 190–207, 214–240
   yandex.py1278335%58–128, 138–176, 184, 190, 196–222
auth_backend/cli
   group.py13130%1–17
   process.py56560%1–72
   scope.py11110%1–20
   user.py20200%1–40
   user_group.py12120%1–25
auth_backend/kafka
   kafka.py502060%30–33, 42–44, 56–59, 72–82, 85, 93, 103
   kafkameta.py9278%8, 12
auth_backend/models
   base.py83594%79, 100–102, 106
   db.py136398%52, 209, 213
   dynamic_settings.py27774%24–31
auth_backend/routes
   base.py26292%20–21
   exc_handlers.py37586%29, 54, 67, 93, 105
   groups.py84396%85, 135, 137
   scopes.py36392%24, 53–54
   user.py872670%51, 53, 55–64, 70–76, 88–108, 155
   user_session.py84693%76–85, 114, 116, 174, 181
auth_backend/schemas
   models.py98397%148–150
auth_backend/schemas/types
   scopes.py24675%30–32, 37, 40, 42
auth_backend/utils
   security.py43295%36, 61
   smtp.py771877%76–79, 101–120
   string.py7186%12
   user_session_control.py33197%28
TOTAL270888068% 

Summary

Tests Skipped Failures Errors Time
58 6 💤 0 ❌ 0 🔥 9.570s ⏱️

Comment thread auth_backend/routes/user.py Outdated
## Изменения
Фикс для Pydantic 2.9...

## Детали реализации
Добавил обязательный хэндлер 

## Check-List
<!-- После сохранения у следующих полей появятся галочки, которые нужно
проставить мышкой -->
- [ ] Вы проверили свой код перед отправкой запроса?
- [ ] Вы написали тесты к реализованным функциям?
- [ ] Вы не забыли применить форматирование `black` и `isort` для
_Back-End_ или `Prettier` для _Front-End_?

## Изменения
<!-- Опишите здесь на языке, понятном каждому, изменения, сделанные в
исходном коде по пунктам. -->

## Детали реализации
<!-- Здесь можно описать технические детали по пунктам. -->

## Check-List
<!-- После сохранения у следующих полей появятся галочки, которые нужно
проставить мышкой -->
- [ ] Вы проверили свой код перед отправкой запроса?
- [ ] Вы написали тесты к реализованным функциям?
- [ ] Вы не забыли применить форматирование `black` и `isort` для
_Back-End_ или `Prettier` для _Front-End_?
@Temmmmmo Temmmmmo merged commit 4a1570d into main Sep 21, 2024
@Temmmmmo Temmmmmo deleted the Fix-delete-user-rout branch September 21, 2024 17:10
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.

Не работает ручка удаления юзеров

2 participants