Start/stop miner for replaceTransaction_already_mined test

Author: kclowes

tests/generate_go_ethereum_fixture.py

@@ -26,7 +26,7 @@
     valmap,
 )
 
-from tests.utils import (
+from utils import (
     get_open_port,
 )
 from web3 import Web3

tests/integration/go_ethereum/test_goethereum_http.py

@@ -41,7 +41,7 @@ def geth_command_arguments(rpc_port,
                 (
                     '--rpc',
                     '--rpcport', rpc_port,
-                    '--rpcapi', 'admin,db,eth,net,web3,personal,shh,web3',
+                    '--rpcapi', 'admin,db,eth,net,web3,personal,shh,miner',
                     '--ipcdisable',
                     '--allow-insecure-unlock',
                 )
@@ -52,7 +52,7 @@ def geth_command_arguments(rpc_port,
                 (
                     '--rpc',
                     '--rpcport', rpc_port,
-                    '--rpcapi', 'admin,db,eth,net,web3,personal,shh,web3',
+                    '--rpcapi', 'admin,db,eth,net,web3,personal,shh,miner',
                     '--ipcdisable',
                 )
             )

tests/integration/go_ethereum/test_goethereum_ws.py

@@ -44,7 +44,7 @@ def geth_command_arguments(geth_binary,
                 (
                     '--ws',
                     '--wsport', ws_port,
-                    '--wsapi', 'admin,db,eth,net,shh,web3,personal,web3',
+                    '--wsapi', 'admin,db,eth,net,shh,web3,personal,miner',
                     '--wsorigins', '*',
                     '--ipcdisable',
                     '--allow-insecure-unlock',
@@ -56,7 +56,7 @@ def geth_command_arguments(geth_binary,
                 (
                     '--ws',
                     '--wsport', ws_port,
-                    '--wsapi', 'admin,db,eth,net,shh,web3,personal,web3',
+                    '--wsapi', 'admin,db,eth,net,shh,web3,personal,miner',
                     '--wsorigins', '*',
                     '--ipcdisable',
                 )

web3/_utils/module_testing/eth_module.py

@@ -463,10 +463,11 @@ def test_eth_replaceTransaction_non_existing_transaction(
                 txn_params
             )
 
-    # auto mine is enabled for this test
     def test_eth_replaceTransaction_already_mined(
         self, web3: "Web3", unlocked_account_dual_type: ChecksumAddress
     ) -> None:
+        if 'v1.9.7' in web3.clientVersion:
+            web3.geth.miner.start()
         txn_params: TxParams = {
             'from': unlocked_account_dual_type,
             'to': unlocked_account_dual_type,
@@ -475,10 +476,13 @@ def test_eth_replaceTransaction_already_mined(
             'gasPrice': web3.eth.gasPrice,
         }
         txn_hash = web3.eth.sendTransaction(txn_params)
+        web3.eth.waitForTransactionReceipt(txn_hash)
 
         txn_params['gasPrice'] = Wei(web3.eth.gasPrice * 2)
         with pytest.raises(ValueError, match="Supplied transaction with hash"):
             web3.eth.replaceTransaction(txn_hash, txn_params)
+        if 'v1.9.7' in web3.clientVersion:
+            web3.geth.miner.stop()
 
     def test_eth_replaceTransaction_incorrect_nonce(
         self, web3: "Web3", unlocked_account: ChecksumAddress