aboutsummaryrefslogtreecommitdiff
path: root/qa/rpc-tests
Commit message (Collapse)AuthorAgeFilesLines
...
| * [qa] Test prioritise_transaction / getblocktemplate interactionSuhas Daftuar2017-04-211-1/+15
| | | | | | | | | | Github-Pull: #10196 Rebased-From: 6a61424c9eed072ae8d79d68aac71f525c1608b5
* | [Wallet] unset change position when there is no change on exact matchGregory Sanders2017-05-311-0/+5
|/
* [qa] Fix mempool_packages.py for the 0.14 branchSuhas Daftuar2017-04-051-2/+2
| | | | | The backport in d947afc0f7868815dff248ad398c3680def5bf81 of this test was incorrect due to an api change in prioritisetransaction.
* Test prioritisetransaction and ancestor fee stateSuhas Daftuar2017-04-051-0/+12
| | | | | | | There is already a similar test for descendant fee state. Github-Pull: #10144 Rebased-From: ba7dd8bf6f41de647a8df0c4570df6ac20b08e48
* Test transaction selection when gbt called without segwit supportSuhas Daftuar2017-03-161-6/+49
| | | | | Github-Pull: #9955 Rebased-From: c85ffe6d8d57132c1825c16a572d3847419030a6
* Don't require segwit in getblocktemplate for segwit signalling or miningSuhas Daftuar2017-03-162-17/+10
| | | | | | | | | Segwit's version bit will be signalled for all invocations of CreateNewBlock, and not specifying segwit only will cause CreateNewBlock to skip transactions with witness from being selected. Github-Pull: #9955 Rebased-From: abe7b3d3abe10e3554b770f40824174b3b217490
* [qa] Make import-rescan.py watchonly check reliableRussell Yanofsky2017-02-271-12/+18
| | | | | | | | | | | Send payments during the test from a different node than the node generating keys to be imported, so the spending node doesn't create transactions that inadvertently involve (spend funds from) the imported keys. Fixes #9826 Github-Pull: #9839 Rebased-From: 864890adf5fefcf969f82f70949199b68b36f165
* qa: Check return code when stopping nodesMarcoFalke2017-02-233-27/+15
| | | | | | | This includes work by jnewbery Github-Pull: #9824 Rebased-From: fa4cd2e998e0ebebe283a6243f7f752123a105ab
* Fix pruning test broken by 2 hour manual prune windowRussell Yanofsky2017-02-231-1/+6
| | | | | | | Regression happened in 91fb506 Add two hour buffer to manual pruning. Github-Pull: #9820 Rebased-From: 874c7363d794c18a3b46d3d2a03f19c88f3a20cf
* QA: Test GBT size/weight limit valuesLuke Dashjr2017-02-201-0/+8
| | | | | Github-Pull: #9619 Rebased-From: 279f944e8d3b437d31210f3ba3e798e98a9334fc
* Use 2 hour grace period for key timestamps in importmulti rescansRussell Yanofsky2017-02-161-2/+5
| | | | | | | | Gregory Maxwell <[email protected]> pointed out the lack of grace period in https://github.com/bitcoin/bitcoin/pull/9490#issue-199407998. The importwallet RPC which uses key timestamps in a similar way already has a 2 hour grace period.
* [qa] Extend import-rescan.py to test imports on pruned nodes.Russell Yanofsky2017-02-161-21/+43
|
* [qa] Extend import-rescan.py to test specific key timestampsRussell Yanofsky2017-02-161-11/+15
|
* [qa] Simplify import-rescan.pyRussell Yanofsky2017-02-161-112/+112
| | | | | | | | | | | | | | Get rid of partial functions so the test can be more easily extended to add more variants of imports with options that affect rescanning (e.g. different key timestamps). Also change the second half of the test to send /to/ the imported addresses, instead of /from/ the imported addresses. The goal of this part of the test was to confirm that the wallet would pick up new transactions after an import regardless of whether or not a rescan happened during the import. But because the wallet can only do this reliably for incoming transactions and not outgoing transactions (which require the wallet to look up transaction inputs) the test previously was less meaningful than it should have been.
* Merge #9765: Harden against mistakes handling invalid blocksWladimir J. van der Laan2017-02-161-2/+2
|\ | | | | | | ba803ef Harden against mistakes handling invalid blocks (Suhas Daftuar)
| * Harden against mistakes handling invalid blocksSuhas Daftuar2017-02-151-2/+2
| | | | | | | | | | | | | | | | Fixes a bug in AcceptBlock() in invoking CheckBlock() with incorrect arguments, and restores a call to CheckBlock() from ProcessNewBlock() as belt-and-suspenders. Updates the (overspecified) tests to match behavior.
* | Merge #9756: Return error when importmulti called with invalid address.Wladimir J. van der Laan2017-02-151-0/+10
|\ \ | | | | | | | | | 9acf25c Return error when importmulti called with invalid address. (Russell Yanofsky)
| * | Return error when importmulti called with invalid address.Russell Yanofsky2017-02-131-0/+10
| | | | | | | | | | | | Lack of error checking noticed by Alex Morcos <[email protected]>
* | | Merge #9108: Use importmulti timestamp when importing watch only keys (on ↵Wladimir J. van der Laan2017-02-151-1/+32
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | top of #9682) a80f98b Use importmulti timestamp when importing watch only keys (Russell Yanofsky) a58370e Dedup nTimeFirstKey update logic (Russell Yanofsky)
| * | | Use importmulti timestamp when importing watch only keysRussell Yanofsky2017-02-101-1/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When importing a watch-only address over importmulti with a specific timestamp, the wallet's nTimeFirstKey is currently set to 1. After this change, the provided timestamp will be used and stored as metadata associated with watch-only key. This can improve wallet performance because it can avoid the need to scan the entire blockchain for watch only addresses when timestamps are provided. Also adds timestamp to validateaddress return value (needed for tests). Fixes #9034.
* | | | Merge #9720: net: fix banning and disallow sending messages before receiving ↵Wladimir J. van der Laan2017-02-141-0/+145
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | verack d943491 qa: add a test to detect leaky p2p messages (Cory Fields) 8650bbb qa: Expose on-connection to mininode listeners (Matt Corallo) 5b5e4f8 qa: mininode learns when a socket connects, not its first action (Matt Corallo) cbfc5a6 net: require a verack before responding to anything else (Cory Fields) 8502e7a net: parse reject earlier (Cory Fields) c45b9fb net: correctly ban before the handshake is complete (Cory Fields)
| * | | | qa: add a test to detect leaky p2p messagesCory Fields2017-02-131-0/+145
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is certainly not exhaustive, but it's better than nothing. Adds checks for: - Any message received before sending a version - Any message received other than version/reject before sending a verack It also tries to goad the remote into sending a pong, address, or block announcement.
| * | | | qa: Expose on-connection to mininode listenersMatt Corallo2017-02-131-0/+2
| | | | |
| * | | | qa: mininode learns when a socket connects, not its first actionMatt Corallo2017-02-131-12/+24
| | | | |
* | | | | Merge #9715: Disconnect peers which we do not receive VERACKs from within 60 secWladimir J. van der Laan2017-02-142-12/+129
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 66f861a Add a test for P2P inactivity timeouts (Matt Corallo) b436f92 qa: Expose on-connection to mininode listeners (Matt Corallo) 8aaba7a qa: mininode learns when a socket connects, not its first action (Matt Corallo) 2cbd119 Disconnect peers which we do not receive VERACKs from within 60 sec (Matt Corallo)
| * | | | | Add a test for P2P inactivity timeoutsMatt Corallo2017-02-091-0/+103
| | | | | |
| * | | | | qa: Expose on-connection to mininode listenersMatt Corallo2017-02-081-0/+2
| | | | | |
| * | | | | qa: mininode learns when a socket connects, not its first actionMatt Corallo2017-02-081-12/+24
| |/ / / /
* | | | | Merge #9682: Require timestamps for importmulti keysWladimir J. van der Laan2017-02-142-3/+37
|\ \ \ \ \ | | |/ / / | |/| / / | |_|/ / |/| | | | | | | 266a811 Use MTP for importmulti "now" timestamps (Russell Yanofsky) 3cf9917 Add test to check new importmulti "now" value (Russell Yanofsky) 442887f Require timestamps for importmulti keys (Russell Yanofsky)
| * | | Use MTP for importmulti "now" timestampsRussell Yanofsky2017-02-101-1/+1
| | | |
| * | | Add test to check new importmulti "now" valueRussell Yanofsky2017-02-101-0/+2
| | | | | | | | | | | | | | | | | | | | Easiest way to test this was to expose the timestamp via the validateaddress RPC (which was already looking up and returning key metadata).
| * | | Require timestamps for importmulti keysRussell Yanofsky2017-02-102-3/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Additionally, accept a "now" timestamp, to allow avoiding rescans for keys which are known never to have been used. Note that the behavior when "now" is specified is slightly different than the previous behavior when no timestamp was specified at all. Previously, when no timestamp was specified, it would avoid rescanning during the importmulti call, but set the key's nCreateTime value to 1, which would not prevent future block reads in later ScanForWalletTransactions calls. With this change, passing a "now" timestamp will set the key's nCreateTime to the current block time instead of 1. Fixes #9491
* | | | Merge #9707: Fix RPC failure testingMarcoFalke2017-02-102-4/+21
|\ \ \ \ | |_|/ / |/| | | | | | | 9db8eec Fix RPC failure testing (John Newbery)
| * | | Fix RPC failure testingJohn Newbery2017-02-082-4/+21
| |/ / | | | | | | | | | | | | | | | | | | Make sure that RPC tests are actually checking failures correctly by: - Catching JSON RPC exceptions and verifying the error codes and messages. - Failing the test case if the JSON RPC exception isn't raised.
* | | Merge #9532: Remove unused variablesWladimir J. van der Laan2017-02-075-6/+2
|\ \ \ | |/ / |/| | | | | | | | 90fd29b Remove unused int64_t nSinceLastSeen (practicalswift) ac4a095 Remove unused Python variables (practicalswift)
| * | Remove unused Python variablespracticalswift2017-01-135-6/+2
| | |
* | | Merge #9650: Better handle invalid parameters to signrawtransactionWladimir J. van der Laan2017-02-061-3/+31
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6dbfe08 [qa] test signrawtransaction merge with missing inputs (Matt Corallo) ec4f7e4 [qa] Add second input to signrawtransaction test case (Matt Corallo) 691710a [qa] Test that decoderawtransaction throws with extra data appended (Matt Corallo) 922bea9 Better handle invalid parameters to signrawtransaction (Matt Corallo) 7ea0ad5 Fail in DecodeHexTx if there is extra data at the end (Matt Corallo)
| * | | [qa] test signrawtransaction merge with missing inputsMatt Corallo2017-02-021-0/+16
| | | |
| * | | [qa] Add second input to signrawtransaction test caseMatt Corallo2017-02-021-3/+5
| | | |
| * | | [qa] Test that decoderawtransaction throws with extra data appendedMatt Corallo2017-02-021-0/+10
| | |/ | |/|
* / | [wallet] Set correct metadata on bumpfee wallet transactionsRussell Yanofsky2017-02-021-0/+9
|/ / | | | | | | | | | | | | | | Preserve comment, order form, and account strings from the original wallet transaction. Also set fTimeReceivedIsTxTime and fFromMe fields for consistency with CWallet::CreateTransaction. The latter two fields don't influence current wallet behavior, but do record that the transaction originated in the wallet instead of coming from the network or sendrawtransaction.
* | qa: Actually test assertions in pruning.pyMarcoFalke2017-01-261-21/+17
| | | | | | | | Also refactor to use wrapper for stop_node
* | Merge #9628: qa: Increase a sync_blocks timeout in pruning.pyMarcoFalke2017-01-261-1/+1
|\ \ | | | | | | | | | 2f10f06 qa: Increase a sync_blocks timeout in pruning.py (Suhas Daftuar)
| * | qa: Increase a sync_blocks timeout in pruning.pySuhas Daftuar2017-01-251-1/+1
| | |
* | | Merge #9516: Bug-fix: listsinceblock: use fork point as reference for blocks ↵Wladimir J. van der Laan2017-01-231-0/+80
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | in reorg'd chains 7ba0a00 Testing: listsinceblock should not use orphan block height. (Karl-Johan Alm) ee5c1ce Bug-fix: listsinceblock: use closest common ancestor when a block hash was provided for a chain that was not the main chain. (Karl-Johan Alm)
| * | | Testing: listsinceblock should not use orphan block height.Karl-Johan Alm2017-01-181-0/+80
| | | |
* | | | Merge #9607: Remove redundant semicolons in Python codeWladimir J. van der Laan2017-01-235-12/+12
|\ \ \ \ | | | | | | | | | | | | | | | 5cdf106 Remove redundant semicolons in Python code (practicalswift)
| * | | | Remove redundant semicolons in Python codepracticalswift2017-01-205-12/+12
| | | | |
* | | | | Merge #9377: fundrawtransaction: Keep change-output keys by default, make it ↵Wladimir J. van der Laan2017-01-201-1/+27
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | optional c9f3062 Add fundrawtransactions new reserveChangeKey option to the release notes (Jonas Schnelli) 9eb325d [QA] Add test for fundrawtransactions new reserveChangeKey option (Jonas Schnelli) 9aa4e6a [Wallet] Add an option to keep the change address key, true by default (Jonas Schnelli)
| * | | | [QA] Add test for fundrawtransactions new reserveChangeKey optionJonas Schnelli2017-01-191-1/+27
| | | | |