minor docstring and code cleanup

Author: claire-peters

coldfront/core/project/templates/project/project_remove_users.html

@@ -19,7 +19,8 @@ <h2>Remove users from project: {{project.title}}</h2>
       <p>
         To be removed from a lab, the user must not have the lab as their primary
         group. If you would like to remove a user that has your lab as their primary
-        group, please contact FASRC support.
+        group, please <a href="https://portal.rc.fas.harvard.edu/rcrt/submit_ticket">
+        contact FASRC support</a>.
       </p>
       <form action="{% url 'project-remove-users' project.pk %}" method="post">
         {% csrf_token %}

coldfront/core/project/test_views.py

@@ -5,9 +5,7 @@
 from coldfront.core.test_helpers import utils
 from coldfront.core.test_helpers.factories import (
     setup_models,
-    UserFactory,
     ProjectFactory,
-    ProjectUserFactory,
     PAttributeTypeFactory,
     ProjectAttributeFactory,
     ProjectStatusChoiceFactory,

coldfront/plugins/ldap/utils.py

@@ -192,7 +192,9 @@ def remove_member_from_group(self, user_name, group_name):
         return result
 
     def users_in_primary_group(self, usernames, groupname):
-        """Return two lists of users based on membership in the specified group.
+        """
+        Return list of usernames representing users that are members of the
+        designated AD Group
         """
         group = self.return_group_by_name(groupname)
         attrs = ['sAMAccountName', 'gidNumber']