test: add two edge case tests for CSubNet #19351

pull vasild wants to merge 1 commits into bitcoin:master from vasild:add_csubnet_test changing 1 files +3 −0
  1. vasild commented at 1:40 pm on June 22, 2020: member
    This is chopped off from #19031. It is needed because later 19031 modifies the related code and the tests ensure that no surprising changes in behavior sneak in.
  2. test: add two edge case tests for CSubNet ccef5d7bf0
  3. fanquake added the label Tests on Jun 22, 2020
  4. practicalswift commented at 2:26 pm on June 22, 2020: contributor
    ACK ccef5d7bf0af8377c6c779295f7b41d5af435c47 – more test coverage is better than less test coverage :)
  5. laanwj commented at 3:53 pm on June 22, 2020: member
    ACK ccef5d7bf0af8377c6c779295f7b41d5af435c47
  6. hebasto approved
  7. hebasto commented at 3:57 pm on June 22, 2020: member
    ACK ccef5d7bf0af8377c6c779295f7b41d5af435c47, I have reviewed the code and it looks OK, I agree it can be merged.
  8. laanwj merged this on Jun 22, 2020
  9. laanwj closed this on Jun 22, 2020

  10. jonatack commented at 4:55 pm on June 22, 2020: member
    ACK ccef5d7bf0af8377c6c779295f7b41d5af435c47
  11. vasild deleted the branch on Jun 22, 2020
  12. PastaPastaPasta referenced this in commit 1b237a541f on Jan 16, 2021
  13. UdjinM6 referenced this in commit f90a52ae45 on Jan 18, 2021
  14. PastaPastaPasta referenced this in commit 3af7c994db on Jan 18, 2021
  15. Fabcien referenced this in commit 88a66b297a on Feb 5, 2021
  16. random-zebra referenced this in commit b4751e10ce on Aug 11, 2021
  17. DrahtBot locked this on Feb 15, 2022

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-07-03 10:13 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me