ci: Avoid boost test warnings #24708

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2203-ci-boost-test-warn-🎈 changing 1 files +1 −1

MarcoFalke commented at 2:08 PM on March 29, 2022: member

This removes boost test warnings from the CI log.

For example https://cirrus-ci.com/task/6029158399606784?logs=ci#L4060:

Boost.Test WARNING: token "/tmp/cirrus-build/ci/scratch/build/bitcoin-x86_64-pc-linux-gnu/src/test/test_bitcoin-addrman_tests.o" does not correspond to the Boost.Test argument 
                    and should be placed after all Boost.Test arguments and the -- separator.
                    For example: test_bitcoin --random -- /tmp/cirrus-build/ci/scratch/build/bitcoin-x86_64-pc-linux-gnu/src/test/test_bitcoin-addrman_tests.o
Boost.Test WARNING: token "/tmp/cirrus-build/ci/scratch/build/bitcoin-x86_64-pc-linux-gnu/src/test/test_bitcoin-allocator_tests.o" does not correspond to the Boost.Test argument 
                    and should be placed after all Boost.Test arguments and the -- separator.
                    For example: test_bitcoin --random -- /tmp/cirrus-build/ci/scratch/build/bitcoin-x86_64-pc-linux-gnu/src/test/test_bitcoin-allocator_tests.o
Boost.Test WARNING: token "/tmp/cirrus-build/ci/scratch/build/bitcoin-x86_64-pc-linux-gnu/src/test/test_bitcoin-amount_tests.o" does not correspond to the Boost.Test argument 
                    and should be placed after all Boost.Test arguments and the -- separator.
...

ci: Avoid boost test warnings fa76b2fbe3
DrahtBot added the label Tests on Mar 29, 2022
MarcoFalke commented at 3:23 PM on March 29, 2022: member

Clean after this patch: https://cirrus-ci.com/task/6318327072030720?logs=ci#L4061
fanquake approved
fanquake commented at 7:54 AM on March 30, 2022: member

ACK fa76b2fbe343af5679263ac9bb3f1bd72fd82b5e - checked that tests are also still being run in other jobs.
fanquake merged this on Mar 30, 2022
fanquake closed this on Mar 30, 2022
MarcoFalke deleted the branch on Mar 30, 2022
MarcoFalke commented at 8:16 AM on March 30, 2022: member

There is only one task right now that sets RUN_UNIT_TESTS_SEQUENTIAL, so the other tasks should be unaffected either way. But thanks for checking.
sidhujag referenced this in commit 5983c9a0be on Apr 3, 2022
DrahtBot locked this on Mar 30, 2023

Contributors

MarcoFalke

fanquake

Labels

Tests