openstack/nova
Code Issues Proposed changes

Merge "Remove get_all method from servicegroup API"

Browse Source
This commit is contained in:
Jenkins
2015-11-12 15:03:06 +00:00
committed by Gerrit Code Review
parent aa141f0d8e eb285ee07b
commit 11bf1bc6a1
2 changed files with 0 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
nova/servicegroup/api.py
View File

@@ -90,9 +90,3 @@ class API(object):
return False return False
return self._driver.is_up(member) return self._driver.is_up(member)
def get_all(self, group_id):
"""Returns ALL members of the given group."""
LOG.debug('Returns ALL members of the [%s] '
'ServiceGroup', group_id)
return self._driver.get_all(group_id)

10
nova/tests/unit/servicegroup/test_api.py
View File

@@ -40,16 +40,6 @@ class ServiceGroupApiTestCase(test.NoDBTestCase):
self.assertIsNone(result) self.assertIsNone(result)
self.driver.join.assert_called_with(member, group, None) self.driver.join.assert_called_with(member, group, None)
def test_get_all(self):
result = "fake response"
group_id = 1
self.driver.get_all = mock.MagicMock(return_value=result)
members = self.servicegroup_api.get_all(group_id)
self.assertEqual(members, result)
self.driver.get_all.assert_called_with(group_id)
def test_service_is_up(self): def test_service_is_up(self):
"""""" """"""
member = {"host": "fake-host", member = {"host": "fake-host",