From f83b1d91746d37f92490a855344685e117930ccf Mon Sep 17 00:00:00 2001 From: JONASXZB <88891196+JONASXZB@users.noreply.github.com> Date: Tue, 2 Jun 2026 12:36:47 +0800 Subject: [PATCH] test: use BGL unit test binary names --- doc/developer-notes.md | 2 +- src/Makefile.test.include | 2 +- src/test/README.md | 10 +++++----- src/test/argsman_tests.cpp | 4 ++-- src/test/main.cpp | 2 +- src/test/settings_tests.cpp | 2 +- 6 files changed, 11 insertions(+), 11 deletions(-) diff --git a/doc/developer-notes.md b/doc/developer-notes.md index 654380a2ce..9d5b7c471c 100644 --- a/doc/developer-notes.md +++ b/doc/developer-notes.md @@ -487,7 +487,7 @@ To enable LCOV report generation during test runs: make make cov -# A coverage report will now be accessible at `./test_bitcoin.coverage/index.html`, +# A coverage report will now be accessible at `./test_BGL.coverage/index.html`, # which covers unit tests, and `./total.coverage/index.html`, which covers # unit and functional tests. ``` diff --git a/src/Makefile.test.include b/src/Makefile.test.include index 1b7c14d361..da1856608d 100644 --- a/src/Makefile.test.include +++ b/src/Makefile.test.include @@ -61,7 +61,7 @@ if USE_NATPMP FUZZ_SUITE_LD_COMMON += $(NATPMP_LIBS) endif -# test_bitcoin binary # +# test_BGL binary # BGL_TESTS =\ test/addrman_tests.cpp \ test/allocator_tests.cpp \ diff --git a/src/test/README.md b/src/test/README.md index eaff49970b..58ec5888bd 100644 --- a/src/test/README.md +++ b/src/test/README.md @@ -58,7 +58,7 @@ standard terminal output. ... or to run just the doubledash test: ```bash -test_ --run_test=getarg_tests/doubledash +test_BGL --run_test=getarg_tests/doubledash ``` `test_BGL` creates a temporary working (data) directory with a randomly @@ -72,7 +72,7 @@ have a `debug.log` file, for example. The location of the temporary data directory can be specified with the `-testdatadir` option. This can make debugging easier. The directory path used is the argument path appended with -`/test_common_Bitcoin Core//datadir`. +`/test_common_BGL Core//datadir`. The directory path is created if necessary. Specifying this argument also causes the data directory not to be removed after the last test. This is useful for looking at @@ -81,12 +81,12 @@ what the test wrote to `debug.log` after it completes, for example. so no leftover state is used.) ```bash -$ test_bitcoin --run_test=getarg_tests/doubledash -- -testdatadir=/somewhere/mydatadir +$ test_BGL --run_test=getarg_tests/doubledash -- -testdatadir=/somewhere/mydatadir Test directory (will not be deleted): "/somewhere/mydatadir/test_common_BGL Core/getarg_tests/doubledash/datadir Running 1 test case... *** No errors detected -$ ls -l '/somewhere/mydatadir/test_common_Bitcoin Core/getarg_tests/doubledash/datadir' +$ ls -l '/somewhere/mydatadir/test_common_BGL Core/getarg_tests/doubledash/datadir' total 8 drwxrwxr-x 2 admin admin 4096 Nov 27 22:45 blocks -rw-rw-r-- 1 admin admin 1003 Nov 27 22:45 debug.log @@ -96,7 +96,7 @@ If you run an entire test suite, such as `--run_test=getarg_tests`, or all the t (by not specifying `--run_test`), a separate directory will be created for each individual test. -Run `test_bitcoin --help` for the full list of tests. +Run `test_BGL --help` for the full list of tests. ### Adding test cases diff --git a/src/test/argsman_tests.cpp b/src/test/argsman_tests.cpp index 5f0318e8c4..4602eefb79 100644 --- a/src/test/argsman_tests.cpp +++ b/src/test/argsman_tests.cpp @@ -906,7 +906,7 @@ BOOST_FIXTURE_TEST_CASE(util_ArgsMerge, ArgsMergeTestingSetup) // If check below fails, should manually dump the results with: // - // ARGS_MERGE_TEST_OUT=results.txt ./test_bitcoin --run_test=argsman_tests/util_ArgsMerge + // ARGS_MERGE_TEST_OUT=results.txt ./test_BGL --run_test=argsman_tests/util_ArgsMerge // // And verify diff against previous results to make sure the changes are expected. // @@ -1009,7 +1009,7 @@ BOOST_FIXTURE_TEST_CASE(util_ChainMerge, ChainMergeTestingSetup) // If check below fails, should manually dump the results with: // - // CHAIN_MERGE_TEST_OUT=results.txt ./test_bitcoin --run_test=argsman_tests/util_ChainMerge + // CHAIN_MERGE_TEST_OUT=results.txt ./test_BGL --run_test=argsman_tests/util_ChainMerge // // And verify diff against previous results to make sure the changes are expected. // diff --git a/src/test/main.cpp b/src/test/main.cpp index 5e0cfe4c68..b79678cd1b 100644 --- a/src/test/main.cpp +++ b/src/test/main.cpp @@ -29,7 +29,7 @@ const std::function G_TEST_LOG_FUN = [](const std::str /** * Retrieve the command line arguments from boost. * Allows usage like: - * `test_bitcoin --run_test="net_tests/cnode_listen_port" -- -checkaddrman=1 -printtoconsole=1` + * `test_BGL --run_test="net_tests/cnode_listen_port" -- -checkaddrman=1 -printtoconsole=1` * which would return `["-checkaddrman=1", "-printtoconsole=1"]`. */ const std::function()> G_TEST_COMMAND_LINE_ARGUMENTS = []() { diff --git a/src/test/settings_tests.cpp b/src/test/settings_tests.cpp index 95f38fc0ce..3f34aedfc5 100644 --- a/src/test/settings_tests.cpp +++ b/src/test/settings_tests.cpp @@ -256,7 +256,7 @@ BOOST_FIXTURE_TEST_CASE(Merge, MergeTestingSetup) // If check below fails, should manually dump the results with: // - // SETTINGS_MERGE_TEST_OUT=results.txt ./test_bitcoin --run_test=settings_tests/Merge + // SETTINGS_MERGE_TEST_OUT=results.txt ./test_BGL --run_test=settings_tests/Merge // // And verify diff against previous results to make sure the changes are expected. //