test: add ripemd160 to test framework modules list #27542

pull theStack wants to merge 1 commits into bitcoin:master from theStack:test-test_runner_add_ripemd160_module changing 1 files +3 −0
  1. theStack commented at 9:43 AM on April 29, 2023: contributor

    Currently test runner doesn't execute the unit tests of the ripemd160 module, so add it to the list. All other framework modules that contain unit tests are included, as can be easily checked via $ git grep unittest.TestCase ./test/functional/test_framework/

    This is a late follow-up to PR #23716 (commit ad3e9e1f214d739e098c6ebbd300da5df1026a44).

  2. DrahtBot commented at 9:43 AM on April 29, 2023: contributor

    <!--e57a25ab6845829454e8d69fc972939a-->

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    <!--021abf342d371248e50ceaed478a90ca-->

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK MarcoFalke

    If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

  3. DrahtBot added the label Tests on Apr 29, 2023
  4. fanquake requested review from maflcko on May 1, 2023
  5. in test/functional/test_runner.py:1 in 768ae178af


    maflcko commented at 10:41 AM on May 2, 2023:

    Maybe add a comment that this should be kept in sync with the output from grep:

    git grep unittest.TestCase ./test/functional/test_frame
    

    theStack commented at 12:13 PM on May 2, 2023:

    Good idea, done.

  6. maflcko commented at 10:41 AM on May 2, 2023: member

    lgtm ACK 768ae178affa5be5960dfff38f7167a13f99e774

  7. DrahtBot removed review request from maflcko on May 2, 2023
  8. test: add ripemd160 to test framework modules list
    Currently test runner doesn't execute the unit tests of the ripemd160
    module, so add it to the list. All other framework modules that contain
    unit tests are included, as can be easily checked via
    `git grep unittest.TestCase ./test/functional/test_framework/`
    
    This is a late follow-up to PR #23716 (commit
    ad3e9e1f214d739e098c6ebbd300da5df1026a44).
    82e6e3cae5
  9. theStack force-pushed on May 2, 2023
  10. maflcko commented at 12:14 PM on May 2, 2023: member

    lgtm ACK 82e6e3cae50d30b28b98f0cb7789c28e68170d5e

  11. fanquake merged this on May 2, 2023
  12. fanquake closed this on May 2, 2023

  13. theStack deleted the branch on May 2, 2023
  14. sidhujag referenced this in commit 8d1c5b5275 on May 4, 2023
  15. luke-jr referenced this in commit 2758bf7335 on Aug 16, 2023
  16. kwvg referenced this in commit 23f74b3c2b on Feb 14, 2024
  17. kwvg referenced this in commit 1af2ca875c on Feb 14, 2024
  18. kwvg referenced this in commit 6437bf0dc0 on Feb 14, 2024
  19. kwvg referenced this in commit e52d6f4174 on Feb 15, 2024
  20. PastaPastaPasta referenced this in commit 74925f94c2 on Feb 19, 2024
  21. PastaPastaPasta referenced this in commit 0f0c53aed3 on Feb 19, 2024
  22. bitcoin locked this on May 1, 2024

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: 2026-04-14 21:13 UTC

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