chore: updated default network

Signed-off-by: Manish Dait <daitmanish88@gmail.com>

Author: manishdait

src/hiero_sdk_python/client/client.py

@@ -18,6 +18,7 @@
 )
 from hiero_sdk_python.hbar import Hbar
 from hiero_sdk_python.logger.logger import Logger, LogLevel
+from hiero_sdk_python.node import _Node
 from hiero_sdk_python.transaction.transaction_id import TransactionId
 
 from .network import Network
@@ -152,6 +153,22 @@ def for_previewnet(cls) -> Client:
         """
         return cls(Network("previewnet"))
 
+    @classmethod
+    def for_network(cls, network_map: dict[str, AccountId], network_name: str | None = "localhost") -> Client:
+        """
+        Create a Client with a custom set of nodes and an optional network label.
+
+        Args:
+            network_map (dict[str, AccountId]): A map where keys are "host:port" strings
+                and values are the node's AccountId.
+            network_name (str): A label for the network. Defaults to "localhost".
+
+        Returns:
+            Client: A Client instance configured with the custom network.
+        """
+        nodes = [_Node(account_id, address, None) for address, account_id in network_map.items()]
+        return cls(Network(network=network_name, nodes=nodes))
+
     def _init_mirror_stub(self) -> None:
         """
         Connect to a mirror node for topic message subscriptions.

src/hiero_sdk_python/client/network.py

@@ -73,7 +73,7 @@ class Network:
 
     def __init__(
         self,
-        network: str = "testnet",
+        network: str | None = None,
         nodes: list[_Node] | None = None,
         mirror_address: str | None = None,
         ledger_id: bytes | None = None,
@@ -96,7 +96,7 @@ def __init__(
             Certificate verification is enabled by default for all networks.
             Use Client.set_transport_security() and Client.set_verify_certificates() to customize.
         """
-        self.network: str = network or "testnet"
+        self.network: str = network or "localhost"
         self.mirror_address: str = mirror_address or self.MIRROR_ADDRESS_DEFAULT.get(network, "localhost:5600")
 
         self.ledger_id = ledger_id or self.LEDGER_ID.get(network, bytes.fromhex("03"))
@@ -128,7 +128,8 @@ def _set_network_nodes(self, nodes: list[_Node] | None = None):
 
         # Apply TLS configuration to all nodes
         for node in final_nodes:
-            node._apply_transport_security(self._transport_security)  # pylint: disable=protected-access
+            if self._transport_security:
+                node._apply_transport_security(self._transport_security)  # pylint: disable=protected-access
             node._set_verify_certificates(self._verify_certificates)  # pylint: disable=protected-access
             node._set_root_certificates(self._root_certificates)  # pylint: disable=protected-access
 

tck/handlers/sdk.py

@@ -1,7 +1,6 @@
 from __future__ import annotations
 
-from hiero_sdk_python import AccountId, Client, Network, PrivateKey
-from hiero_sdk_python.node import _Node
+from hiero_sdk_python import AccountId, Client, PrivateKey
 from tck.handlers.registry import rpc_method
 from tck.param.base import BaseParams
 from tck.param.sdk import SetupParams
@@ -15,11 +14,11 @@ def setup_handler(params: SetupParams) -> SetupResponse:
     operator_private_key = PrivateKey.from_string(params.operatorPrivateKey)
 
     if params.nodeIp and params.nodeAccountId and params.mirrorNetworkIp:
-        client = Client()
-        client.network = Network(
-            nodes=[_Node(AccountId.from_string(params.nodeAccountId), params.nodeIp, None)],
-            mirror_address=params.mirrorNetworkIp,
-        )
+        nodes = {params.nodeIp: AccountId.from_string(params.nodeAccountId)}
+
+        client = Client.for_network(network_map=nodes)
+        client.network.mirror_address = params.mirrorNetworkIp
+
         client.set_operator(operator_account_id, operator_private_key)
 
         client_type = "custom"

tests/unit/client_test.py

@@ -10,8 +10,10 @@
 
 import pytest
 
-from hiero_sdk_python import AccountId, Client, PrivateKey
+from hiero_sdk_python import AccountId, Client
 from hiero_sdk_python.client import client as client_module
+from hiero_sdk_python.client.network import Network
+from hiero_sdk_python.crypto.private_key import PrivateKey
 from hiero_sdk_python.hbar import Hbar
 from hiero_sdk_python.node import _Node
 from hiero_sdk_python.transaction.transaction_id import TransactionId
@@ -527,3 +529,53 @@ def test_get_node_account_ids_raises_when_no_nodes():
         client.get_node_account_ids()
 
     client.close()
+
+
+def test_for_network_initializes_with_custom_map():
+    """Test for_network correctly maps strings to AccountIds and Nodes."""
+    network_map = {"127.0.0.1:50211": AccountId(0, 0, 3), "127.0.0.1:50212": AccountId(0, 0, 4)}
+
+    client = Client.for_network(network_map)
+
+    assert isinstance(client.network, Network)
+    assert len(client.network.nodes) == 2
+
+    node_accounts = [node._account_id.__str__() for node in client.network.nodes]
+    assert "0.0.3" in node_accounts
+    assert "0.0.4" in node_accounts
+
+
+@pytest.mark.parametrize("network", ["mainnet", "testnet", "previewnet"])
+def test_for_network_with_hosted_network_forces_tls(network):
+    """Test that if hosted-net the port 50211 is upgraded to 50212."""
+    network_map = {"34.94.106.61:50211": AccountId(0, 0, 3)}
+    client = Client.for_network(network_map, network_name=network)
+
+    node = client.network.nodes[0]
+    assert str(node._address) == "34.94.106.61:50212"
+    assert node._address._is_transport_security() is True
+    assert client.network.is_transport_security() is True
+
+
+@pytest.mark.parametrize("network", ["local", "localhost", "solo", "custom", None])
+def test_for_network_with_non_hosted_network_not_forces_tls(network):
+    """Test that if non hosted-net the port 50211 does not change."""
+    network_map = {"127.0.0.1:50211": AccountId(0, 0, 3)}
+    client = Client.for_network(network_map, network_name=network)
+
+    node = client.network.nodes[0]
+    assert str(node._address) == "127.0.0.1:50211"
+    assert node._address._is_transport_security() is False
+    assert client.network.is_transport_security() is False
+
+
+@pytest.mark.parametrize("network", ["local", "localhost", "solo", "custom", None])
+def test_for_network_with_non_hosted_network_not_downgrade_tls(network):
+    """Test that if non hosted-net the port 50212 does not change."""
+    network_map = {"127.0.0.1:50212": AccountId(0, 0, 3)}
+    client = Client.for_network(network_map, network_name=network)
+
+    node = client.network.nodes[0]
+    assert str(node._address) == "127.0.0.1:50212"
+    assert node._address._is_transport_security() is True
+    assert client.network.is_transport_security() is False  # Non hosted network.

tests/unit/network_test.py

@@ -449,3 +449,58 @@ def test_resolve_nodes_fallback_to_default(monkeypatch):
     assert all(isinstance(n, _Node) for n in resolved_nodes)
     assert len(resolved_nodes) == expected_count
     assert resolved_nodes[0]._account_id == network.DEFAULT_NODES[network_name][0][1]
+
+
+def test_network_default_is_local():
+    """Test that a new Network defaults to localhost and non-tls."""
+    network = Network()
+    assert network.network == "localhost"
+    assert network._transport_security is False
+
+
+@pytest.mark.parametrize("network", ["mainnet", "previewnet", "testnet"])
+def test_self_hosted_net_auto_converts_port_50211_to_50212(network):
+    """Test that self hosted port 50211 is upgraded to 50212 and TLS is enabled."""
+    node_50211 = _Node(AccountId(0, 0, 3), "34.94.106.61:50211", None)
+
+    network = Network(network=network, nodes=[node_50211])
+
+    assert ":50212" in str(network.nodes[0]._address)
+    assert network.nodes[0]._address._is_transport_security() is True
+    assert network._transport_security is True
+
+
+@pytest.mark.parametrize("network", ["mainnet", "previewnet", "testnet"])
+def test_self_hosted_network_respect_port_50212(network):
+    """Test that on self hosted network respect port 50212"""
+    node_50211 = _Node(AccountId(0, 0, 3), "127.0.0.1:50212", None)
+
+    network = Network(network=network, nodes=[node_50211])
+
+    assert ":50212" in str(network.nodes[0]._address)
+    assert network.nodes[0]._address._is_transport_security() is True
+    assert network._transport_security is True
+
+
+@pytest.mark.parametrize("network", ["local", "localhost", "solo", "custom", None])
+def test_non_hosted_network_respects_port_50211(network):
+    """Test that on non-hosted network, port 50211 stays 50211 and remains non-tls."""
+    node_50211 = _Node(AccountId(0, 0, 3), "127.0.0.1:50211", None)
+
+    network = Network(network=None, nodes=[node_50211])
+
+    assert ":50211" in str(network.nodes[0]._address)
+    assert network.nodes[0]._address._is_transport_security() is False
+    assert network._transport_security is False
+
+
+@pytest.mark.parametrize("network", ["local", "localhost", "solo", "custom", None])
+def test_non_hosted_network_respect_port_50212(network):
+    """Test that on non hosted network respect port 50212"""
+    node_50211 = _Node(AccountId(0, 0, 3), "127.0.0.1:50212", None)
+
+    network = Network(network=network, nodes=[node_50211])
+
+    assert ":50212" in str(network.nodes[0]._address)
+    assert network.nodes[0]._address._is_transport_security() is True
+    assert network._transport_security is False