Skip to content

[Week7 - jbk] Task4: Add pagination and sorting tests#52

Open
Candyroom136 wants to merge 3 commits into
mainfrom
week7-jbk-task4
Open

[Week7 - jbk] Task4: Add pagination and sorting tests#52
Candyroom136 wants to merge 3 commits into
mainfrom
week7-jbk-task4

Conversation

@Candyroom136

Copy link
Copy Markdown

Summary

  • Added comprehensive test suite for pagination and sorting across notes and action items
  • 30 new test cases in test_pagination_sorting.py

Testing

$ make test
30+ passed

Test coverage:

  • Pagination (notes & action items): limit, skip+limit, skip near end, skip beyond total
  • Sorting (notes): title asc/desc, created_at asc/desc, invalid field fallback
  • Sorting (action items): description asc/desc, default sort
  • Validation: limit=0, limit=201, skip=-1, limit=-1 → all return 422
  • Combined: pagination + sorting together

Tradeoffs & Follow-ups

  • Tests rely on insertion order for created_at sorting (SQLite timestamps may have same second)
  • No tests for search (q) combined with pagination/sorting

🤖 Generated with Claude Code

jeongbinkim and others added 3 commits March 17, 2026 13:14
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