eli-366 addressing ruff failures with updated version of ruff

Author: eddalmond1

poetry.lock

@@ -37,7 +37,7 @@ botocore = "^1.40.57"
 aws-xray-sdk = "2.15.0"
 
 [tool.poetry.group.dev.dependencies]
-ruff = "^0.11.13"
+ruff = "^0.14.10"
 docopt = "^0.6.2"
 jsonpath-rw = "^1.4.0"
 semver = "^3.0.4"

pyproject.toml

@@ -129,7 +129,7 @@ def get_best_iteration_result(self, campaign_group: list[CampaignConfig]) -> Bes
         if not iteration_results:
             return None
 
-        (best_iteration_name, best_iteration_result) = max(
+        (_best_iteration_name, best_iteration_result) = max(
             iteration_results.items(),
             key=lambda item: next(iter(item[1].cohort_results.values())).status.value
             # Below handles the case where there are no cohort results

src/eligibility_signposting_api/services/calculators/eligibility_calculator.py

@@ -228,7 +228,7 @@ def iam_role(iam_client: BaseClient) -> Generator[str]:
 
 @pytest.fixture(scope="session")
 def lambda_zip() -> Path:
-    build_result = subprocess.run(["make", "build"], capture_output=True, text=True, check=False)  # Noqa: S603, S607
+    build_result = subprocess.run(["make", "build"], capture_output=True, text=True, check=False)  # noqa: S603, S607
     assert build_result.returncode == 0, f"'make build' failed: {build_result.stderr}"
     return Path("dist/lambda.zip")
 

tests/integration/conftest.py

@@ -1436,7 +1436,7 @@ def test_eligibility_status_with_invalid_tokens_raises_attribute_error(faker: Fa
 
     calculator = EligibilityCalculator(person_rows, campaign_configs)
 
-    with pytest.raises(ValueError, match="Invalid token."):
+    with pytest.raises(ValueError, match=r"Invalid token\."):
         calculator.get_eligibility_status("Y", ["ALL"], "ALL")
 
 

tests/unit/services/calculators/test_eligibility_calculator.py

@@ -38,7 +38,7 @@ def test_parse_valid_tokens(self, token, expected_level, expected_name, expected
         ],
     )
     def test_parse_invalid_tokens_raises_error(self, token):
-        with pytest.raises(ValueError, match="Invalid token."):
+        with pytest.raises(ValueError, match=r"Invalid token\."):
             TokenParser.parse(token)
 
     @pytest.mark.parametrize(
@@ -53,5 +53,5 @@ def test_parse_invalid_tokens_raises_error(self, token):
         ],
     )
     def test_parse_invalid_token_format_raises_error(self, token):
-        with pytest.raises(ValueError, match="Invalid token format."):
+        with pytest.raises(ValueError, match=r"Invalid token format\."):
             TokenParser.parse(token)

tests/unit/services/processors/test_token_parser.py

@@ -351,7 +351,7 @@ def test_valid_token_invalid_format_should_raise_error(self, token_format: str):
             actions=[],
         )
 
-        with pytest.raises(ValueError, match="Invalid token format."):
+        with pytest.raises(ValueError, match=r"Invalid token format\."):
             TokenProcessor.find_and_replace_tokens(person, condition)
 
     @pytest.mark.parametrize(