feat: Add working Batch Transaction e2e test

Signed-off-by: gsstoykov <[email protected]>

Author: gsstoykov

src/batch_transaction.rs

@@ -204,6 +204,8 @@ impl ToTransactionDataProtobuf for BatchTransactionData {
             let signed_transaction_bytes = transaction
                 .to_signed_transaction_bytes()
                 .expect("Inner transaction should be frozen and serializable");
+
+            println!("Signed transaction bytes: {:?}", signed_transaction_bytes);
             builder.transactions.push(signed_transaction_bytes);
         }
 

src/transaction/mod.rs

@@ -543,6 +543,7 @@ impl<D: TransactionExecute> Transaction<D> {
 
     /// Convenience method to mark a transaction as part of a batch transaction.
     /// The Transaction will be frozen and signed by the operator of the client.
+    /// Per HIP-551, inner transactions use node account ID 0.0.0.
     ///
     /// # Errors
     ///
@@ -554,6 +555,8 @@ impl<D: TransactionExecute> Transaction<D> {
     ) -> crate::Result<&mut Self> {
         self.require_not_frozen();
         self.set_batch_key(batch_key);
+        // Set node account ID to 0.0.0 for batch transactions (as per HIP-551)
+        self.node_account_ids([crate::AccountId::new(0, 0, 0)]);
         self.freeze_with(client)?;
         self.sign_with_operator(client)
     }
@@ -833,7 +836,7 @@ impl<D: TransactionExecute> Transaction<D> {
             sig_map: Some(services::SignatureMap { sig_pair: signatures.clone() }),
         };
         services::Transaction {
-            signed_transaction_bytes: Vec::new(),
+            signed_transaction_bytes: signed_transaction.encode_to_vec(),
             body: None,
             sigs: None,
             body_bytes: signed_transaction.body_bytes,

tests/e2e/batch_transaction.rs

@@ -0,0 +1,43 @@
+use std::str::FromStr;
+use hedera::{
+    AccountCreateTransaction,
+    BatchTransaction,
+    Hbar,
+    PrivateKey,
+};
+
+use crate::common::{
+    setup_nonfree,
+    TestEnvironment,
+};
+
+#[tokio::test]
+async fn can_execute_batch_transaction() -> anyhow::Result<()> {
+    let Some(TestEnvironment { config: _, client }) = setup_nonfree() else {
+        return Ok(());
+    };
+
+    // Given
+    let operator_key = PrivateKey::from_str(
+        "302e020100300506032b65700422042091132178e72057a1d7528025956fe39b0b847f200ab59b2fdd367017f3087137"
+    )?;
+
+    let account_key = PrivateKey::generate_ed25519();
+
+    let mut inner_transaction = AccountCreateTransaction::new();
+    inner_transaction
+        .set_key_without_alias(account_key.public_key())
+        .initial_balance(Hbar::new(1));
+
+    // Use batchify to prepare the transaction
+    inner_transaction.batchify(&client, operator_key.public_key().into())?;
+
+    // When / Then
+    let mut batch_transaction = BatchTransaction::new();
+    batch_transaction.add_inner_transaction(inner_transaction.into())?;
+
+    let tx_response = batch_transaction.execute(&client).await?;
+    let _tx_receipt = tx_response.get_receipt(&client).await?;
+
+    Ok(())
+}

tests/e2e/main.rs

@@ -1,5 +1,6 @@
 mod account;
 mod address_book;
+mod batch_transaction;
 mod client;
 mod common;
 mod contract;