Open ldiego08 opened 10 months ago
@CAGS295 seems like merging is blocked due to one workflow needing approval. Is there something else I need to do to get this merged?
Hi @ldiego08, thank you for your contribution. You need to wait for a core maintainer to approve the workflow. @AshtonStephens
Summary of Changes
Add unit tests to
codec.rs
to assertCodec
implementations for the following works as expected:bdk::bitcoin::Amount
bdk::bitcoin::Script
bdk::bitcoin::secp256k1::ecdsa::RecoverableSignature
u64
Add
anyhow
crate tostacks-core
for easy error propagation in unit tests.Testing
Run tests normally via
cargo make test
.Risks
None.
How were these changes tested?
Changes are tests themselves and were verified by running
cargo make test
.What future testing should occur?
None.
Checklist: