rust-bitcoin / rust-bitcoin / 11113679177

83%

Build Type

push

github

Committed by apoelstra

Commit Message

Merge rust-bitcoin/rust-bitcoin#3409: Improve `witness` unit tests

<a class=hub.com/rust-bitcoin/rust-bitcoin/commit/5fab6b178f594db7f609d576b7cf906757c95dda">5fab6b178<a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/7379eba587567cb42257beaf9d71338080e08180"> Rename iter len unit test (Tobin C. Harding)
<a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/f6a74ef4afa03d83c526c5ba9eed5af97bdc3dcd">f6a74ef4a</a><a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/7379eba587567cb42257beaf9d71338080e08180"> Refactor the serde Witness unit tests (Tobin C. Harding)
</a><a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/9860453b5b27164109e1ba580be61b6120ad3828">9860453b5</a><a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/7379eba587567cb42257beaf9d71338080e08180"> Improve Witness consensus encode unit test (Tobin C. Harding)
</a><a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/7e2899d310f2c75f6ddcf934331841fbf9dc61b0">7e2899d31</a><a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/7379eba587567cb42257beaf9d71338080e08180"> Improve Witness::push unit test (Tobin C. Harding)
</a><a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/fe967279e5055f0786b44f13c8efc82bd0794f32">fe967279e Improve witness unit tests for single empty element (Tobin C. Harding)

Pull request description:

  In preparation for moving the `Witness` type over to `primitives` refactor and improve all the unit tests that will be moved, do not touch the ones that will stay behind.

  The first five patches are from #3406, the last is just a re-name of the test function I tried to refactor in https://github.com/rust-bitcoin/rust-bitcoin/commit/ac6fe3a88

ACKs for top commit:
  apoelstra:
    ACK 5fab6b178f594db7f609d576b7cf906757c95dda successfully ran local tests

Tree-SHA512: bc00f81e38cc0735d716af0ec7ce80a0fbf

Coverage Stats

102 of 102 new or added lines in 1 file covered. (100.0%)

1 existing line in 1 file now uncovered.

19022 of 23034 relevant lines covered (82.58%)

18992.54 hits per line