diff options
author | Andrew Chow <achow101-github@achow101.com> | 2020-11-16 14:48:00 -0500 |
---|---|---|
committer | Andrew Chow <achow101-github@achow101.com> | 2021-12-05 13:39:51 -0500 |
commit | cbf0b9f4ff438865a71c7ceb0a543c18a34f41f0 (patch) | |
tree | 7d720babd28d3f56a3cbdb15837aab70955a33f3 | |
parent | 9d1d86da04d5d4768975338841285e90b01130b8 (diff) | |
download | bitcoin-cbf0b9f4ff438865a71c7ceb0a543c18a34f41f0.tar.xz |
scripted-diff: Use SelectionResult in coin selector tests
Replace the CoinSet actual_selection with a SelectionResult
expected_result. We don't use the SelectionResult functions yet, but
will soon.
-BEGIN VERIFY SCRIPT-
sed -i 's/CoinSet actual_selection/SelectionResult expected_result(CAmount(0))/' src/wallet/test/coinselector_tests.cpp
sed -i 's/actual_selection/expected_result.m_selected_inputs/' src/wallet/test/coinselector_tests.cpp
sed -i 's/expected_result.m_selected_inputs.clear/expected_result.Clear/' src/wallet/test/coinselector_tests.cpp
-END VERIFY SCRIPT-
-rw-r--r-- | src/wallet/test/coinselector_tests.cpp | 64 |
1 files changed, 32 insertions, 32 deletions
diff --git a/src/wallet/test/coinselector_tests.cpp b/src/wallet/test/coinselector_tests.cpp index 6b0cbd5fd6..adc2266477 100644 --- a/src/wallet/test/coinselector_tests.cpp +++ b/src/wallet/test/coinselector_tests.cpp @@ -157,7 +157,7 @@ BOOST_AUTO_TEST_CASE(bnb_search_test) // Setup std::vector<CInputCoin> utxo_pool; CoinSet selection; - CoinSet actual_selection; + SelectionResult expected_result(CAmount(0)); CAmount value_ret = 0; ///////////////////////// @@ -175,72 +175,72 @@ BOOST_AUTO_TEST_CASE(bnb_search_test) add_coin(4 * CENT, 4, utxo_pool); // Select 1 Cent - add_coin(1 * CENT, 1, actual_selection); + add_coin(1 * CENT, 1, expected_result.m_selected_inputs); BOOST_CHECK(SelectCoinsBnB(GroupCoins(utxo_pool), 1 * CENT, 0.5 * CENT, selection, value_ret)); - BOOST_CHECK(equivalent_sets(selection, actual_selection)); + BOOST_CHECK(equivalent_sets(selection, expected_result.m_selected_inputs)); BOOST_CHECK_EQUAL(value_ret, 1 * CENT); - actual_selection.clear(); + expected_result.Clear(); selection.clear(); // Select 2 Cent - add_coin(2 * CENT, 2, actual_selection); + add_coin(2 * CENT, 2, expected_result.m_selected_inputs); BOOST_CHECK(SelectCoinsBnB(GroupCoins(utxo_pool), 2 * CENT, 0.5 * CENT, selection, value_ret)); - BOOST_CHECK(equivalent_sets(selection, actual_selection)); + BOOST_CHECK(equivalent_sets(selection, expected_result.m_selected_inputs)); BOOST_CHECK_EQUAL(value_ret, 2 * CENT); - actual_selection.clear(); + expected_result.Clear(); selection.clear(); // Select 5 Cent - add_coin(4 * CENT, 4, actual_selection); - add_coin(1 * CENT, 1, actual_selection); + add_coin(4 * CENT, 4, expected_result.m_selected_inputs); + add_coin(1 * CENT, 1, expected_result.m_selected_inputs); BOOST_CHECK(SelectCoinsBnB(GroupCoins(utxo_pool), 5 * CENT, 0.5 * CENT, selection, value_ret)); - BOOST_CHECK(equivalent_sets(selection, actual_selection)); + BOOST_CHECK(equivalent_sets(selection, expected_result.m_selected_inputs)); BOOST_CHECK_EQUAL(value_ret, 5 * CENT); - actual_selection.clear(); + expected_result.Clear(); selection.clear(); // Select 11 Cent, not possible BOOST_CHECK(!SelectCoinsBnB(GroupCoins(utxo_pool), 11 * CENT, 0.5 * CENT, selection, value_ret)); - actual_selection.clear(); + expected_result.Clear(); selection.clear(); // Cost of change is greater than the difference between target value and utxo sum - add_coin(1 * CENT, 1, actual_selection); + add_coin(1 * CENT, 1, expected_result.m_selected_inputs); BOOST_CHECK(SelectCoinsBnB(GroupCoins(utxo_pool), 0.9 * CENT, 0.5 * CENT, selection, value_ret)); BOOST_CHECK_EQUAL(value_ret, 1 * CENT); - BOOST_CHECK(equivalent_sets(selection, actual_selection)); - actual_selection.clear(); + BOOST_CHECK(equivalent_sets(selection, expected_result.m_selected_inputs)); + expected_result.Clear(); selection.clear(); // Cost of change is less than the difference between target value and utxo sum BOOST_CHECK(!SelectCoinsBnB(GroupCoins(utxo_pool), 0.9 * CENT, 0, selection, value_ret)); - actual_selection.clear(); + expected_result.Clear(); selection.clear(); // Select 10 Cent add_coin(5 * CENT, 5, utxo_pool); - add_coin(5 * CENT, 5, actual_selection); - add_coin(4 * CENT, 4, actual_selection); - add_coin(1 * CENT, 1, actual_selection); + add_coin(5 * CENT, 5, expected_result.m_selected_inputs); + add_coin(4 * CENT, 4, expected_result.m_selected_inputs); + add_coin(1 * CENT, 1, expected_result.m_selected_inputs); BOOST_CHECK(SelectCoinsBnB(GroupCoins(utxo_pool), 10 * CENT, 0.5 * CENT, selection, value_ret)); - BOOST_CHECK(equivalent_sets(selection, actual_selection)); + BOOST_CHECK(equivalent_sets(selection, expected_result.m_selected_inputs)); BOOST_CHECK_EQUAL(value_ret, 10 * CENT); - actual_selection.clear(); + expected_result.Clear(); selection.clear(); // Negative effective value // Select 10 Cent but have 1 Cent not be possible because too small - add_coin(5 * CENT, 5, actual_selection); - add_coin(3 * CENT, 3, actual_selection); - add_coin(2 * CENT, 2, actual_selection); + add_coin(5 * CENT, 5, expected_result.m_selected_inputs); + add_coin(3 * CENT, 3, expected_result.m_selected_inputs); + add_coin(2 * CENT, 2, expected_result.m_selected_inputs); BOOST_CHECK(SelectCoinsBnB(GroupCoins(utxo_pool), 10 * CENT, 5000, selection, value_ret)); BOOST_CHECK_EQUAL(value_ret, 10 * CENT); // FIXME: this test is redundant with the above, because 1 Cent is selected, not "too small" - // BOOST_CHECK(equivalent_sets(selection, actual_selection)); + // BOOST_CHECK(equivalent_sets(selection, expected_result.m_selected_inputs)); // Select 0.25 Cent, not possible BOOST_CHECK(!SelectCoinsBnB(GroupCoins(utxo_pool), 0.25 * CENT, 0.5 * CENT, selection, value_ret)); - actual_selection.clear(); + expected_result.Clear(); selection.clear(); // Iteration exhaustion test @@ -251,11 +251,11 @@ BOOST_AUTO_TEST_CASE(bnb_search_test) // Test same value early bailout optimization utxo_pool.clear(); - add_coin(7 * CENT, 7, actual_selection); - add_coin(7 * CENT, 7, actual_selection); - add_coin(7 * CENT, 7, actual_selection); - add_coin(7 * CENT, 7, actual_selection); - add_coin(2 * CENT, 7, actual_selection); + add_coin(7 * CENT, 7, expected_result.m_selected_inputs); + add_coin(7 * CENT, 7, expected_result.m_selected_inputs); + add_coin(7 * CENT, 7, expected_result.m_selected_inputs); + add_coin(7 * CENT, 7, expected_result.m_selected_inputs); + add_coin(2 * CENT, 7, expected_result.m_selected_inputs); add_coin(7 * CENT, 7, utxo_pool); add_coin(7 * CENT, 7, utxo_pool); add_coin(7 * CENT, 7, utxo_pool); @@ -266,7 +266,7 @@ BOOST_AUTO_TEST_CASE(bnb_search_test) } BOOST_CHECK(SelectCoinsBnB(GroupCoins(utxo_pool), 30 * CENT, 5000, selection, value_ret)); BOOST_CHECK_EQUAL(value_ret, 30 * CENT); - BOOST_CHECK(equivalent_sets(selection, actual_selection)); + BOOST_CHECK(equivalent_sets(selection, expected_result.m_selected_inputs)); //////////////////// // Behavior tests // |