fix:updating file changes

Signed-off-by: Angelina <aceppaluni@gmail.com>

Author: aceppaluni

.github/workflows/test.yml

@@ -22,15 +22,15 @@ jobs:
 
     steps:
       - name: Harden the runner (Audit all outbound calls)
-        uses: step-security/harden-runner@df199fb7be9f65074067a9eb93f12bb4c5547cf2 # v2.13.3
+        uses: step-security/harden-runner@95d9a5deda9de15063e7595e9719c11c38c90ae2 # v2.13.2
         with:
           egress-policy: audit
 
       - name: Checkout repository
-        uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v6.0.1
+        uses: actions/checkout@1af3b93b6815bc44a9784bd300feb67ff0d1eeb3 # v6.0.0
 
       - name: Set up Python ${{ matrix.python-version }}
-        uses: actions/setup-python@83679a892e2d95755f2dac6acb0bfd1e9ac5d548 # v6.1.0
+        uses: actions/setup-python@e797f83bcb11b83ae66e0230d6156d7c80228e7c # v6.0.0
         with:
           python-version: ${{ matrix.python-version }}
           cache: 'pip'
@@ -49,7 +49,7 @@ jobs:
 
       - name: Prepare Hiero Solo
         id: solo
-        uses: hiero-ledger/hiero-solo-action@fbca3e7a99ce9aa8a250563a81187abe115e0dad # v0.15.0
+        uses: hiero-ledger/hiero-solo-action@dd0048139ef1e40fd6067f01bf94eb42a67294f4 # v0.15.0
         with:
           installMirrorNode: true
 
@@ -105,7 +105,14 @@ jobs:
       - name: Fail workflow if any tests failed
         shell: bash
         run: |
-          if [ "${{ steps.integration.outputs.integration_failed }}" != "0" ] || [ "${{ steps.unit.outputs.unit_failed }}" != "0" ]; then
+          integration_failed="${{ steps.integration.outputs.integration_failed }}"
+          unit_failed="${{ steps.unit.outputs.unit_failed }}"
+
+          # Default to 0 if empty
+          integration_failed="${integration_failed:-0}"
+          unit_failed="${unit_failed:-0}"
+
+          if [ "$integration_failed" != "0" ] || [ "$unit_failed" != "0" ]; then
             echo "❌ Some tests failed. Failing workflow."
             exit 1
           else

src/hiero_sdk_python/executable.py

@@ -306,5 +306,4 @@ def _execute_method(method, proto_request):
         return method.transaction(proto_request)
     elif method.query is not None:
         return method.query(proto_request)
-    raise Exception("No method to execute")
-    
+    raise Exception("No method to execute") 

src/hiero_sdk_python/query/query.py

@@ -107,9 +107,6 @@ def _before_execute(self, client: Client) -> None:
         """
         self.operator = self.operator or client.operator
 
-        if not getattr(self, "node_account_ids", None):
-            self.node_account_ids = client.get_node_account_ids()
-            
         # If no payment amount was specified and payment is required for this query,
         # get the cost from the network and set it as the payment amount
         if self.payment_amount is None and self._is_payment_required():
@@ -377,3 +374,4 @@ def _is_payment_required(self) -> bool:
             bool: True if payment is required, False otherwise
         """
         return True
+        

src/hiero_sdk_python/transaction/transaction.py

@@ -9,10 +9,9 @@
 from hiero_sdk_python.crypto.private_key import PrivateKey
 from hiero_sdk_python.exceptions import PrecheckError
 from hiero_sdk_python.executable import _Executable, _ExecutionState
-from hiero_sdk_python.hapi.services import (basic_types_pb2, transaction_pb2, transaction_contents_pb2)
+from hiero_sdk_python.hapi.services import (basic_types_pb2, transaction_pb2, transaction_contents_pb2, transaction_pb2)
 from hiero_sdk_python.hapi.services.schedulable_transaction_body_pb2 import SchedulableTransactionBody
 from hiero_sdk_python.hapi.services.transaction_response_pb2 import (TransactionResponse as TransactionResponseProto)
-from hiero_sdk_python.hbar import Hbar
 from hiero_sdk_python.response_code import ResponseCode
 from hiero_sdk_python.transaction.transaction_id import TransactionId
 from hiero_sdk_python.transaction.transaction_response import TransactionResponse
@@ -62,12 +61,9 @@ def __init__(self) -> None:
         # This allows us to maintain the signatures for each unique transaction
         # and ensures that the correct signatures are used when submitting transactions
         self._signature_map: dict[bytes, basic_types_pb2.SignatureMap] = {}
-        # changed from int: 2_000_000 to Hbar: 0.02
-        self._default_transaction_fee = Hbar(0.02)
-        self.operator_account_id = None  
+        self._default_transaction_fee = 2_000_000
+        self.operator_account_id = None
         self.batch_key: Optional[PrivateKey] = None
-        self.node_account_ids: Optional[List[AccountId]] = None
-        self._used_node_account_id: Optional[AccountId] = None
 
     def _make_request(self):
         """
@@ -423,11 +419,7 @@ def build_base_transaction_body(self) -> transaction_pb2.TransactionBody:
         transaction_body.transactionID.CopyFrom(transaction_id_proto)
         transaction_body.nodeAccountID.CopyFrom(self.node_account_id._to_proto())
 
-        fee = self.transaction_fee or self._default_transaction_fee
-        if hasattr(fee, "to_tinybars"):
-            transaction_body.transactionFee = int(fee.to_tinybars())
-        else:
-            transaction_body.transactionFee = int(fee)
+        transaction_body.transactionFee = self.transaction_fee or self._default_transaction_fee
 
         transaction_body.transactionValidDuration.seconds = self.transaction_valid_duration
         transaction_body.generateRecord = self.generate_record
@@ -449,13 +441,9 @@ def build_base_scheduled_body(self) -> SchedulableTransactionBody:
                 The protobuf SchedulableTransactionBody message with common fields set.
         """
         schedulable_body = SchedulableTransactionBody()
-
-        fee = self.transaction_fee or self._default_transaction_fee
-        if hasattr(fee, "to_tinybars"):
-            schedulable_body.transactionFee = int(fee.to_tinybars())
-        else:
-            schedulable_body.transactionFee = int(fee)
-
+        schedulable_body.transactionFee = (
+            self.transaction_fee or self._default_transaction_fee
+        )
         schedulable_body.memo = self.memo
         custom_fee_limits = [custom_fee._to_proto() for custom_fee in self.custom_fee_limits]
         schedulable_body.max_custom_fees.extend(custom_fee_limits)

tests/unit/test_executable.py

@@ -46,7 +46,7 @@ def test_retry_success_before_max_attempts():
     # First server gives 2 BUSY responses then OK on the 3rd try
     response_sequences = [[busy_response, busy_response, ok_response, receipt_response]]
 
-    with mock_hedera_servers(response_sequences) as client, patch('time.sleep'):
+    with mock_hedera_servers(response_sequences) as client, patch('hiero_sdk_python.executable.time.sleep'):
         # Configure client to allow 3 attempts - should succeed on the last try
         client.max_attempts = 3
 
@@ -70,7 +70,7 @@ def test_retry_failure_after_max_attempts():
 
     response_sequences = [[busy_response, busy_response]]
 
-    with mock_hedera_servers(response_sequences) as client, patch('time.sleep'):
+    with mock_hedera_servers(response_sequences) as client, patch('hiero_sdk_python.executable.time.sleep'):
         client.max_attempts = 2
 
         transaction = (
@@ -112,7 +112,7 @@ def test_node_switching_after_single_grpc_error():
         [error],
     ]
 
-    with mock_hedera_servers(response_sequences) as client, patch('time.sleep'):
+    with mock_hedera_servers(response_sequences) as client, patch('hiero_sdk_python.executable.time.sleep'):
         transaction = (
             AccountCreateTransaction()
             .set_key(PrivateKey.generate().public_key())
@@ -149,7 +149,7 @@ def test_node_switching_after_multiple_grpc_errors():
         [ok_response, receipt_response],
     ]
 
-    with mock_hedera_servers(response_sequences) as client, patch('time.sleep'):
+    with mock_hedera_servers(response_sequences) as client, patch('hiero_sdk_python.executable.time.sleep'):
         transaction = (
             AccountCreateTransaction()
             .set_key(PrivateKey.generate().public_key())
@@ -185,7 +185,7 @@ def test_transaction_with_expired_error_not_retried():
         [error_response]
     ]
 
-    with mock_hedera_servers(response_sequences) as client, patch('time.sleep'):
+    with mock_hedera_servers(response_sequences) as client, patch('hiero_sdk_python.executable.time.sleep'):
         transaction = (
             AccountCreateTransaction()
             .set_key(PrivateKey.generate().public_key())
@@ -216,7 +216,7 @@ def test_transaction_with_fatal_error_not_retried():
         [error_response]
     ]
 
-    with mock_hedera_servers(response_sequences) as client, patch('time.sleep'):
+    with mock_hedera_servers(response_sequences) as client, patch('hiero_sdk_python.executable.time.sleep'):
         transaction = (
             AccountCreateTransaction()
             .set_key(PrivateKey.generate().public_key())
@@ -248,7 +248,7 @@ def test_exponential_backoff_retry():
     response_sequences = [[busy_response, busy_response, busy_response, ok_response, receipt_response]]
 
     # Use a mock for time.sleep to capture the delay values
-    with mock_hedera_servers(response_sequences) as client, patch('time.sleep') as mock_sleep:
+    with mock_hedera_servers(response_sequences) as client, patch('hiero_sdk_python.executable.time.sleep') as mock_sleep:
         client.max_attempts = 5
 
         transaction = (
@@ -288,7 +288,7 @@ def test_retriable_error_does_not_switch_node():
         )
     )
     response_sequences = [[busy_response, ok_response, receipt_response]]
-    with mock_hedera_servers(response_sequences) as client, patch('time.sleep'):
+    with mock_hedera_servers(response_sequences) as client, patch('hiero_sdk_python.executable.time.sleep'):
         transaction = (
             AccountCreateTransaction()
             .set_key(PrivateKey.generate().public_key())
@@ -333,7 +333,7 @@ def test_topic_create_transaction_retry_on_busy():
         [busy_response, ok_response, receipt_response],
     ]
 
-    with mock_hedera_servers(response_sequences) as client, patch('time.sleep') as mock_sleep:
+    with mock_hedera_servers(response_sequences) as client, patch('hiero_sdk_python.executable.time.sleep') as mock_sleep:
         client.max_attempts = 3
 
         tx = (
@@ -367,7 +367,7 @@ def test_topic_create_transaction_fails_on_nonretriable_error():
         [error_response],
     ]
 
-    with mock_hedera_servers(response_sequences) as client, patch('time.sleep'):
+    with mock_hedera_servers(response_sequences) as client, patch('hiero_sdk_python.executable.time.sleep'):
         tx = (
             TopicCreateTransaction()
             .set_memo("Test with error")
@@ -400,7 +400,7 @@ def test_transaction_node_switching_body_bytes():
         [ok_response, receipt_response],
     ]
 
-    with mock_hedera_servers(response_sequences) as client, patch('time.sleep'):
+    with mock_hedera_servers(response_sequences) as client, patch('hiero_sdk_python.executable.time.sleep'):
         # We set the current node to 0
         client.network._node_index = 0
         client.network.current_node = client.network.nodes[0]
@@ -467,8 +467,9 @@ def test_query_retry_on_busy():
         [ok_response],
     ]
 
-    with mock_hedera_servers(response_sequences) as client, patch('time.sleep') as mock_sleep:
+    with mock_hedera_servers(response_sequences) as client, patch('hiero_sdk_python.executable.time.sleep') as mock_sleep:
         # We set the current node to the first node so we are sure it will return BUSY response
+        client.network._node_index = 0
         client.network.current_node = client.network.nodes[0]
 
         query = CryptoGetAccountBalanceQuery()