scriptoriumadmin/litellm
1
0
Fork 0
mirror of https://github.com/BerriAI/litellm.git synced 2026-10-09 03:18:44 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 108

fix(mavvrik_focus): pass required args to _init_custom_logger_compatible_class

Browse source 
Calling it with only logging_integration raised TypeError at proxy startup
because internal_usage_cache and llm_router have no defaults. Also fix test
name to reflect the actual status code (5xx not 4xx) used in the mock.
This commit is contained in:
Praveen Ghuge 2026-06-24 00:03:28 +05:30
parent 16634bb6b8
commit 3a9cf3ad9b
2 changed files with 5 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
litellm/integrations/mavvrik_focus/mavvrik_focus_logger.py
View file

@ -276,7 +276,9 @@ class MavvrikFocusLogger(FocusLogger):
)
instance = _init_custom_logger_compatible_class(
logging_integration="mavvrik"
logging_integration="mavvrik",
internal_usage_cache=None,
llm_router=None,
)
if isinstance(instance, MavvrikFocusLogger):
loggers = [instance]

4
tests/test_litellm/integrations/focus/test_mavvrik_destination.py
View file

@ -774,8 +774,8 @@ async def test_gcs_session_cancelled_on_chunk_failure():
@pytest.mark.asyncio
async def test_update_metrics_marker_warns_on_4xx():
"""_update_metrics_marker must log a warning on 4xx but not raise."""
async def test_update_metrics_marker_warns_on_non_410_error():
"""_update_metrics_marker must log a warning on any >=400 (non-410) status but not raise."""
dest = _dest()
fail_resp = MagicMock()