feat: Add auto set for auto_renew_account (#907)

Signed-off-by: gsstoykov <[email protected]>
Co-authored-by: Ricky Saechao <[email protected]>

Author: gsstoykov

.github/workflows/rust-ci.yml

@@ -117,7 +117,7 @@ jobs:
           repo-token: ${{ secrets.GITHUB_TOKEN }}
 
       - name: Start the local node
-        run: npx @hashgraph/hedera-local start -d --network local
+        run: npx @hashgraph/hedera-local start -d --network local --network-tag=0.60.0-alpha.0
 
       - name: "Create env file"
         run: |

src/token/token_create_transaction.rs

@@ -487,7 +487,15 @@ impl ToTransactionDataProtobuf for TokenCreateTransactionData {
     ) -> services::transaction_body::Data {
         let _ = chunk_info.assert_single_transaction();
 
-        services::transaction_body::Data::TokenCreation(self.to_protobuf())
+        // Generate the protobuf data
+        let mut protobuf_data = self.to_protobuf();
+
+        // Manually assign the auto_renew_account with operator_id if none is set
+        if protobuf_data.auto_renew_account.is_none() {
+            let operator_id = chunk_info.current_transaction_id.account_id;
+            protobuf_data.auto_renew_account = Some(operator_id.to_protobuf());
+        }
+        services::transaction_body::Data::TokenCreation(protobuf_data)
     }
 }
 

src/topic/topic_create_transaction.rs

@@ -240,7 +240,15 @@ impl ToTransactionDataProtobuf for TopicCreateTransactionData {
     ) -> services::transaction_body::Data {
         let _ = chunk_info.assert_single_transaction();
 
-        services::transaction_body::Data::ConsensusCreateTopic(self.to_protobuf())
+        // Generate the protobuf data
+        let mut protobuf_data = self.to_protobuf();
+
+        // Manually assign the auto_renew_account with operator_id if none is set
+        if protobuf_data.auto_renew_account.is_none() {
+            let operator_id = chunk_info.current_transaction_id.account_id;
+            protobuf_data.auto_renew_account = Some(operator_id.to_protobuf());
+        }
+        services::transaction_body::Data::ConsensusCreateTopic(protobuf_data)
     }
 }
 

tests/e2e/token/create.rs

@@ -12,6 +12,7 @@ use hedera::{
     Status,
     TokenCreateTransaction,
     TokenId,
+    TokenInfoQuery,
     TokenType,
 };
 use time::{
@@ -611,3 +612,59 @@ async fn royalty_fee() -> anyhow::Result<()> {
     account.delete(&client).await?;
     Ok(())
 }
+
+#[tokio::test]
+async fn auto_renew_account() -> anyhow::Result<()> {
+    let Some(TestEnvironment { config: _, client }) = setup_nonfree() else {
+        return Ok(());
+    };
+
+    let account = Account::create(Hbar::new(0), &client).await?;
+
+    let token_id = TokenCreateTransaction::new()
+        .name("ffff")
+        .symbol("F")
+        .treasury_account_id(account.id)
+        .auto_renew_account_id(account.id)
+        .expiration_time(OffsetDateTime::now_utc() + Duration::minutes(5))
+        .sign(account.key.clone())
+        .execute(&client)
+        .await?
+        .get_receipt(&client)
+        .await?
+        .token_id
+        .unwrap();
+
+    let info = TokenInfoQuery::new().token_id(token_id).execute(&client).await?;
+
+    // auto renew account should be set to operator account
+    assert_eq!(info.auto_renew_account.unwrap(), account.id);
+    Ok(())
+}
+
+#[tokio::test]
+async fn autoset_auto_renew_account() -> anyhow::Result<()> {
+    let Some(TestEnvironment { config: _, client }) = setup_nonfree() else {
+        return Ok(());
+    };
+
+    let account = Account::create(Hbar::new(0), &client).await?;
+
+    let token_id = TokenCreateTransaction::new()
+        .name("ffff")
+        .symbol("F")
+        .treasury_account_id(account.id)
+        .expiration_time(OffsetDateTime::now_utc() + Duration::minutes(5))
+        .sign(account.key.clone())
+        .execute(&client)
+        .await?
+        .get_receipt(&client)
+        .await?
+        .token_id
+        .unwrap();
+
+    let info = TokenInfoQuery::new().token_id(token_id).execute(&client).await?;
+    // auto renew account should be set to operator account
+    assert_eq!(info.auto_renew_account.unwrap(), client.get_operator_account_id().unwrap());
+    Ok(())
+}

tests/e2e/topic/create.rs

@@ -62,7 +62,27 @@ async fn fieldless() -> anyhow::Result<()> {
         .await?
         .topic_id
         .unwrap();
+    Ok(())
+}
 
+#[tokio::test]
+async fn autoset_auto_renew_account() -> anyhow::Result<()> {
+    let Some(TestEnvironment { config: _, client }) = setup_nonfree() else {
+        return Ok(());
+    };
+
+    let topic_id = TopicCreateTransaction::new()
+        .admin_key(client.get_operator_public_key().unwrap())
+        .topic_memo("[e2e::TopicCreateTransaction]")
+        .execute(&client)
+        .await?
+        .get_receipt(&client)
+        .await?
+        .topic_id
+        .unwrap();
+
+    let info = TopicInfoQuery::new().topic_id(topic_id).execute(&client).await?;
+    assert_eq!(info.auto_renew_account_id.unwrap(), client.get_operator_account_id().unwrap());
     Ok(())
 }
 
@@ -429,7 +449,6 @@ async fn exempts_fee_exempt_keys_from_hbar_fees() -> anyhow::Result<()> {
 
 // Test temporarily taken out until can figure out a solution for a separate freeze
 #[tokio::test]
-#[ignore = "Not Supported. Apply test when 0.60.0 is released to networks (mainnet, testnet, previewnet)"]
 async fn automatically_assign_auto_renew_account_id_on_topic_create() -> anyhow::Result<()> {
     let Some(TestEnvironment { config: _, client }) = setup_nonfree() else {
         return Ok(());
@@ -448,7 +467,6 @@ async fn automatically_assign_auto_renew_account_id_on_topic_create() -> anyhow:
 }
 
 #[tokio::test]
-#[ignore = "Not Supported. Apply test when 0.60.0 is released to networks (mainnet, testnet, previewnet)"]
 async fn create_with_transaction_id_assigns_auto_renew_account_id_to_transaction_id_account_id(
 ) -> anyhow::Result<()> {
     let Some(TestEnvironment { config: _, client }) = setup_nonfree() else {