diff --git a/source/test-java/org/alfresco/rest/api/tests/TestSiteMembers.java b/source/test-java/org/alfresco/rest/api/tests/TestSiteMembers.java index 8537f54ad4..0e2f50829c 100644 --- a/source/test-java/org/alfresco/rest/api/tests/TestSiteMembers.java +++ b/source/test-java/org/alfresco/rest/api/tests/TestSiteMembers.java @@ -41,17 +41,21 @@ import org.alfresco.repo.tenant.TenantUtil.TenantRunAsWork; import org.alfresco.rest.api.tests.RepoService.TestNetwork; import org.alfresco.rest.api.tests.RepoService.TestPerson; import org.alfresco.rest.api.tests.RepoService.TestSite; +import org.alfresco.rest.api.tests.client.HttpResponse; import org.alfresco.rest.api.tests.client.PublicApiClient.ListResponse; import org.alfresco.rest.api.tests.client.PublicApiClient.Paging; import org.alfresco.rest.api.tests.client.PublicApiClient.Sites; import org.alfresco.rest.api.tests.client.PublicApiException; import org.alfresco.rest.api.tests.client.RequestContext; import org.alfresco.rest.api.tests.client.data.Person; +import org.alfresco.rest.api.tests.client.data.Site; +import org.alfresco.rest.api.tests.client.data.SiteImpl; import org.alfresco.rest.api.tests.client.data.SiteMember; import org.alfresco.rest.api.tests.client.data.SiteRole; import org.alfresco.service.cmr.site.SiteVisibility; import org.alfresco.util.GUID; import org.apache.commons.httpclient.HttpStatus; +import org.json.simple.JSONObject; import org.junit.Test; public class TestSiteMembers extends EnterpriseTestApi @@ -113,6 +117,14 @@ public class TestSiteMembers extends EnterpriseTestApi publicApiClient.setRequestContext(new RequestContext(testNetwork.getId(), personId)); ListResponse siteMembers = sitesProxy.getSiteMembers(testSite.getSiteId(), createParams(paging, null)); checkList(expectedSiteMembers.subList(skipCount, skipCount + paging.getExpectedPaging().getCount()), paging.getExpectedPaging(), siteMembers); + + HttpResponse response = sitesProxy.getAll("sites", testSite.getSiteId(), "members", null, createParams(paging,Collections.singletonMap("includeSource", "true")), "Failed to get all site members"); + checkList(expectedSiteMembers.subList(skipCount, skipCount + paging.getExpectedPaging().getCount()), paging.getExpectedPaging(), SiteMember.parseSiteMembers(testSite.getSiteId(), response.getJsonResponse())); + JSONObject source = sitesProxy.parseListSource(response.getJsonResponse()); + Site sourceSite = SiteImpl.parseSite(source); + assertNotNull(sourceSite); + assertEquals(testSite.getSiteId(), sourceSite.getSiteId()); + assertEquals(testSite.getGuid(), sourceSite.getGuid()); } // invalid site id diff --git a/source/test-java/org/alfresco/rest/api/tests/client/PublicApiClient.java b/source/test-java/org/alfresco/rest/api/tests/client/PublicApiClient.java index 16e4974f1c..7a4fe42a5f 100644 --- a/source/test-java/org/alfresco/rest/api/tests/client/PublicApiClient.java +++ b/source/test-java/org/alfresco/rest/api/tests/client/PublicApiClient.java @@ -25,6 +25,8 @@ */ package org.alfresco.rest.api.tests.client; +import static org.junit.Assert.assertNotNull; + import java.io.IOException; import java.io.InputStream; import java.io.Serializable; @@ -750,6 +752,16 @@ public class PublicApiClient throw new PublicApiException(e); } } + + public JSONObject parseListSource(JSONObject jsonResponse) + { + JSONObject jsonList = (JSONObject)jsonResponse.get("list"); + assertNotNull(jsonList); + + JSONObject source = (JSONObject)jsonList.get("source"); + assertNotNull(source); + return source; + } } public static class ListResponse diff --git a/source/test-java/org/alfresco/rest/workflow/api/tests/ListParser.java b/source/test-java/org/alfresco/rest/workflow/api/tests/ListParser.java index 5f452a7d18..cda0a13975 100644 --- a/source/test-java/org/alfresco/rest/workflow/api/tests/ListParser.java +++ b/source/test-java/org/alfresco/rest/workflow/api/tests/ListParser.java @@ -40,7 +40,7 @@ public abstract class ListParser public ListResponse parseList(JSONObject jsonResponse) { - List deployments = new ArrayList(); + List entries = new ArrayList(); JSONObject jsonList = (JSONObject)jsonResponse.get("list"); assertNotNull(jsonList); @@ -52,12 +52,12 @@ public abstract class ListParser { JSONObject jsonEntry = (JSONObject)jsonEntries.get(i); JSONObject entry = (JSONObject)jsonEntry.get("entry"); - deployments.add(parseEntry(entry)); + entries.add(parseEntry(entry)); } ExpectedPaging paging = ExpectedPaging.parsePagination(jsonList); - ListResponse resp = new ListResponse(paging, deployments); + ListResponse resp = new ListResponse(paging, entries); return resp; }