Linting

Author: kathryn-dale

ingestion/data_transfer_models/headline.py

@@ -109,7 +109,7 @@ def validate_period_dates(
         return validation.validate_period_end(
             period_start=input_period_start, period_end=period_end
         )
-    
+
     @model_validator(mode="after")
     def invalidate_non_public_data_for_public_ingestion(self) -> Self:
         """Checks that if this is a public instance of the product then `is_public=False` data is invalidated."""
@@ -155,7 +155,7 @@ def _build_enriched_headline_specific_fields(
             period_end=individual_time_series["period_end"],
             embargo=individual_time_series["embargo"],
             metric_value=individual_time_series["metric_value"],
-            is_public=individual_time_series["is_public"]
+            is_public=individual_time_series["is_public"],
         )
         for individual_time_series in source_data["data"]
         if individual_time_series["metric_value"] is not None

tests/integration/ingestion/test_consumer.py

@@ -266,7 +266,7 @@ def test_can_ingest_duplicated_data_with_force_write_flag(self):
                     "date": "2024-06-17",
                     "metric_value": 71,
                     "embargo": "2024-07-16 09:30:00",
-                    "is_public": True
+                    "is_public": True,
                 }
             ],
         }

tests/system/test_weather_health_alerts.py

@@ -34,7 +34,7 @@ def cold_alert_payload(self) -> dict[str, str | list[dict]]:
                     "period_end": "2025-01-07 08:00:00",
                     "metric_value": 11,
                     "embargo": None,
-                    "is_public": True
+                    "is_public": True,
                 }
             ],
             "refresh_date": "2025-06-11 11:15:00",

tests/unit/ingestion/consumer/test_build_models.py

@@ -104,7 +104,7 @@ def test_build_core_time_series(
                 "date": "2023-08-01",
                 "metric_value": 123,
                 "force_write": True,
-                "is_public": True
+                "is_public": True,
             }
         ]
 

tests/unit/ingestion/data_transfer_models/headline/test_headline.py

@@ -39,7 +39,7 @@ def test_casts_dates_for_period_start_and_end(self):
             period_end=fake_period_end,
             embargo=VALID_DATETIME,
             metric_value=123,
-            is_public=True
+            is_public=True,
         )
 
         # Then
@@ -66,7 +66,7 @@ def test_validates_embargo_of_none(self):
             period_end=fake_period_end,
             embargo=fake_embargo,
             metric_value=123,
-            is_public=True
+            is_public=True,
         )
 
         # Then
@@ -91,7 +91,7 @@ def test_validates_datetime_passed_to_period_start(self):
             period_end=fake_period_end,
             metric_value=123,
             embargo=None,
-            is_public=True
+            is_public=True,
         )
 
         # Then
@@ -116,7 +116,7 @@ def test_validates_when_datetime_passed_to_period_end(self):
             period_end=fake_period_end,
             metric_value=123,
             embargo=None,
-            is_public=True
+            is_public=True,
         )
 
         # Then
@@ -145,7 +145,7 @@ def test_validates_date_passed_to_embargo(self):
             period_end=fake_period_end,
             embargo=fake_embargo,
             metric_value=123,
-            is_public=True
+            is_public=True,
         )
 
         # Then

tests/unit/ingestion/data_transfer_models/headline/test_period_dates.py

@@ -23,7 +23,7 @@ def test_period_end_after_period_start_is_deemed_valid(self):
             period_end=period_end,
             embargo=None,
             metric_value=123,
-            is_public=True
+            is_public=True,
         )
 
         # Then
@@ -45,7 +45,7 @@ def test_period_end_as_same_date_as_period_start_is_deemed_valid(self):
             period_end=period_end,
             embargo=None,
             metric_value=123,
-            is_public=True
+            is_public=True,
         )
 
         # Then

tests/unit/ingestion/data_transfer_models/time_series/test_epiweek.py

@@ -11,7 +11,7 @@ def payload_without_epiweek(self) -> dict[str, str | int | None]:
             "date": "2023-11-01",
             "embargo": None,
             "metric_value": 123,
-            "is_public": True
+            "is_public": True,
         }
 
     @pytest.mark.parametrize("epiweek", list(range(1, 53 + 1)))

tests/unit/ingestion/data_transfer_models/time_series/test_time_series.py

@@ -67,7 +67,7 @@ def test_validates_payload_with_embargo_of_none(self):
                 date=fake_date,
                 embargo=fake_embargo,
                 metric_value=fake_metric_value,
-                is_public=True
+                is_public=True,
             )
         )
 
@@ -137,7 +137,7 @@ def test_raises_error_when_date_passed_to_embargo(self):
                 epiweek=fake_epiweek,
                 embargo=fake_embargo,
                 metric_value=123,
-                is_public=True
+                is_public=True,
             )
         )