style: Fix tox -e quality issues.

Author: feanil

backend/sample_plugin/models.py

@@ -11,7 +11,7 @@ class CourseArchiveStatus(models.Model):
     Model to track the archive status of a course.
 
     Stores information about whether a course has been archived and when it was archived.
-    
+
     .. no_pii: This model does not store PII directly, only references to users via foreign keys.
     """
 
@@ -47,12 +47,14 @@ def __str__(self):
         """
         Return a string representation of the course archive status.
         """
+        # pylint: disable=no-member
         return f"{self.course_id} - {self.user.username} - {'Archived' if self.is_archived else 'Not Archived'}"
 
     class Meta:
         """
         Meta options for the CourseArchiveStatus model.
         """
+
         verbose_name = "Course Archive Status"
         verbose_name_plural = "Course Archive Statuses"
         ordering = ["-updated_at"]

backend/sample_plugin/serializers.py

@@ -10,11 +10,12 @@ class CourseArchiveStatusSerializer(serializers.ModelSerializer):
     """
     Serializer for the CourseArchiveStatus model.
     """
-    
+
     class Meta:
         """
         Meta class for CourseArchiveStatusSerializer.
         """
+
         model = CourseArchiveStatus
         fields = [
             'id',
@@ -25,4 +26,4 @@ class Meta:
             'created_at',
             'updated_at',
         ]
-        read_only_fields = ['id', 'created_at', 'updated_at', 'archive_date']
+        read_only_fields = ['id', 'created_at', 'updated_at', 'archive_date']

backend/sample_plugin/views.py

@@ -2,17 +2,19 @@
 Views for the sample_plugin app.
 """
 import logging
+
 from django.utils import timezone
 from django_filters.rest_framework import DjangoFilterBackend
+from opaque_keys import InvalidKeyError
+from opaque_keys.edx.keys import CourseKey
 from rest_framework import filters, permissions, viewsets
 from rest_framework.exceptions import PermissionDenied, ValidationError
 from rest_framework.pagination import PageNumberPagination
-from rest_framework.throttling import UserRateThrottle, AnonRateThrottle
+from rest_framework.throttling import AnonRateThrottle, UserRateThrottle
 
 from sample_plugin.models import CourseArchiveStatus
 from sample_plugin.serializers import CourseArchiveStatusSerializer
 
-
 logger = logging.getLogger(__name__)
 
 
@@ -47,6 +49,7 @@ class CourseArchiveStatusPagination(PageNumberPagination):
     """
     Pagination class for CourseArchiveStatus.
     """
+
     page_size = 20
     page_size_query_param = 'page_size'
     max_page_size = 100
@@ -56,6 +59,7 @@ class CourseArchiveStatusThrottle(UserRateThrottle):
     """
     Throttle for the CourseArchiveStatus API.
     """
+
     rate = '60/minute'
 
 
@@ -68,6 +72,7 @@ class CourseArchiveStatusViewSet(viewsets.ModelViewSet):
     Filtering is available on course_id, user, and is_archived fields.
     Ordering is available on all fields.
     """
+
     serializer_class = CourseArchiveStatusSerializer
     permission_classes = [IsOwnerOrStaffSuperuser]
     pagination_class = CourseArchiveStatusPagination
@@ -120,10 +125,9 @@ def _is_valid_course_id(self, course_id):
         sophisticated validator from the edx-platform.
         """
         try:
-            from opaque_keys.edx.keys import CourseKey
             CourseKey.from_string(course_id)
             return True
-        except:
+        except InvalidKeyError:
             return False
 
     def perform_create(self, serializer):

backend/tests/__init__.py

@@ -1,4 +1,5 @@
 #!/usr/bin/env python
+# pylint: disable=redefined-outer-name
 """
 Tests for the `sample-plugin` REST API.
 """
@@ -12,7 +13,6 @@
 
 from sample_plugin.models import CourseArchiveStatus
 
-
 User = get_user_model()
 
 
@@ -98,7 +98,7 @@ def test_list_course_archive_status_authenticated(api_client, user, course_archi
 
 
 @pytest.mark.django_db
-def test_list_course_archive_status_unauthenticated(api_client, course_archive_status):
+def test_list_course_archive_status_unauthenticated(api_client):
     """
     Test that an unauthenticated user cannot list course archive statuses.
     """

backend/tests/test_api.py

@@ -1,17 +1,16 @@
 #!/usr/bin/env python
+# pylint: disable=redefined-outer-name
 """
 Tests for the `sample-plugin` models module.
 """
 
 import pytest
 from django.contrib.auth import get_user_model
 from django.db.utils import IntegrityError
-from opaque_keys.edx.django.models import CourseKeyField
 from opaque_keys.edx.keys import CourseKey
 
 from sample_plugin.models import CourseArchiveStatus
 
-
 User = get_user_model()