fix: do not use extra_args for enforced DashTestFramework arguments

test/functional/rpc_net.py

@@ -311,7 +311,7 @@ def test_addpeeraddress(self):
         by first testing adding a tried table entry before testing adding a new table one.
         """
         self.log.info("Test addpeeraddress")
-        self.restart_node(1, self.extra_args[1] + ["-checkaddrman=1"])
+        self.restart_node(1, ["-checkaddrman=1"])
         node = self.nodes[1]
 
         self.log.debug("Test that addpeerinfo is a hidden RPC")

test/functional/test_framework/test_framework.py

@@ -43,6 +43,7 @@
 from .util import (
     PortSeed,
     MAX_NODES,
+    append_config,
     assert_equal,
     check_json_precision,
     copy_datadir,
@@ -635,6 +636,7 @@ def dynamically_initialize_datadir(self, node_p2p_port, node_rpc_port):
             f.write("upnp=0\n")
             f.write("natpmp=0\n")
             f.write("shrinkdebugfile=0\n")
+            f.write("dip3params=2:2\n")
             os.makedirs(os.path.join(new_data_dir, 'stderr'), exist_ok=True)
             os.makedirs(os.path.join(new_data_dir, 'stdout'), exist_ok=True)
 
@@ -1099,6 +1101,15 @@ def run_test(self):
         """Tests must override this method to define test logic"""
         raise NotImplementedError
 
+    def add_nodes(self, num_nodes: int, extra_args=None, *, rpchost=None, binary=None, binary_cli=None, versions=None):
+        old_num_nodes = len(self.nodes)
+        super().add_nodes(num_nodes, extra_args, rpchost=rpchost, binary=binary, binary_cli=binary_cli, versions=versions)
+        for i in range(old_num_nodes, old_num_nodes + num_nodes):
+            append_config(self.nodes[i].datadir, ["dip3params=2:2"])
+        if old_num_nodes == 0:
+            # controller node is the only node that has an extra option allowing it to submit sporks
+            append_config(self.nodes[0].datadir, ["sporkkey=cP4EKFyJsHT39LDqgdcB43Y3YXjNyjb5Fuas1GQSeAtjnZWmZEQK"])
+
     def connect_nodes(self, a, b):
         for mn2 in self.mninfo:
             if mn2.node is not None:
@@ -1119,9 +1130,6 @@ def set_dash_test_params(self, num_nodes, masterodes_count, extra_args=None, evo
             extra_args = [[]] * num_nodes
         assert_equal(len(extra_args), num_nodes)
         self.extra_args = [copy.deepcopy(a) for a in extra_args]
-        self.extra_args[0] += ["-sporkkey=cP4EKFyJsHT39LDqgdcB43Y3YXjNyjb5Fuas1GQSeAtjnZWmZEQK"]
-        for i in range(0, num_nodes):
-            self.extra_args[i].append("-dip3params=2:2")
 
         # LLMQ default test params (no need to pass -llmqtestparams)
         self.llmq_size = 3

test/functional/wallet_mnemonicbits.py

@@ -13,7 +13,6 @@ class WalletMnemonicbitsTest(BitcoinTestFramework):
     def set_test_params(self):
         self.setup_clean_chain = True
         self.num_nodes = 1
-        self.extra_args = [['-usehd=1']]
 
     def skip_test_if_missing_module(self):
         self.skip_if_no_wallet()
@@ -23,18 +22,18 @@ def run_test(self):
 
         self.log.info("Invalid -mnemonicbits should rise an error")
         self.stop_node(0)
-        self.nodes[0].assert_start_raises_init_error(self.extra_args[0] + ['-mnemonicbits=123'], "Error: Invalid '-mnemonicbits'. Allowed values: 128, 160, 192, 224, 256.")
+        self.nodes[0].assert_start_raises_init_error(['-mnemonicbits=123'], "Error: Invalid '-mnemonicbits'. Allowed values: 128, 160, 192, 224, 256.")
         self.start_node(0)
         assert_equal(len(self.nodes[0].dumphdinfo()["mnemonic"].split()), 12)  # 12 words by default
 
         self.log.info("Can have multiple wallets with different mnemonic length loaded at the same time")
-        self.restart_node(0, extra_args=self.extra_args[0] + ["-mnemonicbits=160"])
+        self.restart_node(0, extra_args=["-mnemonicbits=160"])
         self.nodes[0].createwallet("wallet_160")
-        self.restart_node(0, extra_args=self.extra_args[0] + ["-mnemonicbits=192"])
+        self.restart_node(0, extra_args=["-mnemonicbits=192"])
         self.nodes[0].createwallet("wallet_192")
-        self.restart_node(0, extra_args=self.extra_args[0] + ["-mnemonicbits=224"])
+        self.restart_node(0, extra_args=["-mnemonicbits=224"])
         self.nodes[0].createwallet("wallet_224")
-        self.restart_node(0, extra_args=self.extra_args[0] + ["-mnemonicbits=256"])
+        self.restart_node(0, extra_args=["-mnemonicbits=256"])
         self.nodes[0].loadwallet("wallet_160")
         self.nodes[0].loadwallet("wallet_192")
         self.nodes[0].loadwallet("wallet_224")