write unit tests
@nochiel is already working on this.
Since Jun 25, 2021.
Assessment
This issue has not been assessed yet.
Description
Abstract
Write more unit tests for UR serialization and deserialization.
First, there are small changes that need to be done to the UR test vectors (one has an incorrect depth and another is missing a parent fingerprint), see https://github.com/BlockchainCommons/Research/issues/83. To correct these changes open a PR in Research repo. After approved, write unit tests for those test vectors in this repo.
Tag @wolfmcnally in https://github.com/BlockchainCommons/Research/issues/83 to let him know you'll be working on the issue
You'll need the following tool:
And some general knowledge about:
- Dominant language
- Rust
- Stars
- 3
- Forks
- 1
- PR merge metrics
- No merged PRs in 30d
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from BlockchainCommons/sweeptool-cli
-
question
Difficulty 5/5 Over a week Newbie friendliness 25/100
-
internship
Difficulty 3/5 1-2 days Newbie friendliness 25/100
-
internship
Difficulty 3/5 1-2 days Newbie friendliness 45/100
-
internship
BlockchainCommons/sweeptool-cli#2 · 1 comment · 1 assignee ·
All issues in BlockchainCommons/sweeptool-cli
Similar issues
-
bug
Difficulty 1/5 Under an hour Newbie friendliness 85/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
yantrikos/yantrik-os#255 ·
-
bug CLI custom-model
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
raphamorim/rio#1956 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
rust-bitcoin/rust-bitcoin#6930 · 1 comment ·