tiennm99/litellm
1
0
Fork 0
mirror of https://github.com/tiennm99/litellm.git synced 2026-08-05 20:23:18 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #21439 from BerriAI/fix/restore-default-internal-user-params

Browse Source 
fix(test): restore default_internal_user_params to None instead of delattr
This commit is contained in:
jquinter
2026-02-19 02:19:10 -03:00
committed by GitHub
parent ed79d435a4 063238a789
commit 2c1d2fa948
1 changed files with 4 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/test_litellm/proxy/management_endpoints/test_ui_sso.py
+4 -5
View File
@@ -3750,11 +3750,10 @@ async def test_sso_role_preserved_without_role_mappings():
), "user_defined_values should retain the SSO role after insert_sso_user"
finally:
if original_default_params is not None:
litellm.default_internal_user_params = original_default_params
else:
if hasattr(litellm, "default_internal_user_params"):
delattr(litellm, "default_internal_user_params")
# Restore original default_internal_user_params (always assign, never delattr —
# deleting the attribute causes AttributeError in subsequent tests because
# litellm.__getattr__ has no handler for this name)
litellm.default_internal_user_params = original_default_params
class TestSSOReadinessEndpoint: