From 4051d9a369b012ac0c1e967284bcd31d7ba72a40 Mon Sep 17 00:00:00 2001
From: mateo-berri <277851410+mateo-berri@users.noreply.github.com>
Date: Thu, 21 May 2026 01:45:21 +0000
Subject: [PATCH] test(team_members): skip flaky test_duplicate_user_addition

Same /team/info 404-after-add_team_member race that already led to
test_add_multiple_members being skipped in dedc4022. Duplicate-prevention
behavior is covered by test_update_team_members_list_duplicate_prevention
in tests/test_litellm/proxy/management_endpoints/test_team_endpoints.py,
so the e2e proxy variant doesn't add coverage.
---
 tests/test_team_members.py | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/tests/test_team_members.py b/tests/test_team_members.py
index 415b3f07fc9..4cf85af6410 100644
--- a/tests/test_team_members.py
+++ b/tests/test_team_members.py
@@ -206,6 +206,9 @@ def test_error_handling(api_client):
         api_client.get_team_info("invalid-team-id")
 
 
+@pytest.mark.skip(
+    reason="Flaky in CI: /team/info?team_id=... intermittently returns 404 after add_team_member calls, same race documented for test_add_multiple_members. Duplicate-prevention is covered by test_update_team_members_list_duplicate_prevention in tests/test_litellm/proxy/management_endpoints/test_team_endpoints.py."
+)
 def test_duplicate_user_addition(api_client, new_team):
     """Test that adding the same user twice is handled appropriately"""
     # Add user first time