Apply PR Review fixes

Author: iampujan

apps/blogs/migrations/0004_auto_20260222_0834.py

@@ -5,9 +5,12 @@
 
 def rewrite_blog_urls(apps, schema_editor):
     BlogEntry = apps.get_model("blogs", "BlogEntry")
+    entries_to_update = []
     for entry in BlogEntry.objects.filter(url__contains="pythoninsider.blogspot.com"):
         entry.url = entry.url.replace("pythoninsider.blogspot.com", "blog.python.org")
-        entry.save()
+        entries_to_update.append(entry)
+    if entries_to_update:
+        BlogEntry.objects.bulk_update(entries_to_update, ['url'])
 
 
 class Migration(migrations.Migration):

apps/blogs/tests/test_parser.py

@@ -31,13 +31,20 @@ def test_rewrites_blogspot_url(self, mock_parse):
         mock_parse.return_value = {
             "entries": [
                 {
-                    "title": "Test Title",
+                    "title": "Test Title HTTPS",
                     "summary": "Summary",
-                    "published_parsed": (2026, 2, 22, 12, 0, 0, 0, 0, 0),
-                    "link": "https://pythoninsider.blogspot.com/2026/02/test.html",
+                    "published_parsed": (2024, 1, 15, 12, 0, 0, 0, 0, 0),
+                    "link": "https://pythoninsider.blogspot.com/2024/01/test.html",
+                },
+                {
+                    "title": "Test Title HTTP",
+                    "summary": "Summary",
+                    "published_parsed": (2024, 1, 15, 12, 0, 0, 0, 0, 0),
+                    "link": "http://pythoninsider.blogspot.com/2024/01/test2.html",
                 }
             ]
         }
         entries = get_all_entries("http://fake.url")
-        self.assertEqual(len(entries), 1)
-        self.assertEqual(entries[0]["url"], "https://blog.python.org/2026/02/test.html")
+        self.assertEqual(len(entries), 2)
+        self.assertEqual(entries[0]["url"], "https://blog.python.org/2024/01/test.html")
+        self.assertEqual(entries[1]["url"], "http://blog.python.org/2024/01/test2.html")