Skip to content

Test/add tests for groups notes services - #30

Open
CaseAsLimbo wants to merge 13 commits into
mainfrom
test/add-tests-for-groups-notes-services
Open

Test/add tests for groups notes services#30
CaseAsLimbo wants to merge 13 commits into
mainfrom
test/add-tests-for-groups-notes-services

Conversation

@CaseAsLimbo

@CaseAsLimbo CaseAsLimbo commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

Изменения

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

Детали реализации

Check-List

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

@github-actions

Copy link
Copy Markdown

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

@github-actions

github-actions Bot commented Jul 25, 2026

Copy link
Copy Markdown

Code Coverage

Coverage Report
FileStmtsMissCoverMissing
modal_backend
   __main__.py440%1–6
   exceptions.py20290%32, 37
modal_backend/models
   base.py62789%22, 25–28, 57, 87
modal_backend/routes
   exc_handlers.py17288%31, 38
   notes.py451664%84–93, 109–112, 128–131, 147–150, 166–169, 185–188, 210–211
modal_backend/schemas
   base.py12467%6–9
modal_backend/utils
   services.py902671%48–86
TOTAL5306188% 

Summary

Tests Skipped Failures Errors Time
50 0 💤 0 ❌ 0 🔥 5.814s ⏱️

@CaseAsLimbo
CaseAsLimbo force-pushed the test/add-tests-for-groups-notes-services branch from 452ae37 to ed4deb3 Compare July 28, 2026 14:12
@github-actions

Copy link
Copy Markdown

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

@github-actions

Copy link
Copy Markdown

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

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant