diff options
author | Jenkins <jenkins@review.openstack.org> | 2016-11-11 21:38:35 +0000 |
---|---|---|
committer | Gerrit Code Review <review@openstack.org> | 2016-11-11 21:38:35 +0000 |
commit | ff09b4400388e7dafddfaf2d2201144712e3a352 (patch) | |
tree | 060e3ff65d21914f5fb0df344807da663ad7264d | |
parent | be3eb3daaab93805ecda377a8d840227a86bf8cc (diff) | |
parent | f5c7762bb072a48ba5cee62c5c3b587bf220bae6 (diff) | |
download | python-barbicanclient-ff09b4400388e7dafddfaf2d2201144712e3a352.tar.gz |
Merge "Using assertTrue/False() instead of assertEqual(True/False)"
-rw-r--r-- | barbicanclient/tests/test_acls.py | 8 | ||||
-rw-r--r-- | barbicanclient/tests/test_barbican.py | 2 |
2 files changed, 5 insertions, 5 deletions
diff --git a/barbicanclient/tests/test_acls.py b/barbicanclient/tests/test_acls.py index 7f716e6..f3a468e 100644 --- a/barbicanclient/tests/test_acls.py +++ b/barbicanclient/tests/test_acls.py @@ -61,7 +61,7 @@ class WhenTestingACLManager(ACLTestCase): api_resp = self.manager.get(entity_ref=self.secret_ref) self.assertEqual(self.secret_acl_ref, self.responses.last_request.url) - self.assertEqual(False, api_resp.get('read').project_access) + self.assertFalse(api_resp.get('read').project_access) self.assertEqual('read', api_resp.get('read').operation_type) self.assertEqual(self.secret_acl_ref, api_resp.get('read').acl_ref) @@ -80,7 +80,7 @@ class WhenTestingACLManager(ACLTestCase): api_resp = self.manager.get(entity_ref=self.container_ref) self.assertEqual(self.container_acl_ref, self.responses.last_request.url) - self.assertEqual(False, api_resp.get('read').project_access) + self.assertFalse(api_resp.get('read').project_access) self.assertEqual('read', api_resp.get('read').operation_type) self.assertEqual(self.container_acl_ref, api_resp.get('read').acl_ref) @@ -110,7 +110,7 @@ class WhenTestingACLManager(ACLTestCase): read_acl = entity.read # entity ref is kept same as provided input. self.assertEqual(self.secret_ref + '///', read_acl.entity_ref) - self.assertEqual(True, read_acl.project_access) + self.assertTrue(read_acl.project_access) self.assertEqual(self.users1, read_acl.users) self.assertEqual(acls.DEFAULT_OPERATION_TYPE, read_acl.operation_type) # acl ref removes extra trailing slashes if there @@ -130,7 +130,7 @@ class WhenTestingACLManager(ACLTestCase): self.assertEqual(self.container_ref + '///', entity.entity_ref) read_acl = entity.read - self.assertEqual(False, read_acl.project_access) + self.assertFalse(read_acl.project_access) self.assertEqual(self.users2, read_acl.users) self.assertEqual(acls.DEFAULT_OPERATION_TYPE, read_acl.operation_type) # acl ref removes extra trailing slashes if there diff --git a/barbicanclient/tests/test_barbican.py b/barbicanclient/tests/test_barbican.py index 0fa8ab7..0e5bd30 100644 --- a/barbicanclient/tests/test_barbican.py +++ b/barbicanclient/tests/test_barbican.py @@ -117,7 +117,7 @@ class WhenTestingBarbicanCLI(test_client.BaseEntityResource): api_version = argv.os_identity_api_version barbican = Barbican() response = barbican.check_auth_arguments(argv, api_version) - self.assertEqual(True, response) + self.assertTrue(response) def test_should_fail_check_auth_arguments_v2(self): args = ("--os-username bob --os-password jan --os-auth-url boop" |