Skip to content

Commit fa7f65b

Browse files
author
MarcoFalke
committed
test: Use clean chain in MempoolCompatibilityTest
The test creates enough blocks itself, so there is no need to have more. Also, remove os import.
1 parent e35fb7b commit fa7f65b

File tree

1 file changed

+5
-6
lines changed

1 file changed

+5
-6
lines changed

test/functional/mempool_compatibility.py

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,6 @@
1010
Previous releases are required by this test, see test/README.md.
1111
"""
1212

13-
import os
14-
1513
from test_framework.blocktools import COINBASE_MATURITY
1614
from test_framework.test_framework import BitcoinTestFramework
1715
from test_framework.wallet import (
@@ -23,6 +21,7 @@
2321
class MempoolCompatibilityTest(BitcoinTestFramework):
2422
def set_test_params(self):
2523
self.num_nodes = 2
24+
self.setup_clean_chain = True
2625

2726
def skip_test_if_missing_module(self):
2827
self.skip_if_no_previous_releases()
@@ -55,9 +54,9 @@ def run_test(self):
5554
self.stop_node(1)
5655

5756
self.log.info("Move mempool.dat from old to new node")
58-
old_node_mempool = os.path.join(old_node.chain_path, 'mempool.dat')
59-
new_node_mempool = os.path.join(new_node.chain_path, 'mempool.dat')
60-
os.rename(old_node_mempool, new_node_mempool)
57+
old_node_mempool = old_node.chain_path / "mempool.dat"
58+
new_node_mempool = new_node.chain_path / "mempool.dat"
59+
old_node_mempool.rename(new_node_mempool)
6160

6261
self.log.info("Start new node and verify mempool contains the tx")
6362
self.start_node(1)
@@ -70,7 +69,7 @@ def run_test(self):
7069
self.stop_node(1)
7170

7271
self.log.info("Move mempool.dat from new to old node")
73-
os.rename(new_node_mempool, old_node_mempool)
72+
new_node_mempool.rename(old_node_mempool)
7473

7574
self.log.info("Start old node again and verify mempool contains both txs")
7675
self.start_node(0, ['-nowallet'])

0 commit comments

Comments
 (0)