add domain id to expected IdP fields, unskip tests

commit Id18b8b2fe853b97631bc990df8188ed64a6e1275 added domain IDs
to an Identity provider, our functional tests have a hard match
on what to expect when 'showing' an idp, the domain ID was missing.

Change-Id: I87a1fd762918551c533668a9aa94f7c6268b79d6

Author: stevemar

openstackclient/tests/functional/identity/v3/common.py

@@ -42,7 +42,8 @@ class IdentityTests(base.TestCase):
     ENDPOINT_LIST_HEADERS = ['ID', 'Region', 'Service Name', 'Service Type',
                              'Enabled', 'Interface', 'URL']
 
-    IDENTITY_PROVIDER_FIELDS = ['description', 'enabled', 'id', 'remote_ids']
+    IDENTITY_PROVIDER_FIELDS = ['description', 'enabled', 'id', 'remote_ids',
+                                'domain_id']
     IDENTITY_PROVIDER_LIST_HEADERS = ['ID', 'Enabled', 'Description']
 
     SERVICE_PROVIDER_FIELDS = ['auth_url', 'description', 'enabled',

openstackclient/tests/functional/identity/v3/test_idp.py

@@ -11,49 +11,42 @@
 #    under the License.
 
 from tempest.lib.common.utils import data_utils
-import testtools
 
 from openstackclient.tests.functional.identity.v3 import common
 
 
 class IdentityProviderTests(common.IdentityTests):
     # Introduce functional test case for command 'Identity Provider'
 
-    @testtools.skip('domain resource changed')
     def test_idp_create(self):
         self._create_dummy_idp()
 
-    @testtools.skip('domain resource changed')
     def test_idp_delete(self):
         identity_provider = self._create_dummy_idp(add_clean_up=False)
         raw_output = self.openstack('identity provider delete %s'
                                     % identity_provider)
         self.assertEqual(0, len(raw_output))
 
-    @testtools.skip('domain resource changed')
     def test_idp_multi_delete(self):
         idp_1 = self._create_dummy_idp(add_clean_up=False)
         idp_2 = self._create_dummy_idp(add_clean_up=False)
         raw_output = self.openstack(
             'identity provider delete %s %s' % (idp_1, idp_2))
         self.assertEqual(0, len(raw_output))
 
-    @testtools.skip('domain resource changed')
     def test_idp_show(self):
         identity_provider = self._create_dummy_idp(add_clean_up=True)
         raw_output = self.openstack('identity provider show %s'
                                     % identity_provider)
         items = self.parse_show(raw_output)
         self.assert_show_fields(items, self.IDENTITY_PROVIDER_FIELDS)
 
-    @testtools.skip('domain resource changed')
     def test_idp_list(self):
         self._create_dummy_idp(add_clean_up=True)
         raw_output = self.openstack('identity provider list')
         items = self.parse_listing(raw_output)
         self.assert_table_structure(items, self.IDENTITY_PROVIDER_LIST_HEADERS)
 
-    @testtools.skip('domain resource changed')
     def test_idp_set(self):
         identity_provider = self._create_dummy_idp(add_clean_up=True)
         new_remoteid = data_utils.rand_name('newRemoteId')