Skip to content
Snippets Groups Projects
Commit bc5caeee authored by felmer's avatar felmer
Browse files

SSDM-5721: fixing system tests by ignoring name and email because they are...

SSDM-5721: fixing system tests by ignoring name and email because they are changed randomly by the dummy authentication service.

SVN: 38969
parent fdbb9c12
No related branches found
No related tags found
No related merge requests found
......@@ -1212,10 +1212,7 @@ public class AbstractTest extends SystemTestCase
{
builder.append("[inactive] ");
}
builder.append(person.getUserId()).append(":");
appendTo(builder, person.getFirstName());
appendTo(builder, person.getLastName());
appendTo(builder, person.getEmail());
builder.append(person.getUserId());
Space space = person.getSpace();
if (space != null)
{
......@@ -1243,14 +1240,6 @@ public class AbstractTest extends SystemTestCase
return renderedAssignments.toString();
}
private void appendTo(StringBuilder builder, String stringOrNull)
{
if (stringOrNull != null)
{
builder.append(" ").append(stringOrNull);
}
}
protected static String patternContains(String... parts)
{
StringBuilder pattern = new StringBuilder();
......
......@@ -51,9 +51,7 @@ public class GetPersonTest extends AbstractTest
Map<IPersonId, Person> persons = v3api.getPersons(sessionToken, Arrays.asList(id1, id2), fetchOptions);
// Then
assertEquals(renderPerson(persons.get(id1)), "observer: John Observer observer@o.o, home space:CISD, "
+ "[SPACE_OBSERVER Space TESTGROUP]");
assertEquals(renderPerson(persons.get(id2)), "test_role: John 3 Doe test role test_role@in.active, home space:CISD, "
+ "[SPACE_POWER_USER Space CISD], registrator: test");
assertEquals(renderPerson(persons.get(id1)), "observer, home space:CISD, [SPACE_OBSERVER Space TESTGROUP]");
assertEquals(renderPerson(persons.get(id2)), "test_role, home space:CISD, [SPACE_POWER_USER Space CISD], registrator: test");
}
}
......@@ -42,7 +42,7 @@ public class SearchPersonTest extends AbstractTest
searchCriteria.withOrOperator();
searchCriteria.withUserId().thatStartsWith("observer");
searchCriteria.withUserId().thatContains("role");
searchCriteria.withLastName().thatContains("active");
searchCriteria.withUserId().thatEndsWith("active");
PersonFetchOptions fetchOptions = new PersonFetchOptions();
fetchOptions.withSpace();
fetchOptions.withRoleAssignments().withSpace();
......@@ -52,12 +52,9 @@ public class SearchPersonTest extends AbstractTest
List<Person> persons = v3api.searchPersons(sessionToken, searchCriteria, fetchOptions).getObjects();
// When
assertEquals(renderPersons(persons), "[inactive] inactive: John Inactive inactive@in.active, home space:CISD, []\n"
+ "observer: John Observer observer@o.o, home space:CISD, "
+ "[SPACE_OBSERVER Space TESTGROUP]\n"
+ "observer_cisd: John ObserverCISD observer_cisd@o.o, home space:CISD, "
+ "[SPACE_ADMIN Space TESTGROUP, SPACE_OBSERVER Space CISD]\n"
+ "test_role: John 3 Doe test role test_role@in.active, home space:CISD, "
+ "[SPACE_POWER_USER Space CISD], registrator: test\n");
assertEquals(renderPersons(persons), "[inactive] inactive, home space:CISD, []\n"
+ "observer, home space:CISD, [SPACE_OBSERVER Space TESTGROUP]\n"
+ "observer_cisd, home space:CISD, [SPACE_ADMIN Space TESTGROUP, SPACE_OBSERVER Space CISD]\n"
+ "test_role, home space:CISD, [SPACE_POWER_USER Space CISD], registrator: test\n");
}
}
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment