diff options
author | chenaidong1 <chen.aidong@zte.com.cn> | 2016-12-08 14:56:39 +0800 |
---|---|---|
committer | chenaidong1 <chen.aidong@zte.com.cn> | 2016-12-08 09:10:12 +0000 |
commit | a9ebec4fbda82c51bad5eae4bcec3356f7af7f1d (patch) | |
tree | f3d08673136f49bf5d736c5a4263419689d57020 /keystoneclient | |
parent | 2b9d1554cc7ab08dd6606a0338843be268c868df (diff) | |
download | python-keystoneclient-a9ebec4fbda82c51bad5eae4bcec3356f7af7f1d.tar.gz |
Use assertIsNone(...) instead of assertEqual(None, ...)
Refer to:http://docs.openstack.org/developer/hacking/#unit-tests-and-assertraises
[H203] Use assertIs(Not)None to check for None (off by default) Unit
test assertions tend to give better messages for more specific assertions.
As a result, assertIsNone(...) is preferred over assertEqual(None, ...)
and assertIs(None, ...)
Change-Id: I4e60f3f7f3557080669b98cb48627acc40a72606
Diffstat (limited to 'keystoneclient')
-rw-r--r-- | keystoneclient/tests/unit/v2_0/test_client.py | 2 | ||||
-rw-r--r-- | keystoneclient/tests/unit/v3/test_client.py | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/keystoneclient/tests/unit/v2_0/test_client.py b/keystoneclient/tests/unit/v2_0/test_client.py index 90b6450..fc9bf14 100644 --- a/keystoneclient/tests/unit/v2_0/test_client.py +++ b/keystoneclient/tests/unit/v2_0/test_client.py @@ -218,4 +218,4 @@ class KeystoneClientTest(utils.TestCase): # authenticated sess = auth_session.Session() cl = client.Client(session=sess) - self.assertEqual(None, cl.service_catalog) + self.assertIsNone(cl.service_catalog) diff --git a/keystoneclient/tests/unit/v3/test_client.py b/keystoneclient/tests/unit/v3/test_client.py index 42004ff..29a2818 100644 --- a/keystoneclient/tests/unit/v3/test_client.py +++ b/keystoneclient/tests/unit/v3/test_client.py @@ -268,4 +268,4 @@ class KeystoneClientTest(utils.TestCase): # authenticated sess = auth_session.Session() cl = client.Client(session=sess) - self.assertEqual(None, cl.service_catalog) + self.assertIsNone(cl.service_catalog) |