test: implement SDK feature tests for issues #471, #470, #469, #468 - #564
Open
janipauwels-sys wants to merge 4 commits into
Open
test: implement SDK feature tests for issues #471, #470, #469, #468#564janipauwels-sys wants to merge 4 commits into
janipauwels-sys wants to merge 4 commits into
Conversation
- Add tests for preemptive token expiry detection - Add tests for concurrent request queueing during refresh - Add tests for SEP-10 challenge transaction signing - Verify token refresh hooks fire correctly - Test failed refresh scenarios with proper error handling Closes Stellar-split#471
- Add tests for gap detection logic (onChainSeq - lastSubmittedSeq) - Add tests for BumpSequenceOperation submission - Add tests for automatic retry after gap is filled - Add tests for zero-gap edge cases - Verify sequence:gapFilled event emission with correct payloads Closes Stellar-split#470
- Add tests for 2-of-3 signing sessions with threshold weights - Add tests for partial signature collection and weight tracking - Add tests for session token serialization stability - Add tests for duplicate signature rejection - Add tests for automatic submission when ready - Verify multi-sig with varying weight distributions Closes Stellar-split#469
- Add tests for createMockSplitClient() factory function - Verify all public methods are vi.fn() stubs with correct signatures - Add tests for default return values are type-correct - Add tests for selective method override capability - Test mock.calls tracking and mockImplementation support - Verify independent mock instances with separate state - Test @stellar-split/sdk/testing subpath resolution Closes Stellar-split#468
|
@janipauwels-sys 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! 🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Implements comprehensive test suites for four critical SDK features:
Build SDK auth token refresh interceptor for SEP-10 sessions #471: Token Refresh Interceptor for SEP-10 auth sessions
Implement ledger sequence gap detector and automatic gap-filler #470: Sequence Gap Detector with automatic gap-filling
Add invoice batch signing coordinator with multi-sig key aggregation #469: Batch Signing Coordinator for multi-signature transactions
Build SDK mock factory exporting typed test doubles for all public methods #468: SDK Mock Factory with typed test doubles
createMockSplitClient()factory for testingTest Coverage
Each feature includes unit tests covering:
All tests use Vitest with proper mocking and spying patterns.
Closes #471
Closes #470
Closes #469
Closes #468