Add missing removeAdminMembers and removeOwnerships (#633)

uhawaii-system-its-ti-iam · Jan 23, 2025 · ebdf1ad · ebdf1ad
1 parent fbba8ed
commit ebdf1ad
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/src/test/java/edu/hawaii/its/api/service/TestUpdateMemberService.java b/src/test/java/edu/hawaii/its/api/service/TestUpdateMemberService.java
@@ -480,14 +480,16 @@ public void validateOptInActionForAlreadyOptedUser() {
             fail("should not throw an exception if user does self opt in when user already opted in");
         }
 
-         try {
+        try {
             updateMemberService.removeIncludeMember(ADMIN, GROUPING, testUid);
             updateMemberService.optIn(ADMIN, GROUPING, testUid);
             assertTrue(memberService.isMember(GROUPING_INCLUDE, testUid));
         } catch (CommandException e) {
             fail("Should not throw an exception because opt in operation can be executed after removing include member");
         }
-         updateMemberService.removeIncludeMember(ADMIN, GROUPING, testUid);
+        updateMemberService.removeIncludeMember(ADMIN, GROUPING, testUid);
+        updateMemberService.removeAdminMember(ADMIN, testUid);
+        updateMemberService.removeOwnerships(ADMIN, GROUPING, testUids);
     }
 
     @Test
@@ -511,6 +513,9 @@ public void validateOptOutActionForAlreadyOptedUser() {
         } catch (CommandException e) {
             fail("Should not throw an exception because opt out operation is not executed after removing include member");
         }
+
+        updateMemberService.removeAdminMember(ADMIN, testUid);
+        updateMemberService.removeOwnerships(ADMIN, GROUPING, testUids);
     }
 
     private void addGroupMember(String groupPath, String uhIdentifier) {