chore: neat pick

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

Author: manishdait

pyproject.toml

@@ -21,6 +21,8 @@ dependencies = [
     "pycryptodome>=3.18.0,<4",
     "eth-abi>=5.1.0,<6",
     "python-dotenv>=1.2.1,<3",
+    "pytest-repeat>=0.9.4",
+    "pytest-xdist>=3.8.0",
 ]
 classifiers = [
     "Development Status :: 2 - Pre-Alpha",

src/hiero_sdk_python/consensus/topic_message_submit_transaction.py

@@ -253,13 +253,7 @@ def freeze_with(self, client: Client) -> TopicMessageSubmitTransaction:
         if self._transaction_body_bytes:
             return self
 
-        if self.transaction_id is None:
-            if client is None:
-                raise ValueError(
-                    "Transaction ID must be set before freezing. Use freeze_with(client) or set_transaction_id()."
-                )
-
-            self.transaction_id = client.generate_transaction_id()
+        self._resolve_transaction_id(client)
 
         if not self._transaction_ids:
             base_timestamp = self.transaction_id.valid_start

src/hiero_sdk_python/file/file_append_transaction.py

@@ -295,6 +295,9 @@ def freeze_with(self, client: Client) -> FileAppendTransaction:
 
             self.transaction_id = client.generate_transaction_id()
 
+        
+        self._resolve_transaction_id(client)
+
         # Generate transaction IDs for all chunks
         if not self._transaction_ids:
             base_timestamp = self.transaction_id.valid_start

src/hiero_sdk_python/node.py

@@ -126,6 +126,8 @@ def _get_channel(self):
             if self._root_certificates:
                 # Use the certificate that is provided
                 self._node_pem_cert = self._root_certificates
+                print("node cert ", self._node_pem_cert)
+            
             else:
                 # Fetch pem_cert for the node
                 self._node_pem_cert = self._fetch_server_certificate_pem()

src/hiero_sdk_python/transaction/transaction.py

@@ -226,7 +226,28 @@ def _to_proto(self):
 
         signed_transaction = transaction_contents_pb2.SignedTransaction(bodyBytes=body_bytes, sigMap=sig_map)
 
-        return transaction_pb2.Transaction(signedTransactionBytes=signed_transaction.SerializeToString())
+
+        return transaction_pb2.Transaction(
+            signedTransactionBytes=signed_transaction.SerializeToString()
+        )
+    
+    def _resolve_transaction_id(self, client: "Client"):
+        if self.transaction_id is not None:
+            return
+        
+        if client is not None:
+            operator_account_id = client.operator_account_id
+            if operator_account_id is not None:
+                self.transaction_id = client.generate_transaction_id()
+            else:
+                raise ValueError(
+                    "Client must have an operator_account or transactionId must be set."
+                )
+        else:
+            raise ValueError(
+                "Transaction ID must be set before freezing. Use freeze_with(client) or set_transaction_id()."
+            )
+
 
     def freeze(self):
         """
@@ -266,19 +287,7 @@ def freeze_with(self, client: "Client"):
 
 
         # Check transaction_id and node id to be set when using freeze()
-        if self.transaction_id is None:
-            if client is not None:
-                operator_account_id = client.operator_account_id
-                if operator_account_id is not None:
-                    self.transaction_id = client.generate_transaction_id()
-                else:
-                    raise ValueError(
-                        "Client must have an operator_account or transactionId must be set."
-                    )
-            else:
-                raise ValueError(
-                    "Transaction ID must be set before freezing. Use freeze_with(client) or set_transaction_id()."
-                )
+        self._resolve_transaction_id(client)
 
 
         if self.node_account_id is None and len(self.node_account_ids) == 0:

tests/integration/file_append_transaction_e2e_test.py

@@ -292,7 +292,7 @@ def test_integration_file_append_transaction_method_chaining(env):
     assert append_receipt.status == ResponseCode.SUCCESS 
 
 @pytest.mark.integration
-def test_file_append_chuck_transaction_can_execute_with_manual_freeze(env):
+def test_file_append_chunk_transaction_can_execute_with_manual_freeze(env):
     """Test file append transaction can execute  with manual freeze."""
     create_receipt = (
         FileCreateTransaction()
@@ -307,7 +307,7 @@ def test_file_append_chuck_transaction_can_execute_with_manual_freeze(env):
     file_contents = FileContentsQuery().set_file_id(file_id).execute(env.client)
     assert file_contents == b""
 
-    content = "A" * (4000) # content with (1024 * 14) bytes ie 14 chunks
+    content = "A" * (4000) # content with (4000/1024) bytes ie approx 4 chunks
 
     tx = (
         FileAppendTransaction()

tests/unit/mock_server.py

@@ -181,8 +181,11 @@ def mock_hedera_servers(response_sequences):
 
         client = Client(network)
 
+        # Force non-tls channel
         for node in client.network.nodes:
-            node._address = node._address._to_insecure()
+            node._address._is_transport_security = lambda: False
+            node._set_verify_certificates(False)
+            node._close()
 
         client.logger.set_level(LogLevel.DISABLED)
         # Set the operator