fixup!: pylint fixes

openedx_learning/core/publishing/api.py

@@ -95,7 +95,7 @@ def create_publishable_entity_version(
         )
         Draft.objects.update_or_create(
             entity_id=entity_id,
-            defaults=dict(version=version),
+            defaults={"version": version},
         )
     return version
 
@@ -193,10 +193,10 @@ def get_draft_version(publishable_entity_id: int) -> PublishableEntityVersion |
     except Draft.DoesNotExist:
         # No draft was ever created.
         return None
-    
+
     # draft.version could be None if it was set that way by set_draft_version.
     # Setting the Draft.version to None is how we show that we've "deleted" the
-    # content in Studio. 
+    # content in Studio.
     return draft.version
 
 

openedx_learning/lib/admin_utils.py

@@ -5,7 +5,6 @@
 from django.db.models.fields.reverse_related import OneToOneRel
 from django.urls import reverse, NoReverseMatch
 from django.utils.html import format_html, format_html_join
-from django.utils.safestring import mark_safe
 
 
 class ReadOnlyModelAdmin(admin.ModelAdmin):

tests/openedx_learning/core/publishing/test_api.py

@@ -81,6 +81,9 @@ def test_already_exists(self) -> None:
 
 
 class DraftTestCase(TestCase):
+    """
+    Test basic operations with Drafts.
+    """
 
     def test_draft_lifecycle(self):
         """