Support bulk deletion for delete commands in identityv3

Support bulk deletion for delete commands in the list below
identity/v3/consumer
identity/v3/credential
identity/v3/domain
identity/v3/ec2creds
identity/v3/endpoint
identity/v3/federation_protocol
identity/v3/identity_provider
identity/v3/mapping
identity/v3/policy
identity/v3/region
identity/v3/service_provider
identity/v3/service

The unit test in identityv3 need to be refactored, so I add
some functional tests instead. I will add all unit tests at
one time after the refactor completed.

Change-Id: I82367570f59817b47c87b6c7bfeae95ccfe5c50e
Closes-Bug: #1592906

Author: Huanxuan Ao

doc/source/command-objects/consumer.rst

@@ -24,17 +24,17 @@ Create new consumer
 consumer delete
 ---------------
 
-Delete consumer
+Delete consumer(s)
 
 .. program:: consumer delete
 .. code:: bash
 
     os consumer delete
-        <consumer>
+        <consumer> [<consumer> ...]
 
 .. describe:: <consumer>
 
-    Consumer to delete
+    Consumer(s) to delete
 
 consumer list
 -------------

doc/source/command-objects/domain.rst

@@ -43,17 +43,17 @@ Create new domain
 domain delete
 -------------
 
-Delete domain
+Delete domain(s)
 
 .. program:: domain delete
 .. code:: bash
 
     os domain delete
-        <domain>
+        <domain> [<domain> ...]
 
 .. describe:: <domain>
 
-    Domain to delete (name or ID)
+    Domain(s) to delete (name or ID)
 
 domain list
 -----------

doc/source/command-objects/federation-protocol.rst

@@ -34,22 +34,22 @@ Create new federation protocol
 federation protocol delete
 --------------------------
 
-Delete federation protocol
+Delete federation protocol(s)
 
 .. program:: federation protocol delete
 .. code:: bash
 
     os federation protocol delete
         --identity-provider <identity-provider>
-        <federation-protocol>
+        <federation-protocol> [<federation-protocol> ...]
 
 .. option:: --identity-provider <identity-provider>
 
     Identity provider that supports <federation-protocol> (name or ID) (required)
 
 .. describe:: <federation-protocol>
 
-    Federation protocol to delete (name or ID)
+    Federation protocol(s) to delete (name or ID)
 
 federation protocol list
 ------------------------

doc/source/command-objects/identity-provider.rst

@@ -49,17 +49,17 @@ Create new identity provider
 identity provider delete
 ------------------------
 
-Delete identity provider
+Delete identity provider(s)
 
 .. program:: identity provider delete
 .. code:: bash
 
     os identity provider delete
-        <identity-provider>
+        <identity-provider> [<identity-provider> ...]
 
 .. describe:: <identity-provider>
 
-    Identity provider to delete
+    Identity provider(s) to delete
 
 identity provider list
 ----------------------

doc/source/command-objects/mapping.rst

@@ -30,18 +30,18 @@ Create new mapping
 mapping delete
 --------------
 
-Delete mapping
+Delete mapping(s)
 
 .. program:: mapping delete
 .. code:: bash
 
     os mapping delete
-        <mapping>
+        <mapping> [<mapping> ...]
 
 .. _mapping_delete-mapping:
 .. describe:: <mapping>
 
-    Mapping to delete
+    Mapping(s) to delete
 
 mapping list
 ------------

doc/source/command-objects/policy.rst

@@ -27,17 +27,17 @@ Create new policy
 policy delete
 -------------
 
-Delete policy
+Delete policy(s)
 
 .. program:: policy delete
 .. code:: bash
 
     os policy delete
-        <policy>
+        <policy> [<policy> ...]
 
 .. describe:: <policy>
 
-    Policy to delete
+    Policy(s) to delete
 
 policy list
 -----------

doc/source/command-objects/region.rst

@@ -33,18 +33,18 @@ Create new region
 region delete
 -------------
 
-Delete region
+Delete region(s)
 
 .. program:: region delete
 .. code:: bash
 
     os region delete
-        <region-id>
+        <region-id> [<region-id> ...]
 
 .. _region_delete-region-id:
 .. describe:: <region-id>
 
-    Region ID to delete
+    Region ID(s) to delete
 
 region list
 -----------

doc/source/command-objects/service-provider.rst

@@ -48,17 +48,17 @@ Create new service provider
 service provider delete
 -----------------------
 
-Delete service provider
+Delete service provider(s)
 
 .. program:: service provider delete
 .. code:: bash
 
     os service provider delete
-        <service-provider>
+        <service-provider> [<service-provider> ...]
 
 .. describe:: <service-provider>
 
-    Service provider to delete
+    Service provider(s) to delete
 
 service provider list
 ---------------------

functional/tests/identity/v3/test_domain.py

@@ -43,6 +43,18 @@ def test_domain_delete(self):
         raw_output = self.openstack('domain delete %s' % domain_name)
         self.assertEqual(0, len(raw_output))
 
+    def test_domain_multi_delete(self):
+        domain_1 = self._create_dummy_domain(add_clean_up=False)
+        domain_2 = self._create_dummy_domain(add_clean_up=False)
+        # cannot delete enabled domain, disable it first
+        raw_output = self.openstack('domain set --disable %s' % domain_1)
+        self.assertEqual(0, len(raw_output))
+        raw_output = self.openstack('domain set --disable %s' % domain_2)
+        self.assertEqual(0, len(raw_output))
+        raw_output = self.openstack(
+            'domain delete %s %s' % (domain_1, domain_2))
+        self.assertEqual(0, len(raw_output))
+
     def test_domain_delete_failure(self):
         domain_name = self._create_dummy_domain()
         # cannot delete enabled domain

functional/tests/identity/v3/test_endpoint.py

@@ -28,6 +28,13 @@ def test_endpoint_delete(self):
             'endpoint delete %s' % endpoint_id)
         self.assertEqual(0, len(raw_output))
 
+    def test_endpoint_multi_delete(self):
+        endpoint_1 = self._create_dummy_endpoint(add_clean_up=False)
+        endpoint_2 = self._create_dummy_endpoint(add_clean_up=False)
+        raw_output = self.openstack(
+            'endpoint delete %s %s' % (endpoint_1, endpoint_2))
+        self.assertEqual(0, len(raw_output))
+
     def test_endpoint_list(self):
         endpoint_id = self._create_dummy_endpoint()
         raw_output = self.openstack('endpoint list')