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

Move pagination tests to appropiate location #77

Open
kevin-pease opened this issue May 24, 2024 · 0 comments
Open

Move pagination tests to appropiate location #77

kevin-pease opened this issue May 24, 2024 · 0 comments
Labels
enhancement An enhancement of an existing feature

Comments

@kevin-pease
Copy link
Contributor

kevin-pease commented May 24, 2024

Currently, pagination functionality for requests are being tested in assets/all_assets_request.rs. Since all code concerning pagination in requests (i.e. set cursors, order, limit) has been moved to a macro, it would make more sense to test it there.

Identical (and therefore duplicate) tests also occur in some other endpoints (for example all_claimable_balances_request). These can be removed as well.

@kevin-pease kevin-pease added the enhancement An enhancement of an existing feature label May 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement of an existing feature
Projects
None yet
Development

No branches or pull requests

1 participant