16 commits

Author	SHA1	Message	Date
Vasil Dimov	89836a82ee	style: minor improvements as a followup to #19845 ... Address suggestions: https://github.com/bitcoin/bitcoin/pull/19845#discussion_r495486760 https://github.com/bitcoin/bitcoin/pull/19845#discussion_r495488051 https://github.com/bitcoin/bitcoin/pull/19845#discussion_r495730125	2020-10-31 17:02:52 +01:00
Vasil Dimov	fe42411b4b	test: move HasReason so it can be reused ... Move the class `HasReason` from `miner_tests.cpp` to `setup_common.h` so that it can be reused by other tests.	2020-09-17 14:45:17 +02:00
MarcoFalke	fa96574b0d	test: Move doxygen comment to header ... Also, unrelated formatting fixups. Can be reviewed with --word-diff-regex=.	2020-08-21 18:44:27 +02:00
MarcoFalke	5ec19df687	Merge #19277: util: Add Assert identity function ... fab80fef61 refactor: Remove unused EnsureChainman (MarcoFalke) fa34587f1c scripted-diff: Replace EnsureChainman with Assert in unit tests (MarcoFalke) fa6ef701ad util: Add Assert identity function (MarcoFalke) fa457fbd33 move-only: Move NDEBUG compile time check to util/check (MarcoFalke) Pull request description: The utility function is primarily useful to dereference pointer types, which are known to be not null at that time. For example, the ArgsManager is known to exist when the wallets are started: https://github.com/bitcoin/bitcoin/pull/18923/files#diff-fdb2a1a1d8bc790fcddeb6cf5a42ac55R503 . Instead of silently relying on that assumption, `Assert` can be used to abort the program and avoid UB should the assumption ever be violated. ACKs for top commit: promag: Tested ACK fab80fef61. ryanofsky: Code review ACK fab80fef61 Tree-SHA512: 830fba10152ba17d47c4dd42809c7e26f9fe6d38e17a2d5b3f054fd644a5c4c9841286ac421ec9bb28cea9f5faeb659740fcf00de6cc589d423fee7694c42d16	2020-07-04 08:44:45 -04:00
MarcoFalke	3a4a3729d9	Merge #19090: refactor: Misc scheduler cleanups ... fa8337fcdb clang-format scheduler (MarcoFalke) fa3d41b5ab doc: Switch scheduler to doxygen comments (MarcoFalke) fac43f9889 scheduler: Replace stop(true) with StopWhenDrained() (MarcoFalke) fa9cca0550 doc: Remove unused documentation about unimplemented features (MarcoFalke) fab2950d70 doc: Switch boost::thread to std::thread in scheduler (MarcoFalke) fa9819695a test: Remove unused scheduler.h include from the common setup (MarcoFalke) fa609c4f76 scheduler: Remove unused REVERSE_LOCK (MarcoFalke) Pull request description: This accumulates a bunch of cleanup that was long overdue, but I haven't yet gotten around to address. Specifically, but not limited to: * Remove unused code, documentation and includes * Upgrade to doxygen documentation Please refer to the individual commits for more details. ACKs for top commit: jnewbery: utACK fa8337fcdb Tree-SHA512: 0c825ad9767e2697a3ef1ec1be13fdc2b18eeb7493ad0be5b65cc9f209391e78b17ee66e35e094c5e171c12b0f1624f287a110f6bddaf3024b708877afa8552e	2020-06-25 09:24:58 -04:00
MarcoFalke	fa6ef701ad	util: Add Assert identity function ... The utility is primarily useful to dereference pointer types, which are known to be not null at that time. For example, the ArgsManager is known to exist when the wallets are started. Instead of silently relying on that assumption, Assert can be used to abort the program and avoid UB should the assumption ever be violated.	2020-06-15 07:39:08 -04:00
Hennadii Stepanov	89f9fef1f7	refactor: Specify boost/thread/thread.hpp explicitly	2020-06-04 10:05:54 -04:00
MarcoFalke	fa9819695a	test: Remove unused scheduler.h include from the common setup ... The common setup is included in virtually all tests, so it should be as slim as possible.	2020-05-28 09:00:56 -04:00
MarcoFalke	fa488f131f	scripted-diff: Bump copyright headers ... -BEGIN VERIFY SCRIPT- ./contrib/devtools/copyright_header.py update ./ -END VERIFY SCRIPT-	2020-04-16 13:33:09 -04:00
MarcoFalke	fa0cbd48c4	test: Add optional extra_args to testing setup	2020-04-15 15:13:04 -04:00
MarcoFalke	fad4fa7e2f	node: Add args alias for gArgs global	2020-04-15 15:05:18 -04:00
Ben Woosley	d056df033a	Replace std::to_string with locale-independent alternative	2020-03-14 12:23:01 -07:00
Amiti Uttarwar	7c8b6e5b52	[lib] add scheduler to node context ... - also update test setup & access point in denial of service test	2020-02-17 14:49:34 -08:00
MarcoFalke	fa37e0a68b	test: Show debug log on unit test failure	2020-01-02 18:00:05 -05:00
MarcoFalke	772673dfbe	Merge #16978: test: Seed test RNG context for each test case, print seed ... fae43a97ca test: Seed test RNG context for each test case, print seed (MarcoFalke) Pull request description: Debugging failing unit tests is hard if the failure is non-deterministic and the seed is not known. Fix that by printing the seed and making it possible to set the seed from outside. ACKs for top commit: davereikher: Tested ACK fae43a97ca Tree-SHA512: 33d848dd1f4180d3664ecf60e9810c2a93590c05276b2c46b1e4fe6e376b45916a46b90c803bb602750ab666da3a05ce499e550024685a90b8cc38fab6667cb8	2019-11-07 10:18:40 -05:00
MarcoFalke	faec28252c	scripted-diff: test: Move setup_common to test library ... -BEGIN VERIFY SCRIPT- # Move files for f in $(git ls-files src/test/lib/); do git mv $f src/test/util/; done git mv src/test/setup_common.cpp src/test/util/ git mv src/test/setup_common.h src/test/util/ # Replace Windows paths sed -i -e 's|\\setup_common|\\util\\setup_common|g' $(git grep -l '\\setup_common') sed -i -e 's|src\\test\\lib\\|src\\test\\util\\|g' build_msvc/test_bitcoin/test_bitcoin.vcxproj # Everything else sed -i -e 's|/setup_common|/util/setup_common|g' $(git grep -l 'setup_common') sed -i -e 's|test/lib/|test/util/|g' $(git grep -l 'test/lib/') # Fix include guard sed -i -e 's|BITCOIN_TEST_SETUP_COMMON_H|BITCOIN_TEST_UTIL_SETUP_COMMON_H|g' ./src/test/util/setup_common.h sed -i -e 's|BITCOIN_TEST_LIB_|BITCOIN_TEST_UTIL_|g' $(git grep -l 'BITCOIN_TEST_LIB_') -END VERIFY SCRIPT-	2019-11-06 11:56:41 -05:00

Renamed from src/test/setup_common.h (Browse further)