hip-1299 add more integration tests

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

Author: emiliyank

sdk/src/main/java/com/hedera/hashgraph/sdk/Client.java

@@ -473,40 +473,6 @@ public synchronized Client setMirrorNetwork(List<String> network) throws Interru
         return this;
     }
 
-    /**
-     * Trigger an immediate address book update.
-     * This will fetch the address book even if periodic updates are disabled.
-     */
-    public synchronized Client updateNetworkFromAddressBook() {
-        try {
-            var fileId = FileId.getAddressBookFileIdFor(this.shard, this.realm);
-
-            logger.debug("Fetching address book from file {}", fileId);
-            System.out.println("Fetching address book from file " + fileId);
-
-            // Execute synchronously - no async complexity
-            var addressBook = new AddressBookQuery()
-                .setFileId(fileId)
-                .execute(this);  // ← Synchronous!
-
-            logger.debug("Received address book with {} nodes", addressBook.nodeAddresses.size());
-            System.out.println("address book size: " + addressBook.nodeAddresses.size());
-
-            // Update the network
-            this.setNetworkFromAddressBook(addressBook);
-
-            logger.info("Address book update completed successfully");
-            System.out.println("Address book update completed successfully");
-
-        } catch (TimeoutException e) {
-            logger.warn("Failed to fetch address book: {}", e.getMessage());
-        } catch (Exception e) {
-            logger.warn("Failed to update address book", e);
-        }
-
-        return this;
-    }
-
     private synchronized void scheduleNetworkUpdate(@Nullable Duration delay) {
         if (delay == null) {
             networkUpdateFuture = null;
@@ -1445,26 +1411,33 @@ public synchronized Client setNetworkUpdatePeriod(Duration networkUpdatePeriod)
      *
      * @return {@code this}
      */
-//    public synchronized Client updateNetworkFromAddressBook() {
-////        scheduleNetworkUpdate(Duration.ZERO);
-//        updateNetworkFromAddressBook2();
-//
-//        if (networkUpdateFuture != null) {
-//            try {
-//                networkUpdateFuture.get(30, TimeUnit.SECONDS);
-//                logger.debug("Address book update completed successfully");
-//            } catch (TimeoutException e) {
-//                logger.warn("Address book update timed out after 30 seconds", e);
-//            } catch (InterruptedException e) {
-//                Thread.currentThread().interrupt();
-//                logger.warn("Address book update was interrupted", e);
-//            } catch (ExecutionException e) {
-//                logger.warn("Address book update failed", e);
-//            }
-//        }
-//
-//        return this;
-//    }
+    public synchronized Client updateNetworkFromAddressBook() {
+        try {
+            var fileId = FileId.getAddressBookFileIdFor(this.shard, this.realm);
+
+            logger.debug("Fetching address book from file {}", fileId);
+            System.out.println("Fetching address book from file " + fileId);
+
+            // Execute synchronously - no async complexity
+            var addressBook = new AddressBookQuery().setFileId(fileId).execute(this); // ← Synchronous!
+
+            logger.debug("Received address book with {} nodes", addressBook.nodeAddresses.size());
+            System.out.println("address book size: " + addressBook.nodeAddresses.size());
+
+            // Update the network
+            this.setNetworkFromAddressBook(addressBook);
+
+            logger.info("Address book update completed successfully");
+            System.out.println("Address book update completed successfully");
+
+        } catch (TimeoutException e) {
+            logger.warn("Failed to fetch address book: {}", e.getMessage());
+        } catch (Exception e) {
+            logger.warn("Failed to update address book", e);
+        }
+
+        return this;
+    }
 
     public Logger getLogger() {
         return this.logger;

sdk/src/main/java/com/hedera/hashgraph/sdk/Executable.java

@@ -613,11 +613,10 @@ void setNodesFromNodeAccountIds(Client client) {
         if (nodeAccountIds.size() == 1) {
             var nodeProxies = client.network.getNodeProxies(nodeAccountIds.get(0));
             if (nodeProxies == null || nodeProxies.isEmpty()) {
-                logger.warn("Node {} not found in network, fetching latest address book",
-                    nodeAccountIds.get(0));
+                logger.warn("Node {} not found in network, fetching latest address book", nodeAccountIds.get(0));
 
                 try {
-                    client.updateNetworkFromAddressBook();  // Synchronous update
+                    client.updateNetworkFromAddressBook(); // Synchronous update
                     nodeProxies = client.network.getNodeProxies(nodeAccountIds.get(0));
                 } catch (Exception e) {
                     logger.error("Failed to update address book", e);
@@ -915,14 +914,11 @@ ExecutionState getExecutionState(Status status, ResponseT response) {
                 return ExecutionState.SERVER_ERROR;
             case BUSY:
                 return ExecutionState.RETRY;
-                //TODO - use ExecutionState.SUCCESS otherwise there is issue with transaction receipt - raised status INVALID_NODE_ACCOUNT
-//            case INVALID_NODE_ACCOUNT:
-//                // Matches Go SDK's executionStateRetryWithAnotherNode behavior:
-//                // immediately retry with next node without delay
-//                // This occurs when a node's account ID has changed
-//                return ExecutionState.SERVER_ERROR;
-                case INVALID_NODE_ACCOUNT:
-                case OK:
+            case INVALID_NODE_ACCOUNT:
+                return ExecutionState
+                        .SERVER_ERROR; // immediately retry with next node without delay. This occurs when a node's
+            // account ID has changed
+            case OK:
                 return ExecutionState.SUCCESS;
             default:
                 return ExecutionState.REQUEST_ERROR; // user error