Skip to content

Add unit tests for CSeederNode::ProcessMessage()

Dagur Valberg Johannsson requested to merge dagurval/bitcoin-cash-node:d4436 into master

Summary: Adds some unit tests for the CSeederNode class by using a test class wrapper for CSeederNode.

Depends on D4449

Test Plan: ../configure make check cmake -GNinja .. ninja check-bitcoin-seeder

../configure CXX=clang++ CC=clang --with-gui=no make check cmake -DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++ -GNinja .. ninja check-bitcoin-seeder

Expected output of last step:

Final line contains PASSED: bitcoin-seeder test suite

You can check that tests added in this MR are executed by

cd src/seeder/test && ./test_bitcoin-seeder --log_level=all

The output should indicate all 2 test cases (seeder_node_version_test and seeder_node_addr_test) in seeder_tests suite pass.

Reviewers: deadalnix, Fabien, jasonbcox, O1 Bitcoin ABC, #bitcoin_abc

Reviewed By: Fabien, jasonbcox, O1 Bitcoin ABC, #bitcoin_abc

Differential Revision: https://reviews.bitcoinabc.org/D4436

Edited by freetrader

Merge request reports