Rename column to default project id for long listing v3 user

Steve Martinelli · Steve Martinelli · commit e7ec6bc6e447 · 2015-01-02T02:39:44.000-05:00
Previously this column was coming up as empty, since user's
have a `default project id`, not just `project id`.

Change-Id: I3d7f7eb600e9526b9c6cc2a8c5d6009b9100b1f5
diff --git a/openstackclient/identity/v3/user.py b/openstackclient/identity/v3/user.py
@@ -15,6 +15,7 @@
 
 """Identity v3 User action implementations"""
 
+import copy
 import logging
 import six
 
@@ -217,17 +218,21 @@ def take_action(self, parsed_args):
 
         # List users
         if parsed_args.long:
-            columns = ('ID', 'Name', 'Project Id', 'Domain Id',
-                       'Description', 'Email', 'Enabled')
+            columns = ['ID', 'Name', 'Default Project Id', 'Domain Id',
+                       'Description', 'Email', 'Enabled']
+            column_headers = copy.deepcopy(columns)
+            column_headers[2] = 'Project'
+            column_headers[3] = 'Domain'
         else:
-            columns = ('ID', 'Name')
+            columns = ['ID', 'Name']
+            column_headers = copy.deepcopy(columns)
         data = identity_client.users.list(
             domain=domain,
             group=group,
         )
 
         return (
-            columns,
+            column_headers,
             (utils.get_item_properties(
                 s, columns,
                 formatters={},
diff --git a/openstackclient/tests/identity/v3/fakes.py b/openstackclient/tests/identity/v3/fakes.py
@@ -179,7 +179,7 @@
 USER = {
     'id': user_id,
     'name': user_name,
-    'project_id': project_id,
+    'default_project_id': project_id,
     'email': user_email,
     'enabled': True,
     'domain_id': domain_id,
diff --git a/openstackclient/tests/identity/v3/test_user.py b/openstackclient/tests/identity/v3/test_user.py
@@ -102,15 +102,16 @@ def test_user_create_no_options(self):
             **kwargs
         )
 
-        collist = ('domain_id', 'email', 'enabled', 'id', 'name', 'project_id')
+        collist = ('default_project_id', 'domain_id', 'email',
+                   'enabled', 'id', 'name')
         self.assertEqual(columns, collist)
         datalist = (
+            identity_fakes.project_id,
             identity_fakes.domain_id,
             identity_fakes.user_email,
             True,
             identity_fakes.user_id,
             identity_fakes.user_name,
-            identity_fakes.project_id,
         )
         self.assertEqual(data, datalist)
 
@@ -147,15 +148,16 @@ def test_user_create_password(self):
             **kwargs
         )
 
-        collist = ('domain_id', 'email', 'enabled', 'id', 'name', 'project_id')
+        collist = ('default_project_id', 'domain_id', 'email',
+                   'enabled', 'id', 'name')
         self.assertEqual(columns, collist)
         datalist = (
+            identity_fakes.project_id,
             identity_fakes.domain_id,
             identity_fakes.user_email,
             True,
             identity_fakes.user_id,
             identity_fakes.user_name,
-            identity_fakes.project_id,
         )
         self.assertEqual(data, datalist)
 
@@ -195,15 +197,16 @@ def test_user_create_password_prompt(self):
             **kwargs
         )
 
-        collist = ('domain_id', 'email', 'enabled', 'id', 'name', 'project_id')
+        collist = ('default_project_id', 'domain_id', 'email',
+                   'enabled', 'id', 'name')
         self.assertEqual(columns, collist)
         datalist = (
+            identity_fakes.project_id,
             identity_fakes.domain_id,
             identity_fakes.user_email,
             True,
             identity_fakes.user_id,
             identity_fakes.user_name,
-            identity_fakes.project_id,
         )
         self.assertEqual(data, datalist)
 
@@ -239,15 +242,16 @@ def test_user_create_email(self):
             **kwargs
         )
 
-        collist = ('domain_id', 'email', 'enabled', 'id', 'name', 'project_id')
+        collist = ('default_project_id', 'domain_id', 'email',
+                   'enabled', 'id', 'name')
         self.assertEqual(columns, collist)
         datalist = (
+            identity_fakes.project_id,
             identity_fakes.domain_id,
             identity_fakes.user_email,
             True,
             identity_fakes.user_id,
             identity_fakes.user_name,
-            identity_fakes.project_id,
         )
         self.assertEqual(data, datalist)
 
@@ -260,7 +264,7 @@ def test_user_create_project(self):
         )
         # Set up to return an updated user
         USER_2 = copy.deepcopy(identity_fakes.USER)
-        USER_2['project_id'] = identity_fakes.PROJECT_2['id']
+        USER_2['default_project_id'] = identity_fakes.PROJECT_2['id']
         self.users_mock.create.return_value = fakes.FakeResource(
             None,
             USER_2,
@@ -298,15 +302,16 @@ def test_user_create_project(self):
             **kwargs
         )
 
-        collist = ('domain_id', 'email', 'enabled', 'id', 'name', 'project_id')
+        collist = ('default_project_id', 'domain_id', 'email',
+                   'enabled', 'id', 'name')
         self.assertEqual(columns, collist)
         datalist = (
+            identity_fakes.PROJECT_2['id'],
             identity_fakes.domain_id,
             identity_fakes.user_email,
             True,
             identity_fakes.user_id,
             identity_fakes.user_name,
-            identity_fakes.PROJECT_2['id'],
         )
         self.assertEqual(data, datalist)
 
@@ -342,15 +347,16 @@ def test_user_create_domain(self):
             **kwargs
         )
 
-        collist = ('domain_id', 'email', 'enabled', 'id', 'name', 'project_id')
+        collist = ('default_project_id', 'domain_id', 'email',
+                   'enabled', 'id', 'name')
         self.assertEqual(columns, collist)
         datalist = (
+            identity_fakes.project_id,
             identity_fakes.domain_id,
             identity_fakes.user_email,
             True,
             identity_fakes.user_id,
             identity_fakes.user_name,
-            identity_fakes.project_id,
         )
         self.assertEqual(data, datalist)
 
@@ -385,15 +391,16 @@ def test_user_create_enable(self):
             **kwargs
         )
 
-        collist = ('domain_id', 'email', 'enabled', 'id', 'name', 'project_id')
+        collist = ('default_project_id', 'domain_id', 'email',
+                   'enabled', 'id', 'name')
         self.assertEqual(columns, collist)
         datalist = (
+            identity_fakes.project_id,
             identity_fakes.domain_id,
             identity_fakes.user_email,
             True,
             identity_fakes.user_id,
             identity_fakes.user_name,
-            identity_fakes.project_id,
         )
         self.assertEqual(data, datalist)
 
@@ -427,15 +434,16 @@ def test_user_create_disable(self):
             **kwargs
         )
 
-        collist = ('domain_id', 'email', 'enabled', 'id', 'name', 'project_id')
+        collist = ('default_project_id', 'domain_id', 'email',
+                   'enabled', 'id', 'name')
         self.assertEqual(columns, collist)
         datalist = (
+            identity_fakes.project_id,
             identity_fakes.domain_id,
             identity_fakes.user_email,
             True,
             identity_fakes.user_id,
             identity_fakes.user_name,
-            identity_fakes.project_id,
         )
         self.assertEqual(data, datalist)
 
@@ -524,7 +532,7 @@ def test_user_list_no_options(self):
             **kwargs
         )
 
-        collist = ('ID', 'Name')
+        collist = ['ID', 'Name']
         self.assertEqual(columns, collist)
         datalist = ((
             identity_fakes.user_id,
@@ -554,7 +562,7 @@ def test_user_list_domain(self):
             **kwargs
         )
 
-        collist = ('ID', 'Name')
+        collist = ['ID', 'Name']
         self.assertEqual(columns, collist)
         datalist = ((
             identity_fakes.user_id,
@@ -584,7 +592,7 @@ def test_user_list_group(self):
             **kwargs
         )
 
-        collist = ('ID', 'Name')
+        collist = ['ID', 'Name']
         self.assertEqual(columns, collist)
         datalist = ((
             identity_fakes.user_id,
@@ -614,15 +622,15 @@ def test_user_list_long(self):
             **kwargs
         )
 
-        collist = (
+        collist = [
             'ID',
             'Name',
-            'Project Id',
-            'Domain Id',
+            'Project',
+            'Domain',
             'Description',
             'Email',
             'Enabled',
-        )
+        ]
         self.assertEqual(columns, collist)
         datalist = ((
             identity_fakes.user_id,
@@ -1020,14 +1028,15 @@ def test_user_show(self):
 
         self.users_mock.get.assert_called_with(identity_fakes.user_id)
 
-        collist = ('domain_id', 'email', 'enabled', 'id', 'name', 'project_id')
+        collist = ('default_project_id', 'domain_id', 'email',
+                   'enabled', 'id', 'name')
         self.assertEqual(columns, collist)
         datalist = (
+            identity_fakes.project_id,
             identity_fakes.domain_id,
             identity_fakes.user_email,
             True,
             identity_fakes.user_id,
             identity_fakes.user_name,
-            identity_fakes.project_id,
         )
         self.assertEqual(data, datalist)
