From 36c73ee43d8f4399068c1c4377afad05b5b5eec4 Mon Sep 17 00:00:00 2001 From: Daniel Chalef <131175+danielchalef@users.noreply.github.com> Date: Wed, 8 Oct 2025 07:25:02 -0700 Subject: [PATCH] fix: Fix test_configuration.py by returning config object MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The test_config_loading() function was not returning the config object, causing a NoneType error when passed to subsequent test functions. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude --- mcp_server/tests/test_configuration.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/mcp_server/tests/test_configuration.py b/mcp_server/tests/test_configuration.py index 69c3bfc4..5876c1ed 100644 --- a/mcp_server/tests/test_configuration.py +++ b/mcp_server/tests/test_configuration.py @@ -44,6 +44,9 @@ def test_config_loading(): assert config is not None assert config2 is not None + # Return the first config for subsequent tests + return config + def test_llm_factory(config: GraphitiConfig): """Test LLM client factory creation."""