Use *_as_ids instead *_as_list

The parents_as_list and subtree_as_list query parameters limit the
result to only parents and subtree where the user making the call has
role assignments in. Since OSC only displays the IDs, the call would be
the same as the similar *_as_ids queries, the difference is that the
later doesn't enforce the role assignments (making it more useful).

Output example by using this patch:

$ openstack project show --children root
+-------------+------------------------------+
| Field       | Value                        |
+-------------+------------------------------+
| description |                              |
| domain_id   | default                      |
| enabled     | True                         |
| id          | 123                          |
| is_domain   | False                        |
| name        | root                         |
| parent_id   | default                      |
| subtree     | {u'456': None, u'789': None} |
+-------------+------------------------------+

Change-Id: Ib7b37ae8f55190a7efcc375d5be4a2823d02d1a4

Author: Rodrigo Duarte Sousa

openstackclient/identity/v3/project.py

@@ -348,15 +348,8 @@ def take_action(self, parsed_args):
             #                identity manager.get() with kwargs directly.
             project = identity_client.projects.get(
                 project.id,
-                parents_as_list=parsed_args.parents,
-                subtree_as_list=parsed_args.children)
-
-        if project._info.get('parents'):
-            project._info['parents'] = [str(p['project']['id'])
-                                        for p in project._info['parents']]
-        if project._info.get('subtree'):
-            project._info['subtree'] = [str(p['project']['id'])
-                                        for p in project._info['subtree']]
+                parents_as_ids=parsed_args.parents,
+                subtree_as_ids=parsed_args.children)
 
         project._info.pop('links')
         return zip(*sorted(six.iteritems(project._info)))

openstackclient/tests/unit/identity/v3/test_project.py

@@ -845,8 +845,8 @@ def test_project_show_parents(self):
 
         self.projects_mock.get.assert_has_calls([call(self.project.id),
                                                  call(self.project.id,
-                                                      parents_as_list=True,
-                                                      subtree_as_list=False,
+                                                      parents_as_ids=True,
+                                                      subtree_as_ids=False,
                                                       )])
 
         collist = (
@@ -868,7 +868,7 @@ def test_project_show_parents(self):
             self.project.is_domain,
             self.project.name,
             self.project.parent_id,
-            [self.project.parent_id],
+            [{'project': {'id': self.project.parent_id}}]
         )
         self.assertEqual(data, datalist)
 
@@ -904,8 +904,8 @@ def test_project_show_subtree(self):
         columns, data = self.cmd.take_action(parsed_args)
         self.projects_mock.get.assert_has_calls([call(self.project.id),
                                                  call(self.project.id,
-                                                      parents_as_list=False,
-                                                      subtree_as_list=True,
+                                                      parents_as_ids=False,
+                                                      subtree_as_ids=True,
                                                       )])
 
         collist = (
@@ -927,7 +927,7 @@ def test_project_show_subtree(self):
             self.project.is_domain,
             self.project.name,
             self.project.parent_id,
-            ['children-id'],
+            [{'project': {'id': 'children-id'}}]
         )
         self.assertEqual(data, datalist)
 
@@ -965,8 +965,8 @@ def test_project_show_parents_and_children(self):
         columns, data = self.cmd.take_action(parsed_args)
         self.projects_mock.get.assert_has_calls([call(self.project.id),
                                                  call(self.project.id,
-                                                      parents_as_list=True,
-                                                      subtree_as_list=True,
+                                                      parents_as_ids=True,
+                                                      subtree_as_ids=True,
                                                       )])
 
         collist = (
@@ -989,7 +989,7 @@ def test_project_show_parents_and_children(self):
             self.project.is_domain,
             self.project.name,
             self.project.parent_id,
-            [self.project.parent_id],
-            ['children-id'],
+            [{'project': {'id': self.project.parent_id}}],
+            [{'project': {'id': 'children-id'}}]
         )
         self.assertEqual(data, datalist)