Merge 8df0a98 into 7a24096

docs/conf.py

@@ -152,4 +152,4 @@
     (master_doc, 'django-gdpr-assist', 'django-gdpr-assist Documentation',
      author, 'django-gdpr-assist', 'One line description of project.',
      'Miscellaneous'),
-]
+]

gdpr_assist/admin/tool.py

@@ -169,7 +169,7 @@ def handle_export(self, request, querysets):
                     csv_writer.writerow(obj_export)
 
                 # Add CSV file to zip
-                privacy_meta =  getattr(
+                privacy_meta = getattr(
                     model,
                     app_settings.GDPR_PRIVACY_INSTANCE_NAME,
                 )

gdpr_assist/management/commands/anonymise_db.py

@@ -1,8 +1,6 @@
 """
 Anonymise all personal data in the database
 """
-import six
-
 from django.core.management import BaseCommand
 
 from ... import app_settings

gdpr_assist/models.py

@@ -99,7 +99,7 @@ def deconstruct(self):
         """
         # Check bases are as expected from _cast_class
         bases = self.__class__.__bases__
-        if len(bases) != 2:
+        if len(bases) != 2:  # pragma: no cover
             raise ValueError('Unexpected base classes for CastPrivacyManager')
 
         # Original is second - instatiate and deconstruct it
@@ -283,7 +283,7 @@ class EventLog(models.Model):
     model_name = models.CharField(max_length=255)
     target_pk = models.TextField()
 
-    objects = EventLogManager() #.db_manager(app_settings.GDPR_LOG_DATABASE_NAME)
+    objects = EventLogManager()
 
     def get_target(self):
         model = apps.get_model(self.app_label, self.model_name)

tests/test_admin.py

@@ -4,11 +4,9 @@
 from io import BytesIO, TextIOWrapper
 import csv
 import six
-import sys
 import zipfile
 
 import django
-from django.contrib import admin
 from django.contrib.auth.models import User
 from django.contrib.contenttypes.models import ContentType
 from django.test import Client, TestCase
@@ -17,7 +15,6 @@
 
 import gdpr_assist
 
-from .gdpr_assist_tests_app.admin import ModelWithPrivacyMetaAdmin
 from .gdpr_assist_tests_app.models import (
     ModelWithPrivacyMeta,
     FirstSearchModel,
@@ -48,7 +45,7 @@ def setUp(self):
 
 class TestModelAdmin(AdminTestCase):
     def test_changelist__anonymise_action_present(self):
-        obj = mommy.make(ModelWithPrivacyMeta)
+        mommy.make(ModelWithPrivacyMeta)
         response = self.client.get(model_root_url)
         self.assertContains(response, '<option value="anonymise">')
 
@@ -126,13 +123,13 @@ def test_anonymise_view_submit__redirect_to_anonymise_view(self):
 
         self.assertContains(
             response,
-             '<li class="success">2 Model With Privacy Metas anonymised</li>',
+            '<li class="success">2 Model With Privacy Metas anonymised</li>',
         )
 
 
 class TestAdminTool(AdminTestCase):
     def test_tool_is_available(self):
-        obj = mommy.make(FirstSearchModel)
+        mommy.make(FirstSearchModel)
         response = self.client.get(tool_root_url)
         self.assertContains(response, '<h1>Personal Data</h1>')
 
@@ -141,7 +138,7 @@ def test_search__returns_correct_results(self):
             FirstSearchModel,
             email='one@example.com',
         )
-        obj_2 = mommy.make(
+        mommy.make(
             FirstSearchModel,
             email='two@example.com',
         )

tests/test_commands.py

@@ -91,7 +91,6 @@ def test_anonymise_disabled__raises_error(self):
         )
 
 
-
 class TestRerunCommand(CommandTestCase):
     def test_gdpr_delete__deletes_object(self):
         obj_1 = ModelWithPrivacyMeta.objects.create(