fix: update TestGroup and ExecutionType

e2e-test/java/org/alfresco/rest/demo/SampleCommentsTests.java

@@ -32,24 +32,21 @@ public class SampleCommentsTests extends RestTest
         document = dataContent.usingUser(userModel).usingResource(folderModel).createContent(DocumentType.TEXT_PLAIN);
     }
 
-    @TestRail(section={"demo", "sample-section"}, executionType= ExecutionType.SANITY,
-            description= "Verify admin user adds comments with Rest API and status code is 200")
+    @Test
     public void admiShouldAddComment() throws JsonToModelConversionException, Exception
     {
         restClient.withCoreAPI().usingResource(document).addComment("This is a new comment");
         restClient.assertStatusCodeIs(HttpStatus.CREATED);
     }
 
-    @TestRail(section={"demo", "sample-section"}, executionType= ExecutionType.SANITY,
-            description= "Verify admin user gets comments with Rest API and status code is 200")
+    @Test
     public void admiShouldRetrieveComments() throws Exception
     {
         restClient.withCoreAPI().usingResource(document).getNodeComments();
         restClient.assertStatusCodeIs(HttpStatus.OK);
     }
 
-    @TestRail(section={"demo", "sample-section"}, executionType= ExecutionType.SANITY,
-            description= "Verify admin user updates comments with Rest API")
+    @Test
     public void adminShouldUpdateComment() throws JsonToModelConversionException, Exception
     {
         RestCommentModel commentModel = restClient.withCoreAPI().usingResource(document).addComment("This is a new comment");

e2e-test/java/org/alfresco/rest/demo/SamplePeopleTests.java

@@ -24,8 +24,7 @@ public class SamplePeopleTests extends RestTest
         restClient.authenticateUser(adminUser);
     }
 
-    @TestRail(section={"demo", "sample-section"}, executionType= ExecutionType.SANITY,
-            description = "Verify admin user gets person with Rest API and response is not empty")
+    @Test
     public void adminShouldRetrievePerson() throws Exception
     {
         restClient.withCoreAPI().usingUser(userModel).getPerson().assertThat().field("id").isNotEmpty();

e2e-test/java/org/alfresco/rest/demo/SampleSitesTests.java

@@ -26,39 +26,34 @@ public class SampleSitesTests extends RestTest
         siteModel = dataSite.usingUser(userModel).createPublicRandomSite();        
     }
 
-    @TestRail(section={"demo", "sample-section"}, executionType= ExecutionType.SANITY,
-            description = "Verify admin user gets site details with Rest API and response is not empty")
+    @Test
     public void adminShouldGetSiteDetails() throws JsonToModelConversionException, Exception
     {
         restClient.withCoreAPI().usingSite(siteModel).getSite()
             .assertThat().field("id").isNotNull();
     }
 
-    @TestRail(section={"demo", "sample-section"}, executionType= ExecutionType.SANITY,
-            description = "Verify admin user gets site information and gets status code OK (200)")
+    @Test
     public void adminShouldGetSites() throws JsonToModelConversionException, Exception
     {
         restClient.withCoreAPI().usingSite(siteModel).getSite();
         restClient.assertStatusCodeIs(HttpStatus.OK);
     }
 
-    @TestRail(section={"demo", "sample-section"}, executionType= ExecutionType.SANITY,
-            description = "Verify admin user gets sites with Rest API and the response is not empty")
+    @Test
     public void adminShouldAccessSites() throws JsonToModelConversionException, Exception
     {
         restClient.withCoreAPI().getSites().assertThat().entriesListIsNotEmpty();
         restClient.assertStatusCodeIs(HttpStatus.OK);
     }
 
-    @TestRail(section={"demo", "sample-section"}, executionType= ExecutionType.SANITY,
-            description = "Verify admin user gets sites with Rest API and status code is 200")
+    @Test
     public void adminShouldAccessResponsePagination() throws JsonToModelConversionException, Exception
     {
         restClient.withCoreAPI().getSites().assertThat().paginationExist();
     }
 
-    @TestRail(section={"demo", "sample-section"}, executionType= ExecutionType.SANITY,
-            description = "Verify admin user adds site member with Rest API and status code is 201")
+    @Test
     public void adminShouldAddNewSiteMember() throws JsonToModelConversionException, DataPreparationException, Exception
     {
         UserModel testUser = dataUser.createRandomTestUser("testUser");
@@ -67,15 +62,13 @@ public class SampleSitesTests extends RestTest
         restClient.assertStatusCodeIs(HttpStatus.CREATED);
     }
 
-    @TestRail(section={"demo", "sample-section"}, executionType= ExecutionType.SANITY,
-            description = "Verify that site exists from get all sites request")
+    @Test
     public void adminShouldGetSiteFromSitesList() throws JsonToModelConversionException, Exception
     {
         restClient.withCoreAPI().getSites().assertThat().entriesListContains("id", siteModel.getId());    
     }
 
-    @TestRail(section={"demo", "sample-section"}, executionType= ExecutionType.SANITY,
-            description = "Verify site details: response not empty, description, title, visibility")
+    @Test
     public void adminShouldAccessSiteDetails() throws JsonToModelConversionException, Exception
     {
       restClient.withCoreAPI().usingSite(siteModel).getSite()

e2e-test/java/org/alfresco/rest/networks/RestGetNetworkSanityTests.java

@@ -15,7 +15,7 @@ import org.testng.annotations.Test;
 /**
  * Created by Cristina Axinte on 9/26/2016.
  */
-public class RestGetNetworkTest extends RestTest
+public class RestGetNetworkSanityTests extends RestTest
 {
     UserModel adminTenantUser;
     UserModel adminAnotherTenantUser;
@@ -39,7 +39,7 @@ public class RestGetNetworkTest extends RestTest
     @Bug(id = "MNT-16904")
     @TestRail(section = { TestGroup.REST_API, TestGroup.NETWORKS }, executionType = ExecutionType.SANITY,
             description = "Verify non existing user gets another exisiting network with Rest API and checks the forbidden status")
-    @Test(groups = { TestGroup.REST_API, TestGroup.NETWORKS, TestGroup.COMMENTS})
+    @Test(groups = { TestGroup.REST_API, TestGroup.NETWORKS, TestGroup.COMMENTS, TestGroup.SANITY})
     public void nonExistingTenantUserIsNotAuthorizedToRequest() throws Exception
     {
         UserModel tenantUser = new UserModel("nonexisting", "password");
@@ -51,7 +51,7 @@ public class RestGetNetworkTest extends RestTest
 
     @TestRail(section = { TestGroup.REST_API, TestGroup.NETWORKS }, executionType = ExecutionType.SANITY,
             description = "Verify tenant admin user gets specific network with Rest API and response is not empty")
-    @Test(groups = { TestGroup.REST_API, TestGroup.NETWORKS, TestGroup.COMMENTS })
+    @Test(groups = { TestGroup.REST_API, TestGroup.NETWORKS, TestGroup.COMMENTS, TestGroup.SANITY })
     public void adminTenantChecksIfNetworkIsPresent() throws Exception
     {
         restClient.authenticateUser(adminTenantUser);
@@ -59,18 +59,18 @@ public class RestGetNetworkTest extends RestTest
         restClient.assertStatusCodeIs(HttpStatus.OK);
     }
 
-    @TestRail(section = { TestGroup.REST_API, TestGroup.NETWORKS },
+    @TestRail(section = { TestGroup.REST_API, TestGroup.NETWORKS }, executionType = ExecutionType.SANITY,
             description = "Verify tenant admin user gets specific network with Rest API and checks response parameters are correct")
-    @Test(groups = { TestGroup.REST_API, TestGroup.NETWORKS })
+    @Test(groups = { TestGroup.REST_API, TestGroup.NETWORKS, TestGroup.SANITY })
     public void adminTenantChecksNetworkParamsAreCorrect() throws Exception
     {
         restClient.authenticateUser(adminTenantUser);
         restClient.withCoreAPI().usingNetworks().getNetwork().assertNetworkHasName(adminTenantUser).assertNetworkIsEnabled();
     }
 
-    @TestRail(section = { TestGroup.REST_API, TestGroup.NETWORKS },
+    @TestRail(section = { TestGroup.REST_API, TestGroup.NETWORKS }, executionType = ExecutionType.SANITY,
             description = "Verify tenant admin user gets non existing network with Rest API and checks the not found status")
-    @Test(groups = { TestGroup.REST_API, TestGroup.NETWORKS })
+    @Test(groups = { TestGroup.REST_API, TestGroup.NETWORKS, TestGroup.SANITY })
     public void adminTenantChecksIfNonExistingNetworkIsNotFound() throws Exception
     {
         restClient.authenticateUser(adminTenantUser);
@@ -78,9 +78,9 @@ public class RestGetNetworkTest extends RestTest
         restClient.assertStatusCodeIs(HttpStatus.NOT_FOUND);
     }
 
-    @TestRail(section = { TestGroup.REST_API, TestGroup.NETWORKS },
+    @TestRail(section = { TestGroup.REST_API, TestGroup.NETWORKS }, executionType = ExecutionType.SANITY,
             description = "Verify tenant admin user gets another existing network with Rest API and checks the forbidden status")
-    @Test(groups = { TestGroup.REST_API, TestGroup.NETWORKS })
+    @Test(groups = { TestGroup.REST_API, TestGroup.NETWORKS, TestGroup.SANITY })
     public void adminTenantChecksIfAnotherExistingNetworkIsForbidden() throws Exception
     {
         restClient.authenticateUser(adminTenantUser);
@@ -88,9 +88,9 @@ public class RestGetNetworkTest extends RestTest
         restClient.assertStatusCodeIs(HttpStatus.FORBIDDEN);
     }
 
-    @TestRail(section = { TestGroup.REST_API, TestGroup.NETWORKS },
+    @TestRail(section = { TestGroup.REST_API, TestGroup.NETWORKS }, executionType = ExecutionType.SANITY,
             description = "Verify any tenant user gets its network with Rest API and response is not empty")
-    @Test(groups = { TestGroup.REST_API, TestGroup.NETWORKS })
+    @Test(groups = { TestGroup.REST_API, TestGroup.NETWORKS, TestGroup.SANITY })
     public void userTenantChecksIfNetworkIsPresent() throws Exception
     {
         restClient.authenticateUser(tenantUser);
@@ -98,18 +98,18 @@ public class RestGetNetworkTest extends RestTest
         restClient.assertStatusCodeIs(HttpStatus.OK);
     }
 
-    @TestRail(section = { TestGroup.REST_API, TestGroup.NETWORKS },
+    @TestRail(section = { TestGroup.REST_API, TestGroup.NETWORKS }, executionType = ExecutionType.SANITY,
             description = "Verify any tenant user gets specific network with Rest API and checks response parameters are correct")
-    @Test(groups = { TestGroup.REST_API, TestGroup.NETWORKS })
+    @Test(groups = { TestGroup.REST_API, TestGroup.NETWORKS, TestGroup.SANITY })
     public void userTenantChecksNetworkParamsAreCorrect() throws Exception
     {
         restClient.authenticateUser(tenantUser);
         restClient.withCoreAPI().usingNetworks().getNetwork(adminTenantUser).assertNetworkHasName(adminTenantUser).assertNetworkIsEnabled();
     }
 
-    @TestRail(section = { TestGroup.REST_API, TestGroup.NETWORKS },
+    @TestRail(section = { TestGroup.REST_API, TestGroup.NETWORKS }, executionType = ExecutionType.SANITY,
             description = "Verify any tenant user gets non existing network with Rest API and checks the not found status")
-    @Test(groups = { TestGroup.REST_API, TestGroup.NETWORKS })
+    @Test(groups = { TestGroup.REST_API, TestGroup.NETWORKS, TestGroup.SANITY })
     public void userTenantChecksIfNonExistingNetworkIsNotFound() throws Exception
     {
         restClient.authenticateUser(tenantUser);
@@ -117,9 +117,9 @@ public class RestGetNetworkTest extends RestTest
         restClient.assertStatusCodeIs(HttpStatus.NOT_FOUND);
     }
 
-    @TestRail(section = { TestGroup.REST_API, TestGroup.NETWORKS },
+    @TestRail(section = { TestGroup.REST_API, TestGroup.NETWORKS }, executionType = ExecutionType.SANITY,
             description = "Verify any tenant user gets another existing network with Rest API and checks the forbidden status")
-    @Test(groups = { TestGroup.REST_API, TestGroup.NETWORKS, TestGroup.COMMENTS })
+    @Test(groups = { TestGroup.REST_API, TestGroup.NETWORKS, TestGroup.COMMENTS, TestGroup.SANITY })
     public void userTenantChecksIfAnotherExistingNetworkIsForbidden() throws Exception
     {
         restClient.authenticateUser(tenantUser);
@@ -129,7 +129,7 @@ public class RestGetNetworkTest extends RestTest
 
     @TestRail(section = { TestGroup.REST_API, TestGroup.NETWORKS }, executionType = ExecutionType.SANITY,
             description = "Verify manager tenant user gets its network with Rest API and response is not empty")
-    @Test(groups = { TestGroup.REST_API, TestGroup.NETWORKS, TestGroup.COMMENTS })
+    @Test(groups = { TestGroup.REST_API, TestGroup.NETWORKS, TestGroup.COMMENTS, TestGroup.SANITY })
     public void tenantManagerUserChecksIfNetworkIsPresent() throws Exception
     {
         UserModel managerTenantUser = dataUser.usingUser(adminTenantUser).createUserWithTenant("manTenant");
@@ -142,7 +142,7 @@ public class RestGetNetworkTest extends RestTest
 
     @TestRail(section = { TestGroup.REST_API, TestGroup.NETWORKS }, executionType = ExecutionType.SANITY,
             description = "Verify collaborator tenant user gets its network with Rest API and response is not empty")
-    @Test(groups = { TestGroup.REST_API, TestGroup.NETWORKS, TestGroup.COMMENTS })
+    @Test(groups = { TestGroup.REST_API, TestGroup.NETWORKS, TestGroup.COMMENTS, TestGroup.SANITY })
     public void tenantCollaboratorUserChecksIfNetworkIsPresent() throws Exception
     {
         UserModel collaboratorTenantUser = dataUser.usingUser(adminTenantUser).createUserWithTenant("colTenant");
@@ -155,7 +155,7 @@ public class RestGetNetworkTest extends RestTest
 
     @TestRail(section = { TestGroup.REST_API, TestGroup.NETWORKS }, executionType = ExecutionType.SANITY,
             description = "Verify consumer tenant user gets its network with Rest API and response is not empty")
-    @Test(groups = { TestGroup.REST_API, TestGroup.NETWORKS, TestGroup.COMMENTS })
+    @Test(groups = { TestGroup.REST_API, TestGroup.NETWORKS, TestGroup.COMMENTS, TestGroup.SANITY })
     public void tenantConsumerUserChecksIfNetworkIsPresent() throws Exception
     {
         UserModel consumerTenantUser = dataUser.usingUser(adminTenantUser).createUserWithTenant("consTenant");
@@ -168,7 +168,7 @@ public class RestGetNetworkTest extends RestTest
 
     @TestRail(section = { TestGroup.REST_API, TestGroup.NETWORKS }, executionType = ExecutionType.SANITY,
             description = "Verify contributor tenant user gets its network with Rest API and response is not empty")
-    @Test(groups = { TestGroup.REST_API, TestGroup.NETWORKS, TestGroup.COMMENTS })
+    @Test(groups = { TestGroup.REST_API, TestGroup.NETWORKS, TestGroup.COMMENTS, TestGroup.SANITY })
     public void tenantContributorUserChecksIfItsNetworkIsPresent() throws Exception
     {
         UserModel contributorTenantUser = dataUser.usingUser(adminTenantUser).createUserWithTenant("contTenant");