Skip to content

Commit

Permalink
[test] Remove redundant repeated test
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
murchandamus committed Jun 27, 2024
1 parent f07909f commit 4ae1dff
Showing 1 changed file with 0 additions and 9 deletions.
9 changes: 0 additions & 9 deletions src/wallet/test/coinselector_tests.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -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{
Expand Down

0 comments on commit 4ae1dff

Please sign in to comment.