diff --git a/e2e-test/java/org/alfresco/rest/sites/AddSiteMembershipRequestSanityTest.java b/e2e-test/java/org/alfresco/rest/sites/AddSiteMembershipRequestSanityTest.java index 36183ed72..161382860 100644 --- a/e2e-test/java/org/alfresco/rest/sites/AddSiteMembershipRequestSanityTest.java +++ b/e2e-test/java/org/alfresco/rest/sites/AddSiteMembershipRequestSanityTest.java @@ -7,9 +7,9 @@ import org.alfresco.rest.RestTest; import org.alfresco.rest.body.SiteMembership; import org.alfresco.rest.exception.JsonToModelConversionException; 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.UserRole; import org.alfresco.utility.exception.DataPreparationException; import org.alfresco.utility.model.SiteModel; import org.alfresco.utility.model.UserModel; @@ -42,7 +42,7 @@ public class AddSiteMembershipRequestSanityTest extends RestTest { adminUser = dataUser.getAdminUser(); siteModel = dataSite.usingUser(adminUser).createPublicRandomSite(); - usersWithRoles = dataUser.addUsersToSiteWithRoles(siteModel, + usersWithRoles = dataUser.addUsersWithRolesToSite(siteModel, Arrays.asList(UserRole.SiteManager, UserRole.SiteCollaborator, UserRole.SiteConsumer, UserRole.SiteContributor)); siteAPI.useRestClient(restClient); diff --git a/e2e-test/java/org/alfresco/rest/sites/GetSiteMembershipInformationSanityTest.java b/e2e-test/java/org/alfresco/rest/sites/GetSiteMembershipInformationSanityTest.java index 2437bdffa..e395d0177 100644 --- a/e2e-test/java/org/alfresco/rest/sites/GetSiteMembershipInformationSanityTest.java +++ b/e2e-test/java/org/alfresco/rest/sites/GetSiteMembershipInformationSanityTest.java @@ -6,9 +6,10 @@ import java.util.HashMap; import org.alfresco.rest.RestTest; import org.alfresco.rest.exception.JsonToModelConversionException; 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.UserRole; +import org.alfresco.utility.data.TestData; import org.alfresco.utility.exception.DataPreparationException; import org.alfresco.utility.model.SiteModel; import org.alfresco.utility.model.UserModel; @@ -40,7 +41,7 @@ public class GetSiteMembershipInformationSanityTest extends RestTest { adminUser = dataUser.getAdminUser(); siteModel = dataSite.usingUser(adminUser).createPublicRandomSite(); - usersWithRoles = dataUser.addUsersToSiteWithRoles(siteModel, + usersWithRoles = dataUser.addUsersWithRolesToSite(siteModel, Arrays.asList(UserRole.SiteManager, UserRole.SiteCollaborator, UserRole.SiteConsumer, UserRole.SiteContributor)); siteAPI.useRestClient(restClient); @@ -100,5 +101,17 @@ public class GetSiteMembershipInformationSanityTest extends RestTest siteAPI.usingRestWrapper() .assertStatusCodeIs(HttpStatus.OK.toString()); } + + @TestRail(section = { "rest-api", "sites" }, + executionType = ExecutionType.SANITY, + description = "Verify that unauthenticated user is not able to retrieve site membership information") + public void unauthenticatedUserCannotRetrieveSiteMembershipInformation() throws JsonToModelConversionException, Exception + { + UserModel inexistentUser = new UserModel("inexistent user", "wrong password"); + restClient.authenticateUser(inexistentUser); + siteAPI.getSiteMembershipInformation(usersWithRoles.get(UserRole.SiteManager).getUsername()); + siteAPI.usingRestWrapper() + .assertStatusCodeIs(HttpStatus.UNAUTHORIZED.toString()); + } } \ No newline at end of file