fixup!: more formatting

Author: rgraber

kobo/apps/audit_log/models.py

@@ -300,6 +300,7 @@ def create(self, **kwargs):
         )
 
 class ProjectHistoryLog(AuditLog):
+
     objects = ProjectHistoryLogManager()
 
     class Meta:
@@ -532,7 +533,9 @@ def qa_change(_, new_field):
         return AuditAction.UPDATE_QA, {'qa': {NEW: new_field}}
 
     @staticmethod
-    def create_from_related_request(request, label, add_action, delete_action, modify_action):
+    def create_from_related_request(
+        request, label, add_action, delete_action, modify_action
+    ):
         initial_data = getattr(request, 'initial_data', None)
         updated_data = getattr(request, 'updated_data', None)
         asset_uid = request.resolver_match.kwargs['parent_lookup_asset']
@@ -556,8 +559,5 @@ def create_from_related_request(request, label, add_action, delete_action, modif
         else:
             action = modify_action
         ProjectHistoryLog.objects.create(
-            user = request.user,
-            object_id = object_id,
-            action = action,
-            metadata = metadata
+            user=request.user, object_id=object_id, action=action, metadata=metadata
         )

kobo/apps/audit_log/tests/test_project_history_logs.py

@@ -39,7 +39,9 @@ def _check_common_metadata(self, metadata_dict):
         self.assertEqual(metadata_dict['ip_address'], '127.0.0.1')
         self.assertEqual(metadata_dict['source'], 'source')
 
-    def _base_asset_detail_endpoint_test(self, patch, url_name, request_data, expected_action):
+    def _base_asset_detail_endpoint_test(
+        self, patch, url_name, request_data, expected_action
+    ):
         url = reverse(url_name, kwargs={'uid': self.asset.uid})
         method = self.client.patch if patch else self.client.post
         log_metadata = self._base_project_history_log_test(
@@ -478,10 +480,13 @@ def test_modify_service_creates_log(self):
         }
         log_metadata = self._base_project_history_log_test(
             method=self.client.patch,
-            url=reverse('api_v2:hook-detail', kwargs={
-                'parent_lookup_asset': self.asset.uid,
-                'uid': new_hook.uid,
-            }),
+            url=reverse(
+                'api_v2:hook-detail',
+                kwargs={
+                    'parent_lookup_asset': self.asset.uid,
+                    'uid': new_hook.uid,
+                },
+            ),
             request_data=request_data,
             expected_action=AuditAction.MODIFY_SERVICE,
         )

kobo/apps/hook/views/v2/hook.py

@@ -5,7 +5,7 @@
 from django.db.models import Q
 from django.utils import timezone
 from django.utils.translation import gettext as t
-from rest_framework import status, viewsets
+from rest_framework import status
 from rest_framework.decorators import action
 from rest_framework.response import Response
 from rest_framework_extensions.mixins import NestedViewSetMixin
@@ -19,8 +19,9 @@
 from kpi.utils.viewset_mixins import AssetNestedObjectViewsetMixin
 
 
-class HookViewSet(AssetNestedObjectViewsetMixin, NestedViewSetMixin,
-                  AuditLoggedModelViewSet):
+class HookViewSet(
+    AssetNestedObjectViewsetMixin, NestedViewSetMixin, AuditLoggedModelViewSet
+):
     """
 
     ## External services

kpi/models/asset_file.py

@@ -11,10 +11,7 @@
 
 from kpi.fields import KpiUidField
 from kpi.fields.file import PrivateExtendedFileField
-from kpi.interfaces import (
-    OpenRosaManifestInterface,
-    SyncBackendMediaInterface,
-)
+from kpi.interfaces import OpenRosaManifestInterface, SyncBackendMediaInterface
 from kpi.models.abstract_models import AbstractTimeStampedModel
 from kpi.utils.hash import calculate_hash
 from kpi.utils.models import DjangoModelABCMetaclass
@@ -159,9 +156,9 @@ def get_download_url(self, request):
         """
         Implements `OpenRosaManifestInterface.get_download_url()`
         """
-        return drf_reverse('asset-file-content',
-                       args=(self.asset.uid, self.uid),
-                       request=request)
+        return drf_reverse(
+            'asset-file-content', args=(self.asset.uid, self.uid), request=request
+        )
 
     @staticmethod
     def get_path(asset, file_type, filename):

kpi/views/v2/asset_file.py

@@ -1,5 +1,5 @@
 # coding: utf-8
-from django.http import HttpResponseRedirect, Http404
+from django.http import HttpResponseRedirect
 from private_storage.views import PrivateStorageDetailView
 from rest_framework.decorators import action
 from rest_framework_extensions.mixins import NestedViewSetMixin
@@ -8,14 +8,14 @@
 from kpi.constants import PERM_VIEW_ASSET
 from kpi.filters import RelatedAssetPermissionsFilter
 from kpi.models import AssetFile
-from kpi.serializers.v2.asset_file import AssetFileSerializer
 from kpi.permissions import AssetEditorPermission
+from kpi.serializers.v2.asset_file import AssetFileSerializer
 from kpi.utils.viewset_mixins import AssetNestedObjectViewsetMixin
-from kpi.views.no_update_model import NoUpdateModelViewSet
 
 
-class AssetFileViewSet(AssetNestedObjectViewsetMixin, NestedViewSetMixin,
-                       AuditLoggedNoUpdateModelViewSet):
+class AssetFileViewSet(
+    AssetNestedObjectViewsetMixin, NestedViewSetMixin, AuditLoggedNoUpdateModelViewSet
+):
     """
     This endpoint shows uploaded files related to an asset.
 
@@ -130,7 +130,13 @@ class AssetFileViewSet(AssetNestedObjectViewsetMixin, NestedViewSetMixin,
     serializer_class = AssetFileSerializer
     permission_classes = (AssetEditorPermission,)
     log_type = 'project-history'
-    logged_fields = ['uid', 'filename', 'md5_hash', 'download_url', ('object_id', 'asset.id')]
+    logged_fields = [
+        'uid',
+        'filename',
+        'md5_hash',
+        'download_url',
+        ('object_id', 'asset.id'),
+    ]
 
     def get_queryset(self):
         _queryset = self.model.objects.filter(asset__uid=self.asset_uid)