diff --git a/app/main/views/dashboard.py b/app/main/views/dashboard.py index 0f432c439..0913dcfdd 100644 --- a/app/main/views/dashboard.py +++ b/app/main/views/dashboard.py @@ -94,7 +94,7 @@ def service_dashboard(service_id): @main.route("/daily_stats.json") def get_daily_stats(): - service_id = session.get('service_id') + service_id = session.get("service_id") date_range = get_stats_date_range() stats = service_api_client.get_service_notification_statistics_by_day( @@ -105,7 +105,7 @@ def get_daily_stats(): @main.route("/daily_stats_by_user.json") def get_daily_stats_by_user(): - service_id = session.get('service_id') + service_id = session.get("service_id") date_range = get_stats_date_range() user_id = current_user.id stats = service_api_client.get_user_service_notification_statistics_by_day( diff --git a/tests/app/main/views/test_jobs_activity.py b/tests/app/main/views/test_jobs_activity.py index 962a2a378..cfc61b96e 100644 --- a/tests/app/main/views/test_jobs_activity.py +++ b/tests/app/main/views/test_jobs_activity.py @@ -21,7 +21,10 @@ MOCK_JOBS = { "scheduled_for": None, "service": "21b3ee3d-1cb0-4666-bfa0-9c5ac26d3fe3", "service_name": {"name": "Mock Texting Service"}, - "statistics": [{"count": 1, "status": "delivered"}, {"count": 5, "status": "failed"}], + "statistics": [ + {"count": 1, "status": "delivered"}, + {"count": 5, "status": "failed"}, + ], "template": "6a456418-498c-4c86-b0cd-9403c14a216c", "template_name": "Mock Template Name", "template_type": "sms", @@ -63,7 +66,15 @@ def test_all_activity( assert table is not None, "Table not found in the response" headers = [th.get_text(strip=True) for th in table.find_all("th")] - expected_headers = ["Job ID#", "Template", "Time sent", "Sender", "Report", "Delivered", "Failed"] + expected_headers = [ + "Job ID#", + "Template", + "Time sent", + "Sender", + "Report", + "Delivered", + "Failed", + ] assert ( headers == expected_headers @@ -103,9 +114,7 @@ def test_all_activity( ), f"Expected delivered count '1', but got '{delivered_cell}'" failed_cell = cells[6].get_text(strip=True) - assert ( - failed_cell == "5" - ), f"Expected failed count '5', but got '{failed_cell}'" + assert failed_cell == "5", f"Expected failed count '5', but got '{failed_cell}'" mock_get_page_of_jobs.assert_called_with(SERVICE_ONE_ID, page=current_page) diff --git a/tests/app/test_navigation.py b/tests/app/test_navigation.py index aeee32164..a9ce4ea87 100644 --- a/tests/app/test_navigation.py +++ b/tests/app/test_navigation.py @@ -96,6 +96,8 @@ EXCLUDED_ENDPOINTS = tuple( "get_users_report", "get_daily_volumes", "get_daily_sms_provider_volumes", + "get_daily_stats", + "get_daily_stats_by_user", "get_volumes_by_service", "get_example_csv", "get_notifications_as_json",