chore: fix lint issues

Author: navinkarkera

cms/djangoapps/modulestore_migrator/api.py

@@ -182,7 +182,7 @@ def construct_usage_key(lib_key_str: str, component: Component) -> LibraryUsageL
     return {
         obj.source.key: construct_usage_key(obj.target.learning_package.key, obj.target.component)
         for obj in query_set
-        if obj.source.key is not None
+        if obj.source.key is not None and obj.target is not None
     }
 
 
@@ -193,6 +193,10 @@ def get_migration_blocks_info(
     task_uuid: str | None,
     is_failed: bool | None,
 ):
+    """
+    Given the target key, and optional source key, target collection key, task_uuid and is_failed get a dictionary
+    containing information about migration blocks.
+    """
     filters: dict[str, str | UUID | bool] = {
         'overall_migration__target__key': target_key
     }

cms/djangoapps/modulestore_migrator/rest_api/v1/serializers.py

@@ -11,7 +11,10 @@
 from user_tasks.serializers import StatusSerializer
 
 from cms.djangoapps.modulestore_migrator.data import CompositionLevel, RepeatHandlingStrategy
-from cms.djangoapps.modulestore_migrator.models import ModulestoreBlockMigration, ModulestoreMigration, ModulestoreSource
+from cms.djangoapps.modulestore_migrator.models import (
+    ModulestoreMigration,
+    ModulestoreSource,
+)
 
 
 class ModulestoreMigrationSerializer(serializers.Serializer):

cms/djangoapps/modulestore_migrator/rest_api/v1/urls.py

@@ -4,7 +4,13 @@
 from django.urls import include, path
 from rest_framework.routers import SimpleRouter
 
-from .views import BulkMigrationViewSet, LibraryCourseMigrationViewSet, MigrationInfoViewSet, MigrationViewSet, BlockMigrationInfo
+from .views import (
+    BlockMigrationInfo,
+    BulkMigrationViewSet,
+    LibraryCourseMigrationViewSet,
+    MigrationInfoViewSet,
+    MigrationViewSet,
+)
 
 ROUTER = SimpleRouter()
 ROUTER.register(r'migrations', MigrationViewSet, basename='migrations')

cms/djangoapps/modulestore_migrator/rest_api/v1/views.py

@@ -575,7 +575,7 @@ def get(self, request: Request):
         target_key = request.query_params.get("target_key")
         target_collection_key = request.query_params.get("target_collection_key")
         task_uuid = request.query_params.get("task_uuid")
-        is_failed = request.query_params.get("is_failed")
+        is_failed: str | bool | None = request.query_params.get("is_failed")
         if not target_key:
             return Response({"error": "Target key cannot be blank."}, status=400)
         try: