openstack/neutron
Code Issues Proposed changes

Change reference to OvnDbNotifyHandler._watched_events

Browse Source 
Since [1], RowEventHandler._watched_events do not use name mangling.
OvnDbNotifyHandler, inheriting from RowEventHandler, can access to
this variable.

[1]https://review.opendev.org/#/c/752797/

Change-Id: I545c31977799c4ea51e037b631a49306da9c4a3b
Partial-Bug: #1897928
This commit is contained in:
Rodolfo Alonso Hernandez 2020-09-30 15:59:22 +00:00
parent 24590a334f
commit ac08e89124
1 changed files with 6 additions and 1 deletions

7
neutron/tests/unit/plugins/ml2/drivers/ovn/mech_driver/ovsdb/test_ovsdb_monitor.py

@ -99,7 +99,12 @@ class TestOvnDbNotifyHandler(base.BaseTestCase):
def setUp(self): def setUp(self):
super(TestOvnDbNotifyHandler, self).setUp() super(TestOvnDbNotifyHandler, self).setUp()
self.handler = ovsdb_monitor.OvnDbNotifyHandler(mock.ANY) self.handler = ovsdb_monitor.OvnDbNotifyHandler(mock.ANY)
self.watched_events = self.handler._RowEventHandler__watched_events # NOTE(ralonsoh): once the ovsdbapp library version is bumped beyond
# 1.5.0, the first assignation (using name mangling) can be deleted.
try:
self.watched_events = self.handler._RowEventHandler__watched_events
except AttributeError:
self.watched_events = self.handler._watched_events
def test_watch_and_unwatch_events(self): def test_watch_and_unwatch_events(self):
expected_events = set() expected_events = set()