rust-bitcoin / rust-bitcoin / 16077780292

83%

Build Type

push

github

Committed by apoelstra

Commit Message

Merge rust-bitcoin/rust-bitcoin#4666: Kill new mutants

<a class=hub.com/rust-bitcoin/rust-bitcoin/commit/f0562504b7f758a439de60fe5fa76ad9ebfaba00">f0562504b<a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/ca5d1366671c5bb974083079ac35b5f25073f53a"> Exclude deprecated fn from mutation testing (Jamil Lambert, PhD)
<a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/4d31b141a857e4c90a417cc5805105f160846842">4d31b141a</a><a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/ca5d1366671c5bb974083079ac35b5f25073f53a"> Improve is_too_precise test (Jamil Lambert, PhD)
</a><a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/a2bae3bb0b1598fdc7c5a47965c5b0481bd10fc8">a2bae3bb0</a><a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/ca5d1366671c5bb974083079ac35b5f25073f53a"> Add test for impl Display for Script (Jamil Lambert, PhD)

Pull request description:

  Weekly mutation testing found new mutants. There are also some untested mutants in match statements that will be included in future mutation testing once #4654 goes in.
  
  - Add a regression test for the mutants in the `Display` impl for `Script`.
  - Improve the existing test for `is_too_precise` to check all four cases.
  - Exclude the two deprecated functions that have untested mutants.
  
  Closes #4646


ACKs for top commit:
  benalleng:
    **tACK** no missed mutants on f056250 I ran with https://github.com/rust-bitcoin/rust-bitcoin/pull/4654/commits/63b61e949 rebased in to make sure the match arm and match guard mutations were included
  tcharding:
    ACK f0562504b7f758a439de60fe5fa76ad9ebfaba00


Tree-SHA512: d109e30be2ac418fa9e287c962028457dd3

Coverage Stats

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

1 existing line in 1 file now uncovered.

21292 of 25486 relevant lines covered (83.54%)

15262.04 hits per line