Skip to content

Commit 2a1b275

Browse files
committed
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.
1 parent 4781f5c commit 2a1b275

File tree

1 file changed

+0
-9
lines changed

1 file changed

+0
-9
lines changed

src/wallet/test/coinselector_tests.cpp

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -209,15 +209,6 @@ BOOST_AUTO_TEST_CASE(bnb_search_test)
209209
////////////////////
210210
// Behavior tests //
211211
////////////////////
212-
// Select 1 Cent with pool of only greater than 5 Cent
213-
utxo_pool.clear();
214-
for (int i = 5; i <= 20; ++i) {
215-
add_coin(i * CENT, i, utxo_pool);
216-
}
217-
// Run 100 times, to make sure it is never finding a solution
218-
for (int i = 0; i < 100; ++i) {
219-
BOOST_CHECK(!SelectCoinsBnB(GroupCoins(utxo_pool), 1 * CENT, 2 * CENT));
220-
}
221212

222213
// Make sure that effective value is working in AttemptSelection when BnB is used
223214
CoinSelectionParams coin_selection_params_bnb{

0 commit comments

Comments
 (0)