diff --git a/zerver/tests/test_logging_handlers.py b/zerver/tests/test_logging_handlers.py
index bdae3780b5..c38cf4d639 100644
--- a/zerver/tests/test_logging_handlers.py
+++ b/zerver/tests/test_logging_handlers.py
@@ -108,7 +108,7 @@ class AdminZulipHandlerTest(ZulipTestCase):
             record = self.logger.makeRecord('name', logging.ERROR, 'function', 15,  # type: ignore # https://github.com/python/typeshed/pull/1100
                                             'message\nmoremesssage\nmore', None,
                                             None)
-            record.request = captured_request # type: ignore # this field is dynamically added
+            record.request = captured_request  # type: ignore # this field is dynamically added
 
             report = self.run_handler(record)
             self.assertIn("user_email", report)
@@ -131,7 +131,7 @@ class AdminZulipHandlerTest(ZulipTestCase):
             global captured_request
             global captured_exc_info
             record = self.logger.makeRecord('name', logging.ERROR, 'function', 15, 'message', None, captured_exc_info)  # type: ignore # https://github.com/python/typeshed/pull/1100
-            record.request = captured_request # type: ignore # this field is dynamically added
+            record.request = captured_request  # type: ignore # this field is dynamically added
 
             report = self.run_handler(record)
             self.assertIn("user_email", report)
@@ -148,7 +148,7 @@ class AdminZulipHandlerTest(ZulipTestCase):
             self.assertEqual(report["stack_trace"], "See /var/log/zulip/errors.log")
 
             # Check anonymous user is handled correctly
-            record.request.user = AnonymousUser() # type: ignore # this field is dynamically added
+            record.request.user = AnonymousUser()  # type: ignore # this field is dynamically added
             report = self.run_handler(record)
             self.assertIn("host", report)
             self.assertIn("user_email", report)
@@ -159,10 +159,10 @@ class AdminZulipHandlerTest(ZulipTestCase):
             def get_host_error():
                 # type: () -> None
                 raise Exception("Get Host Failure!")
-            orig_get_host = record.request.get_host # type: ignore # this field is dynamically added
-            record.request.get_host = get_host_error # type: ignore # this field is dynamically added
+            orig_get_host = record.request.get_host  # type: ignore # this field is dynamically added
+            record.request.get_host = get_host_error  # type: ignore # this field is dynamically added
             report = self.run_handler(record)
-            record.request.get_host = orig_get_host # type: ignore # this field is dynamically added
+            record.request.get_host = orig_get_host  # type: ignore # this field is dynamically added
             self.assertIn("host", report)
             self.assertIn("user_email", report)
             self.assertIn("message", report)
@@ -171,9 +171,9 @@ class AdminZulipHandlerTest(ZulipTestCase):
             # Test an exception_filter exception
             with patch("zerver.logging_handlers.get_exception_reporter_filter",
                        return_value=15):
-                record.request.method = "POST" # type: ignore # this field is dynamically added
+                record.request.method = "POST"  # type: ignore # this field is dynamically added
                 report = self.run_handler(record)
-                record.request.method = "GET" # type: ignore # this field is dynamically added
+                record.request.method = "GET"  # type: ignore # this field is dynamically added
             self.assertIn("host", report)
             self.assertIn("user_email", report)
             self.assertIn("message", report)
@@ -199,7 +199,7 @@ class AdminZulipHandlerTest(ZulipTestCase):
             self.assertEqual(report["stack_trace"], None)
 
             # Test arbitrary exceptions from request.user
-            record.request.user = None # type: ignore # this field is dynamically added
+            record.request.user = None  # type: ignore # this field is dynamically added
             with patch("zerver.logging_handlers.traceback.print_exc"):
                 report = self.run_handler(record)
             self.assertIn("host", report)