Using assertFalse(A) instead of assertEqual(False, A)

This patch is to replace assertEqual(False, A) with assertFalse(A), which
the latter is more straightforward and easier to understand.
Similar to a merged patch https://review.openstack.org/#/c/473749/

Change-Id: I056c2840ed431e3c792f5f1f70f9e7d9e4e22d0b
This commit is contained in:
Van Hung Pham 2017-06-23 16:38:12 +07:00
parent 40261e5b11
commit 7787ef3b0a
3 changed files with 16 additions and 16 deletions

View File

@ -43,13 +43,13 @@ class TestAttributes(base.BaseTestCase):
def test_is_attr_set(self): def test_is_attr_set(self):
data = attributes.ATTR_NOT_SPECIFIED data = attributes.ATTR_NOT_SPECIFIED
self.assertIs(attributes.is_attr_set(data), False) self.assertFalse(attributes.is_attr_set(data))
data = None data = None
self.assertIs(attributes.is_attr_set(data), False) self.assertFalse(attributes.is_attr_set(data))
data = "I'm set" data = "I'm set"
self.assertIs(attributes.is_attr_set(data), True) self.assertTrue(attributes.is_attr_set(data))
def test_validate_values(self): def test_validate_values(self):
msg = attributes._validate_values(4, [4, 6]) msg = attributes._validate_values(4, [4, 6])
@ -647,7 +647,7 @@ class TestAttributes(base.BaseTestCase):
self.assertIsNone(msg) self.assertIsNone(msg)
# Explicitly comparing with literal 'True' as assertTrue # Explicitly comparing with literal 'True' as assertTrue
# succeeds also for 'true' # succeeds also for 'true'
self.assertIs(True, dictionary['key_bool']) self.assertTrue(dictionary['key_bool'])
def test_subdictionary(self): def test_subdictionary(self):
dictionary, constraints = self._construct_dict_and_constraints() dictionary, constraints = self._construct_dict_and_constraints()
@ -693,23 +693,23 @@ class TestAttributes(base.BaseTestCase):
class TestConvertToBoolean(base.BaseTestCase): class TestConvertToBoolean(base.BaseTestCase):
def test_convert_to_boolean_bool(self): def test_convert_to_boolean_bool(self):
self.assertIs(attributes.convert_to_boolean(True), True) self.assertTrue(attributes.convert_to_boolean(True))
self.assertIs(attributes.convert_to_boolean(False), False) self.assertFalse(attributes.convert_to_boolean(False))
def test_convert_to_boolean_int(self): def test_convert_to_boolean_int(self):
self.assertIs(attributes.convert_to_boolean(0), False) self.assertFalse(attributes.convert_to_boolean(0))
self.assertIs(attributes.convert_to_boolean(1), True) self.assertTrue(attributes.convert_to_boolean(1))
self.assertRaises(n_exc.InvalidInput, self.assertRaises(n_exc.InvalidInput,
attributes.convert_to_boolean, attributes.convert_to_boolean,
7) 7)
def test_convert_to_boolean_str(self): def test_convert_to_boolean_str(self):
self.assertIs(attributes.convert_to_boolean('True'), True) self.assertTrue(attributes.convert_to_boolean('True'))
self.assertIs(attributes.convert_to_boolean('true'), True) self.assertTrue(attributes.convert_to_boolean('true'))
self.assertIs(attributes.convert_to_boolean('False'), False) self.assertFalse(attributes.convert_to_boolean('False'))
self.assertIs(attributes.convert_to_boolean('false'), False) self.assertFalse(attributes.convert_to_boolean('false'))
self.assertIs(attributes.convert_to_boolean('0'), False) self.assertFalse(attributes.convert_to_boolean('0'))
self.assertIs(attributes.convert_to_boolean('1'), True) self.assertTrue(attributes.convert_to_boolean('1'))
self.assertRaises(n_exc.InvalidInput, self.assertRaises(n_exc.InvalidInput,
attributes.convert_to_boolean, attributes.convert_to_boolean,
'7') '7')

View File

@ -65,7 +65,7 @@ class TackerKeystoneContextTestCase(base.BaseTestCase):
self.assertEqual(['role1', 'role2', self.assertEqual(['role1', 'role2',
'role3', 'role4', 'role5'], 'role3', 'role4', 'role5'],
self.context.roles) self.context.roles)
self.assertEqual(False, self.context.is_admin) self.assertFalse(self.context.is_admin)
def test_roles_with_admin(self): def test_roles_with_admin(self):
self.request.headers['X_PROJECT_ID'] = 'testtenantid' self.request.headers['X_PROJECT_ID'] = 'testtenantid'

View File

@ -108,7 +108,7 @@ class PolicyTestCase(base.BaseTestCase):
def test_check_bad_action_noraise(self): def test_check_bad_action_noraise(self):
action = "example:denied" action = "example:denied"
result = policy.check(self.context, action, self.target) result = policy.check(self.context, action, self.target)
self.assertEqual(False, result) self.assertFalse(result)
def test_check_non_existent_action(self): def test_check_non_existent_action(self):
action = "example:idonotexist" action = "example:idonotexist"