Skip to content

Commit 3cdfb0d

Browse files
pranishnepalclaude
andcommitted
test: add integration test coverage for SDK bump regressions (WAL-1484)
Five gaps identified during manual SDK bump regression testing that would go undetected by CI without these changes: 1. eddsa.test.ts — fix wrong coin in test fixture (tsol → tnear) so the test actually exercises the EdDSA key finalization path it claims to cover 2. recoveryWallet.test.ts — add ETH happy path test; ETH recovery calls recover() on the coin object directly rather than building a PSBT and hits a different AWM endpoint shape, so a regression there was invisible 3. mpcFinalize.test.ts — add backup source finalization test; the handler dispatches on source='user' vs source='backup' to set party indexes and signing material field names — only the user path was tested before 4. recoveryConsolidationsWallet.test.ts — replace Prototype-level sinon stubs with nock HTTP interceptors so the real AdvancedWalletManagerClient code path (URL routing, body serialization, codec decoding) is exercised; adds body capture assertions so URL/shape regressions are caught 5. consolidate.test.ts — add walletPubs argument verification on the signing function call; a regression that dropped walletPubs would silently pass because the stub accepts any args Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 9e542e0 commit 3cdfb0d

5 files changed

Lines changed: 288 additions & 51 deletions

File tree

0 commit comments

Comments
 (0)