Последняя неделя #183
Merged
Последняя неделя #183
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
… bug in urls (v1/v1/)
Добавление ER-диаграммы в read.me
Минорные исправление и добавления health-check.
…ect.toml в корень проекта. (#19) * Написан Makefile, дополнены .env.example, README.md, перенесён pyproject.toml в корень проекта. * Разделены make-команды на prod и dev.
…nore статику, чуть переделал README под реалии проекта
…class, everything is stored in cache from now on
…point and if the code sent is correct
…rases; add app to installed apps
…generation, new fields for user model
…file Переделал Makefile, починил docker-compose для прода, добавил в gitignore статику, чуть переделал README под реалии проекта
Feature/auth iter2
…efile, поправил команды в README
…riable Добавил переменную порта в docker-compose.dev.yml, чуть переделал Makefile, поправил команды в README
Добавлены модели для бега
* added minio as s3 server-container * added minio as s3 server-container * added minio as s3 server-container * added minio as s3 server-container * updated env example * updated cd
…out-tracker-backend into develop
…y-workout-tracker/diary-workout-tracker-backend into feature/logic_project_timezone_user
…ect_timezone_user Feature/logic project timezone user
…achievements Повторяющиеся ачивки
…achievements bugfix
freezes test and >0 check
YarikineZ
approved these changes
Apr 7, 2024
There was a problem hiding this comment.
Ребят, ну вы красавчики. Очень рад за вас.
Пара комментов:
- Celery для отправки почты очень хорошая штука, но чтобы разобраться как оно работает внутри, советую попробовать построить свой обработчик очереди на Redis для той же почты. Будет полезно для самообразования + это получится более гибкая штука, в которую можно будет запихнуть например еще пуш уведомления, подключить к админке и пр
- Очень рад что у вас появился S3. Кажется в этот ПР попало не все, не нашел откуда дегаются методы "положить картинку" и "получить картинку". В целом, тут надо просто потрогать технологию. В корпе/стартапе будет все тоже самое, только s3 будет не в компоузе, а внешний
- Про тесты написал чуть выше. Не стоит тестировать вообще все, надо научиться выделять штуки, которые хотите зафиксировать и писать тесты под них
| AWS_S3_SIGNATURE_VERSION = os.getenv("AWS_S3_SIGNATURE_VERSION") | ||
| AWS_S3_USE_SSL = os.getenv("AWS_S3_USE_SSL", False) == "True" | ||
| AWS_S3_SECURE_URLS = os.getenv("AWS_S3_SECURE_URLS", False) == "True" | ||
| AWS_S3_URL_PROTOCOL = os.getenv("AWS_S3_URL_PROTOCOL", "http:") |
|
|
||
|
|
||
| @pytest.mark.django_db | ||
| def test_ios_achievement_is_being_created(user, user_client, load_achievement_fixtures, training_end_data) -> None: |
There was a problem hiding this comment.
Плохо понимаю что вы решили тестить: интерфейс, базу, саму джангу?
Смотрите, тесты обычно пишутся, когда есть какая-то сложная штука, с которой удобно обращаться как с черным ящиком: зафиксировали и если кто-то что-то задел, то не коммитит.
Например на проектах с мобилками можно писать тесты на соответствие ответов свагеру. Не тот свагер, который вы генерируете, а зафиксированный от системного аналитика. И там тестятся и коды ответа и соответствие body ответа
Или есть например риск задеть настройки интеграции с почтовым сервером. Можно перед деплоем послать тестовое сообщение с assert на код ответа
В общем тестирование это плохо объясненная в курсах тема. Наверное главное это понять:
- Писать тесты под все не нужно. Потом просто дольше вносить правки
- Нужно научиться выделять куски, где есть куча пограничных кейсов и есть риск что-то задеть. И уже писать тесты под это
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.
Реализовано все приложение из базовых требований: