From 13f536c599de4ef04aac7f1869031e97caafcfcb Mon Sep 17 00:00:00 2001 From: user <70670632+stuxf@users.noreply.github.com> Date: Thu, 9 Apr 2026 18:12:22 +0000 Subject: [PATCH] fix(test): remove duplicate verbose_logger mock in test_check_migration MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The logger was patched twice — first via mocker.patch() then via mocker.patch.object(autospec=True). The second call fails because autospec cannot inspect an already-mocked attribute. Remove the redundant first patch. Co-Authored-By: Claude Opus 4.6 (1M context) --- tests/test_litellm/proxy/db/test_check_migration.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/tests/test_litellm/proxy/db/test_check_migration.py b/tests/test_litellm/proxy/db/test_check_migration.py index b1936748201..5b182f03c4b 100644 --- a/tests/test_litellm/proxy/db/test_check_migration.py +++ b/tests/test_litellm/proxy/db/test_check_migration.py @@ -20,10 +20,8 @@ def test_check_migration_out_of_sync(mocker): # assignment. from litellm.proxy.db import check_migration - mock_logger = mocker.patch("litellm.proxy.db.check_migration.verbose_logger") - # Mock the helper function to simulate out-of-sync state - mocker.patch.object( + mock_logger = mocker.patch.object( check_migration, "verbose_logger", autospec=True,