Test/add tests for groups notes services

[CaseAsLimbo]

Изменения

• Добавлены тесты для group.py, service.py и notes.py
• Доработаны тесты для note_type.py
• Доработана аннотация типов в ручке note.py::get_notes, чтобы ограничить возможность передачи отрицательных limit и offset и точно валидировать строку status.

Вопросики

• Просто хотел заметить, что в ТЗ сказано, что каждый из типов модалок должен создаваться одной ручкой. Предполагаю, что оно ещё будет меняться.

Check-List

• Вы проверили свой код перед отправкой запроса?
• Вы написали тесты к реализованным функциям?
• Вы не забыли применить форматирование black и isort для Back-End или Prettier для Front-End?

[github-actions]

💩 Code linting failed, use black and isort to fix it.

[github-actions]

Coverage Report

File	Stmts	Miss	Cover	Missing
modal_backend
__main__.py	4	4	0%	1–6
exceptions.py	20	1	95%	37
modal_backend/models
base.py	62	7	89%	22, 25–28, 57, 87
modal_backend/routes
exc_handlers.py	17	1	94%	38
modal_backend/schemas
base.py	12	4	67%	6–9
TOTAL	536	17	97%

Summary

Tests	Skipped	Failures	Errors	Time
77	0 💤	0 ❌	0 🔥	7.610s ⏱️

[github-actions]

💩 Code linting failed, use black and isort to fix it.

[github-actions]

💩 Code linting failed, use black and isort to fix it.

[petrCher]

пока ревью не закончен, посмотрел только базовый функционал, когда полностью закончу ревью напишу в тг
пока что буду периодически кидать новые комменты

[petrCher]

ответ на твой вопрос из коммента к пр: изначально да, планировалось, что одной ручкой будем создавать типы модалок, но сейчас вероятно по причине ненадобности надо будет это убирать
а создание самих модалок происходит же просто отдельными ручками - для каждого типа свое

единственное, не совсем понял зачем этот вопрос, просто из интереса или ты конкретно спрашивал для реализации чего-то?

[CaseAsLimbo]

ответ на твой вопрос из коммента к пр: изначально да, планировалось, что одной ручкой будем создавать типы модалок, но сейчас вероятно по причине ненадобности надо будет это убирать а создание самих модалок происходит же просто отдельными ручками - для каждого типа свое

единственное, не совсем понял зачем этот вопрос, просто из интереса или ты конкретно спрашивал для реализации чего-то?

Просто я когда писал тесты думал, делать один тест на все 5 ручек, потому что они однотипные или 5 тестов на каждую отдельно. Заглянул в ТЗ, чтобы принять решение и соответственно сделал один тест. Но на всякий случай это подсветил, потому что получается несовпадение.

[petrCher]

постарайся распространить мои комменты по кастомному методу и тд на все тесты, не стал прям везде писать одно и тоже
посмотрел пока поверхностно test_notes и conftest, позже подробнее изучу

и еще коммент для себя, чтобы не забыть: не нравится удаление объекта в самом тесте (подумать как упаковать в фикстуру)

[petrCher]

так, по сути все отревьюил, но решил еще клоду дать поревьюить (файл с ревью кину в тг), можешь посмотреть что он думает по этому поводу, всему верить у него точно не надо, надо перепроверять, но мало ли, он иногда действительно хорошие решения предлагает

[petrCher]

и еще на будущее рекомендация, когда делаешь очень много коммитов и пушишь разом больше 5, лучше из них создавать один коммит для пуша, а то много коммитов неудобно, так как ветка засоряется

[petrCher]

еще было бы неплохо структурировать тесты по модели crud
то есть сперва создание потом чтение и тд, чтобы при открытии тестов все было по порядку
это скорее стилистический вопрос, но он повышает читаемость