Feat/test achievements #56
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 30 commits
January 20, 2024 22:04
В фикстуре auth_superuser заменил regicter_client на superuser
Тест запрета создания профиля напрямую без создания юзера Тест запрета удаления профиля
Переписал круд для получения групп с фильтацией по юзеру. Добавил ручку groups/me Добавил ручку патч Для ручки делет установил статус код 204 Добавил схему апдейта группы Добавил фикстуры для групп и тесты для групп
Добавил ручку получения текущим юзером своей группы по id.
Переписал метод гет в круде группы, чтобы подтягивались данные relations.
В модели юзера исправил отношения с группами. Изменил на лист.
Добавил тест запрета доступа юзера к ручке groups/{group_id}.
Добавил тест получения юзером своей группы через ручку groups/me/{group_id},
в этом же тесте запрет получения группы в которой не состоит
и 404 если группа не существует.
Ручка гет всех ачивментс поменял депенденсис на суперюзера Добавил тесты запрета получения всех ачивментс юзером и неавторизованным
added 14 commits
January 30, 2024 21:12
Добавил ручку получения юзером своих ачивментс Переписал get_users_obj для ачивмент
добавил ручку /me/{achievement_id}
переписал круд get
добавил тест на ручку /me/{achievement_id}
рефакторинг теста речки /me
Добавил ручку апдейт Добавил схему для апдейта Добавил тест апдейта суперюзером
получение количества объектов вынесено в отделбную утилиту
ilonka05
reviewed
Feb 3, 2024
| dependencies=[Depends(current_user)] | ||
| ) | ||
| async def get_self_achievements( | ||
| user: User = Depends(current_user), |
There was a problem hiding this comment.
Думаю здесь тоже потребуется пагинация
| ) | ||
| if achievement is None: | ||
| raise HTTPException( | ||
| status_code=404, |
There was a problem hiding this comment.
404 заменить на fastapi status
| ) | ||
| if user.id not in [_.id for _ in achievement.profiles]: | ||
| raise HTTPException( | ||
| status_code=403, |
| if user.id not in [_.id for _ in achievement.profiles]: | ||
| raise HTTPException( | ||
| status_code=403, | ||
| detail='У выс нет этого achievement.' |
There was a problem hiding this comment.
Небольшая опечатка, заменить на 'вас'
| session: AsyncSession = Depends(get_async_session) | ||
| ): | ||
| """Апдейт ачивмент.""" | ||
| _achievement = await achievement_crud.get(achievement_id, session) |
There was a problem hiding this comment.
Надо при апдейте добавить check_obj_exists для проверки, есть ли объект в бд
| self, | ||
| db_session: AsyncSession, | ||
| auth_superuser: TestClient | ||
| ): |
There was a problem hiding this comment.
Докстринг пропущен, лучше чтобы он уже везде здесь был
| db_session: AsyncSession, | ||
| auth_superuser: TestClient | ||
| ): | ||
| """Тест апдейта ачивмент суперюзером.""" |
There was a problem hiding this comment.
Может добавить проверку, если ачивмента нет в бд?
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Добавил ручки и тесты