feat: Getting collaborators allows to specify fields

src/intTest/java/com/box/sdk/BoxFileIT.java

@@ -124,19 +124,21 @@ public void getInfoWithRepresentationsIntegrationTestWithComplexHint() {
     }
 
     @Test
-    public void getRepresentationContentSucceeds() {
+    public void getRepresentationContentSucceeds() throws InterruptedException {
         BoxAPIConnection api = jwtApiForServiceAccount();
         String fileName = "smalltest.pdf";
         BoxFile file = null;
         try {
             file = uploadSampleFileToUniqueFolder(api, fileName);
-
+            final String fileId = file.getID();
             String representationHint = "[jpg?dimensions=32x32]";
             ByteArrayOutputStream outputStream = new ByteArrayOutputStream();
-            file.getRepresentationContent(representationHint, outputStream);
-            byte[] downloadedRepresentationContent = outputStream.toByteArray();
+            Retry.retry(() -> {
+                new BoxFile(api, fileId).getRepresentationContent(representationHint, outputStream);
+                byte[] downloadedRepresentationContent = outputStream.toByteArray();
 
-            assertNotNull(downloadedRepresentationContent);
+                assertNotNull(downloadedRepresentationContent);
+            }, 5, 100);
         } finally {
             deleteFile(file);
         }

src/intTest/java/com/box/sdk/MetadataIT.java

@@ -191,8 +191,6 @@ public void testMultiSelectMetadataCRUD() {
 
             // Update instance multiselect field
             actualMD.test("/" + fieldKey + "/0", "bar");
-            //todo: this check always fails
-//            actualMD.test("/" + fieldKey + "/1", "foo");
             values = new ArrayList<>();
             values.add("two");
             values.add("one");
@@ -203,13 +201,15 @@ public void testMultiSelectMetadataCRUD() {
 
             multiSelectValues = updatedMD.getMultiSelect("/" + fieldKey);
             assertThat(multiSelectValues, Matchers.hasSize(2));
-            assertThat(multiSelectValues, containsInAnyOrder("blargh", "foooooo"));
+            assertThat(multiSelectValues, containsInAnyOrder("foooooo", "bar"));
             multiSelectValues = updatedMD.getMultiSelect("/otherMultiSelect");
             assertThat(multiSelectValues, hasSize(2));
             assertThat(multiSelectValues, containsInAnyOrder("one", "two"));
 
             // Delete metadata template and folder
         } finally {
+            System.out.printf("Template [ID %s] [Key %s]%n", template.getID(), template.getTemplateKey());
+            System.out.printf("Folder [ID %s]%n", folder.getID());
             this.deleteMetadata(api, template);
             deleteFolder(folder);
 

src/main/java/com/box/sdk/BoxCollaboration.java

@@ -130,8 +130,14 @@ protected static BoxCollaboration.Info create(
      * @param api the API connection to use.
      * @return a collection of pending collaboration infos.
      */
-    public static Collection<Info> getPendingCollaborations(BoxAPIConnection api) {
-        URL url = PENDING_COLLABORATIONS_URL.build(api.getBaseURL());
+    public static Collection<Info> getPendingCollaborations(BoxAPIConnection api, String... fields) {
+        QueryStringBuilder queryBuilder = new QueryStringBuilder();
+        queryBuilder.appendParam("status", "pending");
+        if (fields.length > 0) {
+            queryBuilder.appendParam("fields", fields);
+        }
+        URL url = COLLABORATIONS_URL_TEMPLATE.buildWithQuery(api.getBaseURL(), queryBuilder.toString());
+
 
         BoxJSONRequest request = new BoxJSONRequest(api, url, "GET");
         try (BoxJSONResponse response = request.send()) {

src/main/java/com/box/sdk/BoxFile.java

@@ -274,7 +274,7 @@ public BoxTask.Info addTask(BoxTask.Action action, String message, Date dueAt,
      * @return the temporary download URL
      */
     public URL getDownloadURL() {
-        URL url = CONTENT_URL_TEMPLATE.build(this.getAPI().getBaseURL(), this.getID());
+        URL url = getDownloadUrl();
         BoxAPIRequest request = new BoxAPIRequest(this.getAPI(), url, "GET");
         request.setFollowRedirects(false);
 
@@ -305,7 +305,7 @@ public void download(OutputStream output) {
      * @param listener a listener for monitoring the download's progress.
      */
     public void download(OutputStream output, ProgressListener listener) {
-        URL url = CONTENT_URL_TEMPLATE.build(this.getAPI().getBaseURL(), this.getID());
+        URL url = getDownloadUrl();
         BoxAPIRequest request = new BoxAPIRequest(this.getAPI(), url, "GET");
         BoxAPIResponse response = request.send();
         writeStream(response, output, listener);
@@ -342,7 +342,7 @@ public void downloadRange(OutputStream output, long rangeStart, long rangeEnd) {
      * @param listener   a listener for monitoring the download's progress.
      */
     public void downloadRange(OutputStream output, long rangeStart, long rangeEnd, ProgressListener listener) {
-        URL url = CONTENT_URL_TEMPLATE.build(this.getAPI().getBaseURL(), this.getID());
+        URL url = getDownloadUrl();
         BoxAPIRequest request = new BoxAPIRequest(this.getAPI(), url, "GET");
         if (rangeEnd > 0) {
             request.addHeader("Range", String.format("bytes=%s-%s", rangeStart, rangeEnd));
@@ -352,6 +352,14 @@ public void downloadRange(OutputStream output, long rangeStart, long rangeEnd, P
         writeStream(request.send(), output, listener);
     }
 
+    /**
+     * Can be used to override the URL used for file download.
+     * @return URL for file downalod
+     */
+    protected URL getDownloadUrl() {
+        return CONTENT_URL_TEMPLATE.build(this.getAPI().getBaseURL(), this.getID());
+    }
+
     @Override
     public BoxFile.Info copy(BoxFolder destination) {
         return this.copy(destination, null);
@@ -516,6 +524,11 @@ public void getRepresentationContent(String representationHint, String assetPath
                 String repContentURLString = null;
                 while (repContentURLString == null) {
                     repContentURLString = this.pollRepInfo(representation.getInfo().getUrl());
+                    try {
+                        Thread.sleep(100);
+                    } catch (InterruptedException e) {
+                        throw new RuntimeException(e);
+                    }
                 }
 
                 this.makeRepresentationContentRequest(repContentURLString, assetPath, output);
@@ -649,7 +662,7 @@ public boolean canUploadVersion(String name) {
      */
     public boolean canUploadVersion(String name, long fileSize) {
 
-        URL url = CONTENT_URL_TEMPLATE.build(this.getAPI().getBaseURL(), this.getID());
+        URL url = getDownloadUrl();
         BoxJSONRequest request = new BoxJSONRequest(this.getAPI(), url, "OPTIONS");
 
         JsonObject preflightInfo = new JsonObject();

src/main/java/com/box/sdk/BoxFolder.java

@@ -258,9 +258,14 @@ private BoxSharedLink removeCanEditPermissionIfSet(BoxSharedLink sharedLink) {
      *
      * @return a collection of information about the collaborations for this folder.
      */
-    public Collection<BoxCollaboration.Info> getCollaborations() {
+    public Collection<BoxCollaboration.Info> getCollaborations(String... fields) {
         BoxAPIConnection api = this.getAPI();
-        URL url = GET_COLLABORATIONS_URL.build(api.getBaseURL(), this.getID());
+        QueryStringBuilder queryBuilder = new QueryStringBuilder();
+        if (fields.length > 0) {
+            queryBuilder.appendParam("fields", fields);
+        }
+        URL url = GET_COLLABORATIONS_URL.buildWithQuery(api.getBaseURL(), queryBuilder.toString(), this.getID());
+
 
         BoxJSONRequest request = new BoxJSONRequest(api, url, "GET");
         try (BoxJSONResponse response = request.send()) {

src/test/Fixtures/BoxFolder/GetAllFolderCollaborations200.json

@@ -1,5 +1,5 @@
 {
-  "total_count": 1,
+  "total_count": 2,
   "entries": [
     {
       "type": "collaboration",

src/test/Fixtures/BoxFolder/GetAllFolderCollaborations200WithFields.json

@@ -0,0 +1,29 @@
+{
+  "total_count": 2,
+  "entries": [
+    {
+      "type": "collaboration",
+      "id": "12345",
+      "item": {
+        "type": "folder",
+        "id": "3333",
+        "sequence_id": "1",
+        "etag": "1",
+        "name": "Test Folder"
+      },
+      "can_view_path": false
+    },
+    {
+      "type": "collaboration",
+      "id": "124783",
+      "item": {
+        "type": "folder",
+        "id": "3333",
+        "sequence_id": "1",
+        "etag": "1",
+        "name": "Test Folder"
+      },
+      "can_view_path": true
+    }
+  ]
+}

src/test/java/com/box/sdk/BoxCollaborationTest.java

@@ -104,6 +104,26 @@ public void testGetPendingCollaborationInfoSucceeds() {
         assertEquals(BoxCollaboration.Role.EDITOR, pendingCollabInfo.getRole());
     }
 
+    @Test
+    public void testGetPendingCollaborationInfoWithFieldsSucceeds() {
+        final String collaborationURL = "/2.0/collaborations?status=pending&fields=id%2Crole%2Cstatus";
+
+        String result = TestUtils.getFixture("BoxCollaboration/GetPendingCollaborationInfo200");
+
+        wireMockRule.stubFor(WireMock.get(WireMock.urlEqualTo(collaborationURL))
+            .withQueryParam("status", WireMock.containing("pending"))
+            .willReturn(WireMock.aResponse()
+                .withHeader("Content-Type", APPLICATION_JSON)
+                .withBody(result)));
+
+        Collection<BoxCollaboration.Info> pendingCollaborations =
+            BoxCollaboration.getPendingCollaborations(this.api, "id", "role", "status");
+        BoxCollaboration.Info pendingCollabInfo = pendingCollaborations.iterator().next();
+
+        assertEquals(BoxCollaboration.Status.PENDING, pendingCollabInfo.getStatus());
+        assertEquals(BoxCollaboration.Role.EDITOR, pendingCollabInfo.getRole());
+    }
+
     @Test
     public void testGetCollaborationsOnFolderSucceeds() {
         final String folderID = "12345";

src/test/java/com/box/sdk/BoxFolderTest.java

@@ -798,6 +798,33 @@ public void testGetAllFolderCollaborationsSucceeds() {
             collaborationInfo2.getAccessibleBy().getGroupType());
     }
 
+    @Test
+    public void testGetAllFolderCollaborationsWithFieldsSucceeds() {
+        final String folderID = "3333";
+        final String folderCollaborationURL = "/2.0/folders/" + folderID
+            + "/collaborations?fields=id%2Ctype%2Ccan_view_path";
+
+        String result = TestUtils.getFixture("BoxFolder/GetAllFolderCollaborations200WithFields");
+
+        wireMockRule.stubFor(WireMock.get(WireMock.urlEqualTo(folderCollaborationURL))
+            .willReturn(WireMock.aResponse()
+                .withHeader("Content-Type", APPLICATION_JSON)
+                .withBody(result)));
+
+        BoxFolder folder = new BoxFolder(this.api, folderID);
+        Collection<BoxCollaboration.Info> collaborations = folder.getCollaborations("id", "type", "can_view_path");
+
+        Iterator<BoxCollaboration.Info> iterator = collaborations.iterator();
+
+        BoxCollaboration.Info collaborationInfo = iterator.next();
+        assertEquals("12345", collaborationInfo.getID());
+        assertFalse(collaborationInfo.getCanViewPath());
+
+        BoxCollaboration.Info collaborationInfo2 = iterator.next();
+        assertEquals("124783", collaborationInfo2.getID());
+        assertTrue(collaborationInfo2.getCanViewPath());
+    }
+
     @Test
     public void testCreateMetadataOnFolderSucceedsAndSendsCorrectJson() {
         final String folderID = "12345";