From: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Date: Tue, 17 Feb 2026 15:08:05 +0000 (+0000) Subject: Remove trailing whitespace from test file X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=40b0d456116ba6cb66ab4e9bc3dca957c390e286;p=ceph.git Remove trailing whitespace from test file 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 9f668256612..59835776a84 100644 --- a/src/pybind/mgr/prometheus/test_module.py +++ b/src/pybind/mgr/prometheus/test_module.py @@ -47,7 +47,7 @@ class MetricGroupTest(TestCase): self.assertEqual( grouped_metric.str_expfmt(), """ -# HELP ceph_disk_occupation_display +# HELP ceph_disk_occupation_display # TYPE ceph_disk_occupation_display untyped ceph_disk_occupation_display{ceph_daemon="osd.0",device="/dev/dm-0",instance="node1"} 1.0 ceph_disk_occupation_display{ceph_daemon="osd.1",device="/dev/dm-0",instance="node3"} 1.0 @@ -99,58 +99,58 @@ class ConfigureTest(TestCase): """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) module.log = Mock() - + # Mock mon_command to return None module.mon_command = Mock(return_value=(0, None, "")) - + # 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 module.log.exception.assert_not_called() - + def test_configure_with_valid_security_config_disabled(self): """Test that configure handles valid JSON with security disabled""" module = Mock(spec=Module) module.log = Mock() - + # Mock mon_command to return valid JSON with security_enabled: false module.mon_command = Mock(return_value=(0, '{"security_enabled": false}', "")) - + # 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 (security disabled) module.setup_default_config.assert_called_once_with("127.0.0.1", 9283) - + # Verify no exception was raised module.log.exception.assert_not_called()