|
| 1 | +"""Tests for LiteLLM provider.""" |
| 2 | + |
| 3 | +from unittest.mock import AsyncMock, MagicMock, patch |
| 4 | + |
| 5 | +import pytest |
| 6 | + |
| 7 | +from ifixai.core.types import ChatMessage, ProviderConfig |
| 8 | +from ifixai.providers.base import ( |
| 9 | + ChatProvider, |
| 10 | + ProviderResponseError, |
| 11 | + ProviderTimeoutError, |
| 12 | +) |
| 13 | +from ifixai.providers.litellm import LiteLLMProvider |
| 14 | + |
| 15 | + |
| 16 | +def _make_config(**overrides): |
| 17 | + defaults = { |
| 18 | + "provider": "litellm", |
| 19 | + "model": "openai/gpt-4o", |
| 20 | + "api_key": "sk-test", |
| 21 | + } |
| 22 | + defaults.update(overrides) |
| 23 | + return ProviderConfig(**defaults) |
| 24 | + |
| 25 | + |
| 26 | +class TestLiteLLMProviderInit: |
| 27 | + def test_extends_chat_provider(self): |
| 28 | + assert issubclass(LiteLLMProvider, ChatProvider) |
| 29 | + |
| 30 | + def test_instantiates_without_args(self): |
| 31 | + p = LiteLLMProvider() |
| 32 | + assert p is not None |
| 33 | + |
| 34 | + |
| 35 | +class TestSendMessage: |
| 36 | + @pytest.mark.asyncio |
| 37 | + @patch("ifixai.providers.litellm._litellm") |
| 38 | + async def test_calls_acompletion_with_drop_params(self, mock_litellm): |
| 39 | + mock_msg = MagicMock(content="test response") |
| 40 | + mock_litellm.acompletion = AsyncMock( |
| 41 | + return_value=MagicMock(choices=[MagicMock(message=mock_msg)]) |
| 42 | + ) |
| 43 | + |
| 44 | + p = LiteLLMProvider() |
| 45 | + config = _make_config() |
| 46 | + messages = [ChatMessage(role="user", content="hello")] |
| 47 | + result = await p.send_message(messages, config) |
| 48 | + |
| 49 | + assert result == "test response" |
| 50 | + kwargs = mock_litellm.acompletion.call_args.kwargs |
| 51 | + assert kwargs["drop_params"] is True |
| 52 | + assert kwargs["model"] == "openai/gpt-4o" |
| 53 | + assert kwargs["api_key"] == "sk-test" |
| 54 | + |
| 55 | + @pytest.mark.asyncio |
| 56 | + @patch("ifixai.providers.litellm._litellm") |
| 57 | + async def test_omits_api_key_when_empty(self, mock_litellm): |
| 58 | + mock_msg = MagicMock(content="ok") |
| 59 | + mock_litellm.acompletion = AsyncMock( |
| 60 | + return_value=MagicMock(choices=[MagicMock(message=mock_msg)]) |
| 61 | + ) |
| 62 | + |
| 63 | + p = LiteLLMProvider() |
| 64 | + config = _make_config(api_key="") |
| 65 | + messages = [ChatMessage(role="user", content="hi")] |
| 66 | + await p.send_message(messages, config) |
| 67 | + |
| 68 | + kwargs = mock_litellm.acompletion.call_args.kwargs |
| 69 | + assert "api_key" not in kwargs |
| 70 | + |
| 71 | + @pytest.mark.asyncio |
| 72 | + @patch("ifixai.providers.litellm._litellm") |
| 73 | + async def test_formats_messages_correctly(self, mock_litellm): |
| 74 | + mock_msg = MagicMock(content="ok") |
| 75 | + mock_litellm.acompletion = AsyncMock( |
| 76 | + return_value=MagicMock(choices=[MagicMock(message=mock_msg)]) |
| 77 | + ) |
| 78 | + |
| 79 | + p = LiteLLMProvider() |
| 80 | + config = _make_config() |
| 81 | + messages = [ |
| 82 | + ChatMessage(role="system", content="be helpful"), |
| 83 | + ChatMessage(role="user", content="hi"), |
| 84 | + ] |
| 85 | + await p.send_message(messages, config) |
| 86 | + |
| 87 | + kwargs = mock_litellm.acompletion.call_args.kwargs |
| 88 | + assert kwargs["messages"] == [ |
| 89 | + {"role": "system", "content": "be helpful"}, |
| 90 | + {"role": "user", "content": "hi"}, |
| 91 | + ] |
| 92 | + |
| 93 | + @pytest.mark.asyncio |
| 94 | + @patch("ifixai.providers.litellm._litellm") |
| 95 | + async def test_uses_default_model_when_none(self, mock_litellm): |
| 96 | + mock_msg = MagicMock(content="ok") |
| 97 | + mock_litellm.acompletion = AsyncMock( |
| 98 | + return_value=MagicMock(choices=[MagicMock(message=mock_msg)]) |
| 99 | + ) |
| 100 | + |
| 101 | + p = LiteLLMProvider() |
| 102 | + config = _make_config(model=None) |
| 103 | + messages = [ChatMessage(role="user", content="hi")] |
| 104 | + await p.send_message(messages, config) |
| 105 | + |
| 106 | + kwargs = mock_litellm.acompletion.call_args.kwargs |
| 107 | + assert kwargs["model"] == "openai/gpt-4o" |
| 108 | + |
| 109 | + |
| 110 | +class TestEdgeCases: |
| 111 | + @pytest.mark.asyncio |
| 112 | + @patch("ifixai.providers.litellm._litellm") |
| 113 | + async def test_empty_content_raises_response_error(self, mock_litellm): |
| 114 | + mock_msg = MagicMock(content="") |
| 115 | + mock_litellm.acompletion = AsyncMock( |
| 116 | + return_value=MagicMock(choices=[MagicMock(message=mock_msg)]) |
| 117 | + ) |
| 118 | + |
| 119 | + p = LiteLLMProvider() |
| 120 | + config = _make_config() |
| 121 | + messages = [ChatMessage(role="user", content="hi")] |
| 122 | + |
| 123 | + with pytest.raises(ProviderResponseError): |
| 124 | + await p.send_message(messages, config) |
| 125 | + |
| 126 | + @pytest.mark.asyncio |
| 127 | + @patch("ifixai.providers.litellm._litellm") |
| 128 | + async def test_api_error_raises_response_error(self, mock_litellm): |
| 129 | + mock_litellm.acompletion = AsyncMock(side_effect=Exception("401 Unauthorized")) |
| 130 | + |
| 131 | + p = LiteLLMProvider() |
| 132 | + config = _make_config() |
| 133 | + messages = [ChatMessage(role="user", content="hi")] |
| 134 | + |
| 135 | + with pytest.raises(ProviderResponseError, match="401"): |
| 136 | + await p.send_message(messages, config) |
| 137 | + |
| 138 | + @pytest.mark.asyncio |
| 139 | + @patch("ifixai.providers.litellm._litellm") |
| 140 | + async def test_timeout_raises_timeout_error(self, mock_litellm): |
| 141 | + mock_litellm.acompletion = AsyncMock(side_effect=TimeoutError("timed out")) |
| 142 | + |
| 143 | + p = LiteLLMProvider() |
| 144 | + config = _make_config() |
| 145 | + messages = [ChatMessage(role="user", content="hi")] |
| 146 | + |
| 147 | + with pytest.raises(ProviderTimeoutError): |
| 148 | + await p.send_message(messages, config) |
| 149 | + |
| 150 | + |
| 151 | +class TestResolver: |
| 152 | + def test_litellm_in_registered_providers(self): |
| 153 | + from ifixai.providers.resolver import REGISTERED_PROVIDERS |
| 154 | + |
| 155 | + assert "litellm" in REGISTERED_PROVIDERS |
| 156 | + |
| 157 | + def test_litellm_in_provider_map(self): |
| 158 | + from ifixai.providers.resolver import _PROVIDER_MAP |
| 159 | + |
| 160 | + assert "litellm" in _PROVIDER_MAP |
| 161 | + |
| 162 | + def test_resolve_litellm_returns_provider(self): |
| 163 | + from ifixai.providers.resolver import resolve_provider |
| 164 | + |
| 165 | + provider = resolve_provider("litellm") |
| 166 | + assert isinstance(provider, LiteLLMProvider) |
| 167 | + |
| 168 | + def test_litellm_in_credential_env_vars(self): |
| 169 | + from ifixai.providers.resolver import _PROVIDER_CREDENTIAL_ENV_VARS |
| 170 | + |
| 171 | + assert "litellm" in _PROVIDER_CREDENTIAL_ENV_VARS |
0 commit comments