Skip to content

feat(abstract-utxo): refactor backup key recovery tests#8087

Merged
OttoAllmendinger merged 1 commit intomasterfrom
BTC-2866.wasm-utxo-for-backup-tests
Feb 10, 2026
Merged

feat(abstract-utxo): refactor backup key recovery tests#8087
OttoAllmendinger merged 1 commit intomasterfrom
BTC-2866.wasm-utxo-for-backup-tests

Conversation

@OttoAllmendinger
Copy link
Contributor

@OttoAllmendinger OttoAllmendinger commented Feb 9, 2026

Refactor backupKeyRecovery to use wasm-utxo primitives for transaction
building and signing tests. Split the implementation into smaller functions
to improve testability and maintainability:

  1. Extract core PSBT creation logic to backupKeyRecoveryWithWalletUnspents
  2. Create formatBackupKeyRecoveryResult for response formatting
  3. Add type-specific interfaces for improved type safety
  4. Update tests to use wasm-utxo without utxolib dependencies

Issue: BTC-2866

Refactor backupKeyRecovery to use wasm-utxo primitives for transaction
building and signing. Split the implementation into smaller functions
to improve testability and maintainability:

1. Extract core PSBT creation logic to backupKeyRecoveryWithWalletUnspents
2. Create formatBackupKeyRecoveryResult for response formatting
3. Add type-specific interfaces for improved type safety
4. Update tests to use wasm-utxo without utxolib dependencies

Issue: BTC-2866

Co-authored-by: llm-git <llm-git@ttll.de>
@OttoAllmendinger OttoAllmendinger force-pushed the BTC-2866.wasm-utxo-for-backup-tests branch from 773be2f to bba10c3 Compare February 9, 2026 14:23
@OttoAllmendinger OttoAllmendinger marked this pull request as ready for review February 9, 2026 14:42
@OttoAllmendinger OttoAllmendinger requested a review from a team as a code owner February 9, 2026 14:42
@OttoAllmendinger OttoAllmendinger changed the title feat(abstract-utxo): refactor backup key recovery feat(abstract-utxo): refactor backup key recovery tests Feb 9, 2026
@OttoAllmendinger OttoAllmendinger merged commit 147ee61 into master Feb 10, 2026
20 checks 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.

2 participants