Skip to content

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

Open
Sam-Rytech wants to merge 3 commits into
accesslayerorg:mainfrom
Sam-Rytech:fix/502-activity-feed-pagination
Open

test(#502): add integration test for activity feed cursor pagination#522
Sam-Rytech wants to merge 3 commits into
accesslayerorg:mainfrom
Sam-Rytech:fix/502-activity-feed-pagination

Conversation

@Sam-Rytech

Copy link
Copy Markdown

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

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

1 participant