test(lint): fix PT011/PT012 introduced by #37736, blocking lint on every PR

[tin-berri]

TLDR

ruff check --config ruff-tests.toml tests is red on litellm_internal_staging, which blocks the lint job for every open PR into it (including unrelated ones — it's a whole-tree check, not delta-vs-base).

Both violations landed in #37736, merged the day before B017/PT012 started being enforced whole-tree in #37731/#37748, so no delta-vs-base gate caught them at the time.

Changes

• tests/proxy_unit_tests/test_user_api_key_auth.py: pytest.raises(Exception) with no match= (B017) — any Exception subtype, including one from an unrelated regression, satisfies the assert and reads as a pass. Narrowed to match=r"(?i)budget", which preserves the assert "budget" in str(exc.value).lower() it replaces.
• tests/proxy_unit_tests/test_unit_test_max_model_budget_limiter.py: an if/else selecting between two different awaited calls, both inside pytest.raises() (PT012) — the block must hold a single simple statement so a coroutine built in the wrong branch can't silently never run. Coroutine is now built outside the block and awaited inside it.

Verification

• uv run --no-sync ruff check --config ruff-tests.toml tests: clean.
• Both tests pass as edited.
• Mutation-checked each: injecting an unrelated exception / dropping the pre-seeded spend still fails the test, confirming the restructuring didn't weaken what's being asserted.

Pre-Submission checklist

• I have added meaningful tests (N/A — test-only lint fix, no behavior change)
• My PR passes all CI/CD checks (e.g., lint, format, unit tests)
• My PR's scope is as isolated as possible; it only solves 1 specific problem

[greptile-apps]

Greptile Summary

This PR fixes two test lint violations without changing production behavior.

• Restructures a parametrized budget-limiter test so pytest.raises contains one awaited statement.
• Moves the existing case-insensitive budget-message assertion into pytest.raises(match=...).

Confidence Score: 5/5

The PR appears safe to merge because both changes preserve the existing test behavior while satisfying the lint rules.

Both limiter calls are native async methods whose bodies still execute inside pytest.raises, and the revised exception match preserves the removed case-insensitive budget substring assertion.

Important Files Changed

Filename	Overview
tests/proxy_unit_tests/test_unit_test_max_model_budget_limiter.py	Coroutine selection now occurs before pytest.raises, while execution and exception capture remain inside it for both parametrized branches.
tests/proxy_unit_tests/test_user_api_key_auth.py	The exception assertion is expressed through an equivalent case-insensitive regex while retaining the user-ID check.

_{Reviews (1): Last reviewed commit: "test(lint): fix PT011/PT012 in tests int..." | Re-trigger Greptile}