Commit b878014
committed
fix(sdk-core): use newWalletPassphrase in processAcceptShare for bulk operations
Fixed processAcceptShare to correctly use newWalletPassphrase when provided
for both keychainOverrideRequired and userMultiKeyRotationRequired cases in
bulk wallet share operations. Updated validation to accept either
newWalletPassphrase or userLoginPassword for userMultiKeyRotationRequired
shares.
Added comprehensive tests to verify:
- Bulk accept share with userMultiKeyRotationRequired and newWalletPassphrase
- Bulk accept share with keychainOverrideRequired and newWalletPassphrase
- Error handling when neither password is provided
- Correct usage of newWalletPassphrase vs userLoginPassword
Fixed existing keychainOverrideRequired tests to use correct API endpoints
(ofc instead of tbtc) and made nock matchers more flexible to handle dynamic
signatures and payloads.
WP-6769
TICKET: WP-67691 parent f548584 commit b878014
File tree
3 files changed
+250
-43
lines changed- modules
- bitgo/test/v2/unit
- sdk-core/src/bitgo/wallet
3 files changed
+250
-43
lines changed
0 commit comments