torbjorn/my-pal-mcp-server
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor(tests): remove unused setUp method

Browse Source 
The setUp method created provider instances that were never used.
Each test creates its own instance inside the patch context manager,
which is the correct pattern for property mocking.
This commit is contained in:
Robert Hyman
2025-11-29 01:42:27 -05:00
parent 0c3e63c0c7
commit b6a8d682d9
1 changed files with 0 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
tests/test_openrouter_store_parameter.py
View File

@@ -63,11 +63,6 @@ class TestStoreParameterHandling(unittest.TestCase):
**Feature: openrouter-store-parameter-fix, Property 2: Direct OpenAI requests include store parameter** **Feature: openrouter-store-parameter-fix, Property 2: Direct OpenAI requests include store parameter**
""" """
def setUp(self):
"""Set up test fixtures."""
self.openrouter_provider = MockOpenRouterProvider("test-key")
self.openai_provider = MockOpenAIProvider("test-key")
def test_openrouter_responses_omits_store_parameter(self): def test_openrouter_responses_omits_store_parameter(self):
"""Test that OpenRouter provider omits store parameter from responses endpoint. """Test that OpenRouter provider omits store parameter from responses endpoint.