diff --git a/tests/app/main/views/test_dashboard.py b/tests/app/main/views/test_dashboard.py index f92efbb19..d46235668 100644 --- a/tests/app/main/views/test_dashboard.py +++ b/tests/app/main/views/test_dashboard.py @@ -1884,7 +1884,7 @@ def test_service_dashboard_shows_batched_jobs( assert len(rows) == 1 -@pytest.fixture +@pytest.fixture() def app_with_socketio(): app = Flask("app") create_app(app) @@ -1892,10 +1892,18 @@ def app_with_socketio(): @pytest.mark.parametrize( - "service_id, date_range, expected_call_args", + ("service_id", "date_range", "expected_call_args"), [ - (SERVICE_ONE_ID, {"start_date": "2024-01-01", "days": 7}, {"service_id": SERVICE_ONE_ID, "start_date": "2024-01-01", "days": 7}), - (SERVICE_TWO_ID, {"start_date": "2023-06-01", "days": 7}, {"service_id": SERVICE_TWO_ID, "start_date": "2023-06-01", "days": 7}), + ( + SERVICE_ONE_ID, + {"start_date": "2024-01-01", "days": 7}, + {"service_id": SERVICE_ONE_ID, "start_date": "2024-01-01", "days": 7} + ), + ( + SERVICE_TWO_ID, + {"start_date": "2023-06-01", "days": 7}, + {"service_id": SERVICE_TWO_ID, "start_date": "2023-06-01", "days": 7} + ), ] ) def test_fetch_daily_stats( @@ -1914,7 +1922,10 @@ def test_fetch_daily_stats( mock_service_api = mocker.patch( "app.service_api_client.get_service_notification_statistics_by_day", return_value={ - date_range["start_date"]: {"email": {"delivered": 0, "failure": 0, "requested": 0}, "sms": {"delivered": 0, "failure": 1, "requested": 1}}, + date_range["start_date"]: { + "email": {"delivered": 0, "failure": 0, "requested": 0}, + "sms": {"delivered": 0, "failure": 1, "requested": 1} + }, } ) @@ -1929,7 +1940,10 @@ def test_fetch_daily_stats( assert received, "Should receive a response message" assert received[0]['name'] == 'daily_stats_update' assert received[0]['args'][0] == { - date_range["start_date"]: {"email": {"delivered": 0, "failure": 0, "requested": 0}, "sms": {"delivered": 0, "failure": 1, "requested": 1}}, + date_range["start_date"]: { + "email": {"delivered": 0, "failure": 0, "requested": 0}, + "sms": {"delivered": 0, "failure": 1, "requested": 1} + }, } mock_service_api.assert_called_once_with(