mirror of
https://github.com/dashpay/dash.git
synced 2024-12-26 12:32:48 +01:00
4e81732c57
faa137eb9eac5554504b062a6dc865ca87fd572b test: Speed up rpc_blockchain.py by removing miniwallet.generate() (MarcoFalke) fa1fe80c757df0adcbfaf41b5c5c8a468bc07b6f test: Change address type from P2PKH to P2WSH in rpc_blockchain (MarcoFalke) fa4d8f3169e38cbdbae20258efebe7070c49f522 test: Cache 25 mature coins for ADDRESS_BCRT1_P2WSH_OP_TRUE (MarcoFalke) fad25153f5c8e88f72cf666b16b0b0dbdc45d3b1 test: Remove unused bug workaround (MarcoFalke) faabce7d07c5776e4116b1a7ad1f6c408a4a4e46 test: Start only the number of nodes that are needed (MarcoFalke) Pull request description: Speed up various tests: * Remove unused nodes, which only consume time on start/stop * Remove unused "bug workarounds" * Remove the need for `miniwallet.generate()` by adding `miniwallet.scan_blocks()`. (On my system, with valgrind, generating 105 blocks takes 3.31 seconds. Rescanning 5 blocks takes 0.11 seconds.) ACKs for top commit: laanwj: Code review ACK faa137eb9eac5554504b062a6dc865ca87fd572b Tree-SHA512: ead1988d5aaa748ef9f8520af1e0bf812cf1d72e281ad22fbd172b7306d850053040526f8adbcec0b9a971c697a0ee7ee8962684644d65b791663eedd505a025
142 lines
5.8 KiB
Python
Executable File
142 lines
5.8 KiB
Python
Executable File
#!/usr/bin/env python3
|
|
# Copyright (c) 2014-2020 The Bitcoin Core developers
|
|
# Distributed under the MIT software license, see the accompanying
|
|
# file COPYING or http://www.opensource.org/licenses/mit-license.php.
|
|
"""Test the wallet accounts properly when there is a double-spend conflict."""
|
|
from decimal import Decimal
|
|
|
|
from test_framework.test_framework import BitcoinTestFramework
|
|
from test_framework.util import (
|
|
assert_equal,
|
|
find_output,
|
|
)
|
|
|
|
|
|
class TxnMallTest(BitcoinTestFramework):
|
|
def set_test_params(self):
|
|
self.num_nodes = 3
|
|
self.supports_cli = False
|
|
|
|
def skip_test_if_missing_module(self):
|
|
self.skip_if_no_wallet()
|
|
|
|
def add_options(self, parser):
|
|
parser.add_argument("--mineblock", dest="mine_block", default=False, action="store_true",
|
|
help="Test double-spend of 1-confirmed transaction")
|
|
|
|
def setup_network(self):
|
|
# Start with split network:
|
|
super().setup_network()
|
|
self.disconnect_nodes(1, 2)
|
|
|
|
def run_test(self):
|
|
# All nodes should start with 12,500 DASH:
|
|
starting_balance = 12500
|
|
|
|
# All nodes should be out of IBD.
|
|
# If the nodes are not all out of IBD, that can interfere with
|
|
# blockchain sync later in the test when nodes are connected, due to
|
|
# timing issues.
|
|
for n in self.nodes:
|
|
assert n.getblockchaininfo()["initialblockdownload"] == False
|
|
|
|
for i in range(3):
|
|
assert_equal(self.nodes[i].getbalance(), starting_balance)
|
|
|
|
# Assign coins to foo and bar addresses:
|
|
node0_address_foo = self.nodes[0].getnewaddress()
|
|
fund_foo_txid = self.nodes[0].sendtoaddress(node0_address_foo, 12190)
|
|
fund_foo_tx = self.nodes[0].gettransaction(fund_foo_txid)
|
|
|
|
node0_address_bar = self.nodes[0].getnewaddress()
|
|
fund_bar_txid = self.nodes[0].sendtoaddress(node0_address_bar, 290)
|
|
fund_bar_tx = self.nodes[0].gettransaction(fund_bar_txid)
|
|
|
|
assert_equal(self.nodes[0].getbalance(),
|
|
starting_balance + fund_foo_tx["fee"] + fund_bar_tx["fee"])
|
|
|
|
# Coins are sent to node1_address
|
|
node1_address = self.nodes[1].getnewaddress()
|
|
|
|
# First: use raw transaction API to send 12400 DASH to node1_address,
|
|
# but don't broadcast:
|
|
doublespend_fee = Decimal('-.02')
|
|
rawtx_input_0 = {}
|
|
rawtx_input_0["txid"] = fund_foo_txid
|
|
rawtx_input_0["vout"] = find_output(self.nodes[0], fund_foo_txid, 12190)
|
|
rawtx_input_1 = {}
|
|
rawtx_input_1["txid"] = fund_bar_txid
|
|
rawtx_input_1["vout"] = find_output(self.nodes[0], fund_bar_txid, 290)
|
|
inputs = [rawtx_input_0, rawtx_input_1]
|
|
change_address = self.nodes[0].getnewaddress()
|
|
outputs = {}
|
|
outputs[node1_address] = 12400
|
|
outputs[change_address] = 12480 - 12400 + doublespend_fee
|
|
rawtx = self.nodes[0].createrawtransaction(inputs, outputs)
|
|
doublespend = self.nodes[0].signrawtransactionwithwallet(rawtx)
|
|
assert_equal(doublespend["complete"], True)
|
|
|
|
# Create two spends using 1 500 DASH coin each
|
|
txid1 = self.nodes[0].sendtoaddress(node1_address, 400)
|
|
txid2 = self.nodes[0].sendtoaddress(node1_address, 200)
|
|
|
|
# Have node0 mine a block:
|
|
if (self.options.mine_block):
|
|
self.nodes[0].generate(1)
|
|
self.sync_blocks(self.nodes[0:2])
|
|
|
|
tx1 = self.nodes[0].gettransaction(txid1)
|
|
tx2 = self.nodes[0].gettransaction(txid2)
|
|
|
|
# Node0's balance should be starting balance, plus 500DASH for another
|
|
# matured block, minus 400, minus 200, and minus transaction fees:
|
|
expected = starting_balance + fund_foo_tx["fee"] + fund_bar_tx["fee"]
|
|
if self.options.mine_block:
|
|
expected += 500
|
|
expected += tx1["amount"] + tx1["fee"]
|
|
expected += tx2["amount"] + tx2["fee"]
|
|
assert_equal(self.nodes[0].getbalance(), expected)
|
|
|
|
if self.options.mine_block:
|
|
assert_equal(tx1["confirmations"], 1)
|
|
assert_equal(tx2["confirmations"], 1)
|
|
# Node1's balance should be both transaction amounts:
|
|
assert_equal(self.nodes[1].getbalance(), starting_balance - tx1["amount"] - tx2["amount"])
|
|
else:
|
|
assert_equal(tx1["confirmations"], 0)
|
|
assert_equal(tx2["confirmations"], 0)
|
|
|
|
# Now give doublespend and its parents to miner:
|
|
self.nodes[2].sendrawtransaction(fund_foo_tx["hex"])
|
|
self.nodes[2].sendrawtransaction(fund_bar_tx["hex"])
|
|
doublespend_txid = self.nodes[2].sendrawtransaction(doublespend["hex"])
|
|
# ... mine a block...
|
|
self.nodes[2].generate(1)
|
|
|
|
# Reconnect the split network, and sync chain:
|
|
self.connect_nodes(1, 2)
|
|
self.nodes[2].generate(1) # Mine another block to make sure we sync
|
|
self.sync_blocks()
|
|
assert_equal(self.nodes[0].gettransaction(doublespend_txid)["confirmations"], 2)
|
|
|
|
# Re-fetch transaction info:
|
|
tx1 = self.nodes[0].gettransaction(txid1)
|
|
tx2 = self.nodes[0].gettransaction(txid2)
|
|
|
|
# Both transactions should be conflicted
|
|
assert_equal(tx1["confirmations"], -2)
|
|
assert_equal(tx2["confirmations"], -2)
|
|
|
|
# Node0's total balance should be starting balance, plus 1000DASH for
|
|
# two more matured blocks, minus 12400 for the double-spend, plus fees (which are
|
|
# negative):
|
|
expected = starting_balance + 1000 - 12400 + fund_foo_tx["fee"] + fund_bar_tx["fee"] + doublespend_fee
|
|
assert_equal(self.nodes[0].getbalance(), expected)
|
|
|
|
# Node1's balance should be its initial balance (12500 for 25 block rewards) plus the doublespend:
|
|
assert_equal(self.nodes[1].getbalance(), 12500 + 12400)
|
|
|
|
|
|
if __name__ == '__main__':
|
|
TxnMallTest().main()
|