Merge pull request #182 from cloudfoundry-community/features/handle-unmapped-routes

Implement delete unmapped routes
Close #178

Author: antechrestos

main/cloudfoundry_client/client.py

@@ -25,6 +25,7 @@
 from cloudfoundry_client.v2.service_keys import ServiceKeyManager
 from cloudfoundry_client.v2.service_plan_visibilities import ServicePlanVisibilityManager
 from cloudfoundry_client.v2.service_plans import ServicePlanManager as ServicePlanManagerV2
+from cloudfoundry_client.v2.spaces import SpaceManager as SpaceManagerV2
 
 from cloudfoundry_client.v3.apps import AppManager
 from cloudfoundry_client.v3.buildpacks import BuildpackManager
@@ -87,7 +88,7 @@ def __init__(self, target_endpoint: str, credential_manager: "CloudFoundryClient
         self.routes = RouteManager(target_endpoint, credential_manager)
         self.services = EntityManagerV2(target_endpoint, credential_manager, "/v2/services")
         self.shared_domains = EntityManagerV2(target_endpoint, credential_manager, "/v2/shared_domains")
-        self.spaces = EntityManagerV2(target_endpoint, credential_manager, "/v2/spaces")
+        self.spaces = SpaceManagerV2(target_endpoint, credential_manager)
         self.stacks = EntityManagerV2(target_endpoint, credential_manager, "/v2/stacks")
         self.user_provided_service_instances = EntityManagerV2(
             target_endpoint, credential_manager, "/v2/user_provided_service_instances"

main/cloudfoundry_client/v2/spaces.py

@@ -0,0 +1,16 @@
+from typing import TYPE_CHECKING
+
+from cloudfoundry_client.v2.entities import EntityManager
+
+if TYPE_CHECKING:
+    from cloudfoundry_client.client import CloudFoundryClient
+
+
+class SpaceManager(EntityManager):
+    def __init__(self, target_endpoint: str, client: "CloudFoundryClient"):
+        super(SpaceManager, self).__init__(target_endpoint, client, "/v2/spaces")
+
+    def delete_unmapped_routes(self, space_guid: str):
+        url = "%s%s/%s/unmapped_routes" % (self.target_endpoint, self.entity_uri, space_guid)
+        super(SpaceManager, self)._delete(url)
+

test/v2/test_spaces.py

@@ -63,6 +63,12 @@ def test_entity(self):
         self.assertEqual(cpt, 1)
         self.client.get.assert_has_calls([call(side_effect.url) for side_effect in self.client.get.side_effect], any_order=False)
 
+    def test_delete_unmapped_routes(self):
+        self.client.delete.return_value = self.mock_response(
+            "/v2/spaces/space_id/unmapped_routes", HTTPStatus.NO_CONTENT, None)
+        self.client.v2.spaces.delete_unmapped_routes("space_id")
+        self.client.delete.assert_called_with(self.client.delete.return_value.url)
+
     @patch.object(sys, "argv", ["main", "list_spaces"])
     def test_main_list_spaces(self):
         with patch("cloudfoundry_client.main.main.build_client_from_configuration", new=lambda: self.client):