aboutsummaryrefslogtreecommitdiff
path: root/test/functional/p2p_invalid_tx.py
Commit message (Collapse)AuthorAgeFilesLines
* 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.
* scripted-diff: replace p2p with p2ps[0] in p2p_invalid_txgzhao4082020-09-101-6/+6
| | | | | | -BEGIN VERIFY SCRIPT- sed -i 's/\.p2p\./.p2ps[0]./g' test/functional/p2p_invalid_tx.py -END VERIFY SCRIPT-
* test: Update wait_until usage in tests not to use the one from utilsSeleme Topuz2020-08-261-2/+1
| | | | | Replace "wait_until()" usage from utils, with the ones from BitcoinTestFramework and P2PInterface. closes #19080
* scripted-diff: Rename mininode to p2pJohn Newbery2020-08-211-1/+1
| | | | | | | -BEGIN VERIFY SCRIPT- sed -i 's/\.mininode/\.p2p/g' $(git grep -l "mininode") git mv test/functional/test_framework/mininode.py test/functional/test_framework/p2p.py -END VERIFY SCRIPT-
* scripted-diff: Bump copyright headersMarcoFalke2020-04-161-1/+1
| | | | | | -BEGIN VERIFY SCRIPT- ./contrib/devtools/copyright_header.py update ./ -END VERIFY SCRIPT-
* test: Add various low-level p2p testsMarcoFalke2020-04-131-0/+17
|
* 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: Require standard txs in regtestMarcoFalke2019-06-211-0/+3
|
* test: add invalid tx templates for use in functional testsJames O'Beirne2018-11-271-7/+17
| | | | | Add templates for easily constructing different kinds of invalid transactions and use them in feature_block and p2p_invalid_tx.
* Changed functional tests which do not require wallets to run withoutsanket17292018-09-171-4/+1
| | | | | | skipping .Addreses #14216. Changed get_deterministic_priv_key() to a named tuple
* Merge #14180: qa: Run all tests even if wallet is not compiledWladimir J. van der Laan2018-09-131-0/+3
|\ | | | | | | | | | | | | | | | | | | | | fac95398366f644911b58f1605e6bc37fb76782d qa: Run all tests even if wallet is not compiled (MarcoFalke) faa669cbcd1fc799517b523b0f850e01b11bf40a qa: Premine to deterministic address with -disablewallet (MarcoFalke) Pull request description: Currently the test_runner would exit if the wallet was not compiled into the Bitcoin Core executable. However, a lot of the tests run without the wallet just fine and there is no need to globally require the wallet to run the tests. Tree-SHA512: 63177260aa29126fd20f0be217a82b10b62288ab846f96f1cbcc3bd2c52702437703475d91eae3f8d821a3149fc62b725a4c5b2a7b3657b67ffcbc81532a03bb
| * qa: Run all tests even if wallet is not compiledMarcoFalke2018-09-101-0/+3
| |
* | Merge #14119: qa: Read reject reasons from debug log, not p2p messagesWladimir J. van der Laan2018-09-081-13/+2
|\ \ | |/ |/| | | | | | | | | | | | | | | | | | | fac3e22b18cd29053bc17065fd75db7b84ba6f40 qa: Read reject reasons from debug log, not p2p messages (MarcoFalke) Pull request description: For local testing we don't need to rely on p2p messages just to assert a reject reason. Replace reading p2p messages with reading from the debug log file. Tree-SHA512: fa59598ecf5e00cfb420ef1892d90aa415501fd882e1c608894dc577b0d00e93a442326d3a9167fef77d26aafbe345b730b49109982ccad68a5942384564a90b
| * qa: Read reject reasons from debug log, not p2p messagesMarcoFalke2018-08-311-13/+2
| |
* | Fix typos reported by codespellpracticalswift2018-09-041-1/+1
|/
* qa: Add TestNode::assert_debug_logMarcoFalke2018-08-241-2/+7
|
* Merge #13669: Tests: Cleanup create_transaction implementationsMarcoFalke2018-08-091-2/+2
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 44bbceeef1 [Tests] Cleanup feature_block.py, remove unnecessary PreviousSpendableOutput object (Conor Scott) 736f941424 [Tests] Cleanup extra instances of create_transaction (Conor Scott) 157651855f [Tests] Rename create_tx and move to blocktools.py (Conor Scott) Pull request description: There currently exist seven ([1](https://github.com/bitcoin/bitcoin/blob/master/test/functional/feature_cltv.py#L52-L60), [2](https://github.com/bitcoin/bitcoin/blob/master/test/functional/feature_csv_activation.py#L88-L95) [3](https://github.com/bitcoin/bitcoin/blob/master/test/functional/feature_dersig.py#L40-L48), [4](https://github.com/bitcoin/bitcoin/blob/master/test/functional/feature_nulldummy.py#L100-L108), [5](https://github.com/bitcoin/bitcoin/blob/master/test/functional/test_framework/util.py#L529-L535), [6](https://github.com/bitcoin/bitcoin/blob/master/test/functional/test_framework/blocktools.py#L120-L129), [7](https://github.com/bitcoin/bitcoin/blob/master/test/functional/feature_block.py#L1218-L1220)) implementations of a function called something similar to `create_transaction` in the functional tests, some of which are exact copies of each other. This PR aims to clean this up into [three different cases implemented in blocktools.py](https://github.com/conscott/bitcoin/blob/create_tx_cleanup/test/functional/test_framework/blocktools.py#L121-L149) 1. `create_tx_with_script`: Return transaction object spending generic tx output optionally specifying scriptSig and scriptPubKey 2. `create_transaction`: Return transaction object spending coinbase tx 2. `create_raw_transaction`: Return raw transaction (hex string) spending coinbase tx I am not committed to any of these function names, so I'll gladly take suggestions on there. Additionally there are some related cleanups to feature_block.py tests, specifically removing the [PreviousSpendableOutput](https://github.com/conscott/bitcoin/blob/master/test/functional/feature_block.py#L51-L54) object, which seems like an unnecessary layer given that every instance spends the 0 output. Tree-SHA512: 63c6233b6f0942c81ba1ca67ea6770809b8c9409314c6d4cf8e5a3991cb9ee92b22bebe88c0dde45cd71e754eb351230c4c404b70ff118f5f43c034452ada65c
| * [Tests] Cleanup extra instances of create_transactionConor Scott2018-08-091-2/+2
| |
* | Merge #13916: qa: wait_for_verack by defaultMarcoFalke2018-08-091-1/+0
|\ \ | |/ |/| | | | | | | | | | | | | | | fa5587fe71 qa: wait_for_verack by default (MarcoFalke) Pull request description: This removes the need to do so manually every time a connection is added. Tree-SHA512: a46c92cb4df41e30778b42b9fd3dcbd8d2d82aa7503d1213cb1c1165034f648d8caee01c292e2d87d05b0f71696996eef5be8a753f35ab49e5f66b0e3bf29f21
| * qa: wait_for_verack by defaultMarcoFalke2018-08-081-1/+0
| |
* | Update copyright headers to 2018DrahtBot2018-07-271-1/+1
|/
* qa: Avoid start/stop of the network thread mid-testMarcoFalke2018-06-251-3/+1
|
* qa: Avoid checking reject code for nowMarcoFalke2018-05-301-3/+1
| | | | | The node will often disconnect before sending a reject code. A more robust solution would be to read from the debug log.
* Merge #13069: docs: Fix typosMarcoFalke2018-05-301-1/+1
|\ | | | | | | | | | | | | | | | | | | d8c4998f31 Fix typos (practicalswift) Pull request description: Fix typos. Tree-SHA512: 9af52a9799e6892b162e4aa1bcd6585502e10650b8aced59e7346dbb2f08544330081eb79328255fad1d358c095507956e049d354c4383b6965d4d5a7d635425
| * Fix typospracticalswift2018-05-071-1/+1
| |
* | net: Add option `-enablebip61` to configure sending of BIP61 notificationsWladimir J. van der Laan2018-05-131-1/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit adds a boolean option `-enablebip61`, defaulting to `1`, that can be used to disable the sending of BIP61 `reject` messages. This functionality has been requested for various reasons: - security (DoS): reject messages can reveal internal state that can be used to target certain resources such as the mempool more easily. - bandwidth: a typical node sends lots of reject messages; this counts against upstream bandwidth. Also the reject messages tend to be larger than the message that was rejected. On the other hand, reject messages can be useful while developing client software (I found them indispensable while creating bitcoin-submittx), as well as for our own test cases, so whatever the default becomes on the long run, IMO the functionality should be retained as option. But that's a discussion for later.
* | qa: Pad scriptPubKeys to get minimum sized txsMarcoFalke2018-05-051-6/+7
|/
* qa: Add test for orphan handlingMarcoFalke2018-04-161-12/+97
|
* [test] Fix nits leftover from 11771Conor Scott2018-02-131-3/+4
| | | | | Remove unused variable reassignments in p2p_invalid_tx.py and call send_txs_and_test() with valid transaction.
* [tests] Change invalidtxrequest to use BitcoinTestFrameworkJohn Newbery2018-01-301-30/+21
|
* [tests] Fix flake8 warnings in invalidtxrequestJohn Newbery2018-01-301-19/+10
|
* [tests] Rename p2p_* functional tests.Anthony Towns2018-01-251-0/+73
generated by cgit v1.2.3 (git 2.25.1) at 2026-04-21 14:33:50 +0000