Fix: unnecessarily waiting testcase.

[gigi.git] / tests / org / cacert / gigi / TestOrga.java

diff --git a/tests/org/cacert/gigi/TestOrga.java b/tests/org/cacert/gigi/TestOrga.java
index da4ce25c509959ec1e746e3cdf17130ccf79b958..d614371854b93a1b9446a545ed92b2353204c75a 100644 (file)
--- a/tests/org/cacert/gigi/TestOrga.java
+++ b/tests/org/cacert/gigi/TestOrga.java
@@ -15,13 +15,19 @@ public class TestOrga extends ManagedTest {
         User u2 = User.getById(createVerifiedUser("fn", "ln", createUniqueName() + "@email.org", TEST_PASSWORD));
         User u3 = User.getById(createVerifiedUser("fn", "ln", createUniqueName() + "@email.org", TEST_PASSWORD));
         User u4 = User.getById(createVerifiedUser("fn", "ln", createUniqueName() + "@email.org", TEST_PASSWORD));
-        Organisation o1 = new Organisation("name", "ST", "prov", "city", u1);
+        Organisation o1 = new Organisation("name", "ST", "prov", "city", "email", u1);
         assertEquals(0, o1.getAllAdmins().size());
-        o1.addAdmin(u2, u1);
+        o1.addAdmin(u2, u1, false);
         assertEquals(1, o1.getAllAdmins().size());
-        o1.addAdmin(u3, u1);
+        o1.addAdmin(u2, u1, false); // Insert double should be ignored
+        assertEquals(1, o1.getAllAdmins().size());
+        o1.addAdmin(u3, u1, false);
         assertEquals(2, o1.getAllAdmins().size());
-        o1.addAdmin(u4, u1);
+        o1.addAdmin(u4, u1, false);
+        assertEquals(3, o1.getAllAdmins().size());
+        o1.removeAdmin(u3, u1);
+        assertEquals(2, o1.getAllAdmins().size());
+        o1.addAdmin(u3, u1, false); // add again
         assertEquals(3, o1.getAllAdmins().size());
         o1.removeAdmin(u3, u1);
         assertEquals(2, o1.getAllAdmins().size());
@@ -29,4 +35,5 @@ public class TestOrga extends ManagedTest {
         o1.removeAdmin(u2, u1);
         assertEquals(0, o1.getAllAdmins().size());
     }
+
 }