test: add functional test for addresses and deprecated service field

Author: kwvg

test/functional/rpc_netinfo.py

@@ -0,0 +1,257 @@
+#!/usr/bin/env python3
+# Copyright (c) 2025 The Dash Core developers
+# Distributed under the MIT software license, see the accompanying
+# file COPYING or http://www.opensource.org/licenses/mit-license.php.
+"""Test network information fields across RPCs."""
+
+from test_framework.util import (
+    assert_equal
+)
+from test_framework.script import (
+    hash160
+)
+from test_framework.test_framework import (
+    BitcoinTestFramework,
+    MasternodeInfo,
+    p2p_port
+)
+from test_framework.test_node import TestNode
+
+from _decimal import Decimal
+from random import randint
+
+# See CRegTestParams in src/chainparams.cpp
+DEFAULT_PORT_PLATFORM_P2P = 22200
+DEFAULT_PORT_PLATFORM_HTTP = 22201
+
+class Node:
+    mn: MasternodeInfo
+    node: TestNode
+    platform_nodeid: str = ""
+
+    def __init__(self, node: TestNode, is_evo: bool):
+        self.mn = MasternodeInfo(evo=is_evo, legacy=False)
+        self.mn.generate_addresses(node)
+        self.mn.set_node(node.index)
+        self.mn.set_params(nodePort=p2p_port(node.index))
+        self.node = node
+
+    def generate_collateral(self, test: BitcoinTestFramework):
+        assert self.mn.nodeIdx is not None
+
+        # Generate enough coins if we don't have them
+        while self.node.getbalance() < self.mn.get_collateral_value():
+            test.bump_mocktime(1)
+            test.generate(self.node, 10, sync_fun=test.no_op)
+
+        # Create collateral transaction
+        collateral_txid = self.node.sendmany("", {self.mn.collateral_address: self.mn.get_collateral_value(), self.mn.fundsAddr: 1})
+        self.mn.bury_tx(test, self.mn.nodeIdx, collateral_txid, 1)
+        collateral_vout = self.mn.get_collateral_vout(self.node, collateral_txid)
+        self.mn.set_params(collateral_txid=collateral_txid, collateral_vout=collateral_vout)
+
+    def is_mn_visible(self, _protx_hash = None) -> bool:
+        protx_hash = _protx_hash or self.mn.proTxHash
+        mn_list = self.node.masternodelist()
+        mn_visible = False
+        for mn_entry in mn_list:
+            dmn = mn_list.get(mn_entry)
+            if dmn['proTxHash'] == protx_hash:
+                assert_equal(dmn['type'], "Evo" if self.mn.evo else "Regular")
+                mn_visible = True
+        return mn_visible
+
+    def register_mn(self, test: BitcoinTestFramework, submit: bool, addrs_core_p2p, addrs_platform_p2p = None, addrs_platform_http = None) -> str:
+        assert self.mn.nodeIdx is not None
+
+        if self.mn.evo and (not addrs_platform_http or not addrs_platform_p2p):
+            raise AssertionError("EvoNode but platformP2PPort and platformHTTPPort not specified")
+
+        # Evonode-specific fields are ignored if regular masternode
+        self.platform_nodeid = hash160(b'%d' % randint(1, 65535)).hex()
+        protx_output = self.mn.register(self.node, submit=submit, coreP2PAddrs=addrs_core_p2p, operator_reward=0,
+                                        platform_node_id=self.platform_nodeid, platform_p2p_port=addrs_platform_p2p,
+                                        platform_http_port=addrs_platform_http)
+        assert protx_output is not None
+        self.mn.set_params(proTxHash=protx_output, operator_reward=0)
+
+        if not submit:
+            return ""
+
+        # Bury ProTx transaction and check if masternode is online
+        self.mn.bury_tx(test, self.mn.nodeIdx, protx_output, 1)
+        assert_equal(self.is_mn_visible(), True)
+
+        # Update MasternodeInfo
+        test.log.debug(f"Registered {'Evo' if self.mn.evo else 'regular'} masternode with collateral_txid={self.mn.collateral_txid}, "
+                       f"collateral_vout={self.mn.collateral_vout}, provider_txid={self.mn.proTxHash}")
+
+        # Restart masternode to behave like a full masternode
+        test.restart_node(self.mn.nodeIdx, extra_args=self.node.extra_args + [f'-masternodeblsprivkey={self.mn.keyOperator}'])
+        return self.mn.proTxHash
+
+    def update_mn(self, test: BitcoinTestFramework, addrs_core_p2p, addrs_platform_p2p = None, addrs_platform_http = None) -> str:
+        assert self.mn.nodeIdx is not None
+
+        if self.mn.evo and (not addrs_platform_http or not addrs_platform_p2p):
+            raise AssertionError("EvoNode but platformP2PPort and platformHTTPPort not specified")
+
+        # Evonode-specific fields are ignored if regular masternode
+        protx_output = self.mn.update_service(self.node, submit=True, coreP2PAddrs=addrs_core_p2p, platform_node_id=self.platform_nodeid,
+                                              platform_p2p_port=addrs_platform_p2p, platform_http_port=addrs_platform_http)
+        assert protx_output is not None
+
+        # Bury ProTx transaction and check if masternode is online
+        self.mn.bury_tx(test, self.mn.nodeIdx, protx_output, 1)
+        assert_equal(self.is_mn_visible(), True)
+
+        test.log.debug(f"Updated {'Evo' if self.mn.evo else 'regular'} masternode with collateral_txid={self.mn.collateral_txid}, "
+                       f"collateral_vout={self.mn.collateral_vout}, provider_txid={self.mn.proTxHash}")
+        return protx_output
+
+    def destroy_mn(self, test: BitcoinTestFramework):
+        assert self.mn.nodeIdx is not None
+
+        # Get UTXO from address used to pay fees
+        address_funds_unspent = self.node.listunspent(0, 99999, [self.mn.fundsAddr])[0]
+        address_funds_value = address_funds_unspent['amount']
+
+        # Reserve new address for collateral and fee spending
+        new_address_collateral = self.node.getnewaddress()
+        new_address_funds = self.node.getnewaddress()
+
+        # Create transaction to spend old collateral and fee change
+        raw_tx = self.node.createrawtransaction([
+                { 'txid': self.mn.collateral_txid, 'vout': self.mn.collateral_vout },
+                { 'txid': address_funds_unspent['txid'], 'vout': address_funds_unspent['vout'] }
+            ], [
+                {new_address_collateral: float(self.mn.get_collateral_value())},
+                {new_address_funds: float(address_funds_value - Decimal(0.001))}
+            ])
+        raw_tx = self.node.signrawtransactionwithwallet(raw_tx)['hex']
+
+        # Send that transaction, resulting txid is new collateral
+        new_collateral_txid = self.node.sendrawtransaction(raw_tx)
+        self.mn.bury_tx(test, self.mn.nodeIdx, new_collateral_txid, 1)
+        new_collateral_vout = self.mn.get_collateral_vout(self.node, new_collateral_txid)
+        self.mn.set_params(collateral_txid=new_collateral_txid, collateral_vout=new_collateral_vout)
+
+        # Old masternode entry should be dead
+        assert_equal(self.is_mn_visible(self.mn.proTxHash), False)
+        test.log.debug(f"Destroyed {'Evo' if self.mn.evo else 'regular'} masternode with collateral_txid={self.mn.collateral_txid}, "
+                       f"collateral_vout={self.mn.collateral_vout}, provider_txid={self.mn.proTxHash}")
+
+        # Generate fresh addresses (and overwrite some of them with addresses used here)
+        self.mn.generate_addresses(self.node, True)
+        self.collateral_address = new_address_collateral
+        self.fundsAddr = new_address_funds
+        self.collateral_txid = new_collateral_txid
+        self.collateral_vout = new_collateral_vout
+        self.proTxHash = ""
+
+        # Restart node sans masternodeblsprivkey
+        test.restart_node(self.mn.nodeIdx, extra_args=self.node.extra_args)
+
+class NetInfoTest(BitcoinTestFramework):
+    def set_test_params(self):
+        self.num_nodes = 2
+        self.extra_args = [
+            ["-dip3params=2:2"],
+            ["-deprecatedrpc=service", "-dip3params=2:2"]
+        ]
+
+    def skip_test_if_missing_module(self):
+        self.skip_if_no_wallet()
+
+    def check_netinfo_fields(self, val, core_p2p_port: int):
+        assert_equal(val[0], f"127.0.0.1:{core_p2p_port}")
+
+    def run_test(self):
+        self.node_evo: Node = Node(self.nodes[0], True)
+        self.node_simple: TestNode = self.nodes[1]
+
+        self.node_evo.generate_collateral(self)
+
+        # netInfo is represented with JSON in CProRegTx, CProUpServTx, CDeterministicMNState and CSimplifiedMNListEntry,
+        # so we need to test calls that rely on these underlying implementations. Start by collecting RPC responses.
+        self.log.info("Collect JSON RPC responses from node")
+
+        # CProRegTx::ToJson() <- TxToUniv() <- TxToJSON() <- getrawtransaction
+        proregtx_hash = self.node_evo.register_mn(self, True, f"127.0.0.1:{self.node_evo.mn.nodePort}", DEFAULT_PORT_PLATFORM_P2P, DEFAULT_PORT_PLATFORM_HTTP)
+        proregtx_rpc = self.node_evo.node.getrawtransaction(proregtx_hash, True)
+
+        # CDeterministicMNState::ToJson() <- CDeterministicMN::pdmnState <- masternode_status
+        masternode_status = self.node_evo.node.masternode('status')
+
+        # Generate deprecation-disabled response to avoid having to re-create a masternode again later on
+        self.restart_node(self.node_evo.mn.nodeIdx, extra_args=self.node_evo.node.extra_args +
+                          [f'-masternodeblsprivkey={self.node_evo.mn.keyOperator}', '-deprecatedrpc=service'])
+        self.connect_nodes(self.node_evo.mn.nodeIdx, self.node_simple.index) # Needed as restarts don't reconnect nodes
+        masternode_status_depr = self.node_evo.node.masternode('status')
+
+        # Stop actively running the masternode so we can issue a CProUpServTx (and enable the deprecation)
+        self.restart_node(self.node_evo.mn.nodeIdx, extra_args=self.node_evo.node.extra_args)
+        self.connect_nodes(self.node_evo.mn.nodeIdx, self.node_simple.index) # Needed as restarts don't reconnect nodes
+
+        # CProUpServTx::ToJson() <- TxToUniv() <- TxToJSON() <- getrawtransaction
+        proupservtx_hash = self.node_evo.update_mn(self, f"127.0.0.1:{self.node_evo.mn.nodePort+1}", DEFAULT_PORT_PLATFORM_P2P, DEFAULT_PORT_PLATFORM_HTTP)
+        proupservtx_rpc = self.node_evo.node.getrawtransaction(proupservtx_hash, True)
+
+        # We need to update *twice*, the first time to incorrect values and the second time, back to correct values.
+        # This is to make sure that the fields we need to check against are reflected in the diff.
+        proupservtx_hash = self.node_evo.update_mn(self, f"127.0.0.1:{self.node_evo.mn.nodePort}", DEFAULT_PORT_PLATFORM_P2P, DEFAULT_PORT_PLATFORM_HTTP)
+        proupservtx_rpc = self.node_evo.node.getrawtransaction(proupservtx_hash, True)
+
+        # CSimplifiedMNListEntry::ToJson() <- CSimplifiedMNListDiff::mnList <- CSimplifiedMNListDiff::ToJson() <- protx_diff
+        masternode_active_height: int = masternode_status['dmnState']['registeredHeight']
+        protx_diff_rpc = self.node_evo.node.protx('diff', masternode_active_height - 1, masternode_active_height)
+
+        # CDeterministicMNStateDiff::ToJson() <- CDeterministicMNListDiff::updatedMns <- protx_listdiff
+        proupservtx_height = proupservtx_rpc['height']
+        protx_listdiff_rpc = self.node_evo.node.protx('listdiff', proupservtx_height - 1, proupservtx_height)
+
+        self.log.info("Test RPCs return an 'addresses' field")
+        assert "addresses" in proregtx_rpc['proRegTx'].keys()
+        assert "addresses" in masternode_status['dmnState'].keys()
+        assert "addresses" in proupservtx_rpc['proUpServTx'].keys()
+        assert "addresses" in protx_diff_rpc['mnList'][0].keys()
+        assert "addresses" in protx_listdiff_rpc['updatedMNs'][0][proregtx_hash].keys()
+
+        self.log.info("Test 'addresses' report correctly")
+        self.check_netinfo_fields(proregtx_rpc['proRegTx']['addresses'], self.node_evo.mn.nodePort)
+        self.check_netinfo_fields(masternode_status['dmnState']['addresses'], self.node_evo.mn.nodePort)
+        self.check_netinfo_fields(proupservtx_rpc['proUpServTx']['addresses'], self.node_evo.mn.nodePort)
+        self.check_netinfo_fields(protx_diff_rpc['mnList'][0]['addresses'], self.node_evo.mn.nodePort)
+        self.check_netinfo_fields(protx_listdiff_rpc['updatedMNs'][0][proregtx_hash]['addresses'], self.node_evo.mn.nodePort)
+
+        self.log.info("Test RPCs by default no longer return a 'service' field")
+        assert "service" not in proregtx_rpc['proRegTx'].keys()
+        assert "service" not in masternode_status['dmnState'].keys()
+        assert "service" not in proupservtx_rpc['proUpServTx'].keys()
+        assert "service" not in protx_diff_rpc['mnList'][0].keys()
+        assert "service" not in protx_listdiff_rpc['updatedMNs'][0][proregtx_hash].keys()
+        # "service" in "masternode status" is exempt from the deprecation as the primary address is
+        # relevant on the host node as opposed to expressing payload information in most other RPCs.
+        assert "service" in masternode_status.keys()
+
+        self.node_evo.destroy_mn(self) # Shut down previous masternode
+        self.connect_nodes(self.node_evo.mn.nodeIdx, self.node_simple.index) # Needed as restarts don't reconnect nodes
+
+        self.log.info("Collect RPC responses from node with -deprecatedrpc=service")
+
+        # Re-use chain activity from earlier
+        proregtx_rpc = self.node_simple.getrawtransaction(proregtx_hash, True)
+        proupservtx_rpc = self.node_simple.getrawtransaction(proupservtx_hash, True)
+        protx_diff_rpc = self.node_simple.protx('diff', masternode_active_height - 1, masternode_active_height)
+        masternode_status = masternode_status_depr # Pull in response generated from earlier
+        protx_listdiff_rpc = self.node_simple.protx('listdiff', proupservtx_height - 1, proupservtx_height)
+
+        self.log.info("Test RPCs return 'service' with -deprecatedrpc=service")
+        assert "service" in proregtx_rpc['proRegTx'].keys()
+        assert "service" in masternode_status['dmnState'].keys()
+        assert "service" in proupservtx_rpc['proUpServTx'].keys()
+        assert "service" in protx_diff_rpc['mnList'][0].keys()
+        assert "service" in protx_listdiff_rpc['updatedMNs'][0][proregtx_hash].keys()
+
+if __name__ == "__main__":
+    NetInfoTest().main()

test/functional/test_runner.py

@@ -233,6 +233,7 @@
     'p2p_addrfetch.py',
     'rpc_net.py --v1transport',
     'rpc_net.py --v2transport',
+    'rpc_netinfo.py',
     'wallet_keypool.py --legacy-wallet',
     'wallet_keypool_hd.py --legacy-wallet',
     'wallet_keypool_hd.py --descriptors',