Document webhook setters (#888)

box · Apr 30, 2021 · 63c9f8c · 63c9f8c
1 parent 7d55c66
commit 63c9f8c
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 6 deletions.
diff --git a/doc/webhooks.md b/doc/webhooks.md
@@ -89,8 +89,8 @@ A webhook can be updated by calling the [`update(BoxWebHook.Info fieldsToUpdate)
 <!-- sample put_webhooks_id -->
 ```java
 BoxWebHook webhook = new BoxWebHook(api, id);
-BoxWebHook.Info info = webhook.getInfo();
-info.addPendingChange("address", url);
+BoxWebHook.Info info = webhook.new Info();
+info.setAddress(url);
 webhook.update(info);
 ```
 

diff --git a/src/test/java/com/box/sdk/BoxWebHookTest.java b/src/test/java/com/box/sdk/BoxWebHookTest.java
@@ -250,8 +250,12 @@ public void testUpdateWebhookInfoSucceeds() throws IOException {
         final String webhookID = "12345";
         final String createdByLogin = "test@user.com";
         final String webhookURL = "/webhooks/" + webhookID;
+
+        JsonArray triggers = new JsonArray()
+            .add("FILE.UPLOADED");
+
         JsonObject updateObject = new JsonObject()
-                .add("triggers", "FILE.UPLOADED")
+                .add("triggers", triggers)
                 .add("address", newAddress);
 
         getResult = TestConfig.getFixture("BoxWebhook/GetWebhook200");
@@ -270,9 +274,9 @@ public void testUpdateWebhookInfoSucceeds() throws IOException {
                         .withBody(result)));
 
         BoxWebHook webhook = new BoxWebHook(this.api, webhookID);
-        BoxWebHook.Info info = webhook.getInfo();
-        info.addPendingChange("address", newAddress);
-        info.addPendingChange("triggers", "FILE.UPLOADED");
+        BoxWebHook.Info info = webhook.new Info();
+        info.setAddress(new URL(newAddress));
+        info.setTriggers(BoxWebHook.Trigger.FILE_UPLOADED);
         webhook.updateInfo(info);
 
         Assert.assertEquals(new URL(newAddress), info.getAddress());