highperfocused/bitcoin
1
0
Fork 0
mirror of https://github.com/bitcoin/bitcoin.git synced 2026-01-18 22:35:39 +01:00
Code Issues Packages Projects Releases Wiki Activity

wallet, rpc: show mempool conflicts in gettransaction result

Browse Source
This commit is contained in:
ishaanam
2023-07-16 21:42:33 -04:00
parent 54e07ee22f
commit 5952292133
3 changed files with 28 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
test/functional/wallet_basic.py
View File

@@ -679,7 +679,7 @@ class WalletTest(BitcoinTestFramework):
"category": baz["category"],
"vout": baz["vout"]}
expected_fields = frozenset({'amount', 'bip125-replaceable', 'confirmations', 'details', 'fee',
'hex', 'lastprocessedblock', 'time', 'timereceived', 'trusted', 'txid', 'wtxid', 'walletconflicts'})
'hex', 'lastprocessedblock', 'time', 'timereceived', 'trusted', 'txid', 'wtxid', 'walletconflicts', 'mempoolconflicts'})
verbose_field = "decoded"
expected_verbose_fields = expected_fields | {verbose_field}

19
test/functional/wallet_conflicts.py
View File

@@ -178,6 +178,8 @@ class TxConflicts(BitcoinTestFramework):
assert_equal(alice.listunspent(), [unspents[0]])
assert_equal(alice.getbalance(), 25)
assert_equal(alice.gettransaction(tx1_txid)["mempoolconflicts"], [tx2_txid])
self.log.info("Test scenario where a mempool conflict is removed")
# broadcast tx3, replaces tx2 in mempool
@@ -187,6 +189,7 @@ class TxConflicts(BitcoinTestFramework):
# tx1 is no longer conflicted.
alice.sendrawtransaction(tx3)
assert_equal(alice.gettransaction(tx1_txid)["mempoolconflicts"], [])
assert tx1_txid not in self.nodes[0].getrawmempool()
# now all of alice's outputs should be considered spent
@@ -262,6 +265,10 @@ class TxConflicts(BitcoinTestFramework):
assert tx2_txid in bob.getrawmempool()
assert tx1_conflict_txid in bob.getrawmempool()
assert_equal(bob.gettransaction(tx1_txid)["mempoolconflicts"], [tx1_conflict_txid])
assert_equal(bob.gettransaction(tx2_txid)["mempoolconflicts"], [])
assert_equal(bob.gettransaction(tx3_txid)["mempoolconflicts"], [tx1_conflict_txid])
# check that tx3 is now conflicted, so the output from tx2 can now be spent
assert_equal(bob.getbalances()["mine"]["untrusted_pending"], Decimal("24.99990000"))
@@ -348,6 +355,8 @@ class TxConflicts(BitcoinTestFramework):
assert_equal(alice.getbalance(), 25)
assert_equal(bob.getbalances()["mine"]["untrusted_pending"], Decimal("24.99990000"))
assert_equal(bob.gettransaction(tx1_txid)["mempoolconflicts"], [])
raw_tx = bob.createrawtransaction(inputs=[bob.listunspent(minconf=0)[0]], outputs=[{carol.getnewaddress() : 24.999}])
# Bob creates a child to tx1
tx1_child = bob.signrawtransactionwithwallet(raw_tx)['hex']
@@ -356,6 +365,8 @@ class TxConflicts(BitcoinTestFramework):
self.sync_mempools()
# Currently neither tx1 nor tx1_child should have any conflicts
assert_equal(bob.gettransaction(tx1_txid)["mempoolconflicts"], [])
assert_equal(bob.gettransaction(tx1_child_txid)["mempoolconflicts"], [])
assert tx1_txid in bob.getrawmempool()
assert tx1_child_txid in bob.getrawmempool()
assert_equal(len(bob.getrawmempool()), 2)
@@ -378,6 +389,10 @@ class TxConflicts(BitcoinTestFramework):
assert tx1_conflict_txid in bob.getrawmempool()
assert_equal(len(bob.getrawmempool()), 1)
# Now both tx1 and tx1_child are conflicted by tx1_conflict
assert_equal(bob.gettransaction(tx1_txid)["mempoolconflicts"], [tx1_conflict_txid])
assert_equal(bob.gettransaction(tx1_child_txid)["mempoolconflicts"], [tx1_conflict_txid])
# Now create a conflict to tx1_conflict, so that it gets kicked out of the mempool
raw_tx = alice.createrawtransaction(inputs=[unspents[1]], outputs=[{carol.getnewaddress() : 24.9895}])
tx1_conflict_conflict = alice.signrawtransactionwithwallet(raw_tx)['hex']
@@ -385,6 +400,10 @@ class TxConflicts(BitcoinTestFramework):
self.sync_mempools()
# Now that tx1_conflict has been removed, both tx1 and tx1_child
assert_equal(bob.gettransaction(tx1_txid)["mempoolconflicts"], [])
assert_equal(bob.gettransaction(tx1_child_txid)["mempoolconflicts"], [])
# Both tx1 and tx1_child are still not in the mempool because they have not be re-broadcasted
assert tx1_txid not in bob.getrawmempool()
assert tx1_child_txid not in bob.getrawmempool()