Skip to content

Add search and pagination behavior tests#102

Merged
SeanTAllen merged 1 commit intomainfrom
add-search-pagination-tests
Mar 3, 2026
Merged

Add search and pagination behavior tests#102
SeanTAllen merged 1 commit intomainfrom
add-search-pagination-tests

Conversation

@SeanTAllen
Copy link
Copy Markdown
Member

Tests the last untested area identified in #98: pagination navigation (next_page/prev_page making follow-up HTTP requests) for both SearchResults and PaginatedList.

Four unit tests verify the converter layer correctly parses Link headers and exposes pagination state (present vs absent). Four mock HTTP server tests verify the full chain: fetch page, extract link, follow link, fetch next page, verify results.

Tests the last untested area identified in Discussion #98:
pagination navigation (next_page/prev_page making follow-up HTTP
requests) for both SearchResults and PaginatedList.

Four unit tests verify the converter layer correctly parses Link
headers and exposes pagination state (present vs absent). Four mock
HTTP server tests verify the full chain: fetch page, extract link,
follow link, fetch next page, verify results.

Design: #98
@ponylang-main ponylang-main added the discuss during sync Should be discussed during an upcoming sync label Mar 3, 2026
@SeanTAllen SeanTAllen merged commit cc0fc8c into main Mar 3, 2026
10 checks passed
@SeanTAllen SeanTAllen deleted the add-search-pagination-tests branch March 3, 2026 22:54
@ponylang-main ponylang-main removed the discuss during sync Should be discussed during an upcoming sync label Mar 3, 2026
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.

2 participants