fix cleanup in test

Author: simonLeary42

test/functional/NewUserTest.php

@@ -89,6 +89,7 @@ private function approveGroup($uid)
 
     // delete requests made by that user
     // delete user entry
+    // delete user group entry
     // remove user from org group
     // remove user from "all users" group
     // does not remove user from PI groups
@@ -105,6 +106,10 @@ private function ensureUserDoesNotExist()
             $LDAP->getUserEntry($USER->uid)->delete();
             assert(!$USER->exists());
         }
+        if ($USER->getGroupEntry()->exists()) {
+            $USER->getGroupEntry()->delete();
+            assert(!$USER->getGroupEntry()->exists());
+        }
         $all_users_group = $LDAP->getUserGroup();
         $all_member_uids = $all_users_group->getAttribute("memberuid");
         $new_uids = array_diff($all_member_uids, [$USER->uid]);