Fixed comments and parameters

Author: TetyanaYahodska

compute/cloud-client/src/main/java/compute/disks/AttachRegionalDiskForce.java

@@ -32,22 +32,24 @@ public static void main(String[] args)
     // TODO(developer): Replace these variables before running the sample.
     // Project ID or project number of the Cloud project you want to use.
     String projectId = "YOUR_PROJECT_ID";
-    // Name of the zone in which the instance you want to use resides.
-    String zone = "zone-name";
-    // Name of the compute instance you want to attach a disk to.
+    // Name of the zone of your compute instance.
+    String zone = "us-central1-a";
+    // The name of the compute instance where you are adding the replicated disk.
     String instanceName = "YOUR_INSTANCE_NAME";
-    // Full or partial URL of a persistent disk that you want to attach.
-    String diskLink = String.format(
-            "projects/%s/regions/DISK_REGION/disks/YOUR_DISK_NAME", projectId);
+    // The region where your replicated disk is located.
+    String region = "us-central1";
+    // The name of the replicated disk.
+    String diskName = "YOUR_DISK_NAME";
 
-    attachRegionalDiskForce(projectId, zone, instanceName, diskLink);
+    attachRegionalDiskForce(projectId, zone, instanceName, region, diskName);
   }
 
   // Attaches a regional disk to the instance,
   // forcing the attachment even if other VMs are using the disk.
-  public static void attachRegionalDiskForce(
-          String projectId, String zone, String instanceName, String diskLink)
+  public static Operation.Status attachRegionalDiskForce(
+          String projectId, String zone, String instanceName, String region, String diskName)
           throws IOException, InterruptedException, ExecutionException, TimeoutException {
+    String diskLink = String.format("projects/%s/regions/%s/disks/%s", projectId, region, diskName);
     // Initialize client that will be used to send requests. This client only needs to be created
     // once, and can be reused for multiple requests.
     try (InstancesClient instancesClient = InstancesClient.create()) {
@@ -69,10 +71,9 @@ public static void attachRegionalDiskForce(
               .get(3, TimeUnit.MINUTES);
 
       if (response.hasError()) {
-        System.out.printf("Error attaching regional disk: %s%n", response.getError());
-        return;
+        throw new Error("Error attaching regional disk! " + response);
       }
-      System.out.println("Regional disk attached successfully.");
+      return response.getStatus();
     }
   }
 }

compute/cloud-client/src/test/java/compute/disks/DisksIT.java

@@ -317,11 +317,9 @@ public void testDiskAttachResize()
   @Order(2)
   public void testAttachRegionalDiskForceAttach()
           throws IOException, ExecutionException, InterruptedException, TimeoutException {
-    String diskLink = String.format("projects/%s/regions/%s/disks/%s",
-            PROJECT_ID, REGION, REGIONAL_ATTACHED_DISK);
+    Operation.Status status = AttachRegionalDiskForce
+          .attachRegionalDiskForce(PROJECT_ID, ZONE, INSTANCE_NAME, REGION, REGIONAL_ATTACHED_DISK);
 
-    AttachRegionalDiskForce.attachRegionalDiskForce(PROJECT_ID, ZONE, INSTANCE_NAME, diskLink);
-
-    assertThat(stdOut.toString()).contains("Regional disk attached successfully.");
+    assertThat(status).isEqualTo(Operation.Status.DONE);
   }
 }