From 40019a657a09a7fe3da7f7300c8b094af20fefd2 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Mon, 16 Feb 2026 15:45:28 +0000 Subject: [PATCH] fix: correct patch paths in config loader tests Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com> --- tests/test_config_loader.py | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/tests/test_config_loader.py b/tests/test_config_loader.py index 75230318..b7ce1ebb 100644 --- a/tests/test_config_loader.py +++ b/tests/test_config_loader.py @@ -166,7 +166,7 @@ class TestLoadSettingsFromDb: class TestReloadSettingsFromDb: """Tests for reload_settings_from_db function.""" - @patch("app.utils.config_loader.SessionLocal") + @patch("app.database.SessionLocal") @patch("app.utils.config_loader.load_settings_from_db") def test_reload_success(self, mock_load, mock_session_local): """Test successful settings reload.""" @@ -180,7 +180,7 @@ class TestReloadSettingsFromDb: mock_load.assert_called_once_with(mock_settings, mock_db) mock_db.close.assert_called_once() - @patch("app.utils.config_loader.SessionLocal") + @patch("app.database.SessionLocal") def test_reload_database_error(self, mock_session_local): """Test reload handling database error.""" mock_settings = MagicMock() @@ -190,13 +190,15 @@ class TestReloadSettingsFromDb: assert result is False - @patch("app.utils.config_loader.SessionLocal") - def test_reload_closes_session_on_error(self, mock_session_local): + @patch("app.utils.config_loader.load_settings_from_db") + @patch("app.database.SessionLocal") + def test_reload_closes_session_on_error(self, mock_session_local, mock_load): """Test that session is closed even on error.""" mock_settings = MagicMock() mock_db = MagicMock() - mock_db.query.side_effect = Exception("Query error") mock_session_local.return_value = mock_db + # Make load_settings_from_db raise an exception + mock_load.side_effect = Exception("Load error") result = reload_settings_from_db(mock_settings)