openstack/keystoneauth
Code Issues Proposed changes

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

Browse Source 
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: I20c8b8b1ced23fb324c46ce46d7a554126c64d86
This commit is contained in:
Van Hung Pham 2017-06-23 15:34:00 +07:00
parent 0a62f6acd5
commit b0dcfa8a40
1 changed files with 1 additions and 1 deletions

2
keystoneauth1/tests/unit/loading/test_loading.py

@ -124,7 +124,7 @@ class LoadingTests(utils.TestCase):
# only the options not passed by kwargs should get passed to getter
self.assertEqual(set(('a-bool', 'a-float')), called_opts)
self.assertEqual(False, p['a_bool'])
self.assertFalse(p['a_bool'])
self.assertEqual(99.99, p['a_float'])
self.assertEqual('another', p['a_str'])
self.assertEqual(66, p['a_int'])