From bcb21726b399d321bb0fc2bc1fe05af2d50f111c Mon Sep 17 00:00:00 2001 From: Jakub Libosvar Date: Tue, 21 Oct 2025 19:01:12 +0000 Subject: [PATCH] Patch Stream methods with mock The original code did not patch the methods but assigned mocks effectively leaving the mock objects existent even after the test has finished. Closes-Bug: #2129284 Change-Id: I61da9b061ed4c0c78c2a84d889ed6f39928329cc Signed-off-by: Jakub Libosvar --- .../ml2/drivers/ovn/mech_driver/ovsdb/test_ovsdb_monitor.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/neutron/tests/unit/plugins/ml2/drivers/ovn/mech_driver/ovsdb/test_ovsdb_monitor.py b/neutron/tests/unit/plugins/ml2/drivers/ovn/mech_driver/ovsdb/test_ovsdb_monitor.py index cd041066e87..76c12f52157 100644 --- a/neutron/tests/unit/plugins/ml2/drivers/ovn/mech_driver/ovsdb/test_ovsdb_monitor.py +++ b/neutron/tests/unit/plugins/ml2/drivers/ovn/mech_driver/ovsdb/test_ovsdb_monitor.py @@ -187,9 +187,9 @@ class TestOvnConnection(base.BaseTestCase): def test_connection_nb_start(self): ovn_conf.cfg.CONF.set_override('ovn_nb_private_key', 'foo-key', 'ovn') - Stream.ssl_set_private_key_file = mock.Mock() - Stream.ssl_set_certificate_file = mock.Mock() - Stream.ssl_set_ca_cert_file = mock.Mock() + mock.patch.object(Stream, 'ssl_set_private_key_file').start() + mock.patch.object(Stream, 'ssl_set_certificate_file').start() + mock.patch.object(Stream, 'ssl_set_ca_cert_file').start() self._test_connection_start(idl_class=ovsdb_monitor.OvnNbIdl, schema='OVN_Northbound')