openstack/mistral
Code Issues Proposed changes

Merge "Use assertCountEqual instead of assertItemsEqual"

Browse Source
This commit is contained in:
Zuul 2020-08-10 05:19:22 +00:00 committed by Gerrit Code Review
commit b728e9cfd4
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
mistral/tests/unit/api/v2/test_services.py
View File

@ -39,7 +39,7 @@ class TestServicesController(base.APITest):
self.assertEqual(1, len(resp.json['services'])) self.assertEqual(1, len(resp.json['services']))
srv_ret = [{"name": "service1", "type": "api_group"}] srv_ret = [{"name": "service1", "type": "api_group"}]
self.assertItemsEqual(srv_ret, resp.json['services']) self.assertCountEqual(srv_ret, resp.json['services'])
def test_get_all_without_backend(self): def test_get_all_without_backend(self):
cfg.CONF.set_default('backend_url', None, 'coordination') cfg.CONF.set_default('backend_url', None, 'coordination')

2
mistral/tests/unit/test_coordination.py
View File

@ -93,7 +93,7 @@ class ServiceCoordinatorTest(base.BaseTest):
members = coordinator.get_members('fake_group') members = coordinator.get_members('fake_group')
self.assertEqual(1, len(members)) self.assertEqual(1, len(members))
self.assertItemsEqual((six.b('fake_id'),), members) self.assertCountEqual((six.b('fake_id'),), members)
def test_join_group_and_leave_group(self): def test_join_group_and_leave_group(self):
cfg.CONF.set_default( cfg.CONF.set_default(