openstack
/
neutron
Code Issues Proposed changes

Merge "Allow __new__ method to accept extra arguments"

This commit is contained in:
Zuul 2017-12-19 00:40:36 +00:00 committed by Gerrit Code Review
parent b6f8a0562f e862d28068
commit bfd55d86fb
2 changed files with 12 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
neutron/db/l3_db.py
View File

@ -89,8 +89,8 @@ class L3_NAT_dbonly_mixin(l3.RouterPluginBase,
_fip_qos = None
def __new__(cls):
inst = super(L3_NAT_dbonly_mixin, cls).__new__(cls)
def __new__(cls, *args, **kwargs):
inst = super(L3_NAT_dbonly_mixin, cls).__new__(cls, *args, **kwargs)
inst._start_janitor()
return inst

10
neutron/tests/unit/db/test_l3_db.py
View File

@ -42,6 +42,16 @@ class TestL3_NAT_dbonly_mixin(base.BaseTestCase):
filtered = l3_db.L3_NAT_dbonly_mixin._each_port_having_fixed_ips(None)
self.assertEqual([], list(filtered))
def test__new__passes_args(self):
class T(l3_db.L3_NAT_db_mixin):
def __init__(self, *args, **kwargs):
self.args = args
self.kwargs = kwargs
t = T(1, 2, a=3)
self.assertEqual((1, 2), t.args)
self.assertEqual({'a': 3}, t.kwargs)
def test__each_port_having_fixed_ips(self):
"""Basic test that ports without fixed ips are filtered out"""
ports = [{'id': 'a', 'fixed_ips': [mock.sentinel.fixedip]},