diff options
author | Cao Xuan Hoang <hoangcx@vn.fujitsu.com> | 2016-09-26 15:33:18 +0700 |
---|---|---|
committer | Cao Xuan Hoang <hoangcx@vn.fujitsu.com> | 2016-09-26 15:33:18 +0700 |
commit | 09131e136227a8dfb40a8b248aa7faf52d299547 (patch) | |
tree | fa399e91fe7439074ab32834b24e237962b8702e /keystone/tests/unit/test_config.py | |
parent | cb2b548f6b4abb83b21bf2d09a4e983735a9d4e8 (diff) | |
download | keystone-09131e136227a8dfb40a8b248aa7faf52d299547.tar.gz |
Using assertIsNone() instead of assertIs(None)
Following OpenStack Style Guidelines[1]:
[H203] Unit test assertions tend to give better messages for more
specific assertions. As a result, assertIsNone(...) is preferred over
assertEqual(None, ...) and assertIs(..,None)
[1] http://docs.openstack.org/developer/hacking/#unit-tests-and-assertraises
Change-Id: I9b0cedae367798ce282b0229c135b3f4a72f353a
Diffstat (limited to 'keystone/tests/unit/test_config.py')
-rw-r--r-- | keystone/tests/unit/test_config.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/keystone/tests/unit/test_config.py b/keystone/tests/unit/test_config.py index 52dfdae22..36fa73343 100644 --- a/keystone/tests/unit/test_config.py +++ b/keystone/tests/unit/test_config.py @@ -45,8 +45,8 @@ class ConfigTestCase(unit.TestCase): wsgi.find_paste_config()) def test_config_default(self): - self.assertIs(None, CONF.auth.password) - self.assertIs(None, CONF.auth.token) + self.assertIsNone(CONF.auth.password) + self.assertIsNone(CONF.auth.token) def test_profiler_config_default(self): """Check config.set_config_defaults() has set [profiler]enabled.""" |