REPO-1646: V1 REST API - cannot unset optional fields (eg. when updating person / site details ...)

- fix test fallout (Person returns empty Company object by default - as per existing V1 getPerson API)
- TODO review the way we match expected results QueriesPeopleApiTest (vs other test patterns)

git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/BRANCHES/DEV/5.2.N/root@133296 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
This commit is contained in:
Jan Vonka
2016-11-30 15:51:00 +00:00
parent 78a7208ad7
commit e3355b881b

View File

@@ -435,9 +435,9 @@ public class QueriesPeopleApiTest extends AbstractSingleNetworkSiteTest
term = LAST_A;
expectedPeople = new String[]
{
"Person ["+ "lastName=LastA, ]", // USER5
"Person ["+"firstName=FirstA, lastName=LastA, ]", // USER1
"Person ["+"firstName=FirstB, lastName=LastA, ]", // USER3
"Person ["+ "lastName=LastA, company=Company [address1=null, address2=null, address3=null, postcode=null, telephone=null, fax=null, email=null], ]", // USER5
"Person ["+"firstName=FirstA, lastName=LastA, company=Company [address1=null, address2=null, address3=null, postcode=null, telephone=null, fax=null, email=null], ]", // USER1
"Person ["+"firstName=FirstB, lastName=LastA, company=Company [address1=null, address2=null, address3=null, postcode=null, telephone=null, fax=null, email=null], ]", // USER3
};
checkApiCall(term, orderBy, fields, paging, expectedStatus, expectedPeople);
@@ -450,9 +450,9 @@ public class QueriesPeopleApiTest extends AbstractSingleNetworkSiteTest
term = LAST_A;
expectedPeople = new String[]
{
"Person [id=user5@org.alfresco.rest.api.tests.queriespeopleapitest, ]", // USER5
"Person [id=user1@org.alfresco.rest.api.tests.queriespeopleapitest, ]", // USER1
"Person [id=user3@org.alfresco.rest.api.tests.queriespeopleapitest, ]", // USER3
"Person [id=user5@org.alfresco.rest.api.tests.queriespeopleapitest, company=Company [address1=null, address2=null, address3=null, postcode=null, telephone=null, fax=null, email=null], ]", // USER5
"Person [id=user1@org.alfresco.rest.api.tests.queriespeopleapitest, company=Company [address1=null, address2=null, address3=null, postcode=null, telephone=null, fax=null, email=null], ]", // USER1
"Person [id=user3@org.alfresco.rest.api.tests.queriespeopleapitest, company=Company [address1=null, address2=null, address3=null, postcode=null, telephone=null, fax=null, email=null], ]", // USER3
};
checkApiCall(term, orderBy, fields, paging, expectedStatus, expectedPeople);
@@ -528,9 +528,9 @@ public class QueriesPeopleApiTest extends AbstractSingleNetworkSiteTest
term = LAST_A;
expectedPeople = new String[]
{
"Person ["+ "lastName=LastA, ]", // USER5
"Person ["+"firstName=FirstA, lastName=LastA, ]", // USER1
"Person ["+"firstName=FirstB, lastName=LastA, ]" // USER3
"Person ["+ "lastName=LastA, company=Company [address1=null, address2=null, address3=null, postcode=null, telephone=null, fax=null, email=null], ]", // USER5
"Person ["+"firstName=FirstA, lastName=LastA, company=Company [address1=null, address2=null, address3=null, postcode=null, telephone=null, fax=null, email=null], ]", // USER1
"Person ["+"firstName=FirstB, lastName=LastA, company=Company [address1=null, address2=null, address3=null, postcode=null, telephone=null, fax=null, email=null], ]" // USER3
};
checkApiCall(term, orderBy, fields, paging, expectedStatus, expectedPeople);