test: Add Codec trait unit tests

[ldiego08]

Summary of Changes

• Add unit tests to codec.rs to assert Codec implementations for the following works as expected:

  • bdk::bitcoin::Amount
  • bdk::bitcoin::Script
  • bdk::bitcoin::secp256k1::ecdsa::RecoverableSignature
  • u64

• Add anyhow crate to stacks-core for easy error propagation in unit tests.

NOTE: There are some assertions against hex representations of the expected value to keep the tests as readable as possible and to avoid replicating the serialization or deserialization code. Formal snapshot testing using a crate like insta can prove a more scalable alternative asserting between complex values.

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:

• My code follows the style guidelines of this project
• I have performed a self-review of my code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• Any dependent changes have been merged and published in downstream modules

[CLAassistant]

All committers have signed the CLA.

[ldiego08]

The recovery ID can only be 0 - 3 i32. A greater number is out of bounds and should fail to deserialize.

[ldiego08]

The signature data is ECDSA, hence if the R or S components are out of bounds, deserialization should fail.

[ldiego08]

@CAGS295 seems like merging is blocked due to one workflow needing approval. Is there something else I need to do to get this merged?

[CAGS295]

Hi @ldiego08, thank you for your contribution. You need to wait for a core maintainer to approve the workflow. @AshtonStephens