better format remote IDs for identity providers

remote-ids are a list, and we should format these values as such,
rather than python representations of lists/arrays.

Closes-Bug: 1478995

Change-Id: Ia6ced0fab2435b8cb486822c676c0dee32613abe

Author: Steve Martinelli

openstackclient/identity/v3/identity_provider.py

@@ -88,6 +88,8 @@ def take_action(self, parsed_args):
             enabled=parsed_args.enabled)
 
         idp._info.pop('links', None)
+        remote_ids = utils.format_list(idp._info.pop('remote_ids', []))
+        idp._info['remote_ids'] = remote_ids
         return zip(*sorted(six.iteritems(idp._info)))
 
 
@@ -221,9 +223,11 @@ def get_parser(self, prog_name):
     @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity
-        identity_provider = utils.find_resource(
+        idp = utils.find_resource(
             identity_client.federation.identity_providers,
             parsed_args.identity_provider)
 
-        identity_provider._info.pop('links', None)
-        return zip(*sorted(six.iteritems(identity_provider._info)))
+        idp._info.pop('links', None)
+        remote_ids = utils.format_list(idp._info.pop('remote_ids', []))
+        idp._info['remote_ids'] = remote_ids
+        return zip(*sorted(six.iteritems(idp._info)))

openstackclient/tests/identity/v3/fakes.py

@@ -261,6 +261,7 @@
 idp_id = 'test_idp'
 idp_description = 'super exciting IdP description'
 idp_remote_ids = ['entity1', 'entity2']
+formatted_idp_remote_ids = 'entity1, entity2'
 
 IDENTITY_PROVIDER = {
     'id': idp_id,

openstackclient/tests/identity/v3/test_identity_provider.py

@@ -69,7 +69,7 @@ def test_create_identity_provider_no_options(self):
             identity_fakes.idp_description,
             True,
             identity_fakes.idp_id,
-            identity_fakes.idp_remote_ids
+            identity_fakes.formatted_idp_remote_ids
         )
         self.assertEqual(datalist, data)
 
@@ -103,7 +103,7 @@ def test_create_identity_provider_description(self):
             identity_fakes.idp_description,
             True,
             identity_fakes.idp_id,
-            identity_fakes.idp_remote_ids
+            identity_fakes.formatted_idp_remote_ids
         )
         self.assertEqual(datalist, data)
 
@@ -137,7 +137,7 @@ def test_create_identity_provider_remote_id(self):
             identity_fakes.idp_description,
             True,
             identity_fakes.idp_id,
-            identity_fakes.idp_remote_ids
+            identity_fakes.formatted_idp_remote_ids
         )
         self.assertEqual(datalist, data)
 
@@ -172,7 +172,7 @@ def test_create_identity_provider_remote_ids_multiple(self):
             identity_fakes.idp_description,
             True,
             identity_fakes.idp_id,
-            identity_fakes.idp_remote_ids
+            identity_fakes.formatted_idp_remote_ids
         )
         self.assertEqual(datalist, data)
 
@@ -211,7 +211,7 @@ def test_create_identity_provider_remote_ids_file(self):
             identity_fakes.idp_description,
             True,
             identity_fakes.idp_id,
-            identity_fakes.idp_remote_ids
+            identity_fakes.formatted_idp_remote_ids
         )
         self.assertEqual(datalist, data)
 
@@ -253,7 +253,7 @@ def test_create_identity_provider_disabled(self):
             None,
             False,
             identity_fakes.idp_id,
-            identity_fakes.idp_remote_ids
+            identity_fakes.formatted_idp_remote_ids
         )
         self.assertEqual(datalist, data)
 
@@ -363,7 +363,7 @@ def test_identity_provider_show(self):
             identity_fakes.idp_description,
             True,
             identity_fakes.idp_id,
-            identity_fakes.idp_remote_ids
+            identity_fakes.formatted_idp_remote_ids
         )
         self.assertEqual(datalist, data)