I know this has been here before, but I think we should delete the extra addUser() call. I don't see why we'd need it. if we want to verify that double-adding a user to a group does not generate some error, then it should be a separate test. can you please extract that to a separate test method?