Skip to content

test(#502): add integration test for activity feed cursor pagination#522

Merged
Chucks1093 merged 4 commits into
accesslayerorg:mainfrom
Sam-Rytech:fix/502-activity-feed-pagination
Jul 2, 2026
Merged

test(#502): add integration test for activity feed cursor pagination#522
Chucks1093 merged 4 commits into
accesslayerorg:mainfrom
Sam-Rytech:fix/502-activity-feed-pagination

Conversation

@Sam-Rytech

Copy link
Copy Markdown
Contributor

Description

This PR resolves the missing integration test and cursor pagination functionality for the wallet activity feed.

Changes included:

  • Updated WalletActivityQuerySchema to accept an optional cursor string.
  • Modified fetchWalletActivity to decode the cursor, adjust the Prisma pagination parameters to use cursor with skip: 1 when provided, and encode a nextCursor for the last item.
  • Updated httpGetWalletActivity to expose nextCursor in the response envelope's meta block.
  • Added a full integration test (wallet-activity-cursor-pagination.integration.test.ts) that asserts:
    • First page returns a valid next cursor.
    • Second page fetches correctly using the cursor, with no duplicates or skipped events across the two pages.

Closes #502

@drips-wave

drips-wave Bot commented Jun 26, 2026

Copy link
Copy Markdown

@Sam-Rytech Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Chucks1093 Chucks1093 merged commit ced015e into accesslayerorg:main Jul 2, 2026
1 check passed
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.

Add integration test for activity feed pagination returning correct next cursor on first page

2 participants