From a66b1de4775a5fc2c0035dd2902c0c1c993ebcee Mon Sep 17 00:00:00 2001 From: Mohammed Aqib Date: Fri, 26 Jun 2026 10:18:22 +0530 Subject: [PATCH] refactor: Use pytest caplog instead of logger patching in test_spec.py (#37997) Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com> --- .../controllers/console/test_spec.py | 18 ++++++------------ 1 file changed, 6 insertions(+), 12 deletions(-) diff --git a/api/tests/unit_tests/controllers/console/test_spec.py b/api/tests/unit_tests/controllers/console/test_spec.py index 84c2004ec70..ed02923caf6 100644 --- a/api/tests/unit_tests/controllers/console/test_spec.py +++ b/api/tests/unit_tests/controllers/console/test_spec.py @@ -22,23 +22,17 @@ class TestSpecSchemaDefinitionsApi: assert status == 200 assert resp == schema_definitions - def test_get_exception_returns_empty_list(self): + def test_get_exception_returns_empty_list(self, caplog): api = spec_module.SpecSchemaDefinitionsApi() method = unwrap(api.get) - with ( - patch.object( - spec_module, - "SchemaManager", - side_effect=Exception("boom"), - ), - patch.object( - spec_module.logger, - "exception", - ) as log_exception, + with patch.object( + spec_module, + "SchemaManager", + side_effect=Exception("boom"), ): resp, status = method(api) assert status == 200 assert resp == [] - log_exception.assert_called_once() + assert "boom" in caplog.text