Skip to content

test: add route-level tests for annotation endpoints#347

Merged
leotrs merged 1 commit intomainfrom
std-kztr
Mar 27, 2026
Merged

test: add route-level tests for annotation endpoints#347
leotrs merged 1 commit intomainfrom
std-kztr

Conversation

@leotrs
Copy link
Copy Markdown
Collaborator

@leotrs leotrs commented Mar 27, 2026

Summary

  • Add 36 route-level tests for all 10 annotation API endpoints (file_annotations.py)
  • Cover annotation CRUD (create, list, get, update, delete) and message/note CRUD
  • Test privacy filters (own private + all shared), ownership rules, shared annotation constraints, soft-delete behavior, and auth guards

Test plan

  • All 36 new tests pass locally
  • Full backend test suite passes (650 passed)
  • Ruff lint passes

🤖 Generated with Claude Code

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 27, 2026

Deploy Preview for rsm-studio-site canceled.

Name Link
🔨 Latest commit fa837b8
🔍 Latest deploy log https://app.netlify.com/projects/rsm-studio-site/deploys/69c654fd5546010008825348

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 27, 2026

Deploy Preview for rsm-studio-frontend canceled.

Name Link
🔨 Latest commit fa837b8
🔍 Latest deploy log https://app.netlify.com/projects/rsm-studio-frontend/deploys/69c654fd8642630008025d2d

@leotrs
Copy link
Copy Markdown
Collaborator Author

leotrs commented Mar 27, 2026

CI is failing after 5 work cycles. Needs human intervention to unblock.

Cover all 10 annotation API endpoints (5 annotation CRUD + 5 message CRUD)
with 36 tests covering auth, privacy filters, ownership rules, shared
annotation constraints, and soft-delete behavior.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@leotrs leotrs merged commit 6c23059 into main Mar 27, 2026
18 checks passed
@leotrs leotrs deleted the std-kztr branch March 27, 2026 10:46
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