From 7f796fce9d7917ce50c86a1348e9a602fb4a5a20 Mon Sep 17 00:00:00 2001 From: Conduct AI Date: Thu, 10 Sep 2026 20:45:22 -0500 Subject: [PATCH] style: explicit assert on noop test (TQ001 zero-assert budget) BerriAI/litellm CI's test-quality budget flagged test_raise_if_missing_package_is_noop_when_present as a zero-assert test (TQ001). Make the intent explicit: raise_if_missing_package() must return None when the package IS installed. --- tests/test_litellm/proxy/guardrails/test_conduct_guardrail.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/test_litellm/proxy/guardrails/test_conduct_guardrail.py b/tests/test_litellm/proxy/guardrails/test_conduct_guardrail.py index 11338772db9..ce7c888ca6e 100644 --- a/tests/test_litellm/proxy/guardrails/test_conduct_guardrail.py +++ b/tests/test_litellm/proxy/guardrails/test_conduct_guardrail.py @@ -206,4 +206,4 @@ def test_raise_if_missing_package_is_noop_when_present() -> None: raise_if_missing_package, ) - raise_if_missing_package() # must not raise + assert raise_if_missing_package() is None