Add "make check" target to build and run tests #37

pull ryanofsky wants to merge 1 commits into bitcoin-core:master from ryanofsky:pr/check changing 7 files +69 −48

ryanofsky commented at 11:32 AM on May 13, 2020: collaborator
fanquake reported https://github.com/chaincodelabs/libmultiprocess/issues/36 that instructions for running tests in the README are wrong, and there isn't a convenient way to build and run tests, so this PR adds a "make check" custom target to make it more straightforward.

The new "make check" target builds and runs all available tests, while the previous "make test" target (provided by CTest) only runs existing test binaries without building anything.

This PR also:
- Moves all test code and build rules to a new test/ subdirectory to separate test code from non-test code
- Adds a new "make tests" target used internally by "make check" to build tests without running them

Add "make check" target to build and run tests

CTest already provides a "make test" target, but this only runs previously
built tests.

Also in this commit:

- Move all test code and build rules to a new test/ subdirectory to separate
  test code from non-test code
- Add a new "make tests" target used internally by "make check" to build tests
  without running them

037835320b

ryanofsky force-pushed on May 13, 2020
fanquake commented at 12:13 PM on May 13, 2020: member

Thanks. Tested 037835320bd28d0416913d597f1f6298e8eaea00 and that running make check works for me.
ryanofsky merged this on May 13, 2020
ryanofsky closed this on May 13, 2020
ryanofsky referenced this in commit 4c3cac5440 on Dec 8, 2020
ryanofsky referenced this in commit af5bc75feb on Dec 11, 2020
ryanofsky referenced this in commit b15c4f8bdc on Dec 11, 2020
ryanofsky referenced this in commit dbe1df98fc on Dec 18, 2020
ryanofsky referenced this in commit 8877892266 on Dec 18, 2020
ryanofsky referenced this in commit 29e574aed8 on Jan 28, 2021
ryanofsky referenced this in commit c4ee00e947 on Jan 28, 2021
ryanofsky referenced this in commit 97232e7c62 on Mar 4, 2021
ryanofsky referenced this in commit b1cacee1b4 on Mar 4, 2021
ryanofsky referenced this in commit 1ecf3dcc52 on Mar 4, 2021
ryanofsky referenced this in commit 35cfc59500 on Mar 17, 2021
ryanofsky referenced this in commit 7e43e7a556 on Mar 17, 2021
ryanofsky referenced this in commit 1930323ae9 on Mar 17, 2021
ryanofsky referenced this in commit 4cb21574b8 on Mar 30, 2021
ryanofsky referenced this in commit 85f47de8f0 on Mar 30, 2021
ariard referenced this in commit 14ec1dec98 on Apr 6, 2021
ryanofsky referenced this in commit f1f62325ec on Apr 23, 2021
ryanofsky referenced this in commit 5d62d7f6cd on Apr 23, 2021
bitcoin-core locked this on Jun 25, 2025

Contributors

ryanofsky

fanquake

Linked (view graph)

#115 MSAN Error during connection shutdown #219 SIGABRT in ~ProxyClientBase with #29409 and rust client