aboutsummaryrefslogtreecommitdiff
path: root/test/functional/feature_pruning.py
Commit message (Collapse)AuthorAgeFilesLines
* scripted-diff: test: Replace uses of (dis)?connect_nodes globalPrayank2020-10-201-16/+14
| | | | | | | | | | | | | | | | | | | | | | -BEGIN VERIFY SCRIPT- # max-depth=0 excludes test/functional/test_framework/... FILES=$(git grep -l --max-depth 0 "connect_nodes" test/functional) # Replace (dis)?connect_nodes(self.nodes[a], b) with self.(dis)?connect_nodes(a, b) sed -i 's/\b\(dis\)\?connect_nodes(self\.nodes\[\(.*\)\]/self.\1connect_nodes(\2/g' $FILES # Remove imports in the middle of a line sed -i 's/\(dis\)\?connect_nodes, //g' $FILES sed -i 's/, \(dis\)\?connect_nodes//g' $FILES # Remove imports on a line by themselves sed -i '/^\s*\(dis\)\?connect_nodes,\?$/d' $FILES sed -i '/^from test_framework\.util import connect_nodes$/d' $FILES -END VERIFY SCRIPT- Co-authored-by: Elliott Jin <[email protected]>
* test: Get rid of default wallet hacksRussell Yanofsky2020-09-291-7/+6
| | | | | | | | | | | | | | | | | | - Get rid of hardcoded wallet "" names and -wallet="" args - Get rid of setup_nodes (-wallet, -nowallet, -disablewallet) argument rewriting Motivation: - Simplify test framework behavior so it's easier to write new tests without having arguments mangled by the framework - Make tests more readable, replacing unexplained "" string literals with default_wallet_name references - Make it trivial to update default wallet name and wallet data filename for sqlite #19077 testing - Stop relying on -wallet arguments to create wallets, so it is easy to change -wallet option in the future to only load existing wallets not create new ones (to avoid accidental wallet creation, and encourage use of wallet encryption and descriptor features)
* test: create default wallet in extended testsSjors Provoost2020-09-181-5/+5
| | | | This was omitted from #15454
* test: Update wait_until usage in tests not to use the one from utilsSeleme Topuz2020-08-261-3/+2
| | | | | Replace "wait_until()" usage from utils, with the ones from BitcoinTestFramework and P2PInterface. closes #19080
* refactor: test: use _ variable for unused loop countersSebastian Falbesoner2020-08-061-1/+1
| | | | | | 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
* test: refactor functional tests to use restart_nodeChristopher Coverdale2020-06-221-8/+4
|
* scripted-diff: Bump copyright headersMarcoFalke2020-04-161-1/+1
| | | | | | -BEGIN VERIFY SCRIPT- ./contrib/devtools/copyright_header.py update ./ -END VERIFY SCRIPT-
* Merge #16681: Tests: Use self.chain instead of 'regtest' in all current testsMarcoFalke2020-02-041-2/+2
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1abcecc40c518a98b7d17880657ec0247abdf125 Tests: Use self.chain instead of 'regtest' in almost all current tests (Jorge Timón) Pull request description: Simply avoiding the hardcoded string in more places for consistency. It can also allow for more easily reusing tests for other chains other than regtest. Separated from #8994 . Continues #16509 . It is still not complete (ie to be complete, we need the -chain parameter in #16680 and make whether acceptnonstdtxs is allowed for that chain or not customizable for regtest [or for custom chains like in #8994 ] ). But while being incomplete like #16509 , it's quite simple to review and another step forward IMO. ACKs for top commit: Sjors: re-ACK 1abcecc. I think it's an improvement even if incomplete and if some PR's might accidentally bring "regtest" back. Subsequent improvements hopefully don't have to touch 16 files. elichai: Code review ACK 1abcecc40c518a98b7d17880657ec0247abdf125 ryanofsky: Code review ACK 1abcecc40c518a98b7d17880657ec0247abdf125. ryanofsky: Code review ACK 1abcecc40c518a98b7d17880657ec0247abdf125 Tree-SHA512: 5620de6dab235ca8bd8670d6366c7b9f04f0e3ca9c5e7f87765b38e16ed80c17d7d1630c0d5fd7c5526f070830d94dc74cc2096d8ede87dc7180ed20569509ee
| * Tests: Use self.chain instead of 'regtest' in almost all current testsJorge Timón2019-10-261-2/+2
| |
* | tests: Mark functional tests not supporting bitcoin-cli (--usecli) as suchpracticalswift2019-12-061-0/+1
|/
* test: Bump timeouts in slow running testsMarcoFalke2019-09-161-0/+1
|
* test: Add missing sync_blocks to feature_pruningMarcoFalke2019-08-021-0/+1
|
* fixup: Fix prunning testJoão Barbosa2019-06-111-16/+3
|
* Merge #15696: [qa] test_runner: Move feature_pruning to base testsMarcoFalke2019-04-301-6/+7
|\ | | | | | | | | | | | | | | | | | | | | fafb55e2c2 [qa] test_runner: Move feature_pruning to base tests (MarcoFalke) 8728a66782 [tests] fix block time in feature_pruning.py (John Newbery) Pull request description: ACKs for commit fafb55: Tree-SHA512: 88abef94379fbad6629da11dccb080d5f0644490d6f2cc2756a33fac34bcf72e84245cef596dfae5a40f7a99b3f4da0dd85d306d4c1b452d310d3f36eef75a8b
| * [tests] fix block time in feature_pruning.pyJohn Newbery2019-04-291-6/+7
| |
* | test: Remove unused importsMarcoFalke2019-04-091-1/+8
| |
* | scripted-diff: use self.sync_* methodsMarcoFalke2019-04-091-6/+6
|/ | | | | | | | | | -BEGIN VERIFY SCRIPT- sed -i -e 's/sync_blocks(self.nodes)/self.sync_blocks()/g' $(git grep -l 'sync_blocks(self.nodes)' ./test/functional/*.py) sed -i -e 's/sync_mempools(self.nodes)/self.sync_mempools()/g' $(git grep -l 'sync_mempools(self.nodes)' ./test/functional/*.py) sed -i -e 's/ sync_blocks(/ self.sync_blocks(/g' $(git grep -l sync_blocks ./test/functional/*.py) sed -i -e 's/ sync_mempools(/ self.sync_mempools(/g' $(git grep -l sync_mempools ./test/functional/*.py) -END VERIFY SCRIPT-
* [tests] make pruning test fasterJohn Newbery2019-03-291-47/+60
| | | | | | | | | | | | | | | | This commit makes the pruning.py much faster. Key insights to do this: - pruning.py doesn't care what kind of transactions make up the big blocks that are pruned in the test. Instead of making blocks with several large, expensive to construct and validate transactions, instead make the large blocks contain a single coinbase transaction with a huge OP_RETURN txout. - avoid stop-starting nodes where possible. This test could probably be made even faster by using the P2P interface for submitting blocks instead of the submitblock RPC.
* [tests] style fixes in feature_pruning.pyJohn Newbery2019-03-291-47/+34
| | | | Minor style fixups. No functional change.
* scripted-diff: Update copyright in ./testMarcoFalke2019-03-021-1/+1
| | | | | | -BEGIN VERIFY SCRIPT- ./contrib/devtools/copyright_header.py update ./test/ -END VERIFY SCRIPT-
* scripted-diff: test: Remove brackets after assertMarcoFalke2019-03-021-5/+5
| | | | | | -BEGIN VERIFY SCRIPT- sed -i --regexp-extended -e 's/assert ?\((.+)\)(( )*)?(#.*)?$/assert \1\3\3\4/g' $(git grep -l --extended-regexp 'assert ?\(' test) -END VERIFY SCRIPT-
* Correct units for "-dbcache" and "-prune"Hennadii Stepanov2019-01-301-1/+1
| | | | | | | | | All dbcache-related values in the code are measured in MiB (not in megabytes, MB) or in bytes. The GUI "-prune" values in GB are translated to the node values in MiB correctly. The maximum of the "-prune" QSpinBox is not limited by the default value of 99 (GB). Also, this improves log readability.
* Merge #15026: [test] Rename rpc_timewait to rpc_timeoutMarcoFalke2018-12-291-1/+1
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4999992c34 whitespace: Split ~300 char line into multiple ones (MarcoFalke) fa71b38168 scripted-diff: Rename rpc_timewait to rpc_timeout (MarcoFalke) fa3e5786d0 scripted-diff: Remove unused 'split' parameter to setup_network (MarcoFalke) Pull request description: This is a bugfix, since wallet_dump currently uses the wrong name: https://github.com/bitcoin/bitcoin/blob/18857b4c4034af54e7ad3cbd78ff6f87f4f22567/test/functional/wallet_dump.py#L89-L92 Rename all to the same name with a scripted diff (and some unrelated cleanups). Tree-SHA512: 338ddd20dae12e6cf7aa7adbcfb239cf648017a1572b373f8431fecb184bd2a65492846d81e75a023864d9e41c94afb53044c16b79651a5937d34a5a6b772f81
| * 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-
* | test: pruning: Check that verifychain can be called when prunedMarcoFalke2018-12-111-0/+2
|/
* rpc: Correctly name argumentsJon Layton2018-11-131-1/+1
|
* [tests] Move deterministic address import to setup_nodesJohn Newbery2018-11-011-0/+2
| | | | | | This requires a small changes to a few tests, but means that deterministic addresses will always be imported (unless setup_nodes behaviour is explicitly overridden).
* qa: Run all tests even if wallet is not compiledMarcoFalke2018-09-101-7/+12
|
* tests: Use explicit importspracticalswift2018-08-131-1/+2
|
* Merge #13780: 0.17: Pre-branch maintenanceWladimir J. van der Laan2018-08-081-1/+1
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | 3fc20632a3ad30809356a58d2cf0ea4a4ad4cec3 qt: Set BLOCK_CHAIN_SIZE = 220 (DrahtBot) 2b6a2f4a28792f2fe9dc1be843b1ff1ecae35e8a Regenerate manpages (DrahtBot) eb7daf4d600eeb631427c018a984a77a34aca66e Update copyright headers to 2018 (DrahtBot) Pull request description: Some trivial maintenance to avoid having to do it again after the 0.17 branch off. (The scripts to do this are in `./contrib/`) Tree-SHA512: 16b2af45e0351b1c691c5311d48025dc6828079e98c2aa2e600dc5910ee8aa01858ca6c356538150dc46fe14c8819ed8ec8e4ec9a0f682b9950dd41bc50518fa
| * Update copyright headers to 2018DrahtBot2018-07-271-1/+1
| |
* | qa: Extract rpc_timewait as test paramMarcoFalke2018-08-011-1/+2
|/ | | | Also increase it for wallet_dump and wallet_groups
* expose CBlockIndex::nTx in getblock(header)Gregory Sanders2018-06-131-0/+7
|
* Remove Safe modeAndrew Chow2018-04-261-2/+2
|
* [Tests] Use blockmaxweight where tests previously had blockmaxsizeConor Scott2018-03-261-2/+2
|
* [config] Remove blockmaxsize optionJohn Newbery2018-03-221-5/+5
| | | | | | | | | | The blockmaxsize option was marked as deprecated in V0.15.1, and code was added to convert provided blockmaxsize into blockmaxweight. However, this code was incorrectly implemented, and blockmaxsize was silently ignored. No users have complained about blockmaxsize being ignored, so just remove it in V0.17.
* Merge #12076: qa: Use node.datadir instead of tmpdir in test frameworkWladimir J. van der Laan2018-03-221-1/+1
|\ | | | | | | | | | | | | | | | | | | | | | | c8330d4 qa: Use node.datadir instead of tmpdir in test framework (MarcoFalke) Pull request description: Commit c53c9831eedaf3b311bb942945268830f9ba3abc introduced the utility function `get_datadir_path`, however not all places in the code use this util function. Using the util function everywhere makes it easier to review pull requests related to the datadir. This commit replaces datadir path creation with the `datadir` member of `TestNode`, which itself uses `get_datadir_path`. Tree-SHA512: c75707ab7149d732a6d56152a5813138a33459d3d07577b60b89f2a207c83b7663fac5f203593677c9892d1c23a5eba4bd45c5c4ababf040d720b437240fcddf
| * qa: Use node.datadir instead of tmpdir in test frameworkMarcoFalke2018-03-191-1/+1
| |
* | Merge #12716: Fix typos and cleanup in various filesMarcoFalke2018-03-211-2/+2
|\ \ | |/ |/| | | | | | | | | | | | | | | 4d9b4256d8 Fix typos (Dimitris Apostolou) Pull request description: Unfortunately I messed up my repo while trying to squash #12593 so I created a PR with just the correct fixes. Tree-SHA512: 295d77b51bd2a9381f1802c263de7ffb2edd670d9647391e32f9a414705b3c8b483bb0e469a9b85ab6a70919ea13397fa8dfda2aea7a398b64b187f178fe6a06
| * Fix typosDimitris Apostolou2018-03-211-2/+2
| |
* | test: Use os.path.join consistently in feature_pruning testsBen Woosley2018-03-151-4/+4
| |
* | test: Use wait_until in tests where time was used for pollingBen Woosley2018-03-151-11/+4
|/
* [tests] Rename feature_* functional tests.Anthony Towns2018-01-251-0/+454
generated by cgit v1.2.3 (git 2.25.1) at 2026-04-15 02:10:42 +0000