rust-bitcoin / rust-bitcoin / 13738866432

83%

push

github

Merge rust-bitcoin/rust-bitcoin#4161: Improve ops for amount types

<a class=hub.com/rust-bitcoin/rust-bitcoin/commit/0a9f14f7b036c5232449d058fb6d425c8376d87a">0a9f14f7b<a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/86266d2dad7bd1f7574505c0034b0558a05a3d60"> Implement Div by amount for amount types (Tobin C. Harding)
<a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/b57bfb9bc53107d7a2f6f9f7cb2f4c6c5d733534">b57bfb9bc</a><a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/86266d2dad7bd1f7574505c0034b0558a05a3d60"> Add missing Mul impls for amount types (Tobin C. Harding)
</a><a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/501c9ab89e6cee63b4ba8e493393bfc1c544a225">501c9ab89</a><a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/86266d2dad7bd1f7574505c0034b0558a05a3d60"> Test amount ops that involve an integer (Tobin C. Harding)
</a><a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/851080d3b19eecea147224f8a36f9788d3e7d328">851080d3b</a><a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/86266d2dad7bd1f7574505c0034b0558a05a3d60"> Add more add/sub tests (Tobin C. Harding)
47923957b Improve add/sub tests for amount types (Tobin C. Harding)
8bb9ce3e4 Add tests for amount op int (Tobin C. Harding)

Pull request description:

  Improve the test coverage and add missing implementations of math operations for the amount types.

  Along the way close #4030.

ACKs for top commit:
  apoelstra:
    ACK 0a9f14f7b036c5232449d058fb6d425c8376d87a; successfully ran local tests; nice!

Tree-SHA512: f303b2a90dfe242d84d7c0e91fa62878cdf

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

21700 of 26118 relevant lines covered (83.08%)

16900.54 hits per line