Merge "Removes trailing blank in trust show"

Author: Jenkins

openstackclient/identity/v3/trust.py

@@ -144,7 +144,7 @@ def take_action(self, parsed_args):
 
         # Format roles into something sensible
         roles = trust._info.pop('roles')
-        msg = ''.join([r['name'] + ' ' for r in roles])
+        msg = ' '.join(r['name'] for r in roles)
         trust._info['roles'] = msg
 
         return zip(*sorted(six.iteritems(trust._info)))
@@ -215,7 +215,7 @@ def take_action(self, parsed_args):
 
         # Format roles into something sensible
         roles = trust._info.pop('roles')
-        msg = ''.join([r['name'] + ' ' for r in roles])
+        msg = ' '.join(r['name'] for r in roles)
         trust._info['roles'] = msg
 
         return zip(*sorted(six.iteritems(trust._info)))

openstackclient/tests/identity/v3/test_trust.py

@@ -107,7 +107,7 @@ def test_trust_create_basic(self):
             identity_fakes.trust_id,
             identity_fakes.trust_impersonation,
             identity_fakes.project_id,
-            identity_fakes.role_name + ' ',
+            identity_fakes.role_name,
             identity_fakes.user_id,
             identity_fakes.user_id
         )
@@ -222,7 +222,7 @@ def test_trust_show(self):
             identity_fakes.trust_id,
             identity_fakes.trust_impersonation,
             identity_fakes.project_id,
-            identity_fakes.role_name + ' ',
+            identity_fakes.role_name,
             identity_fakes.user_id,
             identity_fakes.user_id
         )