[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Author: pre-commit-ci[bot]

tests/model_serving/maas_billing/maas_subscription/conftest.py

@@ -814,7 +814,7 @@ def ephemeral_api_key(
     request_session_http: requests.Session,
     base_url: str,
     ocp_token_for_actor: str,
-) -> Generator[dict[str, Any], None, None]:
+) -> Generator[dict[str, Any]]:
     """Create an ephemeral API key and revoke it on teardown."""
     create_resp, create_body = create_api_key(
         base_url=base_url,

tests/model_serving/maas_billing/maas_subscription/test_api_key_ephemeral_cleanup.py

@@ -37,15 +37,11 @@ def test_cronjob_exists_and_configured(self, admin_client: DynamicClient) -> Non
             name=MAAS_CLEANUP_CRONJOB_NAME,
             namespace=applications_namespace,
         )
-        assert cronjob.exists, (
-            f"CronJob {MAAS_CLEANUP_CRONJOB_NAME} not found in {applications_namespace}"
-        )
+        assert cronjob.exists, f"CronJob {MAAS_CLEANUP_CRONJOB_NAME} not found in {applications_namespace}"
 
         spec = cronjob.instance.spec
 
-        assert spec.schedule == "*/15 * * * *", (
-            f"Expected schedule '*/15 * * * *', got '{spec.schedule}'"
-        )
+        assert spec.schedule == "*/15 * * * *", f"Expected schedule '*/15 * * * *', got '{spec.schedule}'"
         assert spec.concurrencyPolicy == "Forbid", (
             "CronJob should use Forbid concurrency policy to prevent overlapping runs"
         )
@@ -61,13 +57,9 @@ def test_cronjob_exists_and_configured(self, admin_client: DynamicClient) -> Non
         sec_ctx = getattr(container_spec, "securityContext", None)
         assert sec_ctx is not None, "Cleanup container should have securityContext configured"
         assert sec_ctx.runAsNonRoot is True, "Cleanup container should run as non-root"
-        assert sec_ctx.readOnlyRootFilesystem is True, (
-            "Cleanup container should have read-only root filesystem"
-        )
+        assert sec_ctx.readOnlyRootFilesystem is True, "Cleanup container should have read-only root filesystem"
 
-        LOGGER.info(
-            f"[ephemeral] CronJob validated: schedule={spec.schedule}, concurrency={spec.concurrencyPolicy}"
-        )
+        LOGGER.info(f"[ephemeral] CronJob validated: schedule={spec.schedule}, concurrency={spec.concurrencyPolicy}")
 
     @pytest.mark.tier1
     def test_cleanup_networkpolicy_exists(self, admin_client: DynamicClient) -> None:
@@ -143,9 +135,7 @@ def test_create_ephemeral_key(
             f"Expected 200 from default search, got {r_default.status_code}: {(r_default.text or '')[:200]}"
         )
         default_body = r_default.json()
-        default_items: list[dict[str, Any]] = (
-            default_body.get("items") or default_body.get("data") or []
-        )
+        default_items: list[dict[str, Any]] = default_body.get("items") or default_body.get("data") or []
         assert key_id not in [item["id"] for item in default_items], (
             "Ephemeral key should be excluded from default search (includeEphemeral defaults to False)"
         )
@@ -187,9 +177,7 @@ def test_trigger_cleanup_preserves_active_keys(
         )
         cleanup_resp = cleanup_response.json()
         deleted_count = cleanup_resp.get("deletedCount", -1)
-        assert deleted_count >= 0, (
-            f"Cleanup response should have non-negative deletedCount, got: {cleanup_resp}"
-        )
+        assert deleted_count >= 0, f"Cleanup response should have non-negative deletedCount, got: {cleanup_resp}"
         LOGGER.info(f"[ephemeral] Cleanup completed: deletedCount={deleted_count}")
 
         r_get = request_session_http.get(