Skip to content

test(abstract-utxo): simplify prebuildAndSign test setup#8659

Merged
OttoAllmendinger merged 3 commits into
masterfrom
BTC-0.improve-prebuildAndSign-test
Apr 30, 2026
Merged

test(abstract-utxo): simplify prebuildAndSign test setup#8659
OttoAllmendinger merged 3 commits into
masterfrom
BTC-0.improve-prebuildAndSign-test

Conversation

@OttoAllmendinger
Copy link
Copy Markdown
Contributor

Remove encrypted keychains and webauthn fixtures from prebuildAndSign
tests. Use plain private keys directly instead of testing passphrase
decryption logic.

Move webauthn passphrase fallback tests to dedicated test file where
they belong.

Remove the txFormat parameter from the run function and hardcode
nockOutputAddresses to false since all tests now use PSBT format.

Update test to include replay protection pubkey configuration when
creating transaction. Uses getReplayProtectionPubkeys helper to fetch
appropriate key for coin network.

Issue: BTC-0

OttoAllmendinger and others added 3 commits April 30, 2026 11:11
Remove encrypted keychains and webauthn fixtures from prebuildAndSign
tests. Use plain private keys directly instead of testing passphrase
decryption logic.

Move webauthn passphrase fallback tests to dedicated test file where
they belong.

Issue: BTC-0

Co-authored-by: llm-git <llm-git@ttll.de>
Remove the `txFormat` parameter from the `run` function and hardcode
`nockOutputAddresses` to `false` since all tests now use PSBT format.

Issue: BTC-0

Co-authored-by: llm-git <llm-git@ttll.de>
Update test to include replay protection pubkey configuration when
creating transaction. Uses getReplayProtectionPubkeys helper to fetch
appropriate key for coin network.

Issue: BTC-0

Co-authored-by: llm-git <llm-git@ttll.de>
@OttoAllmendinger OttoAllmendinger marked this pull request as ready for review April 30, 2026 09:34
@OttoAllmendinger OttoAllmendinger requested a review from a team as a code owner April 30, 2026 09:34
@OttoAllmendinger OttoAllmendinger merged commit a8478ae into master Apr 30, 2026
22 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