554 pagination test coverage#663
Conversation
utksh1
left a comment
There was a problem hiding this comment.
The Pagination coverage is relevant, but this PR also carries the preferred-export-format hook test that has already been merged via #634. Please rebase on latest main and remove frontend/testing/unit/hooks/usePreferredExportFormat.test.ts from this PR so it stays focused on Pagination coverage only.
|
Hi @utksh1 , thank you for the review! I'm working on this right now. |
142019e to
8ff7ebc
Compare
utksh1
left a comment
There was a problem hiding this comment.
The latest net diff is now focused on Pagination component coverage only, resolving the earlier unrelated hook-test concern. Approved pending a fresh branch update.
Description
Adds direct unit test coverage for the Pagination component. Tests cover record range calculations and disabled states for prev/next buttons across empty, first-page, and last-page scenarios.
Related Issues
Fixes #554
Type of Change
How Has This Been Tested?
Ran tests locally using Vitest:
npm test -- Pagination
All 9 tests passed:
Checklist