openstack
/
keystone
Code Issues Proposed changes

Merge "Use assertDictEqual instead of assertEqualPolicies"

This commit is contained in:
Jenkins 2015-12-10 01:43:05 +00:00 committed by Gerrit Code Review
parent 9c79319cf2 1f0466e5c2
commit 79f141b81a
1 changed files with 4 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
keystone/tests/unit/test_backend.py
View File

@ -5306,20 +5306,17 @@ class CatalogTests(object):
class PolicyTests(object):
def assertEqualPolicies(self, a, b):
self.assertDictEqual(a, b)
def test_create(self):
ref = unit.new_policy_ref()
res = self.policy_api.create_policy(ref['id'], ref)
self.assertEqualPolicies(ref, res)
self.assertDictEqual(ref, res)
def test_get(self):
ref = unit.new_policy_ref()
res = self.policy_api.create_policy(ref['id'], ref)
res = self.policy_api.get_policy(ref['id'])
self.assertEqualPolicies(ref, res)
self.assertDictEqual(ref, res)
def test_list(self):
ref = unit.new_policy_ref()
@ -5327,7 +5324,7 @@ class PolicyTests(object):
res = self.policy_api.list_policies()
res = [x for x in res if x['id'] == ref['id']][0]
self.assertEqualPolicies(ref, res)
self.assertDictEqual(ref, res)
def test_update(self):
ref = unit.new_policy_ref()
@ -5344,7 +5341,7 @@ class PolicyTests(object):
ref['id'] = orig['id']
res = self.policy_api.update_policy(orig['id'], ref)
self.assertEqualPolicies(ref, res)
self.assertDictEqual(ref, res)
def test_delete(self):
ref = unit.new_policy_ref()