From: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Date: Tue, 17 Feb 2026 15:10:39 +0000 (+0000) Subject: Remove redundant test case X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=58a74aa4fa01b82e2ac75209183a5f8e8e89eda3;p=ceph.git Remove redundant test case Co-authored-by: epuertat <37327689+epuertat@users.noreply.github.com> --- diff --git a/src/pybind/mgr/prometheus/test_module.py b/src/pybind/mgr/prometheus/test_module.py index 7f894c9a47d..f56b86a0c04 100644 --- a/src/pybind/mgr/prometheus/test_module.py +++ b/src/pybind/mgr/prometheus/test_module.py @@ -119,26 +119,6 @@ class ConfigureTest(TestCase): # Verify no exception was raised and log was not called with exception module.log.exception.assert_not_called() - def test_configure_with_empty_security_config(self): - """Test that configure handles empty string from orch get-security-config""" - module = Mock(spec=Module) - module.log = Mock() - - # Mock mon_command to return empty string (the problematic case) - module.mon_command = Mock(return_value=(0, "", "")) - - # Mock setup_default_config - module.setup_default_config = Mock() - - # Call the actual configure method with our mocks - Module.configure(module, "127.0.0.1", 9283) - - # Verify that setup_default_config was called (fallback behavior) - module.setup_default_config.assert_called_once_with("127.0.0.1", 9283) - - # Verify no exception was raised and log was not called with exception - module.log.exception.assert_not_called() - def test_configure_with_none_security_config(self): """Test that configure handles None from orch get-security-config""" module = Mock(spec=Module)