Skip to content

Conversation

theStack
Copy link
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 ad3e9e1).

@DrahtBot
Copy link
Contributor

DrahtBot commented Apr 29, 2023

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

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.

@DrahtBot DrahtBot added the Tests label Apr 29, 2023
@fanquake fanquake requested a review from maflcko May 1, 2023 13:43
@maflcko
Copy link
Member

maflcko commented May 2, 2023

lgtm ACK 768ae17

@DrahtBot DrahtBot removed the request for review from maflcko May 2, 2023 10:41
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 bitcoin#23716 (commit
ad3e9e1).
@theStack theStack force-pushed the test-test_runner_add_ripemd160_module branch from 768ae17 to 82e6e3c Compare May 2, 2023 12:13
@maflcko
Copy link
Member

maflcko commented May 2, 2023

lgtm ACK 82e6e3c

@fanquake fanquake merged commit cfe5da4 into bitcoin:master May 2, 2023
@theStack theStack deleted the test-test_runner_add_ripemd160_module branch May 2, 2023 12:31
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request May 4, 2023
82e6e3c test: add ripemd160 to test framework modules list (Sebastian Falbesoner)

Pull request description:

  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 bitcoin#23716 (commit ad3e9e1).

ACKs for top commit:
  MarcoFalke:
    lgtm ACK 82e6e3c

Tree-SHA512: 10940e215f728291c7149931a356bfc42795c098bda76d760dfa68f86443a3755e1cd35cb9a8a7b2f48880beb53f3bee3842de2d74bcadd45c7b05c13ff04203
luke-jr pushed a commit to luke-jr/bitcoin that referenced this pull request Aug 16, 2023
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 bitcoin#23716 (commit
ad3e9e1).

Github-Pull: bitcoin#27542
Rebased-From: 82e6e3c
kwvg added a commit to kwvg/dash that referenced this pull request Feb 14, 2024
kwvg added a commit to kwvg/dash that referenced this pull request Feb 14, 2024
kwvg added a commit to kwvg/dash that referenced this pull request Feb 14, 2024
kwvg added a commit to kwvg/dash that referenced this pull request Feb 15, 2024
@bitcoin bitcoin locked and limited conversation to collaborators May 1, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants