From 4954d740fa3ef7638847fb758fe9c419d5ff2c93 Mon Sep 17 00:00:00 2001 From: Iulia Nechita Date: Fri, 30 Sep 2016 10:21:05 +0300 Subject: [PATCH 1/7] TAS-978 getSiteContainer with Manager Role --- .../sites/GetSiteContainerSanityTests.java | 61 +++++++++++++++++++ 1 file changed, 61 insertions(+) create mode 100644 e2e-test/java/org/alfresco/rest/sites/GetSiteContainerSanityTests.java diff --git a/e2e-test/java/org/alfresco/rest/sites/GetSiteContainerSanityTests.java b/e2e-test/java/org/alfresco/rest/sites/GetSiteContainerSanityTests.java new file mode 100644 index 000000000..b25291a87 --- /dev/null +++ b/e2e-test/java/org/alfresco/rest/sites/GetSiteContainerSanityTests.java @@ -0,0 +1,61 @@ +package org.alfresco.rest.sites; + +import java.util.List; + +import org.alfresco.rest.RestTest; +import org.alfresco.rest.exception.JsonToModelConversionException; +import org.alfresco.rest.model.RestSiteContainerModel; +import org.alfresco.rest.requests.RestSitesApi; +import org.alfresco.utility.constants.UserRole; +import org.alfresco.utility.data.DataSite; +import org.alfresco.utility.data.DataUser; +import org.alfresco.utility.data.DataUser.ListUserWithRoles; +import org.alfresco.utility.model.SiteModel; +import org.alfresco.utility.model.UserModel; +import org.alfresco.utility.testrail.ExecutionType; +import org.alfresco.utility.testrail.annotation.TestRail; +import org.springframework.beans.factory.annotation.Autowired; +import org.springframework.http.HttpStatus; +import org.testng.annotations.BeforeClass; +import org.testng.annotations.Test; + +/** + * @author iulia.cojocea + */ +@Test(groups = { "rest-api", "sites", "sanity" }) +public class GetSiteContainerSanityTests extends RestTest +{ + @Autowired + RestSitesApi siteAPI; + + @Autowired + DataUser dataUser; + + @Autowired + DataSite dataSite; + + private UserModel adminUserModel; + private SiteModel siteModel; + private ListUserWithRoles usersWithRoles; + private List listOfFoldersIds; + + @BeforeClass(alwaysRun=true) + public void initTest() throws Exception + { + adminUserModel = dataUser.getAdminUser(); + siteAPI.useRestClient(restClient); + siteModel = dataSite.usingUser(adminUserModel).createPublicRandomSite(); + usersWithRoles = dataUser.addUsersWithRolesToSite(siteModel, UserRole.SiteManager, UserRole.SiteCollaborator, UserRole.SiteConsumer, + UserRole.SiteContributor); + } + + @TestRail(section = { "rest-api", "sites" }, executionType = ExecutionType.SANITY, + description = "Verify user with Manager role gets site container and gets status code OK (200)") + public void getSiteContainerWithManagerRole() throws JsonToModelConversionException, Exception + { + restClient.authenticateUser(usersWithRoles.getOneUserWithRole(UserRole.SiteManager)); + listOfFoldersIds = siteAPI.getSiteContainers(siteModel).getSiteContainersList(); + siteAPI.getSiteContainer(siteModel, listOfFoldersIds.get(0)); + siteAPI.usingRestWrapper().assertStatusCodeIs(HttpStatus.OK); + } +} From 761a41e28e77e3488799214f02445de19a3ba03e Mon Sep 17 00:00:00 2001 From: Iulia Nechita Date: Fri, 30 Sep 2016 10:23:50 +0300 Subject: [PATCH 2/7] TAS-979 getSiteContainer with Collaborator role --- .../rest/sites/GetSiteContainerSanityTests.java | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/e2e-test/java/org/alfresco/rest/sites/GetSiteContainerSanityTests.java b/e2e-test/java/org/alfresco/rest/sites/GetSiteContainerSanityTests.java index b25291a87..36ceea1e6 100644 --- a/e2e-test/java/org/alfresco/rest/sites/GetSiteContainerSanityTests.java +++ b/e2e-test/java/org/alfresco/rest/sites/GetSiteContainerSanityTests.java @@ -58,4 +58,14 @@ public class GetSiteContainerSanityTests extends RestTest siteAPI.getSiteContainer(siteModel, listOfFoldersIds.get(0)); siteAPI.usingRestWrapper().assertStatusCodeIs(HttpStatus.OK); } + + @TestRail(section = { "rest-api", "sites" }, executionType = ExecutionType.SANITY, + description = "Verify user with Collaborator role gets site container and gets status code OK (200)") + public void getSiteContainerWithCollaboratorRole() throws JsonToModelConversionException, Exception + { + restClient.authenticateUser(usersWithRoles.getOneUserWithRole(UserRole.SiteCollaborator)); + listOfFoldersIds = siteAPI.getSiteContainers(siteModel).getSiteContainersList(); + siteAPI.getSiteContainer(siteModel, listOfFoldersIds.get(0)); + siteAPI.usingRestWrapper().assertStatusCodeIs(HttpStatus.OK); + } } From 0bf3be833993c5a8d298acd7a1d6ab4706755eef Mon Sep 17 00:00:00 2001 From: Iulia Nechita Date: Fri, 30 Sep 2016 10:27:39 +0300 Subject: [PATCH 3/7] TAS-980 getSiteContainer with Contributor role --- .../rest/sites/GetSiteContainerSanityTests.java | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/e2e-test/java/org/alfresco/rest/sites/GetSiteContainerSanityTests.java b/e2e-test/java/org/alfresco/rest/sites/GetSiteContainerSanityTests.java index 36ceea1e6..2c1ce48ce 100644 --- a/e2e-test/java/org/alfresco/rest/sites/GetSiteContainerSanityTests.java +++ b/e2e-test/java/org/alfresco/rest/sites/GetSiteContainerSanityTests.java @@ -58,7 +58,7 @@ public class GetSiteContainerSanityTests extends RestTest siteAPI.getSiteContainer(siteModel, listOfFoldersIds.get(0)); siteAPI.usingRestWrapper().assertStatusCodeIs(HttpStatus.OK); } - + @TestRail(section = { "rest-api", "sites" }, executionType = ExecutionType.SANITY, description = "Verify user with Collaborator role gets site container and gets status code OK (200)") public void getSiteContainerWithCollaboratorRole() throws JsonToModelConversionException, Exception @@ -68,4 +68,14 @@ public class GetSiteContainerSanityTests extends RestTest siteAPI.getSiteContainer(siteModel, listOfFoldersIds.get(0)); siteAPI.usingRestWrapper().assertStatusCodeIs(HttpStatus.OK); } + + @TestRail(section = { "rest-api", "sites" }, executionType = ExecutionType.SANITY, + description = "Verify user with Contributor role gets site container and gets status code OK (200)") + public void getSiteContainerWithContributorRole() throws JsonToModelConversionException, Exception + { + restClient.authenticateUser(usersWithRoles.getOneUserWithRole(UserRole.SiteContributor)); + listOfFoldersIds = siteAPI.getSiteContainers(siteModel).getSiteContainersList(); + siteAPI.getSiteContainer(siteModel, listOfFoldersIds.get(0)); + siteAPI.usingRestWrapper().assertStatusCodeIs(HttpStatus.OK); + } } From ba186fea036488509ea1dc8e799157c81b600fc7 Mon Sep 17 00:00:00 2001 From: Iulia Nechita Date: Fri, 30 Sep 2016 10:28:40 +0300 Subject: [PATCH 4/7] TAS-981 getSiteContainer with Consumer role --- .../rest/sites/GetSiteContainerSanityTests.java | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/e2e-test/java/org/alfresco/rest/sites/GetSiteContainerSanityTests.java b/e2e-test/java/org/alfresco/rest/sites/GetSiteContainerSanityTests.java index 2c1ce48ce..96324027f 100644 --- a/e2e-test/java/org/alfresco/rest/sites/GetSiteContainerSanityTests.java +++ b/e2e-test/java/org/alfresco/rest/sites/GetSiteContainerSanityTests.java @@ -78,4 +78,14 @@ public class GetSiteContainerSanityTests extends RestTest siteAPI.getSiteContainer(siteModel, listOfFoldersIds.get(0)); siteAPI.usingRestWrapper().assertStatusCodeIs(HttpStatus.OK); } + + @TestRail(section = { "rest-api", "sites" }, executionType = ExecutionType.SANITY, + description = "Verify user with Consumer role gets site container and gets status code OK (200)") + public void getSiteContainerWithConsumerRole() throws JsonToModelConversionException, Exception + { + restClient.authenticateUser(usersWithRoles.getOneUserWithRole(UserRole.SiteConsumer)); + listOfFoldersIds = siteAPI.getSiteContainers(siteModel).getSiteContainersList(); + siteAPI.getSiteContainer(siteModel, listOfFoldersIds.get(0)); + siteAPI.usingRestWrapper().assertStatusCodeIs(HttpStatus.OK); + } } From d16a8618d259e15335b7f3f187e1fd2c964c0575 Mon Sep 17 00:00:00 2001 From: Iulia Nechita Date: Fri, 30 Sep 2016 10:30:27 +0300 Subject: [PATCH 5/7] TAS-982 getSiteContainer with admin user --- .../rest/sites/GetSiteContainerSanityTests.java | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/e2e-test/java/org/alfresco/rest/sites/GetSiteContainerSanityTests.java b/e2e-test/java/org/alfresco/rest/sites/GetSiteContainerSanityTests.java index 96324027f..060214fe8 100644 --- a/e2e-test/java/org/alfresco/rest/sites/GetSiteContainerSanityTests.java +++ b/e2e-test/java/org/alfresco/rest/sites/GetSiteContainerSanityTests.java @@ -88,4 +88,14 @@ public class GetSiteContainerSanityTests extends RestTest siteAPI.getSiteContainer(siteModel, listOfFoldersIds.get(0)); siteAPI.usingRestWrapper().assertStatusCodeIs(HttpStatus.OK); } + + @TestRail(section = { "rest-api", "sites" }, executionType = ExecutionType.SANITY, + description = "Verify user with admin user gets site container and gets status code OK (200)") + public void getSiteContainerWithAdminUser() throws JsonToModelConversionException, Exception + { + restClient.authenticateUser(adminUserModel); + listOfFoldersIds = siteAPI.getSiteContainers(siteModel).getSiteContainersList(); + siteAPI.getSiteContainer(siteModel, listOfFoldersIds.get(0)); + siteAPI.usingRestWrapper().assertStatusCodeIs(HttpStatus.OK); + } } From 69f7ec4e6c5cab6e151a7ecc63b885f1745367b5 Mon Sep 17 00:00:00 2001 From: Iulia Nechita Date: Fri, 30 Sep 2016 10:35:25 +0300 Subject: [PATCH 6/7] TAS-983 getSiteContainer call returns status code 401 with Manager role --- .../rest/sites/GetSiteContainerSanityTests.java | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/e2e-test/java/org/alfresco/rest/sites/GetSiteContainerSanityTests.java b/e2e-test/java/org/alfresco/rest/sites/GetSiteContainerSanityTests.java index 060214fe8..1286ab2e4 100644 --- a/e2e-test/java/org/alfresco/rest/sites/GetSiteContainerSanityTests.java +++ b/e2e-test/java/org/alfresco/rest/sites/GetSiteContainerSanityTests.java @@ -38,6 +38,7 @@ public class GetSiteContainerSanityTests extends RestTest private SiteModel siteModel; private ListUserWithRoles usersWithRoles; private List listOfFoldersIds; + private UserModel userModel; @BeforeClass(alwaysRun=true) public void initTest() throws Exception @@ -98,4 +99,18 @@ public class GetSiteContainerSanityTests extends RestTest siteAPI.getSiteContainer(siteModel, listOfFoldersIds.get(0)); siteAPI.usingRestWrapper().assertStatusCodeIs(HttpStatus.OK); } + + @TestRail(section = { "rest-api", "sites" }, executionType = ExecutionType.SANITY, + description = "Failed authentication get site container call returns status code 401 with Manager role") + public void unauthenticatedUserIsNotAuthorizedToRetrieveSiteContainer() throws JsonToModelConversionException, Exception + { + restClient.authenticateUser(usersWithRoles.getOneUserWithRole(UserRole.SiteManager)); + userModel = dataUser.createRandomTestUser(); + userModel.setPassword("user wrong password"); + dataUser.addUserToSite(userModel, siteModel, UserRole.SiteManager); + restClient.authenticateUser(userModel); + listOfFoldersIds = siteAPI.getSiteContainers(siteModel).getSiteContainersList(); + siteAPI.getSiteContainer(siteModel, listOfFoldersIds.get(0)); + siteAPI.usingRestWrapper().assertStatusCodeIs(HttpStatus.UNAUTHORIZED); + } } From a649e39e7beb79ad8992d7c16ef9b157e2d642df Mon Sep 17 00:00:00 2001 From: cnechifor Date: Mon, 3 Oct 2016 17:28:17 +0300 Subject: [PATCH 7/7] sample commit --- .../org/alfresco/rest/sites/GetSiteContainerSanityTests.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/e2e-test/java/org/alfresco/rest/sites/GetSiteContainerSanityTests.java b/e2e-test/java/org/alfresco/rest/sites/GetSiteContainerSanityTests.java index 1286ab2e4..50a735b1d 100644 --- a/e2e-test/java/org/alfresco/rest/sites/GetSiteContainerSanityTests.java +++ b/e2e-test/java/org/alfresco/rest/sites/GetSiteContainerSanityTests.java @@ -55,7 +55,7 @@ public class GetSiteContainerSanityTests extends RestTest public void getSiteContainerWithManagerRole() throws JsonToModelConversionException, Exception { restClient.authenticateUser(usersWithRoles.getOneUserWithRole(UserRole.SiteManager)); - listOfFoldersIds = siteAPI.getSiteContainers(siteModel).getSiteContainersList(); + listOfFoldersIds = siteAPI.getSiteContainers(siteModel).getEntries(); siteAPI.getSiteContainer(siteModel, listOfFoldersIds.get(0)); siteAPI.usingRestWrapper().assertStatusCodeIs(HttpStatus.OK); }