From 451de0ace2cdb78bb9e11ac1c562f62d8459b4a0 Mon Sep 17 00:00:00 2001
From: mateo-berri <277851410+mateo-berri@users.noreply.github.com>
Date: Tue, 8 Sep 2026 11:39:10 -0700
Subject: [PATCH] test(otel v2): drop narrating docstrings from the langfuse
 routing tests

---
 .../integrations/otel/test_otel_v2_dynamic.py  | 18 ------------------
 1 file changed, 18 deletions(-)

diff --git a/tests/test_litellm/integrations/otel/test_otel_v2_dynamic.py b/tests/test_litellm/integrations/otel/test_otel_v2_dynamic.py
index e37d4b3ff1f..03d97d06340 100644
--- a/tests/test_litellm/integrations/otel/test_otel_v2_dynamic.py
+++ b/tests/test_litellm/integrations/otel/test_otel_v2_dynamic.py
@@ -686,12 +686,6 @@ LANGFUSE_CREDS = {"langfuse_public_key": "pk", "langfuse_secret_key": "sk"}
 
 
 def test_langfuse_dynamic_endpoint_follows_the_key_host():
-    """A key that names its own Langfuse host must move the export destination.
-
-    Without this the preset swapped the key's credentials in and left the exporter
-    aimed at the process-wide env host, so a tenant's spans were POSTed to the
-    operator's Langfuse signed with keys that host does not know.
-    """
     assert (
         dynamic_otlp_endpoint(
             "langfuse_otel", {**LANGFUSE_CREDS, "langfuse_host": "http://team-b-langfuse:3100"}
@@ -708,8 +702,6 @@ def test_langfuse_dynamic_endpoint_normalizes_a_bare_host():
 
 
 def test_langfuse_dynamic_endpoint_is_none_without_a_host():
-    # No host on the key means "do not move the destination": the preset's
-    # env-resolved endpoint stands, matching V1's fallback to the env host.
     assert dynamic_otlp_endpoint("langfuse_otel", LANGFUSE_CREDS) is None
 
 
@@ -722,18 +714,10 @@ def test_langfuse_dynamic_endpoint_is_none_without_a_host():
     ],
 )
 def test_langfuse_dynamic_endpoint_is_none_without_the_key_pair(partial):
-    """A host without both keys must not move the destination.
-
-    The headers builder needs both keys, so a host on its own would leave the
-    exporter carrying the operator's env-derived Authorization header while
-    pointing it at the caller's host. V1 returns None in the same state.
-    """
     assert dynamic_otlp_endpoint("langfuse_otel", {**partial, "langfuse_host": "attacker.example.com"}) is None
 
 
 def test_operator_credentials_stay_on_the_operator_endpoint():
-    """A team that sets only a host, and reaches routing via its service name,
-    must not redirect the exporter that still carries the operator's header."""
     cache = _cache(
         "langfuse_otel",
         exporters=[
@@ -759,8 +743,6 @@ def test_operator_credentials_stay_on_the_operator_endpoint():
 
 
 def test_langfuse_host_stamped_onto_owned_exporter_only():
-    """A Langfuse key's host must never repoint a co-configured exporter owned by
-    a different backend."""
     cache = _cache(
         "langfuse_otel",
         exporters=[