diff options
author | Jenkins <jenkins@review.openstack.org> | 2017-01-02 12:22:07 +0000 |
---|---|---|
committer | Gerrit Code Review <review@openstack.org> | 2017-01-02 12:22:07 +0000 |
commit | 22eade1b11c35c48f133ad221a8aa11fafbaab59 (patch) | |
tree | 5a9c36e28198b864ce650dc28623a7de274e4d10 | |
parent | 084b0abb755ac36415058f485fc6b0f318d9c041 (diff) | |
parent | ce184797f07675d403245a541de42cd48450d320 (diff) | |
download | python-novaclient-22eade1b11c35c48f133ad221a8aa11fafbaab59.tar.gz |
Merge "Use more specific asserts in tests"
-rw-r--r-- | novaclient/tests/unit/test_api_versions.py | 12 | ||||
-rw-r--r-- | novaclient/tests/unit/test_shell.py | 4 | ||||
-rw-r--r-- | novaclient/tests/unit/v2/contrib/test_tenant_networks.py | 2 | ||||
-rw-r--r-- | novaclient/tests/unit/v2/test_list_extensions.py | 4 |
4 files changed, 11 insertions, 11 deletions
diff --git a/novaclient/tests/unit/test_api_versions.py b/novaclient/tests/unit/test_api_versions.py index 00eb2b6f..89d80607 100644 --- a/novaclient/tests/unit/test_api_versions.py +++ b/novaclient/tests/unit/test_api_versions.py @@ -85,12 +85,12 @@ class APIVersionTestCase(utils.TestCase): v4 = api_versions.APIVersion("2.0") v_null = api_versions.APIVersion() - self.assertTrue(v1 < v2) - self.assertTrue(v3 > v2) - self.assertTrue(v1 != v2) - self.assertTrue(v1 == v4) - self.assertTrue(v1 != v_null) - self.assertTrue(v_null == v_null) + self.assertTrue(v1.__lt__(v2)) + self.assertTrue(v3.__gt__(v2)) + self.assertTrue(v1.__ne__(v2)) + self.assertTrue(v1.__eq__(v4)) + self.assertTrue(v1.__ne__(v_null)) + self.assertTrue(v_null.__eq__(v_null)) self.assertRaises(TypeError, v1.__le__, "2.1") def test_version_matches(self): diff --git a/novaclient/tests/unit/test_shell.py b/novaclient/tests/unit/test_shell.py index b42ddeb6..161b4547 100644 --- a/novaclient/tests/unit/test_shell.py +++ b/novaclient/tests/unit/test_shell.py @@ -173,7 +173,7 @@ class DeprecatedActionTest(utils.TestCase): self.assertEqual(result.emitted, set()) self.assertIsNone(result.use) - self.assertEqual(result.real_action_args, False) + self.assertIs(result.real_action_args, False) self.assertIsNone(result.real_action) mock_init.assert_called_once_with( 'option_strings', 'dest', help='Deprecated', a=1, b=2, c=3) @@ -200,7 +200,7 @@ class DeprecatedActionTest(utils.TestCase): self.assertEqual(result.emitted, set()) self.assertIsNone(result.use) - self.assertEqual(result.real_action_args, False) + self.assertIs(result.real_action_args, False) self.assertEqual(result.real_action, action.return_value) mock_init.assert_called_once_with( 'option_strings', 'dest', help='Deprecated', a=1, b=2, c=3) diff --git a/novaclient/tests/unit/v2/contrib/test_tenant_networks.py b/novaclient/tests/unit/v2/contrib/test_tenant_networks.py index 99e0b270..ba3998be 100644 --- a/novaclient/tests/unit/v2/contrib/test_tenant_networks.py +++ b/novaclient/tests/unit/v2/contrib/test_tenant_networks.py @@ -35,7 +35,7 @@ class TenantNetworkExtensionTests(utils.TestCase): nets = self.cs.tenant_networks.list() self.assert_request_id(nets, fakes.FAKE_REQUEST_ID_LIST) self.cs.assert_called('GET', '/os-tenant-networks') - self.assertTrue(len(nets) > 0) + self.assertGreater(len(nets), 0) def test_get_tenant_network(self): net = self.cs.tenant_networks.get(1) diff --git a/novaclient/tests/unit/v2/test_list_extensions.py b/novaclient/tests/unit/v2/test_list_extensions.py index f473ed75..60783b31 100644 --- a/novaclient/tests/unit/v2/test_list_extensions.py +++ b/novaclient/tests/unit/v2/test_list_extensions.py @@ -25,6 +25,6 @@ class ListExtensionsTests(utils.TestCase): all_exts = self.cs.list_extensions.show_all() self.assert_request_id(all_exts, fakes.FAKE_REQUEST_ID_LIST) self.cs.assert_called('GET', '/extensions') - self.assertTrue(len(all_exts) > 0) + self.assertGreater(len(all_exts), 0) for r in all_exts: - self.assertTrue(len(r.summary) > 0) + self.assertGreater(len(r.summary), 0) |