aboutsummaryrefslogtreecommitdiff
path: root/test/functional/wallet_groups.py
Commit message (Collapse)AuthorAgeFilesLines
* Generate legacy addresses by defaultRoss Nicoll2021-06-121-7/+7
| | | | Generate legacy addresses instead of SegWit by default.
* Dogecoin: Update coinbase maturityRoss Nicoll2021-05-201-1/+1
| | | | | | | * Change coinbase maturity to 240 blocks in most cases, with main/test early chains allowing 30 blocks. I've kept the 240 consistent in regtest to avoid having to redesign a lot of the test cases. * Disabled mining unit test which require COINBASE_MATURITY worth of pre-calculated blocks, as we'd otherwise be constantly refactoring them. * Moved functional test which uses the Bitcoin testnet block data as its reference, as it completely breaks as we introduce Dogecoin data. * Updated standard blockchains for tests from 100/200 to 240/480 as appropriate.
* -maxapsfee: follow-up fixesKarl-Johan Alm2020-08-181-11/+42
| | | | | Co-authored-by: Jon Atack <[email protected]> Co-authored-by: Samuel Dobson <[email protected]>
* tests: add sync_all to fix race condition in wallet groups testKarl-Johan Alm2020-08-181-0/+1
|
* Merge #14582: wallet: always do avoid partial spends if fees are within a ↵Samuel Dobson2020-08-171-2/+48
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | specified range 7f13dfb587dd6a7a5b7dfbfe689ae0ce818fe5c9 test: test the implicit avoid partial spends functionality (Karl-Johan Alm) b82067bf696c53f22536f9ca2e51949c164f6b06 wallet: try -avoidpartialspends mode and use its result if fees are below threshold (Karl-Johan Alm) Pull request description: The `-avoidpartialspends` feature is normally disabled, as it may affect the optimal fee for payments. This PR introduces a new parameter `-maxapsfee` (max avoid partial spends fee) which acts on the following values: * -1: disable partial spend avoidance completely (do not even try it) * 0: only do partial spend avoidance if fees are the same or better as the regular coin selection * 1..∞: use APS variant if the absolute fee difference is less than or equal to the max APS fee For values other than -1, the code will now try partial spend avoidance once, and if that gives a value within the accepted range, it will use that. Example: -maxapsfee=0.00001000 means the wallet will do regular coin select, APS coin select, and then pick AKS iff the absolute fee difference is <= 1000 satoshi. Edit: updated this to reflect the fact we are now using a max fee. ACKs for top commit: fjahr: tested ACK 7f13dfb587dd6a7a5b7dfbfe689ae0ce818fe5c9 achow101: ACK 7f13dfb587dd6a7a5b7dfbfe689ae0ce818fe5c9 jonatack: ACK 7f13dfb58, code review, debug build, verified the test fails with `AssertionError: not(2 == 1)` for the number of vouts when `-maxapsfee=0.0001` is changed to 0, and verified the new logging with an added assertion. meshcollider: Code review ACK 7f13dfb587dd6a7a5b7dfbfe689ae0ce818fe5c9 Tree-SHA512: 475929df57f6191bb4e36bfbcad5a280a64bb0ecd8767b76cb2e44e2301235d0eb294a3f2fac5bbf15d35d7ecfba47acb2285feadb883c9ce31c08377e3afb3c
| * test: test the implicit avoid partial spends functionalityKarl-Johan Alm2020-08-061-2/+48
| | | | | | | | Co-authored-by: Fabian Jahr <[email protected]>
* | refactor: test: use _ variable for unused loop countersSebastian Falbesoner2020-08-061-3/+3
|/ | | | | | substitutes "for x in range(N):" by "for _ in range(N):" indicates to the reader that a block is just repeated N times, and that the loop counter is not used in the body
* scripted-diff: Bump copyright headersMarcoFalke2020-04-161-1/+1
| | | | | | -BEGIN VERIFY SCRIPT- ./contrib/devtools/copyright_header.py update ./ -END VERIFY SCRIPT-
* test: Bump timeouts to avoid valgrind failuresMarcoFalke2020-03-141-2/+4
|
* Merge #17770: test: bump test timeouts so that functional tests run in valgrindMarcoFalke2020-01-251-1/+1
|\ | | | | | | | | | | | | | | | | | | | | | | 2d23082cbe4641175d752a5969f67cdadf1afcea bump test timeouts so that functional tests run in valgrind (Micky Yun Chan) Pull request description: ci/tests: Bump timeouts so all functional tests run on travis in valgrind #17763 Top commit has no ACKs. Tree-SHA512: 5a8c6e2ea02b715facfcb58c761577be15ae58c45a61654beb98c2c2653361196c2eec521bcae4a9a1bab8e409d6807de771ef4c46d3d05996ae47a22d499d54
| * bump test timeouts so that functional tests run in valgrindMicky Yun Chan2020-01-251-1/+1
| |
* | scripted-diff: Bump copyright of files changed in 2019MarcoFalke2019-12-301-1/+1
|/ | | | | | -BEGIN VERIFY SCRIPT- ./contrib/devtools/copyright_header.py update ./ -END VERIFY SCRIPT-
* Test: Move common function assert_approx() into util.pyfridokus2019-09-191-6/+1
|
* scripted-diff: Rename rpc_timewait to rpc_timeoutMarcoFalke2018-12-221-1/+1
| | | | | | -BEGIN VERIFY SCRIPT- sed -i -e 's/self.rpc_timewait/self.rpc_timeout/g' $(git grep -l self.rpc_timewait) -END VERIFY SCRIPT-
* qa: Run all tests even if wallet is not compiledMarcoFalke2018-09-101-1/+4
|
* tests: Use explicit importspracticalswift2018-08-131-2/+1
|
* qa: Extract rpc_timewait as test paramMarcoFalke2018-08-011-0/+1
| | | | Also increase it for wallet_dump and wallet_groups
* [qa] Add test for too-large wallet output groupsSuhas Daftuar2018-07-301-0/+26
|
* test: Add basic testing for wallet groupsKarl-Johan Alm2018-07-241-0/+67
generated by cgit v1.2.3 (git 2.25.1) at 2026-04-19 19:41:11 +0000