rust-bitcoin / rust-bitcoin / 11279738624
83%

DEFAULT BRANCH: master
Ran
Jobs 1
Files 99
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 82.653% (+0.03%) from 82.623%
11279738624

push

github

apoelstra 
Merge rust-bitcoin/rust-bitcoin#3430: Add checked div by weight to amount

<a class=hub.com/rust-bitcoin/rust-bitcoin/commit/<a class="double-link" href="https://git"><a class=hub.com/rust-bitcoin/rust-bitcoin/commit/a0c58a4a8b4244d7c541906c61d1343dd6acdccd">a0c58a4a8<a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/27f6f1797421bfaef87fdde1377102a4204ab5f5"> Add checked weight division to Amount (yancy)
<a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/8def40a991c81118e9a775cd3f28813145cfec2a">8def40a99</a><a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/27f6f1797421bfaef87fdde1377102a4204ab5f5"> Add assertions to checked_weight_mul test (yancy)
</a><a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/16ce70d3a6815aef5129274adaa77141e69d4956">16ce70d3a</a> Add div_by_weight test to fee_rate (yancy)

Pull request description:

  Adds the checked variant of `amount / weight`.  I also added a test to the non-checked version for comparison so the reviewer knows they compute the same way (integer division rounded down).

  Also added assertion to `checked_weight_mul test` showing the results are rounded up.

ACKs for top commit:
  tcharding:
    ACK a0c58a4a8b4244d7c541906c61d1343dd6acdccd
  apoelstra:
    ACK a0c58a4a8b4244d7c541906c61d1343dd6acdccd successfully ran local tests

Tree-SHA512: cf14123ed25e32eaa2e08a8146a534d2663

35 of 35 new or added lines in 2 files covered. (100.0%)

19140 of 23157 relevant lines covered (82.65%)

18825.4 hits per line

Jobs
ID Job ID Ran Files Coverage
1 11279738624.1 0
82.65
 GitHub Action Run
Source Files on build 11279738624
Detailed source file information is not available for this build.