Fix the log level in the run-generate-reports celery task (#2490)

Author: smcmurtry

app/celery/scheduled_tasks.py

@@ -393,7 +393,7 @@ def beat_inbox_sms_priority():
 @notify_celery.task(name="run-generate-reports")
 @statsd(namespace="tasks")
 def run_generate_reports():
-    current_app.logger.error("starting run-generate-reports")
+    current_app.logger.info("starting run-generate-reports")
 
     # query for reports that have been requested but haven't been generated yet
     requested_reports = Report.query.filter(Report.status.in_([ReportStatus.REQUESTED.value])).order_by(Report.requested_at).all()

tests/app/celery/test_scheduled_tasks.py

@@ -630,7 +630,6 @@ def test_mark_jobs_complete(
 
 def test_run_generate_reports(mocker, notify_db_session, sample_user, sample_service):
     mock_logger = mocker.patch("app.celery.scheduled_tasks.current_app.logger.info")
-    mock_error_logger = mocker.patch("app.celery.scheduled_tasks.current_app.logger.error")
     mock_generate_report = mocker.patch("app.celery.scheduled_tasks.generate_report.apply_async")
 
     # Create some reports in REQUESTED status
@@ -665,8 +664,8 @@ def test_run_generate_reports(mocker, notify_db_session, sample_user, sample_ser
     scheduled_tasks.run_generate_reports()
 
     # Check logging
-    mock_error_logger.assert_called_once_with("starting run-generate-reports")
-    assert mock_logger.call_count == 2
+    assert mock_logger.call_count == 3
+    mock_logger.assert_any_call("starting run-generate-reports")
     mock_logger.assert_any_call(f"calling generate_report for Report ID {report1.id}")
     mock_logger.assert_any_call(f"calling generate_report for Report ID {report2.id}")