chore: updated unit test

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

Author: manishdait

tests/integration/file_append_transaction_e2e_test.py

@@ -7,7 +7,9 @@
 from hiero_sdk_python.file.file_append_transaction import FileAppendTransaction
 from hiero_sdk_python.file.file_contents_query import FileContentsQuery
 from hiero_sdk_python.file.file_create_transaction import FileCreateTransaction
+from hiero_sdk_python.hbar import Hbar
 from hiero_sdk_python.response_code import ResponseCode
+from hiero_sdk_python.transaction.transaction import Transaction
 from hiero_sdk_python.transaction.transaction_id import TransactionId
 
 
@@ -324,3 +326,42 @@ def test_file_append_chunk_transaction_can_execute_with_manual_freeze(env):
 
     file_contents = FileContentsQuery().set_file_id(file_id).execute(env.client)
     assert file_contents == bytes(content, "utf-8")
+
+
+@pytest.mark.integration
+def test(env):
+    create_receipt = (
+        FileCreateTransaction()
+        .set_keys(env.client.operator_private_key.public_key())
+        .set_contents(b"")
+        .execute(env.client)
+    )
+
+    assert create_receipt.status == ResponseCode.SUCCESS
+    file_id = create_receipt.file_id
+
+    file_contents = FileContentsQuery().set_file_id(file_id).execute(env.client)
+    assert file_contents == b""
+
+    content = "A" * (8192)  # content with (4000/1024) bytes ie approx 4 chunks
+
+    tx = (
+        FileAppendTransaction()
+        .set_file_id(file_id)
+        .set_contents(content)
+        .set_transaction_id(TransactionId.generate(AccountId(0, 0, 101)))
+        .set_node_account_ids([AccountId(0, 0, 3), AccountId(0, 0, 4)])
+    )
+
+    tx._default_transaction_fee = Hbar.from_hbars(10).to_tinybars()
+
+    print(tx.get_required_chunks())
+    print(tx.transaction_id)
+    print(tx.node_account_ids)
+
+    bytestx = tx.to_bytes()
+    ntx = Transaction.from_bytes(bytestx)
+
+    print(ntx.get_required_chunks())
+    print(ntx.transaction_id)
+    print(ntx.node_account_ids)

tests/integration/topic_message_submit_transaction_e2e_test.py

@@ -19,6 +19,7 @@
 from hiero_sdk_python.response_code import ResponseCode
 from hiero_sdk_python.tokens.custom_fixed_fee import CustomFixedFee
 from hiero_sdk_python.transaction.custom_fee_limit import CustomFeeLimit
+from hiero_sdk_python.transaction.transaction import Transaction
 from hiero_sdk_python.transaction.transaction_id import TransactionId
 
 
@@ -328,3 +329,33 @@ def test_topic_message_submit_transaction_can_submit_a_large_message_manual_free
     assert info.sequence_number == 14
 
     delete_topic(env.client, topic_id)
+
+
+@pytest.mark.integration
+def test(env):
+    topic_id = create_topic(client=env.client, admin_key=env.operator_key)
+
+    info = TopicInfoQuery().set_topic_id(topic_id).execute(env.client)
+    assert info.sequence_number == 0
+
+    message = "ABCD"  # message with (1024 * 14) bytes ie 14 chunks
+
+    message_tx = (
+        TopicMessageSubmitTransaction()
+        .set_topic_id(topic_id)
+        .set_chunk_size(1)
+        .set_message(message)
+        .freeze_with(env.client)
+    )
+    print(len(message_tx._transaction_ids))
+    print(len(message_tx._transaction_body_bytes.keys()))
+
+    bytestx = message_tx.to_bytes()
+    tx = Transaction.from_bytes(bytestx)
+
+    print(isinstance(tx, TopicMessageSubmitTransaction))
+    print(len(tx._transaction_ids))
+    print(len(tx._transaction_body_bytes.keys()))
+    tx.execute(env.client)
+
+    print(tx.message)

tests/integration/transaction_freeze_e2e_test.py

@@ -2,9 +2,11 @@
 
 import pytest
 
+from hiero_sdk_python.account.account_create_transaction import AccountCreateTransaction
 from hiero_sdk_python.account.account_id import AccountId
 from hiero_sdk_python.client.client import Client
 from hiero_sdk_python.consensus.topic_create_transaction import TopicCreateTransaction
+from hiero_sdk_python.crypto.private_key import PrivateKey
 from hiero_sdk_python.response_code import ResponseCode
 from hiero_sdk_python.transaction.transaction import Transaction
 from hiero_sdk_python.transaction.transaction_id import TransactionId
@@ -151,3 +153,22 @@ def test_transaction_with_secondary_client_without_operator_can_execute_successf
     receipt = tx2.execute(executor_client)
 
     assert receipt.status == ResponseCode.SUCCESS, "Transaction must execute successfully"
+
+
+@pytest.mark.integration
+def test(env):
+    key = PrivateKey.generate_ecdsa()
+
+    tx = (
+        AccountCreateTransaction()
+        .set_key_without_alias(key)
+        .set_initial_balance(1)
+        .set_account_memo("test account")
+        .freeze_with(env.client)
+    )
+    print(tx._transaction_body_bytes)
+
+    tx_bytes = tx.to_bytes()
+
+    tx2 = Transaction.from_bytes(tx_bytes)
+    tx2.execute(env.client)

tests/unit/account_create_transaction_test.py

@@ -108,6 +108,7 @@ def test_account_create_transaction_build_scheduled_body(mock_account_ids):
 def test_account_create_transaction_sign(mock_account_ids, mock_client):
     """Test signing the account create transaction."""
     operator_id, node_account_id = mock_account_ids
+    transaction_id = generate_transaction_id(operator_id)
 
     new_private_key = PrivateKey.generate()
     new_public_key = new_private_key.public_key()
@@ -119,20 +120,20 @@ def test_account_create_transaction_sign(mock_account_ids, mock_client):
         .set_initial_balance(100000000)
         .set_account_memo("Test account")
     )
-    account_tx.transaction_id = generate_transaction_id(operator_id)
+    account_tx.transaction_id = transaction_id
     account_tx.node_account_id = node_account_id
     account_tx.freeze_with(mock_client)
 
     # Add first signiture
     account_tx.sign(mock_client.operator_private_key)
-    body_bytes = account_tx._transaction_body_bytes[node_account_id]
+    body_bytes = account_tx._transaction_body_bytes[transaction_id][node_account_id]
 
     assert body_bytes in account_tx._signature_map, "Body bytes should be a key in the signature map dictionary"
     assert len(account_tx._signature_map[body_bytes].sigPair) == 1, "Transaction should have exactly one signature"
 
     # Add second signiture
     account_tx.sign(operator_private_key)
-    body_bytes = account_tx._transaction_body_bytes[node_account_id]
+    body_bytes = account_tx._transaction_body_bytes[transaction_id][node_account_id]
 
     assert body_bytes in account_tx._signature_map, "Body bytes should be a key in the signature map dictionary"
     assert len(account_tx._signature_map[body_bytes].sigPair) == 2, "Transaction should have exactly two signatures"

tests/unit/account_delete_transaction_test.py

@@ -188,7 +188,7 @@ def test_sign_transaction(mock_client, delete_params):
     delete_tx.sign(private_key)
 
     node_id = mock_client.network.current_node._account_id
-    body_bytes = delete_tx._transaction_body_bytes[node_id]
+    body_bytes = delete_tx._transaction_body_bytes[delete_tx.transaction_id][node_id]
 
     assert len(delete_tx._signature_map[body_bytes].sigPair) == 1
     sig_pair = delete_tx._signature_map[body_bytes].sigPair[0]

tests/unit/account_update_transaction_test.py

@@ -354,7 +354,7 @@ def test_sign_transaction(mock_client):
     account_tx.sign(private_key)
 
     node_id = mock_client.network.current_node._account_id
-    body_bytes = account_tx._transaction_body_bytes[node_id]
+    body_bytes = account_tx._transaction_body_bytes[account_tx.transaction_id][node_id]
 
     assert len(account_tx._signature_map[body_bytes].sigPair) == 1
     sig_pair = account_tx._signature_map[body_bytes].sigPair[0]

tests/unit/batch_transaction_test.py

@@ -312,7 +312,7 @@ def test_sign_transaction(mock_client, mock_tx):
     batch_tx.sign(private_key)
 
     node_id = mock_client.network.current_node._account_id
-    body_bytes = batch_tx._transaction_body_bytes[node_id]
+    body_bytes = batch_tx._transaction_body_bytes[batch_tx.transaction_id][node_id]
 
     assert body_bytes in batch_tx._signature_map, "signature map must contain an entry for the tx body bytes"
     sig_pairs = batch_tx._signature_map[body_bytes].sigPair

tests/unit/contract_delete_transaction_test.py

@@ -296,7 +296,7 @@ def test_sign_transaction(mock_client, delete_params):
     delete_tx.sign(private_key)
 
     node_id = mock_client.network.current_node._account_id
-    body_bytes = delete_tx._transaction_body_bytes[node_id]
+    body_bytes = delete_tx._transaction_body_bytes[delete_tx.transaction_id][node_id]
 
     assert len(delete_tx._signature_map[body_bytes].sigPair) == 1
     sig_pair = delete_tx._signature_map[body_bytes].sigPair[0]

tests/unit/contract_execute_transaction_test.py

@@ -325,7 +325,7 @@ def test_sign_transaction(mock_client, execute_params):
     execute_tx.sign(private_key)
 
     node_id = mock_client.network.current_node._account_id
-    body_bytes = execute_tx._transaction_body_bytes[node_id]
+    body_bytes = execute_tx._transaction_body_bytes[execute_tx.transaction_id][node_id]
 
     assert len(execute_tx._signature_map[body_bytes].sigPair) == 1
     sig_pair = execute_tx._signature_map[body_bytes].sigPair[0]

tests/unit/ethereum_transaction_test.py

@@ -170,7 +170,7 @@ def test_sign_transaction(mock_client, ethereum_params):
     ethereum_tx.sign(private_key)
 
     node_id = mock_client.network.current_node._account_id
-    body_bytes = ethereum_tx._transaction_body_bytes[node_id]
+    body_bytes = ethereum_tx._transaction_body_bytes[ethereum_tx.transaction_id][node_id]
 
     assert len(ethereum_tx._signature_map[body_bytes].sigPair) == 1
     sig_pair = ethereum_tx._signature_map[body_bytes].sigPair[0]