Fix: tests and improve code quality

Author: SecCodeSmith

BlogApi/admin.py

@@ -1,55 +1,55 @@
 from django.contrib import admin
-from .models import Author, Category, Tag, Post, Comment
+
+from .models import Author, Category, Comment, Post, Tag
 
 
 class CommentInline(admin.TabularInline):
     model = Comment
     extra = 0
-    readonly_fields = ('created_at',)
-    fields = ('name', 'email', 'content', 'created_at', 'is_public')
+    readonly_fields = ("created_at",)
+    fields = ("name", "email", "content", "created_at", "is_public")
 
 
 @admin.register(Author)
 class AuthorAdmin(admin.ModelAdmin):
-    list_display = ('name', 'email')
-    search_fields = ('name', 'email')
+    list_display = ("name", "email")
+    search_fields = ("name", "email")
     readonly_fields = ("image_tag",)
 
 
 @admin.register(Category)
 class CategoryAdmin(admin.ModelAdmin):
-    list_display = ('title', 'slug')
-    prepopulated_fields = {'slug': ('title',)}
-    search_fields = ('title',)
-    ordering = ('title',)
+    list_display = ("title", "slug")
+    prepopulated_fields = {"slug": ("title",)}
+    search_fields = ("title",)
+    ordering = ("title",)
 
 
 @admin.register(Tag)
 class TagAdmin(admin.ModelAdmin):
-    list_display = ('name', 'slug')
-    prepopulated_fields = {'slug': ('name',)}
-    search_fields = ('name',)
-    ordering = ('name',)
+    list_display = ("name", "slug")
+    prepopulated_fields = {"slug": ("name",)}
+    search_fields = ("name",)
+    ordering = ("name",)
 
 
 @admin.register(Post)
 class PostAdmin(admin.ModelAdmin):
-    list_display = ('title', 'author', 'category', 'published_at', 'featured')
-    list_filter = ('featured', 'category', 'published_at', 'tags')
-    search_fields = ('title', 'excerpt', 'content')
-    prepopulated_fields = {'slug': ('title',)}
-    date_hierarchy = 'published_at'
-    autocomplete_fields = ('author', 'category')
-    filter_horizontal = ('tags',)
+    list_display = ("title", "author", "category", "published_at", "featured")
+    list_filter = ("featured", "category", "published_at", "tags")
+    search_fields = ("title", "excerpt", "content")
+    prepopulated_fields = {"slug": ("title",)}
+    date_hierarchy = "published_at"
+    autocomplete_fields = ("author", "category")
+    filter_horizontal = ("tags",)
     inlines = [CommentInline]
     readonly_fields = ("image_tag",)
 
 
 @admin.register(Comment)
 class CommentAdmin(admin.ModelAdmin):
-    list_display = ('name', 'post', 'created_at', 'is_public')
-    list_filter = ('is_public', 'created_at')
-    search_fields = ('name', 'email', 'content')
-    date_hierarchy = 'created_at'
-    autocomplete_fields = ('post',)
-
+    list_display = ("name", "post", "created_at", "is_public")
+    list_filter = ("is_public", "created_at")
+    search_fields = ("name", "email", "content")
+    date_hierarchy = "created_at"
+    autocomplete_fields = ("post",)

BlogApi/apps.py

@@ -2,5 +2,5 @@
 
 
 class BlogapiConfig(AppConfig):
-    default_auto_field = 'django.db.models.BigAutoField'
-    name = 'BlogApi'
+    default_auto_field = "django.db.models.BigAutoField"
+    name = "BlogApi"

BlogApi/models.py

@@ -3,28 +3,29 @@
 from django.utils.html import format_html
 from django.utils.text import slugify
 
+
 class Author(models.Model):
     """
     Represents an author of a post.
     If you use Django’s built-in User, you can instead
     point to settings.AUTH_USER_MODEL via ForeignKey.
     """
+
     name = models.CharField(max_length=100)
     email = models.EmailField(unique=True)
     bio = models.TextField(blank=True)
     avatar = models.ImageField(
-        upload_to='authors/avatars/',
+        upload_to="authors/avatars/",
         null=True,
         blank=True,
-        help_text="Optional profile picture for the author"
+        help_text="Optional profile picture for the author",
     )
 
     @admin.display
     def image_tag(self):
-        return format_html('<img src="{}" alt="author img" height="100" />',
-                           self.image.url)
+        return format_html('<img src="{}" alt="author img" height="100" />', self.image.url)
 
-    image_tag.short_description = 'Image'
+    image_tag.short_description = "Image"
     image_tag.allow_tags = True
 
     def __str__(self):
@@ -36,6 +37,7 @@ class Category(models.Model):
     A simple Category model. If you prefer to keep category
     as a CharField, you can skip this and use a choices tuple instead.
     """
+
     title = models.CharField(max_length=50, unique=True)
     slug = models.SlugField(max_length=60, unique=True)
 
@@ -56,6 +58,7 @@ class Tag(models.Model):
     """
     Tags for posts. Many-to-many relationship from Post to Tag.
     """
+
     name = models.CharField(max_length=30, unique=True)
     slug = models.SlugField(max_length=40, unique=True)
 
@@ -85,49 +88,35 @@ class Post(models.Model):
       tags: string[];
       content: string;
     """
+
     slug = models.SlugField(
         max_length=150,
         unique=True,
-        help_text="A URL-friendly identifier derived from title."
+        help_text="A URL-friendly identifier derived from title.",
     )
     title = models.CharField(max_length=200)
-    excerpt = models.TextField(
-        help_text="Short summary of the post (e.g. first 1–2 sentences)."
-    )
+    excerpt = models.TextField(help_text="Short summary of the post (e.g. first 1–2 sentences).")
     image = models.ImageField(
-        upload_to='posts/images/',
-        null=True, blank=True,)
+        upload_to="posts/images/",
+        null=True,
+        blank=True,
+    )
     category = models.ForeignKey(
         Category,
         on_delete=models.PROTECT,
         related_name="posts",
-        help_text="Select a category for this post."
+        help_text="Select a category for this post.",
     )
     published_at = models.DateTimeField(
         db_index=True,
         help_text="When the post was (or will be) published.",
         null=True,
-        blank=True
-    )
-    author = models.ForeignKey(
-        Author,
-        on_delete=models.CASCADE,
-        related_name="posts"
-    )
-    featured = models.BooleanField(
-        default=False,
-        help_text="Mark as featured post (e.g. for homepage slider)."
-    )
-    read_time = models.CharField(
-        max_length=20,
         blank=True,
-        help_text="Estimated read time, e.g. '5 min read'."
-    )
-    tags = models.ManyToManyField(
-        Tag,
-        related_name="posts",
-        blank=True
     )
+    author = models.ForeignKey(Author, on_delete=models.CASCADE, related_name="posts")
+    featured = models.BooleanField(default=False, help_text="Mark as featured post (e.g. for homepage slider).")
+    read_time = models.CharField(max_length=20, blank=True, help_text="Estimated read time, e.g. '5 min read'.")
+    tags = models.ManyToManyField(Tag, related_name="posts", blank=True)
     content = models.TextField(help_text="Full HTML or Markdown content of the post.")
 
     class Meta:
@@ -142,10 +131,9 @@ def __str__(self):
 
     @admin.display
     def image_tag(self):
-        return format_html('<img src="{}" height="100" />',
-                           self.image.url)
+        return format_html('<img src="{}" height="100" />', self.image.url)
 
-    image_tag.short_description = 'Image'
+    image_tag.short_description = "Image"
     image_tag.allow_tags = True
 
     @property
@@ -165,20 +153,14 @@ class Comment(models.Model):
     Represents a comment on a Post. Adjust fields as needed
     (e.g. if you want to link comments to registered users).
     """
-    post = models.ForeignKey(
-        Post,
-        on_delete=models.CASCADE,
-        related_name="comments"
-    )
+
+    post = models.ForeignKey(Post, on_delete=models.CASCADE, related_name="comments")
     name = models.CharField(max_length=80, help_text="Display name of the commenter")
     email = models.EmailField(help_text="Email of the commenter")
     content = models.TextField(help_text="Comment text")
     created_at = models.DateTimeField(auto_now_add=True)
 
-    is_public = models.BooleanField(
-        default=True,
-        help_text="Uncheck to hide comment without deleting."
-    )
+    is_public = models.BooleanField(default=True, help_text="Uncheck to hide comment without deleting.")
 
     class Meta:
         ordering = ["created_at"]