REPO-1027: V1 REST API - minor: tweak TestSiteMembers

- to make it repeatable (TODO further splitting/refactoring)

git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/BRANCHES/DEV/5.2.N/root@130918 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
This commit is contained in:
Jan Vonka
2016-09-26 15:26:57 +00:00
parent 6e857ad3bf
commit 114ce8b02c
2 changed files with 26 additions and 20 deletions

View File

@@ -1400,8 +1400,11 @@ public class RepoService
public TestPerson createUser()
{
String username = "user" + System.currentTimeMillis();
PersonInfo personInfo = new PersonInfo("FirstName", "LastName", username, "password", null, "skype", "location",
long timeMillis = System.currentTimeMillis();
String firstname = "first" + timeMillis;
String lastname = "last" + timeMillis;
String username = "user" + timeMillis;
PersonInfo personInfo = new PersonInfo(firstname, lastname, username, "password", null, "skype", "location",
"telephone", "mob", "instant", "google");
TestPerson person = createUser(personInfo);
return person;

View File

@@ -240,6 +240,10 @@ public class TestSiteMembers extends EnterpriseTestApi
}
}, network1.getId());
final TestPerson person1 = people.get(0);
final TestPerson person2 = people.get(1);
final TestPerson person3 = people.get(2);
TenantUtil.runAsSystemTenant(new TenantRunAsWork<Void>()
{
@Override
@@ -252,9 +256,6 @@ public class TestSiteMembers extends EnterpriseTestApi
}
}, network2.getId());
final TestPerson person1 = people.get(0);
final TestPerson person2 = people.get(1);
final TestPerson person3 = people.get(2);
final TestPerson person4 = people.get(3);
// Create site
@@ -641,17 +642,19 @@ public class TestSiteMembers extends EnterpriseTestApi
person1.expected(ret.getMember());
// check site membership in GET
expectedSiteMembers = site.getMembers();
SiteMember toCheck = null;
for(SiteMember sm1 : expectedSiteMembers)
{
if(sm1.getMemberId().equals(person1.getId()))
{
toCheck = sm1;
}
}
assertNotNull(toCheck); // check that the update site membership is present
assertEquals(sm.getRole(), toCheck.getRole()); // check that the role is correct
SiteMember smToCheck = sitesProxy.getSingleSiteMember(site.getSiteId(), person1.getId());
assertNotNull(smToCheck); // check that the update site membership is present
assertEquals(sm.getRole(), smToCheck.getRole()); // check that the role is correct
expectedSiteMembers = new ArrayList<>();
SiteMember sm1 = new SiteMember(person1.getId(), person1, site.getSiteId(), SiteRole.SiteCollaborator.toString());
expectedSiteMembers.add(sm1);
SiteMember sm2 = new SiteMember(person2.getId(), person2, site.getSiteId(), SiteRole.SiteManager.toString());
expectedSiteMembers.add(sm2);
SiteMember sm3 = new SiteMember(person3.getId(), person3, site.getSiteId(), SiteRole.SiteCollaborator.toString());
expectedSiteMembers.add(sm3);
Collections.sort(expectedSiteMembers);
int skipCount = 0;
int maxItems = Integer.MAX_VALUE;