chore(tests): format node unit tests with black (#1615)

Signed-off-by: SubhraSameerDash <[email protected]>
Signed-off-by: Subhra Sameer Dash <[email protected]>

Author: Subhrasameerdash

CHANGELOG.md

@@ -17,6 +17,7 @@ This changelog is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.
 - Style: formatted `tests/unit/prng_transaction_test.py` with black (#1546)
 - Formatted contract unit tests with black for consistent style. (#1523)
 - Format account test files with Black (#1519)
+- Format `tests/unit/node*.py` with Black for consistent code style (#1545)
 - Improve unit test coverage for Hbar, including edge cases, validation, comparisons, and hashing. (#1483)
 - Standardize formatting of evm_address_test.py using Black for improved consistency and readability (#1529)
 - Formatted unit test files using Black.
@@ -217,7 +218,6 @@ This changelog is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.
 - Enhance TopicInfo `__str__` method and tests with additional coverage, and update the format_key function in `key_format.py` to handle objects with a \_to_proto method.
 - Update changelog workflow to trigger automatically on pull requests instead of manual dispatch (#1567)
 - Formatted key-related unit test files (`key_utils_test.py`, `test_key_format.py`, `test_key_list.py`) using the black formatter
-- chore: update maintainer guidelines link in MAINTAINERS.md (#1605)
 
 ### Fixed
 

tests/unit/node_address_test.py

@@ -3,26 +3,34 @@
 from hiero_sdk_python.account.account_id import AccountId
 from hiero_sdk_python.address_book.endpoint import Endpoint
 from hiero_sdk_python.address_book.node_address import NodeAddress
-from hiero_sdk_python.hapi.services.basic_types_pb2 import NodeAddress as NodeAddressProto
+from hiero_sdk_python.hapi.services.basic_types_pb2 import (
+    NodeAddress as NodeAddressProto,
+)
+
 pytestmark = pytest.mark.unit
 
+
 def test_init():
     """Test initialization of _NodeAddress."""
     # Create test data
     account_id = AccountId(0, 0, 123)
-    addresses = [Endpoint(address=bytes("192.168.1.1", 'utf-8'), port=8080, domain_name="example.com")]
-    cert_hash = b'sample-cert-hash'
-    
+    addresses = [
+        Endpoint(
+            address=bytes("192.168.1.1", "utf-8"), port=8080, domain_name="example.com"
+        )
+    ]
+    cert_hash = b"sample-cert-hash"
+
     # Initialize _NodeAddress
     node_address = NodeAddress(
         public_key="sample-public-key",
         account_id=account_id,
         node_id=1234,
         cert_hash=cert_hash,
         addresses=addresses,
-        description="Sample Node"
+        description="Sample Node",
     )
-    
+
     # Assert properties are set correctly
     assert node_address._public_key == "sample-public-key"
     assert node_address._account_id == account_id
@@ -31,30 +39,35 @@ def test_init():
     assert node_address._addresses == addresses
     assert node_address._description == "Sample Node"
 
+
 def test_string_representation():
     """Test string representation of _NodeAddress."""
     # Create AccountId
     account_id = AccountId(0, 0, 123)
-    
-    # Create    
-    endpoint = Endpoint(address=bytes("192.168.1.1", 'utf-8'), port=8080, domain_name="example.com")
-    
+
+    # Create
+    endpoint = Endpoint(
+        address=bytes("192.168.1.1", "utf-8"), port=8080, domain_name="example.com"
+    )
+
     # Create NodeAddress
     node_address = NodeAddress(
         public_key="sample-public-key",
         account_id=account_id,
         node_id=1234,
-        cert_hash=b'sample-cert-hash',
+        cert_hash=b"sample-cert-hash",
         addresses=[endpoint],
-        description="Sample Node"
+        description="Sample Node",
     )
-    
+
     # Get string representation
     result = str(node_address)
-    
+
     # Check if expected fields are in the result
     assert "NodeAccountId: 0.0.123" in result
-    assert "CertHash: 73616d706c652d636572742d68617368" in result  # hex representation of sample-cert-hash
+    assert (
+        "CertHash: 73616d706c652d636572742d68617368" in result
+    )  # hex representation of sample-cert-hash
     assert "NodeId: 1234" in result
     assert "PubKey: sample-public-key" in result
 
@@ -63,17 +76,23 @@ def test_to_proto():
     """Test conversion of NodeAddress to protobuf with endpoints."""
     account_id = AccountId(0, 0, 123)
     endpoints = [
-        Endpoint(address=bytes("192.168.1.1", "utf-8"), port=8080, domain_name="example1.com"),
-        Endpoint(address=bytes("192.168.1.2", "utf-8"), port=8081, domain_name="example2.com"),
-        Endpoint(address=bytes("192.168.1.3", "utf-8"), port=8082, domain_name="example3.com"),
+        Endpoint(
+            address=bytes("192.168.1.1", "utf-8"), port=8080, domain_name="example1.com"
+        ),
+        Endpoint(
+            address=bytes("192.168.1.2", "utf-8"), port=8081, domain_name="example2.com"
+        ),
+        Endpoint(
+            address=bytes("192.168.1.3", "utf-8"), port=8082, domain_name="example3.com"
+        ),
     ]
     node_address = NodeAddress(
         public_key="sample-public-key",
         account_id=account_id,
         node_id=1234,
         cert_hash=b"sample-cert-hash",
         addresses=endpoints,
-        description="Sample Node"
+        description="Sample Node",
     )
 
     node_address_proto = node_address._to_proto()
@@ -146,9 +165,7 @@ def test_from_proto():
     """Test creation of NodeAddress from protobuf with endpoint."""
     account_id_proto = AccountId(0, 0, 123)._to_proto()
     endpoint_proto = Endpoint(
-        address=bytes("192.168.1.1", "utf-8"),
-        port=8080,
-        domain_name="example.com"
+        address=bytes("192.168.1.1", "utf-8"), port=8080, domain_name="example.com"
     )._to_proto()
 
     # Create NodeAddressProto
@@ -178,9 +195,7 @@ def test_round_trip():
     """Ensure NodeAddress → Proto → NodeAddress round trip works."""
     account_id = AccountId(0, 0, 123)
     endpoint = Endpoint(
-        address=bytes("192.168.1.1", "utf-8"),
-        port=8080,
-        domain_name="example.com"
+        address=bytes("192.168.1.1", "utf-8"), port=8080, domain_name="example.com"
     )
 
     # Create NodeAddress
@@ -190,7 +205,7 @@ def test_round_trip():
         node_id=1234,
         cert_hash=b"sample-cert-hash",
         addresses=[endpoint],
-        description="Sample Node"
+        description="Sample Node",
     )
 
     # Convert to proto
@@ -220,12 +235,13 @@ def test_empty_addresses():
         node_id=1234,
         cert_hash=b"sample-cert-hash",
         addresses=[],
-        description="No endpoints"
+        description="No endpoints",
     )
 
     proto = node_address._to_proto()
     assert len(proto.serviceEndpoint) == 0
 
+
 def test_to_proto_none_account_id():
     """Test _to_proto handles None account_id gracefully."""
     node_address = NodeAddress(
@@ -234,10 +250,10 @@ def test_to_proto_none_account_id():
         node_id=1234,
         cert_hash=b"sample-cert-hash",
         addresses=[],
-        description="No account"
+        description="No account",
     )
 
     proto = node_address._to_proto()
 
     # Should not have nodeAccountId set
-    assert not proto.HasField('nodeAccountId')
+    assert not proto.HasField("nodeAccountId")

tests/unit/node_create_transaction_test.py

@@ -104,7 +104,8 @@ def test_build_transaction_body_with_valid_parameters(mock_account_ids, node_par
     )
     assert len(node_create.service_endpoint) == 1
     assert (
-        node_create.service_endpoint[0] == node_params["service_endpoints"][0]._to_proto()
+        node_create.service_endpoint[0]
+        == node_params["service_endpoints"][0]._to_proto()
     )
     assert node_create.gossip_ca_certificate == node_params["gossip_ca_certificate"]
     assert node_create.grpc_certificate_hash == node_params["grpc_certificate_hash"]
@@ -139,7 +140,8 @@ def test_build_scheduled_body(node_params):
     )
     assert len(node_create.service_endpoint) == 1
     assert (
-        node_create.service_endpoint[0] == node_params["service_endpoints"][0]._to_proto()
+        node_create.service_endpoint[0]
+        == node_params["service_endpoints"][0]._to_proto()
     )
     assert node_create.gossip_ca_certificate == node_params["gossip_ca_certificate"]
     assert node_create.grpc_certificate_hash == node_params["grpc_certificate_hash"]