6578133bb7
* Fix user-team association issues in LiteLLM proxy - Update list_team function to properly filter teams using user's teams array instead of only checking members_with_roles field - Add Field descriptions and docstring to TeamMemberAddRequest and related models for better Swagger/OpenAPI documentation - Maintain backward compatibility with fallback to members_with_roles if user lookup fails This ensures users created with teams parameter appear correctly in team views and improves API documentation. * Fix duplicate member checking in team_member_add endpoint - Enhanced team_member_add_duplication_check to check both user_id and user_email - Added additional duplicate prevention logic after user creation/lookup - Fixed issue where users added by email could be duplicated in teams - Added logging for debugging duplicate detection This addresses the bug where adding the same user by email multiple times would create duplicate entries in the team's members_with_roles array. * Improve duplicate member prevention in team_member_add - Enhanced early duplicate check to handle both user_id and user_email - Added late-stage duplicate prevention after user lookup/creation - Fixed issue where users could be added multiple times by email - Cleaned up debug logging Note: There's still an edge case where the duplicate prevention may not work correctly in all scenarios. This needs further investigation and testing. * Refactor team_member_add endpoint for improved member management - Split team_member_add functionality into smaller, dedicated functions for permission validation, member processing, and team member list updates. - Enhanced permission checks to ensure only authorized users can add members. - Streamlined member addition logic to reduce redundancy and improve readability. - Maintained existing functionality while improving code structure and maintainability. * Add tests for team_member_add helper functions - Add test for _validate_team_member_add_permissions with admin user - Add test for _validate_team_member_add_permissions with non-admin user - Add test for _process_team_members with single member - Add test for _process_team_members with multiple members - Add test for _update_team_members_list with new member - Add test for _update_team_members_list duplicate prevention These tests ensure the refactored helper functions work correctly after fixing the PLR0915 linting error. |
||
|---|---|---|
| .. | ||
| caching | ||
| completion_extras/litellm_responses_transformation | ||
| enterprise/enterprise_callbacks | ||
| experimental_mcp_client | ||
| google_genai | ||
| integrations | ||
| litellm_core_utils | ||
| llms | ||
| passthrough | ||
| proxy | ||
| responses | ||
| router_strategy | ||
| router_utils | ||
| secret_managers | ||
| types | ||
| vector_stores | ||
| __init__.py | ||
| conftest.py | ||
| log.txt | ||
| readme.md | ||
| test_constants.py | ||
| test_cost_calculation_log_level.py | ||
| test_cost_calculator.py | ||
| test_logging.py | ||
| test_main.py | ||
| test_router.py | ||
| test_utils.py | ||
Testing for litellm/
This directory 1:1 maps the the litellm/ directory, and can only contain mocked tests.
The point of this is to:
- Increase test coverage of
litellm/ - Make it easy for contributors to add tests for the
litellm/package and easily run tests without needing LLM API keys.
File name conventions
litellm/proxy/test_caching_routes.pymaps tolitellm/proxy/caching_routes.pytest_<filename>.pymaps tolitellm/<filename>.py