Allow & account for pre-existing users other than ROOT to avoid unnecessary failures in src/test/java/com/arangodb/ArangoDBTest: fixes issue #196.

Reşat SABIQ · Reşat SABIQ · commit d96558aeb633 · 2018-06-13T17:24:05.000+06:00
diff --git a/src/test/java/com/arangodb/ArangoDBTest.java b/src/test/java/com/arangodb/ArangoDBTest.java
@@ -32,11 +32,14 @@
 import static org.junit.Assert.assertThat;
 import static org.junit.Assert.fail;
 
+import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.Collection;
 import java.util.HashMap;
+import java.util.List;
 import java.util.Map;
 
+import org.hamcrest.Matcher;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.junit.runners.Parameterized;
@@ -191,12 +194,25 @@ public void getUsersOnlyRoot() {
 	@Test
 	public void getUsers() {
 		try {
+			// Allow & account for pre-existing users other than ROOT
+			final Collection<UserEntity> initialUsers = arangoDB.getUsers();
 			arangoDB.createUser(USER, PW, null);
 			final Collection<UserEntity> users = arangoDB.getUsers();
 			assertThat(users, is(notNullValue()));
-			assertThat(users.size(), is(2));
+			assertThat(users.size(), is(initialUsers.size()+1));
+			
+			List<Matcher<? super String>> matchers = new ArrayList<Matcher<? super String>>(users.size());
+			// Add initial users, including root
+			for (UserEntity userEntity : initialUsers) {
+				Matcher<String> matcher = is(userEntity.getUser());
+				if (!matchers.contains(matcher))
+					matchers.add(matcher);
+			}
+			// Add USER
+			matchers.add(is(USER));
+			
 			for (final UserEntity user : users) {
-				assertThat(user.getUser(), anyOf(is(ROOT), is(USER)));
+				assertThat(user.getUser(), anyOf(matchers));
 			}
 		} finally {
 			arangoDB.deleteUser(USER);
