Page MenuHomePhabricator

test: move HasReason so it can be reused
ClosedPublic

Authored by Fabien on Tue, Feb 9, 12:29.

Details

Reviewers
majcosta
Group Reviewers
Restricted Project
Commits
rABC4adf771f68a1: test: move HasReason so it can be reused
Summary
Move the class `HasReason` from `miner_tests.cpp` to
`setup_common.h` so that it can be reused by other tests.

Partial backport (2/4) of core PR19485:
https://github.com/bitcoin/bitcoin/pull/19845/commits/fe42411b4b07b99c591855f5f00ad45dfeec8e30

Depends on D9186.

Test Plan
ninja check-bitcoin-miner_tests

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Fabien requested review of this revision.Tue, Feb 9, 12:29

Tail of the build log:

[384/440] Running utility command for check-bitcoin-script_commitment_tests
[385/440] bitcoin: testing sync_tests
[386/440] Running utility command for check-bitcoin-sighashtype_tests
[387/440] bitcoin: testing torcontrol_tests
[388/440] Running utility command for check-bitcoin-sync_tests
[389/440] Running utility command for check-bitcoin-torcontrol_tests
[390/440] bitcoin: testing scriptpubkeyman_tests
[391/440] bitcoin: testing settings_tests
[392/440] Running utility command for check-bitcoin-scriptpubkeyman_tests
[393/440] Running utility command for check-bitcoin-settings_tests
[394/440] bitcoin: testing txvalidationcache_tests
[395/440] Running utility command for check-bitcoin-txvalidationcache_tests
[396/440] bitcoin: testing uint256_tests
[397/440] Running utility command for check-bitcoin-uint256_tests
[398/440] bitcoin: testing validation_block_tests
[399/440] bitcoin: testing serialize_tests
[400/440] bitcoin: testing walletdb_tests
[401/440] Running utility command for check-bitcoin-validation_block_tests
[402/440] bitcoin: testing undo_tests
[403/440] Running utility command for check-bitcoin-serialize_tests
[404/440] Running utility command for check-bitcoin-walletdb_tests
[405/440] Running utility command for check-bitcoin-undo_tests
[406/440] bitcoin: testing blockcheck_tests
[407/440] bitcoin: testing radix_tests
[408/440] Running utility command for check-bitcoin-radix_tests
[409/440] Running utility command for check-bitcoin-blockcheck_tests
[410/440] bitcoin: testing validation_chainstatemanager_tests
[411/440] bitcoin: testing schnorr_tests
[412/440] bitcoin: testing ismine_tests
[413/440] Running utility command for check-bitcoin-validation_chainstatemanager_tests
[414/440] Running utility command for check-bitcoin-schnorr_tests
[415/440] Running utility command for check-bitcoin-ismine_tests
[416/440] bitcoin: testing crypto_tests
[417/440] bitcoin: testing cashaddr_tests
[418/440] Running utility command for check-bitcoin-crypto_tests
[419/440] Running utility command for check-bitcoin-cashaddr_tests
[420/440] bitcoin: testing ref_tests
[421/440] Running utility command for check-bitcoin-ref_tests
[422/440] bitcoin: testing cuckoocache_tests
[423/440] Running utility command for check-bitcoin-cuckoocache_tests
[424/440] bitcoin: testing validation_tests
[425/440] Running utility command for check-bitcoin-validation_tests
[426/440] bitcoin: testing script_tests
[427/440] bitcoin: testing util_tests
[428/440] bitcoin: testing skiplist_tests
[429/440] bitcoin: testing coinselector_tests
FAILED: src/test/CMakeFiles/check-bitcoin-coinselector_tests 
cd /work/abc-ci-builds/build-clang/src/test && /usr/bin/cmake -E make_directory /work/abc-ci-builds/build-clang/test/junit && /usr/bin/cmake -E make_directory /work/abc-ci-builds/build-clang/test/log && /usr/bin/cmake -E env /work/cmake/utils/log-and-print-on-failure.sh /work/abc-ci-builds/build-clang/test/log/bitcoin-coinselector_tests.log /work/abc-ci-builds/build-clang/src/test/test_bitcoin --run_test=coinselector_tests --logger=HRF,message:JUNIT,message,bitcoin-coinselector_tests.xml --catch_system_errors=no
Segmentation fault (core dumped)
[430/440] Running utility command for check-bitcoin-script_tests
[431/440] Running utility command for check-bitcoin-skiplist_tests
[432/440] Running utility command for check-bitcoin-util_tests
[433/440] bitcoin: testing op_reversebytes_tests
[434/440] Running utility command for check-bitcoin-op_reversebytes_tests
[435/440] bitcoin: testing transaction_tests
[436/440] Running utility command for check-bitcoin-transaction_tests
[437/440] bitcoin: testing coins_tests
[438/440] Running utility command for check-bitcoin-coins_tests
ninja: build stopped: cannot make progress due to previous errors.
Build build-clang failed with exit code 1
majcosta edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Tue, Feb 9, 15:32
This revision was automatically updated to reflect the committed changes.