* Cleanup unused functions.

django_kala/auth/views/users.py

@@ -15,7 +15,7 @@ def get_context_data(self, **kwargs):
         if organization:
             try:
                 organization = Organization.objects.get(name=organization)
-                organization_users = organization.get_people(self.request.user)
+                organization_users = self.request.user.get_users()
                 if organization_users:
                     users = users.filter(pk__in=organization_users.values_list('pk'))
                 else:

django_kala/documents/models.py

@@ -85,38 +85,6 @@ def get_latest(self):
     def list_versions(self):
         return self.documentversion_set.all()
 
-    def get_users(self, user):
-        if user.is_superuser:
-            return User.objects.all()
-        # If you have permissions for the org, or permissions for the
-        # project, then you can see everyone in the org.
-        if Permissions.has_perms([
-            'change_organization',
-            'add_organization',
-            'delete_organization'
-        ], user, self.project.organization.uuid) or Permissions.has_perms([
-            'add_project',
-            'change_project',
-            'delete_project'
-        ], user, self.project.uuid):
-            return self.project.organization.user_set.all()
-        if Permissions.has_perms([
-            'add_document',
-            'change_document',
-            'delete_document'
-        ], user, self.uuid):
-            project_users = DocumentVersion.objects.filter(
-                document__project=self.project
-            ).prefetch_related(
-                'document_set'
-            ).select_related(
-                'user_id'
-            ).values_list(
-                'user_id', flat=True
-            )
-            return User.objects.filter(id__in=project_users)
-        return None
-
     def can(self, user, _permissions):
         if user.is_superuser:
             return True

django_kala/projects/models.py

@@ -50,11 +50,6 @@ def get_documents(self, user):
             id__in=user.get_documents().values_list('id', flat=True)
         )
 
-    def get_users(self, user):
-        if user.is_superuser:
-            return User.objects.all()
-        return user.get_users()
-
     def can(self, user, _permissions):
         if user.is_superuser:
             return True
@@ -150,7 +145,7 @@ class ProjectPermission(models.Model):
     user = models.ForeignKey(User, on_delete=models.CASCADE)
     permission = models.ForeignKey(Permission, on_delete=models.CASCADE)
 
-    def __str__(self):
+    def __str__(self): # pragma: no cover
         return '{0} | {1} | {2}'.format(self.project, self.user, self.permission.codename)
 
 

django_kala/projects/views/projects/settings/tests/test_manage_access.py

@@ -42,7 +42,6 @@ def test_project_manage_access():
     )
     assert response.redirect_chain[0][0] == reverse('projects:manage_access', args=[project.pk])
     assert response.redirect_chain[0][1] == HTTP_302_FOUND
-    print(ProjectPermission.objects.all())
     assert project.can_create(other_user) == True
 
     # Remove Create

django_kala/projects/views/projects/tests/test_projects.py

@@ -3,7 +3,7 @@
 from django.urls import reverse
 from rest_framework.status import HTTP_200_OK, HTTP_302_FOUND
 
-from organizations.views.tests import setup, user_permissions_test_invite, login
+from organizations.views.tests import setup, login
 
 User = get_user_model()