shinghim / rust-bitcoin / 14947945345

coverage: 84.481% (+0.3%) from 84.188%
14947945345

push

github

apoelstra 
Merge rust-bitcoin/rust-bitcoin#4473: Assert num op error

<a class=hub.com/shinghim/rust-bitcoin/commit/<a class="double-link" href="https://git"><a class=hub.com/shinghim/rust-bitcoin/commit/a7d059151eb47bf4202302604309c95c0d66371d">a7d059151<a href="https://github.com/shinghim/rust-bitcoin/commit/e43c5741466cce60af81978f425390f294995844"> Assert error type (yancy)
<a class="double-link" href="https://github.com/shinghim/rust-bitcoin/commit/2f7e74da45a76449ef7820972c1ae8ee65a0984d">2f7e74da4</a> Add MathOp helper methods (yancy)

Pull request description:

  Follow up from https://github.com/rust-bitcoin/rust-bitcoin/pull/4428 to assert the error type which I agree improves the test.

  Added some helper functions since it can be nice to see what type of overflow happened.

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

Tree-SHA512: e1b3eba64e1eebef290474c39f813f873f7

6 of 9 new or added lines in 2 files covered. (66.67%)

779 existing lines in 36 files now uncovered.

23157 of 27411 relevant lines covered (84.48%)

16207.4 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

85.94
/primitives/src/sequence.rs


Source Not Available