- [N326] Python 3: do not use basestring.
- [N327] Python 3: do not use dict.iteritems.
- [N328] Detect wrong usage with assertEqual
+- [N329] Method's default argument shouldn't be mutable
Creating Unit Tests
-------------------
return False
def init_l3(self, device_name, ip_cidrs, namespace=None,
- preserve_ips=[],
- clean_connections=False):
+ preserve_ips=None, clean_connections=False):
"""Set the L3 settings for the interface using data from the port.
ip_cidrs: list of 'X.X.X.X/YY' strings
clean_connections: Boolean to indicate if we should cleanup connections
associated to removed ips
"""
+ preserve_ips = preserve_ips or []
device = ip_lib.IPDevice(device_name, namespace=namespace)
# The LLA generated by the operating system is not known to
self.devices_with_udpated_sg_members[sg_id].append(device)
def security_group_updated(self, action_type, sec_group_ids,
- device_ids=[]):
+ device_ids=None):
+ device_ids = device_ids or []
if action_type == 'sg_rule':
self.updated_rule_sg_ids.update(sec_group_ids)
elif action_type == 'sg_member':
LOG = logging.getLogger(__name__)
-def get_filters(request, attr_info, skips=[]):
+def get_filters(request, attr_info, skips=None):
"""Extracts the filters from the request string.
Returns a dict of lists for the filters:
becomes:
{'check': [u'a', u'b'], 'name': [u'Bob']}
"""
+ skips = skips or []
res = {}
for key, values in six.iteritems(request.GET.dict_of_lists()):
if key in skips:
"""Add top level resources to the OpenStack API in Neutron."""
def __init__(self, collection, controller, parent=None, path_prefix="",
- collection_actions={}, member_actions={}, attr_map={}):
+ collection_actions=None, member_actions=None, attr_map=None):
+ collection_actions = collection_actions or {}
+ member_actions = member_actions or {}
+ attr_map = attr_map or {}
self.collection = collection
self.controller = controller
self.parent = parent
'exception': '_LE',
}
_all_hints = set(_all_log_levels.values())
+mutable_default_args = re.compile(r"^\s*def .+\((.+=\{\}|.+=\[\])")
def _regex_for_level(level, hint):
yield (0, msg)
+def no_mutable_default_args(logical_line):
+ msg = "N329: Method's default argument shouldn't be mutable!"
+ if mutable_default_args.match(logical_line):
+ yield (0, msg)
+
+
def factory(register):
register(validate_log_translations)
register(use_jsonutils)
register(check_no_basestring)
register(check_python3_no_iteritems)
register(check_asserttrue)
+ register(no_mutable_default_args)
mgr.initialize(connection, 'sriov')
return mgr
- def setup_eswitch_mgr(self, device_mappings, exclude_devices={}):
+ def setup_eswitch_mgr(self, device_mappings, exclude_devices=None):
+ exclude_devices = exclude_devices or {}
self.eswitch_mgr = esm.ESwitchManager()
self.eswitch_mgr.discover_devices(device_mappings, exclude_devices)
'aa:bb:cc:dd:ee:ff',
internal=True)
- def _test_plug(self, additional_expectation=[], bridge=None,
+ def _test_plug(self, additional_expectation=None, bridge=None,
namespace=None):
-
+ additional_expectation = additional_expectation or []
if not bridge:
bridge = 'br-int'
class ExtensionsTestApp(base_wsgi.Router):
- def __init__(self, options={}):
+ def __init__(self, options=None):
+ options = options or {}
mapper = routes.Mapper()
controller = ext_stubs.StubBaseAppController()
mapper.resource("dummy_resource", "/dummy_resources",
fields.extend(policy_attrs)
return fields
- def _get_collection_kwargs(self, skipargs=[], **kwargs):
+ def _get_collection_kwargs(self, skipargs=None, **kwargs):
+ skipargs = skipargs or []
args_list = ['filters', 'fields', 'sorts', 'limit', 'marker',
'page_reverse']
args_dict = dict(
self, expected_status=webob.exc.HTTPOk.code,
expected_error='StateInvalid', subnet=None,
device_owner=DEVICE_OWNER_COMPUTE, updated_fixed_ips=None,
- host_arg={}, arg_list=[]):
+ host_arg=None, arg_list=None):
+ host_arg = host_arg or {}
+ arg_list = arg_list or []
with self.port(device_owner=device_owner, subnet=subnet,
arg_list=arg_list, **host_arg) as port:
self.assertIn('mac_address', port['port'])
class StubPlugin(object):
- def __init__(self, supported_extensions=[]):
+ def __init__(self, supported_extensions=None):
+ supported_extensions = supported_extensions or []
self.supported_extension_aliases = supported_extensions
self._verify_dns_assigment(res['port'],
ips_list=['10.0.0.2'])
- def _verify_dns_assigment(self, port, ips_list=[], exp_ips_ipv4=0,
- exp_ips_ipv6=0, ipv4_cidrs=[], ipv6_cidrs=[],
+ def _verify_dns_assigment(self, port, ips_list=None, exp_ips_ipv4=0,
+ exp_ips_ipv6=0, ipv4_cidrs=None, ipv6_cidrs=None,
dns_name=''):
+ ips_list = ips_list or []
+ ipv4_cidrs = ipv4_cidrs or []
+ ipv6_cidrs = ipv6_cidrs or []
self.assertEqual(port['dns_name'], dns_name)
dns_assignment = port['dns_assignment']
if ips_list:
def _add_external_gateway_to_router(self, router_id, network_id,
expected_code=exc.HTTPOk.code,
- neutron_context=None, ext_ips=[]):
+ neutron_context=None, ext_ips=None):
+ ext_ips = ext_ips or []
body = {'router':
{'external_gateway_info': {'network_id': network_id}}}
if ext_ips:
network)
def get_ports(self, context, filters=None, fields=None,
- sorts=[], limit=None, marker=None,
+ sorts=None, limit=None, marker=None,
page_reverse=False):
+ sorts = sorts or []
neutron_lports = super(SecurityGroupTestPlugin, self).get_ports(
context, filters, sorts=sorts, limit=limit, marker=marker,
page_reverse=page_reverse)
self.assertEqual(
0, len(list(checks.check_asserttrue(pass_code,
"neutron/tests/test_assert.py"))))
+
+ def test_no_mutable_default_args(self):
+ self.assertEqual(1, len(list(checks.no_mutable_default_args(
+ " def fake_suds_context(calls={}):"))))
+
+ self.assertEqual(1, len(list(checks.no_mutable_default_args(
+ "def get_info_from_bdm(virt_type, bdm, mapping=[])"))))
+
+ self.assertEqual(0, len(list(checks.no_mutable_default_args(
+ "defined = []"))))
+
+ self.assertEqual(0, len(list(checks.no_mutable_default_args(
+ "defined, undefined = [], {}"))))
return FakeContext(subnet)
def _get_port_context(self, tenant_id, net_id, port_id,
- fixed_ips=[]):
+ fixed_ips=None):
+ fixed_ips = fixed_ips or []
# sample data for testing purpose only
port = {'device_id': '1234',
'name': 'FakePort',
Code Review / openstack-build / neutron-build.git / commitdiff