Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tests for KeysetAwarePage #214

Merged
merged 1 commit into from
Aug 22, 2023

Conversation

njr-11
Copy link
Contributor

@njr-11 njr-11 commented Aug 21, 2023

Write TCK tests for the following scenarios from #133

  • Request the first KeysetAwarePage of 8 results, expecting to find all 8, then request the next KeysetAwarePage after the keyset of the last entry of the page, expecting to find the next 8. Or validate that UnsupportedOperationException is raised to indicate that the underlying database is incapable of keyset pagination.

  • Request a KeysetAwarePage of 7 results after the keyset of the 20th result, expecting to find the next 7 results. Then request the KeysetAwarePage before the keyset of the first entry of the page, expecting to find the previous 7 results. Then request the KeysetAwarePage after the last entry of the page, expecting to find the next 7. Or validate that UnsupportedOperationException is raised to indicate that the underlying database is incapable of keyset pagination.

  • Request a KeysetAwarePage of results where none match the query, expecting an empty KeysetAwarePage with 0 results. Or validate that UnsupportedOperationException is raised to indicate that the underlying database is incapable of keyset pagination.

Signed-off-by: Nathan Rauh <[email protected]>
@njr-11 njr-11 added the test Something test-related label Aug 21, 2023
@njr-11 njr-11 added this to the Jakarta Data 1.0 milestone Aug 21, 2023
@otaviojava otaviojava merged commit 30307f8 into jakartaee:main Aug 22, 2023
2 checks passed
@njr-11 njr-11 mentioned this pull request Aug 19, 2023
79 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test Something test-related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants