-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feat/endpoints tariff #57
Merged
Merged
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
В фикстуре auth_superuser заменил regicter_client на superuser
Тест запрета создания профиля напрямую без создания юзера Тест запрета удаления профиля
Переписал круд для получения групп с фильтацией по юзеру. Добавил ручку groups/me Добавил ручку патч Для ручки делет установил статус код 204 Добавил схему апдейта группы Добавил фикстуры для групп и тесты для групп
Добавил ручку получения текущим юзером своей группы по id. Переписал метод гет в круде группы, чтобы подтягивались данные relations. В модели юзера исправил отношения с группами. Изменил на лист. Добавил тест запрета доступа юзера к ручке groups/{group_id}. Добавил тест получения юзером своей группы через ручку groups/me/{group_id}, в этом же тесте запрет получения группы в которой не состоит и 404 если группа не существует.
Ручка гет всех ачивментс поменял депенденсис на суперюзера Добавил тесты запрета получения всех ачивментс юзером и неавторизованным
Добавил ручку апдейт Добавил схему для апдейта Добавил тест апдейта суперюзером
В ручке post добавил статус код 201 Добавил тесты на ручку post
На ручке гет убрал депенденсис, потому что просмотр тарифов должен быть доступен всем Добавил тесты на ручку гет
добавил ручку получения тарифа по id добавил тест на эту ручку
Валидатор check_obj_exists теперь возвращает объект, если он есть в базе. Заменил пут на патч и переписал, чтобы эта ручка работала именно с тарифом, а не с юзером. Добавил схему для апдейта. Написал тесты на патч.
В эндпоинт делет добавлен статус код. Добавлены тесты на делет.
получение количества объектов вынесено в отделбную утилиту
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.
Эндпоинты и тесты по тарифам