Skip to content

Commit 7734a01

Browse files
committed
Merge bitcoin/bitcoin#26640: test: Run mempool_compatibility.py with MiniWallet
fa43f60 test: Run mempool_compatibility.py with MiniWallet (MarcoFalke) Pull request description: By using the already existing miniwallet, the test can be run even when no wallet is compiled. ACKs for top commit: glozow: ACK fa43f60 achow101: ACK fa43f60 Tree-SHA512: 6877b3f2f364663f04c28ab9f3d69780de6d1b77cc862379bba8c8242bbcfb0d26eb84c56cf721141407c393f1f3b49f667ae4fb32b3566108d71250e8b5d7bc
2 parents f0c4807 + fa43f60 commit 7734a01

File tree

1 file changed

+1
-7
lines changed

1 file changed

+1
-7
lines changed

test/functional/mempool_compatibility.py

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -21,12 +21,8 @@
2121

2222

2323
class MempoolCompatibilityTest(BitcoinTestFramework):
24-
def add_options(self, parser):
25-
self.add_wallet_options(parser)
26-
2724
def set_test_params(self):
2825
self.num_nodes = 2
29-
self.wallet_names = [None]
3026

3127
def skip_test_if_missing_module(self):
3228
self.skip_if_no_previous_releases()
@@ -37,7 +33,6 @@ def setup_network(self):
3733
None,
3834
])
3935
self.start_nodes()
40-
self.import_deterministic_coinbase_privkeys()
4136

4237
def run_test(self):
4338
self.log.info("Test that mempool.dat is compatible between versions")
@@ -51,11 +46,10 @@ def run_test(self):
5146
# unbroadcasted_tx won't pass old_node's `MemPoolAccept::PreChecks`.
5247
self.connect_nodes(0, 1)
5348
self.sync_blocks()
54-
recipient = old_node.getnewaddress()
5549
self.stop_node(1)
5650

5751
self.log.info("Add a transaction to mempool on old node and shutdown")
58-
old_tx_hash = old_node.sendtoaddress(recipient, 0.0001)
52+
old_tx_hash = new_wallet.send_self_transfer(from_node=old_node)["txid"]
5953
assert old_tx_hash in old_node.getrawmempool()
6054
self.stop_node(0)
6155

0 commit comments

Comments
 (0)