Skip to content
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

Добавить валидаторы к моделям #12

Open
b0g3r opened this issue Aug 17, 2017 · 0 comments
Open

Добавить валидаторы к моделям #12

b0g3r opened this issue Aug 17, 2017 · 0 comments

Comments

@b0g3r
Copy link
Member

b0g3r commented Aug 17, 2017

хочу метод Model.validate_json(json_string), который должен:

  • возвращать True, dict - если всё ок
  • возвращать False, "error" - если произошла ошибка (не-джсон, есть лишние поля, не хватает полей, поля неправильного формата, ...). Возможно пригодится json-schema и иже с ними
  • .validate_json должен проверять только джсон-строку на соответствие формату, а не, например, конфликты с существующими записями в базе.

Нужно также

  • написать тесты, покрывающие методы валидации
  • из тестов апи выкинуть все тесты, связанные с валидацией входных данных. Оставить по одной штуке с замоканным .validate_json() чтобы точно знать что апи-эндпоинт не потерял вызов .validate_json()

Будет также очень удобно, если ты создаешь базовый класс для моделей BaseModel, куда внесешь save и delete (сейчас они у тебя повторяются в двух моделях), а также validate_json с raise NotImplementedError - это используется как указание на "абстрактность" метода (выбросит исключение, если в наследнике попытаться использовать не переопределенный метод)

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

No branches or pull requests

1 participant