From b6a8d682d920c2283724b588818bc1162a865d74 Mon Sep 17 00:00:00 2001 From: Robert Hyman <13157542+brt-h@users.noreply.github.com> Date: Sat, 29 Nov 2025 01:42:27 -0500 Subject: [PATCH] refactor(tests): remove unused setUp method 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. --- tests/test_openrouter_store_parameter.py | 5 ----- 1 file changed, 5 deletions(-) diff --git a/tests/test_openrouter_store_parameter.py b/tests/test_openrouter_store_parameter.py index 953f183..88c54e5 100644 --- a/tests/test_openrouter_store_parameter.py +++ b/tests/test_openrouter_store_parameter.py @@ -63,11 +63,6 @@ class TestStoreParameterHandling(unittest.TestCase): **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): """Test that OpenRouter provider omits store parameter from responses endpoint.