From 20b4700893345ebacdf5d333415bc1d052493713 Mon Sep 17 00:00:00 2001 From: Shuquan Huang Date: Thu, 17 Dec 2015 21:36:58 +0800 Subject: [PATCH] Replace assertEqual(None, *) with assertIsNone in tests Replace assertEqual(None, *) with assertIsNone in tests to have more clear messages in case of failure. Change-Id: I429c8ab3eb4e79a7556ea6f966dd7aa32ee9b86d Closes-bug: #1280522 --- congressclient/tests/v1/test_datasource.py | 4 ++-- congressclient/tests/v1/test_policy.py | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/congressclient/tests/v1/test_datasource.py b/congressclient/tests/v1/test_datasource.py index b0790ea..b8b4d32 100644 --- a/congressclient/tests/v1/test_datasource.py +++ b/congressclient/tests/v1/test_datasource.py @@ -321,7 +321,7 @@ class TestDeleteDatasourceDriver(common.TestCongressBase): return_value="id"): result = cmd.take_action(parsed_args) mocker.assert_called_with("id") - self.assertEqual(None, result) + self.assertIsNone(result) class TestDatasourceRequestRefresh(common.TestCongressBase): @@ -341,4 +341,4 @@ class TestDatasourceRequestRefresh(common.TestCongressBase): return_value="id"): result = cmd.take_action(parsed_args) mocker.assert_called_with("id", {}) - self.assertEqual(None, result) + self.assertIsNone(result) diff --git a/congressclient/tests/v1/test_policy.py b/congressclient/tests/v1/test_policy.py index ae34c0d..477d20c 100644 --- a/congressclient/tests/v1/test_policy.py +++ b/congressclient/tests/v1/test_policy.py @@ -97,7 +97,7 @@ class TestDeletePolicy(common.TestCongressBase): result = cmd.take_action(parsed_args) mocker.assert_called_with(policy_id) - self.assertEqual(None, result) + self.assertIsNone(result) class TestCreatePolicyRule(common.TestCongressBase): @@ -173,7 +173,7 @@ class TestDeletePolicyRule(common.TestCongressBase): result = cmd.take_action(parsed_args) mocker.assert_called_with(policy_name, rule_id) - self.assertEqual(None, result) + self.assertIsNone(result) class TestListPolicyRules(common.TestCongressBase):