fix: patch correct LLMClient.get_model_config path in test

- Fix test_memory_prompt_with_model_name to patch agent_memory_server.api.LLMClient.get_model_config
- Remove commented-out code from tests/conftest.py, tests/test_long_term_memory.py, tests/test_llm_client.py

Author: nkanu17

tests/conftest.py

@@ -33,9 +33,6 @@
 from agent_memory_server.vectorstore_adapter import VectorStoreAdapter
 
 
-# from agent_memory_server.utils.redis import ensure_search_index_exists  # Not used currently
-
-
 load_dotenv()
 
 
@@ -88,9 +85,6 @@ async def search_index(async_redis_client):
             if "unknown index name".lower() not in str(e).lower():
                 pass
 
-        # Skip ensure_search_index_exists for now - let LangChain handle it
-        # await ensure_search_index_exists(async_redis_client)
-
     except Exception:
         raise
 

tests/test_api.py

@@ -1103,7 +1103,7 @@ async def test_memory_prompt_session_not_found(
         assert data["messages"][0]["content"]["text"] == query
 
     @patch("agent_memory_server.api.working_memory.get_working_memory")
-    @patch("agent_memory_server.api.get_model_config")
+    @patch("agent_memory_server.api.LLMClient.get_model_config")
     @pytest.mark.asyncio
     async def test_memory_prompt_with_model_name(
         self, mock_get_model_config, mock_get_working_memory, client

tests/test_llm_client.py

@@ -105,39 +105,6 @@ def test_frozen_dataclass(self):
 
 
 # TODO: Gateway tests disabled pending integration test setup.
-# These tests will be re-enabled when gateway functionality is implemented.
-# See agent_memory_server/llm/client.py for the gateway design.
-#
-# class TestLLMClientGatewayConfiguration:
-#     """Tests for gateway configuration."""
-#
-#     def test_configure_gateway(self):
-#         """configure_gateway should set gateway URL and API key."""
-#         LLMClient.configure_gateway(
-#             base_url="https://gateway.example.com/v1",
-#             api_key="gateway-key",
-#         )
-#         assert LLMClient._gateway_base_url == "https://gateway.example.com/v1"
-#         assert LLMClient._gateway_api_key == "gateway-key"
-#
-#     def test_reset_gateway(self):
-#         """reset_gateway should clear gateway configuration."""
-#         LLMClient.configure_gateway("https://gateway.example.com/v1", "key")
-#         LLMClient.reset_gateway()
-#         assert LLMClient._gateway_base_url is None
-#         assert LLMClient._gateway_api_key is None
-#
-#     def test_reset_clears_all(self):
-#         """reset() should clear backend and gateway."""
-#         mock = MockLLMBackend()
-#         LLMClient.set_backend(mock)
-#         LLMClient.configure_gateway("https://gateway.example.com/v1", "key")
-#         LLMClient.reset()
-#         assert LLMClient._backend is None
-#         assert LLMClient._gateway_base_url is None
-#         assert LLMClient._gateway_api_key is None
-
-
 class TestLLMClientReset:
     """Tests for reset functionality."""
 

tests/test_long_term_memory.py

@@ -26,9 +26,6 @@
 from agent_memory_server.utils.recency import generate_memory_hash
 
 
-# from agent_memory_server.utils.redis import ensure_search_index_exists  # Not used currently
-
-
 class TestLongTermMemory:
     @pytest.mark.asyncio
     async def test_index_memories(self, mock_async_redis_client, session):
@@ -752,8 +749,6 @@ class TestLongTermMemoryIntegration:
     @pytest.mark.asyncio
     async def test_search_messages(self, async_redis_client):
         """Test searching messages"""
-        # await ensure_search_index_exists(async_redis_client)  # Let LangChain handle index
-
         long_term_memories = [
             MemoryRecord(
                 id="memory-1", text="Paris is the capital of France", session_id="123"
@@ -785,8 +780,6 @@ async def test_search_messages(self, async_redis_client):
     @pytest.mark.asyncio
     async def test_search_messages_with_distance_threshold(self, async_redis_client):
         """Test searching messages with a distance threshold"""
-        # await ensure_search_index_exists(async_redis_client)  # Let LangChain handle index
-
         long_term_memories = [
             MemoryRecord(
                 id="memory-1", text="Paris is the capital of France", session_id="123"