From c415496c22382c162cfbd87da828bd830dcb5717 Mon Sep 17 00:00:00 2001 From: Van Hung Pham Date: Fri, 23 Jun 2017 15:21:35 +0700 Subject: [PATCH] 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 --- .../tests/unit/auth_token/test_auth_token_middleware.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/keystonemiddleware/tests/unit/auth_token/test_auth_token_middleware.py b/keystonemiddleware/tests/unit/auth_token/test_auth_token_middleware.py index 1899cb07..3b9a283e 100644 --- a/keystonemiddleware/tests/unit/auth_token/test_auth_token_middleware.py +++ b/keystonemiddleware/tests/unit/auth_token/test_auth_token_middleware.py @@ -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):