From: YAMAMOTO Takashi Date: Fri, 16 Jan 2015 00:31:17 +0000 (+0900) Subject: Fix some assertEqual argument orders X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=f591042d2cfc2a546cad2bbcb7553a29753b1f23;p=openstack-build%2Fneutron-build.git Fix some assertEqual argument orders Perform assertEqual(expected, observed) police in neutron.tests.unit.test_attributes. This is a preparation to introduce more assertions there in a later patch. Change-Id: I87d7e356798b108952535b351e30c2f2cf01f98d --- diff --git a/neutron/tests/unit/test_attributes.py b/neutron/tests/unit/test_attributes.py index 163bbf544..64d7a9b8b 100644 --- a/neutron/tests/unit/test_attributes.py +++ b/neutron/tests/unit/test_attributes.py @@ -59,18 +59,18 @@ class TestAttributes(base.BaseTestCase): self.assertIsNone(msg) msg = attributes._validate_values(7, [4, 6]) - self.assertEqual(msg, "'7' is not in [4, 6]") + self.assertEqual("'7' is not in [4, 6]", msg) msg = attributes._validate_values(7, (4, 6)) - self.assertEqual(msg, "'7' is not in (4, 6)") + self.assertEqual("'7' is not in (4, 6)", msg) def test_validate_not_empty_string(self): msg = attributes._validate_not_empty_string(' ', None) - self.assertEqual(msg, u"' ' Blank strings are not permitted") + self.assertEqual(u"' ' Blank strings are not permitted", msg) def test_validate_not_empty_string_or_none(self): msg = attributes._validate_not_empty_string_or_none(' ', None) - self.assertEqual(msg, u"' ' Blank strings are not permitted") + self.assertEqual(u"' ' Blank strings are not permitted", msg) msg = attributes._validate_not_empty_string_or_none(None, None) self.assertIsNone(msg) @@ -84,7 +84,7 @@ class TestAttributes(base.BaseTestCase): def test_validate_string(self): msg = attributes._validate_string(None, None) - self.assertEqual(msg, "'None' is not a valid string") + self.assertEqual("'None' is not a valid string", msg) # 0 == len(data) == max_len msg = attributes._validate_string("", 0) @@ -104,7 +104,7 @@ class TestAttributes(base.BaseTestCase): # 0 < max_len < len(data) msg = attributes._validate_string("1234567890", 9) - self.assertEqual(msg, "'1234567890' exceeds maximum length of 9") + self.assertEqual("'1234567890' exceeds maximum length of 9", msg) msg = attributes._validate_string("123456789", None) self.assertIsNone(msg) @@ -112,7 +112,7 @@ class TestAttributes(base.BaseTestCase): def test_validate_no_whitespace(self): data = 'no_white_space' result = attributes._validate_no_whitespace(data) - self.assertEqual(result, data) + self.assertEqual(data, result) self.assertRaises(n_exc.InvalidInput, attributes._validate_no_whitespace, @@ -150,14 +150,13 @@ class TestAttributes(base.BaseTestCase): self.assertIsNone(msg) msg = attributes._validate_range(0, [1, 9]) - self.assertEqual(msg, "'0' is too small - must be at least '1'") + self.assertEqual("'0' is too small - must be at least '1'", msg) msg = attributes._validate_range(10, (1, 9)) - self.assertEqual(msg, - "'10' is too large - must be no larger than '9'") + self.assertEqual("'10' is too large - must be no larger than '9'", msg) msg = attributes._validate_range("bogus", (1, 9)) - self.assertEqual(msg, "'bogus' is not an integer") + self.assertEqual("'bogus' is not an integer", msg) msg = attributes._validate_range(10, (attributes.UNLIMITED, attributes.UNLIMITED)) @@ -170,11 +169,10 @@ class TestAttributes(base.BaseTestCase): self.assertIsNone(msg) msg = attributes._validate_range(-1, (0, attributes.UNLIMITED)) - self.assertEqual(msg, "'-1' is too small - must be at least '0'") + self.assertEqual("'-1' is too small - must be at least '0'", msg) msg = attributes._validate_range(10, (attributes.UNLIMITED, 9)) - self.assertEqual(msg, - "'10' is too large - must be no larger than '9'") + self.assertEqual("'10' is too large - must be no larger than '9'", msg) def _test_validate_mac_address(self, validator, allow_none=False): mac_addr = "ff:16:3e:4f:00:00" @@ -184,22 +182,22 @@ class TestAttributes(base.BaseTestCase): mac_addr = "ffa:16:3e:4f:00:00" msg = validator(mac_addr) err_msg = "'%s' is not a valid MAC address" - self.assertEqual(msg, err_msg % mac_addr) + self.assertEqual(err_msg % mac_addr, msg) mac_addr = "123" msg = validator(mac_addr) - self.assertEqual(msg, err_msg % mac_addr) + self.assertEqual(err_msg % mac_addr, msg) mac_addr = None msg = validator(mac_addr) if allow_none: self.assertIsNone(msg) else: - self.assertEqual(msg, err_msg % mac_addr) + self.assertEqual(err_msg % mac_addr, msg) mac_addr = "ff:16:3e:4f:00:00\r" msg = validator(mac_addr) - self.assertEqual(msg, err_msg % mac_addr) + self.assertEqual(err_msg % mac_addr, msg) def test_validate_mac_address(self): self._test_validate_mac_address(attributes._validate_mac_address) @@ -215,29 +213,29 @@ class TestAttributes(base.BaseTestCase): ip_addr = '1111.1.1.1' msg = attributes._validate_ip_address(ip_addr) - self.assertEqual(msg, "'%s' is not a valid IP address" % ip_addr) + self.assertEqual("'%s' is not a valid IP address" % ip_addr, msg) ip_addr = '1.1.1.1 has whitespace' msg = attributes._validate_ip_address(ip_addr) - self.assertEqual(msg, "'%s' is not a valid IP address" % ip_addr) + self.assertEqual("'%s' is not a valid IP address" % ip_addr, msg) ip_addr = '111.1.1.1\twhitespace' msg = attributes._validate_ip_address(ip_addr) - self.assertEqual(msg, "'%s' is not a valid IP address" % ip_addr) + self.assertEqual("'%s' is not a valid IP address" % ip_addr, msg) ip_addr = '111.1.1.1\nwhitespace' msg = attributes._validate_ip_address(ip_addr) - self.assertEqual(msg, "'%s' is not a valid IP address" % ip_addr) + self.assertEqual("'%s' is not a valid IP address" % ip_addr, msg) for ws in string.whitespace: ip_addr = '%s111.1.1.1' % ws msg = attributes._validate_ip_address(ip_addr) - self.assertEqual(msg, "'%s' is not a valid IP address" % ip_addr) + self.assertEqual("'%s' is not a valid IP address" % ip_addr, msg) for ws in string.whitespace: ip_addr = '111.1.1.1%s' % ws msg = attributes._validate_ip_address(ip_addr) - self.assertEqual(msg, "'%s' is not a valid IP address" % ip_addr) + self.assertEqual("'%s' is not a valid IP address" % ip_addr, msg) def test_validate_ip_pools(self): pools = [[{'end': '10.0.0.254'}], @@ -265,8 +263,8 @@ class TestAttributes(base.BaseTestCase): pools = [[{'end': '10.0.0.254', 'start': invalid_ip}]] for pool in pools: msg = attributes._validate_ip_pools(pool) - self.assertEqual(msg, - "'%s' is not a valid IP address" % invalid_ip) + self.assertEqual("'%s' is not a valid IP address" % invalid_ip, + msg) def test_validate_fixed_ips(self): fixed_ips = [ @@ -294,7 +292,7 @@ class TestAttributes(base.BaseTestCase): 'error_msg': "Duplicate IP address '1.1.1.1'"}] for fixed in fixed_ips: msg = attributes._validate_fixed_ips(fixed['data']) - self.assertEqual(msg, fixed['error_msg']) + self.assertEqual(fixed['error_msg'], msg) fixed_ips = [[{'subnet_id': '00000000-ffff-ffff-ffff-000000000000', 'ip_address': '1.1.1.1'}], @@ -369,7 +367,7 @@ class TestAttributes(base.BaseTestCase): ip_addr = '1111.1.1.1' msg = attributes._validate_ip_address_or_none(ip_addr) - self.assertEqual(msg, "'%s' is not a valid IP address" % ip_addr) + self.assertEqual("'%s' is not a valid IP address" % ip_addr, msg) def test_hostname_pattern(self): bad_values = ['@openstack', 'ffff.abcdefg' * 26, 'f' * 80, '-hello', @@ -497,7 +495,7 @@ class TestAttributes(base.BaseTestCase): error = _("'%(data)s' isn't a recognized IP subnet cidr," " '%(cidr)s' is recommended") % {"data": cidr, "cidr": "10.0.2.0/32"} - self.assertEqual(msg, error) + self.assertEqual(error, msg) # Valid - IPv4 with non-zero masked bits is ok for i in range(1, 255): @@ -511,7 +509,7 @@ class TestAttributes(base.BaseTestCase): error = _("'%(data)s' isn't a recognized IP subnet cidr," " '%(cidr)s' is recommended") % {"data": cidr, "cidr": "fe80::/128"} - self.assertEqual(msg, error) + self.assertEqual(error, msg) # Invalid - IPv6 with final octets, missing mask cidr = "fe80::0" @@ -519,13 +517,13 @@ class TestAttributes(base.BaseTestCase): error = _("'%(data)s' isn't a recognized IP subnet cidr," " '%(cidr)s' is recommended") % {"data": cidr, "cidr": "fe80::/128"} - self.assertEqual(msg, error) + self.assertEqual(error, msg) # Invalid - Address format error cidr = 'invalid' msg = validator(cidr, None) error = "'%s' is not a valid IP subnet" % cidr - self.assertEqual(msg, error) + self.assertEqual(error, msg) cidr = None msg = validator(cidr, None) @@ -533,13 +531,13 @@ class TestAttributes(base.BaseTestCase): self.assertIsNone(msg) else: error = "'%s' is not a valid IP subnet" % cidr - self.assertEqual(msg, error) + self.assertEqual(error, msg) # Invalid - IPv4 with trailing CR cidr = "10.0.2.0/24\r" msg = validator(cidr, None) error = "'%s' is not a valid IP subnet" % cidr - self.assertEqual(msg, error) + self.assertEqual(error, msg) def test_validate_subnet(self): self._test_validate_subnet(attributes._validate_subnet) @@ -556,11 +554,11 @@ class TestAttributes(base.BaseTestCase): if allow_none: self.assertIsNone(msg) else: - self.assertEqual(msg, "'None' is not a valid input") + self.assertEqual("'None' is not a valid input", msg) data = 'bat' msg = validator(data, pattern) - self.assertEqual(msg, "'%s' is not a valid input" % data) + self.assertEqual("'%s' is not a valid input" % data, msg) data = 'hat' msg = validator(data, pattern) @@ -579,7 +577,7 @@ class TestAttributes(base.BaseTestCase): def test_validate_uuid(self): msg = attributes._validate_uuid('garbage') - self.assertEqual(msg, "'garbage' is not a valid UUID") + self.assertEqual("'garbage' is not a valid UUID", msg) msg = attributes._validate_uuid('00000000-ffff-ffff-ffff-000000000000') self.assertIsNone(msg) @@ -594,7 +592,7 @@ class TestAttributes(base.BaseTestCase): for uuid in uuids: msg = attributes._validate_uuid_list(uuid) error = "'%s' is not a list" % uuid - self.assertEqual(msg, error) + self.assertEqual(error, msg) # check invalid uuid in a list invalid_uuid_lists = [[None], @@ -608,7 +606,7 @@ class TestAttributes(base.BaseTestCase): for uuid_list in invalid_uuid_lists: msg = attributes._validate_uuid_list(uuid_list) error = "'%s' is not a valid UUID" % uuid_list[0] - self.assertEqual(msg, error) + self.assertEqual(error, msg) # check duplicate items in a list duplicate_uuids = ['e5069610-744b-42a7-8bd8-ceac1a229cd4', @@ -617,7 +615,7 @@ class TestAttributes(base.BaseTestCase): msg = attributes._validate_uuid_list(duplicate_uuids) error = ("Duplicate items in the list: " "'%s'" % ', '.join(duplicate_uuids)) - self.assertEqual(msg, error) + self.assertEqual(error, msg) # check valid uuid lists valid_uuid_lists = [['e5069610-744b-42a7-8bd8-ceac1a229cd4'], @@ -630,8 +628,8 @@ class TestAttributes(base.BaseTestCase): def test_validate_dict_type(self): for value in (None, True, '1', []): - self.assertEqual(attributes._validate_dict(value), - "'%s' is not a dictionary" % value) + self.assertEqual("'%s' is not a dictionary" % value, + attributes._validate_dict(value)) def test_validate_dict_without_constraints(self): msg = attributes._validate_dict({}) @@ -652,7 +650,7 @@ class TestAttributes(base.BaseTestCase): constraints['key1'] = {'type:unsupported': None, 'required': True} msg = attributes._validate_dict(dictionary, constraints) - self.assertEqual(msg, "Validator 'type:unsupported' does not exist.") + self.assertEqual("Validator 'type:unsupported' does not exist.", msg) def test_validate_dict_not_required_keys(self): dictionary, constraints = self._construct_dict_and_constraints() @@ -723,8 +721,8 @@ class TestAttributes(base.BaseTestCase): def test_validate_non_negative(self): for value in (-1, '-2'): - self.assertEqual(attributes._validate_non_negative(value), - "'%s' should be non-negative" % value) + self.assertEqual("'%s' should be non-negative" % value, + attributes._validate_non_negative(value)) for value in (0, 1, '2', True, False): msg = attributes._validate_non_negative(value) @@ -759,19 +757,19 @@ class TestConvertToBoolean(base.BaseTestCase): class TestConvertToInt(base.BaseTestCase): def test_convert_to_int_int(self): - self.assertEqual(attributes.convert_to_int(-1), -1) - self.assertEqual(attributes.convert_to_int(0), 0) - self.assertEqual(attributes.convert_to_int(1), 1) + self.assertEqual(-1, attributes.convert_to_int(-1)) + self.assertEqual(0, attributes.convert_to_int(0)) + self.assertEqual(1, attributes.convert_to_int(1)) def test_convert_to_int_if_not_none(self): - self.assertEqual(attributes.convert_to_int_if_not_none(-1), -1) - self.assertEqual(attributes.convert_to_int_if_not_none(0), 0) - self.assertEqual(attributes.convert_to_int_if_not_none(1), 1) + self.assertEqual(-1, attributes.convert_to_int_if_not_none(-1)) + self.assertEqual(0, attributes.convert_to_int_if_not_none(0)) + self.assertEqual(1, attributes.convert_to_int_if_not_none(1)) self.assertIsNone(attributes.convert_to_int_if_not_none(None)) def test_convert_to_int_str(self): - self.assertEqual(attributes.convert_to_int('4'), 4) - self.assertEqual(attributes.convert_to_int('6'), 6) + self.assertEqual(4, attributes.convert_to_int('4')) + self.assertEqual(6, attributes.convert_to_int('6')) self.assertRaises(n_exc.InvalidInput, attributes.convert_to_int, 'garbage') @@ -782,12 +780,10 @@ class TestConvertToInt(base.BaseTestCase): None) def test_convert_none_to_empty_list_none(self): - self.assertEqual( - [], attributes.convert_none_to_empty_list(None)) + self.assertEqual([], attributes.convert_none_to_empty_list(None)) def test_convert_none_to_empty_dict(self): - self.assertEqual( - {}, attributes.convert_none_to_empty_dict(None)) + self.assertEqual({}, attributes.convert_none_to_empty_dict(None)) def test_convert_none_to_empty_list_value(self): values = ['1', 3, [], [1], {}, {'a': 3}] @@ -832,16 +828,16 @@ class TestConvertToList(base.BaseTestCase): def test_convert_to_empty_list(self): for item in (None, [], (), {}): - self.assertEqual(attributes.convert_to_list(item), []) + self.assertEqual([], attributes.convert_to_list(item)) def test_convert_to_list_string(self): for item in ('', 'foo'): - self.assertEqual(attributes.convert_to_list(item), [item]) + self.assertEqual([item], attributes.convert_to_list(item)) def test_convert_to_list_iterable(self): for item in ([None], [1, 2, 3], (1, 2, 3), set([1, 2, 3]), ['foo']): - self.assertEqual(attributes.convert_to_list(item), list(item)) + self.assertEqual(list(item), attributes.convert_to_list(item)) def test_convert_to_list_non_iterable(self): for item in (True, False, 1, 1.2, object()): - self.assertEqual(attributes.convert_to_list(item), [item]) + self.assertEqual([item], attributes.convert_to_list(item))