Merge "Fix node list"

This commit is contained in:
Jenkins 2015-04-14 10:47:49 +00:00 committed by Gerrit Code Review
commit 37bca2b85c
2 changed files with 9 additions and 2 deletions

View File

@ -229,7 +229,8 @@ class Node(base.APIResourceWrapper):
server = servers_dict.get(n.instance_uuid, None)
nodes_with_instance.append(cls(n, instance=server,
request=request))
return nodes_with_instance
return [cls.get(request, node.uuid)
for node in nodes_with_instance]
return [cls.get(request, node.uuid) for node in nodes]
@classmethod

View File

@ -97,9 +97,15 @@ class NodeAPITests(test.APITestCase):
node = self.ironicclient_nodes.first()
nodes = self.ironicclient_nodes.list()
with mock_ironicclient(node=node, nodes=nodes), mock.patch(
with mock_ironicclient(
node=node,
nodes=nodes
), mock.patch(
'openstack_dashboard.api.nova.server_list',
return_value=(instances, None),
), mock.patch(
'openstack_dashboard.api.nova.server_get',
return_value=instances[0],
):
ret_val = api.node.Node.list(self.request)