Skip to content

Commit cf9a07f

Browse files
committed
fix: mock OpenAI provider in factory tests
- Add @patch decorator to test_openai_url_style - Add @patch decorator to test_default_to_openai - Pass api_key='test-key' to avoid OpenAI API key requirement - Fixes CI test failures on all platforms
1 parent ceacc13 commit cf9a07f

1 file changed

Lines changed: 6 additions & 4 deletions

File tree

tests/test_providers.py

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -59,9 +59,10 @@ def test_return_existing_provider(self):
5959
result = create_provider(provider)
6060
assert result is provider
6161

62-
def test_openai_url_style(self):
62+
@patch('openai.OpenAI')
63+
def test_openai_url_style(self, mock_openai):
6364
"""Test creating OpenAI provider with URL style"""
64-
provider = create_provider("openai://gpt-4o-mini")
65+
provider = create_provider("openai://gpt-4o-mini", api_key="test-key")
6566
assert isinstance(provider, OpenAIProvider)
6667
assert provider.model == "gpt-4o-mini"
6768

@@ -81,9 +82,10 @@ def test_mistral_auto_detection(self):
8182
provider = create_provider("mistral")
8283
assert isinstance(provider, OllamaProvider)
8384

84-
def test_default_to_openai(self):
85+
@patch('openai.OpenAI')
86+
def test_default_to_openai(self, mock_openai):
8587
"""Test unknown models default to OpenAI"""
86-
provider = create_provider("gpt-4o-mini")
88+
provider = create_provider("gpt-4o-mini", api_key="test-key")
8789
assert isinstance(provider, OpenAIProvider)
8890

8991
def test_api_key_passed_through(self):

0 commit comments

Comments
 (0)