openstack
/
keystonemiddleware
Code Issues Proposed changes

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: I899fe822e4a37dda8e14b9c1dfd31c07b7d24000
This commit is contained in:
Van Hung Pham 2017-06-23 15:21:35 +07:00
parent b9e01bda73
commit c415496c22
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
keystonemiddleware/tests/unit/auth_token/test_auth_token_middleware.py
View File

@ -478,7 +478,7 @@ class GeneralAuthTokenMiddlewareTest(BaseAuthTokenMiddlewareTest,
middleware = auth_token.AuthProtocol(self.fake_app, conf)
self.assertEqual(datetime.timedelta(seconds=24),
middleware._revocations._cache_timeout)
self.assertEqual(False, middleware._include_service_catalog)
self.assertFalse(middleware._include_service_catalog)
self.assertEqual('0', middleware._conf.get('nonexsit_option'))
def test_deprecated_conf_values(self):