From 2a1b2754f14a2686e78547b2e8a51a4fb35e3816 Mon Sep 17 00:00:00 2001 From: Murch Date: Fri, 1 Mar 2024 15:42:08 -0500 Subject: [PATCH] test: Remove redundant repeated test We do not need to repeat the same test multiple times because BnB is deterministic and will therefore always have the same outcome. Additionally, this test was redundant because it repeats the "Smallest combination too big" test. --- src/wallet/test/coinselector_tests.cpp | 9 --------- 1 file changed, 9 deletions(-) diff --git a/src/wallet/test/coinselector_tests.cpp b/src/wallet/test/coinselector_tests.cpp index 141664d1fb8..4e9c094ab8d 100644 --- a/src/wallet/test/coinselector_tests.cpp +++ b/src/wallet/test/coinselector_tests.cpp @@ -209,15 +209,6 @@ BOOST_AUTO_TEST_CASE(bnb_search_test) //////////////////// // Behavior tests // //////////////////// - // Select 1 Cent with pool of only greater than 5 Cent - utxo_pool.clear(); - for (int i = 5; i <= 20; ++i) { - add_coin(i * CENT, i, utxo_pool); - } - // Run 100 times, to make sure it is never finding a solution - for (int i = 0; i < 100; ++i) { - BOOST_CHECK(!SelectCoinsBnB(GroupCoins(utxo_pool), 1 * CENT, 2 * CENT)); - } // Make sure that effective value is working in AttemptSelection when BnB is used CoinSelectionParams coin_selection_params_bnb{