Skip to content
This repository has been archived by the owner on Mar 19, 2024. It is now read-only.

тесты сервиса с картинками #60

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

nopilei
Copy link

@nopilei nopilei commented Sep 16, 2023

Успел сделать немного, для меня кучу всего было незнакомо, но было очень полезно. Надеюсь к следующей сдаче буду готов лучше)

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

httpretty и mimesis по логике лучше вынести в dev зависимости, ибо в текущем виде toml файла они будут устанавливаться и на проде

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Немного духоты) Небольшая опечатка в названии функции test_limit_query_params_limits_response_size - либо если param в единственном числе, то он без s, либо если он во множественном, тогда глагол limits после него без s

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants