Skip to content

Comments

test(abstract-utxo): add integration tests for custom change wallet key#8184

Merged
OttoAllmendinger merged 3 commits intomasterfrom
BTC-2650.update-custom-change-wallet-tests
Feb 20, 2026
Merged

test(abstract-utxo): add integration tests for custom change wallet key#8184
OttoAllmendinger merged 3 commits intomasterfrom
BTC-2650.update-custom-change-wallet-tests

Conversation

@OttoAllmendinger
Copy link
Contributor

Add integration tests for parseTransaction to verify custom change
wallet key fetching and signature verification:

  • Custom change wallet keys are fetched and signatures verified
  • Transactions without customChangeWalletId skip custom change
  • Invalid signatures are rejected

Replace SDK-dependent tests with pure utxo-lib implementation using
PSBT construction and explanation. Use synchronous wasm-utxo message
signing to improve test performance and simplify test setup.

Issue: BTC-2650

@OttoAllmendinger OttoAllmendinger force-pushed the BTC-2650.update-custom-change-wallet-tests branch from f372087 to 79399e4 Compare February 20, 2026 09:45
OttoAllmendinger and others added 3 commits February 20, 2026 13:38
Replace async key signing with synchronous wasm-utxo message
signing to improve test performance and simplify test setup.

BTC-2650

Co-authored-by: llm-git <llm-git@ttll.de>
Replace SDK-dependent test with pure utxo-lib implementation using PSBT
construction and explanation. Add signature verification tests using
wasm-utxo message signing. Support both utxolib and wasm-utxo backends.

BTC-2650

Co-authored-by: llm-git <llm-git@ttll.de>
Add parseTransaction tests to verify custom change wallet key fetching,
signature verification, and proper handling of transactions without
customChangeWalletId. Tests cover both valid and invalid signatures.

BTC-2650

Co-authored-by: llm-git <llm-git@ttll.de>
@OttoAllmendinger OttoAllmendinger force-pushed the BTC-2650.update-custom-change-wallet-tests branch from 79399e4 to afce442 Compare February 20, 2026 12:38
@OttoAllmendinger OttoAllmendinger marked this pull request as ready for review February 20, 2026 12:59
@OttoAllmendinger OttoAllmendinger requested a review from a team as a code owner February 20, 2026 12:59
@OttoAllmendinger OttoAllmendinger merged commit aa02f2f into master Feb 20, 2026
21 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