Review

tests/ext/test_application.py

@@ -2454,6 +2454,7 @@ async def callback(update, context):
             )
             assert record.levelno == logging.CRITICAL
 
+            # Let's also check that no critical log is produced when the exception is not raised
             caplog.clear()
             with caplog.at_level(logging.CRITICAL):
                 await app.process_update(2)
@@ -2493,6 +2494,7 @@ async def callback(update, context):
             )
             assert record.levelno == logging.CRITICAL
 
+            # Let's also check that no critical log is produced when the exception is not raised
             caplog.clear()
             with caplog.at_level(logging.CRITICAL):
                 await app.process_error(update=None, error=2)

tests/ext/test_jobqueue.py

@@ -655,7 +655,6 @@ async def test_run_job_exception_in_building_context(
         original_from_job = CallbackContext.from_job
 
         def raise_exception(job, application):
-            print(job, job.data, application)
             if job.data == 1:
                 raise exception
             return original_from_job(job, application)
@@ -680,6 +679,7 @@ async def job_callback(context):
         )
         assert record.levelno == logging.CRITICAL
 
+        # Let's also check that no critical log is produced when the exception is not raised
         caplog.clear()
         with caplog.at_level(logging.CRITICAL):
             job_queue.run_once(job_callback, 0.1, data=2)