fix: update deprecated notificationconfig for healthcare api (GoogleCloudPlatform#9523)

noerog · web-flow · commit df19da6e9d23 · 2024-09-11T22:02:27.000-07:00
* fix: update deprecated notificationconfig for healthcare api

* fix lint
diff --git a/healthcare/v1/src/main/java/snippets/healthcare/fhir/FhirStorePatch.java b/healthcare/v1/src/main/java/snippets/healthcare/fhir/FhirStorePatch.java
@@ -24,8 +24,8 @@
 import com.google.api.services.healthcare.v1.CloudHealthcare;
 import com.google.api.services.healthcare.v1.CloudHealthcare.Projects.Locations.Datasets.FhirStores;
 import com.google.api.services.healthcare.v1.CloudHealthcareScopes;
+import com.google.api.services.healthcare.v1.model.FhirNotificationConfig;
 import com.google.api.services.healthcare.v1.model.FhirStore;
-import com.google.api.services.healthcare.v1.model.NotificationConfig;
 import com.google.auth.http.HttpCredentialsAdapter;
 import com.google.auth.oauth2.GoogleCredentials;
 import java.io.IOException;
@@ -38,51 +38,58 @@ public class FhirStorePatch {
 
   public static void fhirStorePatch(String fhirStoreName, String pubsubTopic) throws IOException {
     // String fhirStoreName =
-    //    String.format(
-    //        FHIR_NAME, "your-project-id", "your-region-id", "your-dataset-id", "your-fhir-id");
+    // String.format(
+    // FHIR_NAME, "your-project-id", "your-region-id", "your-dataset-id",
+    // "your-fhir-id");
     // String pubsubTopic = "projects/your-project-id/topics/your-pubsub-topic";
 
     // Initialize the client, which will be used to interact with the service.
     CloudHealthcare client = createClient();
 
     // Fetch the initial state of the FHIR store.
-    FhirStores.Get getRequest =
-        client.projects().locations().datasets().fhirStores().get(fhirStoreName);
+    FhirStores.Get getRequest = client
+        .projects()
+        .locations()
+        .datasets()
+        .fhirStores()
+        .get(fhirStoreName);
     FhirStore store = getRequest.execute();
 
-    // Update the FhirStore fields as needed as needed. For a full list of FhirStore fields, see:
+    // Update the FhirStore fields as needed as needed. For a full list of FhirStore
+    // fields, see:
     // https://cloud.google.com/healthcare/docs/reference/rest/v1/projects.locations.datasets.fhirStores#FhirStore
-    store.setNotificationConfig(new NotificationConfig().setPubsubTopic(pubsubTopic));
+    FhirNotificationConfig notificationConfig = new FhirNotificationConfig()
+        .setPubsubTopic(pubsubTopic);
+    store.setNotificationConfigs(Collections.singletonList(notificationConfig));
 
     // Create request and configure any parameters.
-    FhirStores.Patch request =
-        client
-            .projects()
-            .locations()
-            .datasets()
-            .fhirStores()
-            .patch(fhirStoreName, store)
-            .setUpdateMask("notificationConfig");
+    FhirStores.Patch request = client
+        .projects()
+        .locations()
+        .datasets()
+        .fhirStores()
+        .patch(fhirStoreName, store)
+        .setUpdateMask("notificationConfigs");
 
     // Execute the request and process the results.
     store = request.execute();
-    System.out.println("Fhir store patched: \n" + store.toPrettyString());
+    System.out.println("FHIR store patched: \n" + store.toPrettyString());
   }
 
   private static CloudHealthcare createClient() throws IOException {
     // Use Application Default Credentials (ADC) to authenticate the requests
-    // For more information see https://cloud.google.com/docs/authentication/production
-    GoogleCredentials credential =
-        GoogleCredentials.getApplicationDefault()
-            .createScoped(Collections.singleton(CloudHealthcareScopes.CLOUD_PLATFORM));
+    // For more information see
+    // https://cloud.google.com/docs/authentication/production
+    GoogleCredentials credential = GoogleCredentials.getApplicationDefault()
+        .createScoped(Collections.singleton(CloudHealthcareScopes.CLOUD_PLATFORM));
 
-    // Create a HttpRequestInitializer, which will provide a baseline configuration to all requests.
-    HttpRequestInitializer requestInitializer =
-        request -> {
-          new HttpCredentialsAdapter(credential).initialize(request);
-          request.setConnectTimeout(60000); // 1 minute connect timeout
-          request.setReadTimeout(60000); // 1 minute read timeout
-        };
+    // Create a HttpRequestInitializer, which will provide a baseline configuration
+    // to all requests.
+    HttpRequestInitializer requestInitializer = request -> {
+      new HttpCredentialsAdapter(credential).initialize(request);
+      request.setConnectTimeout(60000); // 1 minute connect timeout
+      request.setReadTimeout(60000); // 1 minute read timeout
+    };
 
     // Build the client for interacting with the service.
     return new CloudHealthcare.Builder(HTTP_TRANSPORT, JSON_FACTORY, requestInitializer)
diff --git a/healthcare/v1/src/test/java/snippets/healthcare/FhirStoreTests.java b/healthcare/v1/src/test/java/snippets/healthcare/FhirStoreTests.java
@@ -76,8 +76,11 @@ public static void checkRequirements() {
   @BeforeClass
   public static void setUp() throws IOException {
     String datasetId = "dataset-" + UUID.randomUUID().toString().replaceAll("-", "_");
-    datasetName =
-        String.format("projects/%s/locations/%s/datasets/%s", PROJECT_ID, REGION_ID, datasetId);
+    datasetName = String.format(
+        "projects/%s/locations/%s/datasets/%s",
+        PROJECT_ID,
+        REGION_ID,
+        datasetId);
     DatasetCreate.datasetCreate(PROJECT_ID, REGION_ID, datasetId);
   }
 
@@ -159,7 +162,7 @@ public void test_FhirStorePatch() throws Exception {
     FhirStorePatch.fhirStorePatch(fhirStoreName, GCLOUD_PUBSUB_TOPIC);
 
     String output = bout.toString();
-    assertThat(output, containsString("Fhir store patched:"));
+    assertThat(output, containsString("FHIR store patched:"));
   }
 
   @Test
