openstack/python-barbicanclient
Code Issues Proposed changes

Fix order of arguments in assertEqual

Browse Source 
The correct order expected by testtools is assertEqual(expected, observed).

Change-Id: I8457af6e15a9000dd6c54ad8199331e75bd3c7fd
Partial-Bug: #1259292
This commit is contained in:
Jiong Liu
2016-09-28 22:59:54 +08:00
committed by Jeremy Liu
parent 628d9fdc11
commit bd97bedb3f
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
functionaltests/client/v1/functional/test_acl.py
View File

@@ -349,7 +349,7 @@ class ACLsTestCase(BaseACLsTestCase):
entity.submit
)
if hasattr(e, 'status_code'):
self.assertEqual(e.status_code, kwargs.get('error_code'))
self.assertEqual(kwargs.get('error_code'), e.status_code)
@utils.parameterized_dataset(ACL_DELETE_DATA)
def test_acl_delete(self, users, project_access, entity_ref_method,
@@ -476,7 +476,7 @@ class ACLsTestCase(BaseACLsTestCase):
server_acl.submit
)
if hasattr(e, 'status_code'):
self.assertEqual(e.status_code, kwargs.get('error_code'))
self.assertEqual(kwargs.get('error_code'), e.status_code)
@utils.parameterized_dataset(ACL_REMOVE_USERS_DATA_POSITIVE)
@testcase.attr('positive')
@@ -568,4 +568,4 @@ class ACLsTestCase(BaseACLsTestCase):
server_acl.submit
)
if hasattr(e, 'status_code'):
self.assertEqual(e.status_code, kwargs.get('error_code'))
self.assertEqual(kwargs.get('error_code'), e.status_code)