Merge "Add support for endpoing filter commands"

4 files changed, 209 insertions, 0 deletions

diff --git a/openstackclient/tests/functional/identity/v3/common.py b/openstackclient/tests/functional/identity/v3/common.py
index 6d7896d8..33cb5d86 100644
--- a/openstackclient/tests/functional/identity/v3/common.py
+++ b/openstackclient/tests/functional/identity/v3/common.py
@@ -42,6 +42,7 @@ class IdentityTests(base.TestCase):
     REGION_LIST_HEADERS = ['Region', 'Parent Region', 'Description']
     ENDPOINT_LIST_HEADERS = ['ID', 'Region', 'Service Name', 'Service Type',
                              'Enabled', 'Interface', 'URL']
+    ENDPOINT_LIST_PROJECT_HEADERS = ['ID', 'Name']
 
     IDENTITY_PROVIDER_FIELDS = ['description', 'enabled', 'id', 'remote_ids',
                                 'domain_id']
diff --git a/openstackclient/tests/functional/identity/v3/test_endpoint.py b/openstackclient/tests/functional/identity/v3/test_endpoint.py
index 22dc1b65..41f0b4c8 100644
--- a/openstackclient/tests/functional/identity/v3/test_endpoint.py
+++ b/openstackclient/tests/functional/identity/v3/test_endpoint.py
@@ -42,6 +42,29 @@ class EndpointTests(common.IdentityTests):
         items = self.parse_listing(raw_output)
         self.assert_table_structure(items, self.ENDPOINT_LIST_HEADERS)
 
+    def test_endpoint_list_filter(self):
+        endpoint_id = self._create_dummy_endpoint(add_clean_up=False)
+        project_id = self._create_dummy_project(add_clean_up=False)
+        raw_output = self.openstack(
+            'endpoint add project '
+            '%(endpoint_id)s '
+            '%(project_id)s' % {
+                'project_id': project_id,
+                'endpoint_id': endpoint_id})
+        self.assertEqual(0, len(raw_output))
+        raw_output = self.openstack(
+            'endpoint list --endpoint %s' % endpoint_id)
+        self.assertIn(project_id, raw_output)
+        items = self.parse_listing(raw_output)
+        self.assert_table_structure(items,
+                                    self.ENDPOINT_LIST_PROJECT_HEADERS)
+
+        raw_output = self.openstack(
+            'endpoint list --project %s' % project_id)
+        self.assertIn(endpoint_id, raw_output)
+        items = self.parse_listing(raw_output)
+        self.assert_table_structure(items, self.ENDPOINT_LIST_HEADERS)
+
     def test_endpoint_set(self):
         endpoint_id = self._create_dummy_endpoint()
         new_endpoint_url = data_utils.rand_url()
@@ -65,3 +88,22 @@ class EndpointTests(common.IdentityTests):
         raw_output = self.openstack('endpoint show %s' % endpoint_id)
         items = self.parse_show(raw_output)
         self.assert_show_fields(items, self.ENDPOINT_FIELDS)
+
+    def test_endpoint_add_remove_project(self):
+        endpoint_id = self._create_dummy_endpoint(add_clean_up=False)
+        project_id = self._create_dummy_project(add_clean_up=False)
+        raw_output = self.openstack(
+            'endpoint add project '
+            '%(endpoint_id)s '
+            '%(project_id)s' % {
+                'project_id': project_id,
+                'endpoint_id': endpoint_id})
+        self.assertEqual(0, len(raw_output))
+
+        raw_output = self.openstack(
+            'endpoint remove project '
+            '%(endpoint_id)s '
+            '%(project_id)s' % {
+                'project_id': project_id,
+                'endpoint_id': endpoint_id})
+        self.assertEqual(0, len(raw_output))
diff --git a/openstackclient/tests/unit/identity/v3/fakes.py b/openstackclient/tests/unit/identity/v3/fakes.py
index 7de25152..3e2caf01 100644
--- a/openstackclient/tests/unit/identity/v3/fakes.py
+++ b/openstackclient/tests/unit/identity/v3/fakes.py
@@ -493,6 +493,8 @@ class FakeIdentityv3Client(object):
         self.credentials.resource_class = fakes.FakeResource(None, {})
         self.endpoints = mock.Mock()
         self.endpoints.resource_class = fakes.FakeResource(None, {})
+        self.endpoint_filter = mock.Mock()
+        self.endpoint_filter.resource_class = fakes.FakeResource(None, {})
         self.groups = mock.Mock()
         self.groups.resource_class = fakes.FakeResource(None, {})
         self.oauth1 = mock.Mock()
@@ -911,6 +913,31 @@ class FakeEndpoint(object):
                                       loaded=True)
         return endpoint
 
+    @staticmethod
+    def create_one_endpoint_filter(attrs=None):
+        """Create a fake endpoint project relationship.
+
+        :param Dictionary attrs:
+            A dictionary with all attributes of endpoint filter
+        :return:
+            A FakeResource object with project, endpoint and so on
+        """
+        attrs = attrs or {}
+
+        # Set default attribute
+        endpoint_filter_info = {
+            'project': 'project-id-' + uuid.uuid4().hex,
+            'endpoint': 'endpoint-id-' + uuid.uuid4().hex,
+        }
+
+        # Overwrite default attributes if there are some attributes set
+        endpoint_filter_info.update(attrs)
+
+        endpoint_filter = fakes.FakeModel(
+            copy.deepcopy(endpoint_filter_info))
+
+        return endpoint_filter
+
 
 class FakeService(object):
     """Fake one or more service."""
diff --git a/openstackclient/tests/unit/identity/v3/test_endpoint.py b/openstackclient/tests/unit/identity/v3/test_endpoint.py
index fad53fcb..bfe930d6 100644
--- a/openstackclient/tests/unit/identity/v3/test_endpoint.py
+++ b/openstackclient/tests/unit/identity/v3/test_endpoint.py
@@ -22,11 +22,23 @@ class TestEndpoint(identity_fakes.TestIdentityv3):
         # Get a shortcut to the EndpointManager Mock
         self.endpoints_mock = self.app.client_manager.identity.endpoints
         self.endpoints_mock.reset_mock()
+        self.ep_filter_mock = (
+            self.app.client_manager.identity.endpoint_filter
+        )
+        self.ep_filter_mock.reset_mock()
 
         # Get a shortcut to the ServiceManager Mock
         self.services_mock = self.app.client_manager.identity.services
         self.services_mock.reset_mock()
 
+        # Get a shortcut to the DomainManager Mock
+        self.domains_mock = self.app.client_manager.identity.domains
+        self.domains_mock.reset_mock()
+
+        # Get a shortcut to the ProjectManager Mock
+        self.projects_mock = self.app.client_manager.identity.projects
+        self.projects_mock.reset_mock()
+
 
 class TestEndpointCreate(TestEndpoint):
 
@@ -750,3 +762,130 @@ class TestEndpointShowServiceWithoutName(TestEndpointShow):
 
         # Get the command object to test
         self.cmd = endpoint.ShowEndpoint(self.app, None)
+
+
+class TestAddProjectToEndpoint(TestEndpoint):
+
+    project = identity_fakes.FakeProject.create_one_project()
+    domain = identity_fakes.FakeDomain.create_one_domain()
+    service = identity_fakes.FakeService.create_one_service()
+    endpoint = identity_fakes.FakeEndpoint.create_one_endpoint(
+        attrs={'service_id': service.id})
+
+    new_ep_filter = identity_fakes.FakeEndpoint.create_one_endpoint_filter(
+        attrs={'endpoint': endpoint.id,
+               'project': project.id}
+    )
+
+    def setUp(self):
+        super(TestAddProjectToEndpoint, self).setUp()
+
+        # This is the return value for utils.find_resource()
+        self.endpoints_mock.get.return_value = self.endpoint
+
+        # Update the image_id in the MEMBER dict
+        self.ep_filter_mock.create.return_value = self.new_ep_filter
+        self.projects_mock.get.return_value = self.project
+        self.domains_mock.get.return_value = self.domain
+        # Get the command object to test
+        self.cmd = endpoint.AddProjectToEndpoint(self.app, None)
+
+    def test_add_project_to_endpoint_no_option(self):
+        arglist = [
+            self.endpoint.id,
+            self.project.id,
+        ]
+        verifylist = [
+            ('endpoint', self.endpoint.id),
+            ('project', self.project.id),
+        ]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        result = self.cmd.take_action(parsed_args)
+        self.ep_filter_mock.add_endpoint_to_project.assert_called_with(
+            project=self.project.id,
+            endpoint=self.endpoint.id
+        )
+        self.assertIsNone(result)
+
+    def test_add_project_to_endpoint_with_option(self):
+        arglist = [
+            self.endpoint.id,
+            self.project.id,
+            '--project-domain', self.domain.id,
+        ]
+        verifylist = [
+            ('endpoint', self.endpoint.id),
+            ('project', self.project.id),
+            ('project_domain', self.domain.id),
+        ]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        result = self.cmd.take_action(parsed_args)
+        self.ep_filter_mock.add_endpoint_to_project.assert_called_with(
+            project=self.project.id,
+            endpoint=self.endpoint.id
+        )
+        self.assertIsNone(result)
+
+
+class TestRemoveProjectEndpoint(TestEndpoint):
+
+    project = identity_fakes.FakeProject.create_one_project()
+    domain = identity_fakes.FakeDomain.create_one_domain()
+    service = identity_fakes.FakeService.create_one_service()
+    endpoint = identity_fakes.FakeEndpoint.create_one_endpoint(
+        attrs={'service_id': service.id})
+
+    def setUp(self):
+        super(TestRemoveProjectEndpoint, self).setUp()
+
+        # This is the return value for utils.find_resource()
+        self.endpoints_mock.get.return_value = self.endpoint
+
+        self.projects_mock.get.return_value = self.project
+        self.domains_mock.get.return_value = self.domain
+        self.ep_filter_mock.delete.return_value = None
+
+        # Get the command object to test
+        self.cmd = endpoint.RemoveProjectFromEndpoint(self.app, None)
+
+    def test_remove_project_endpoint_no_options(self):
+        arglist = [
+            self.endpoint.id,
+            self.project.id,
+        ]
+        verifylist = [
+            ('endpoint', self.endpoint.id),
+            ('project', self.project.id),
+        ]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        result = self.cmd.take_action(parsed_args)
+
+        self.ep_filter_mock.delete_endpoint_from_project.assert_called_with(
+            project=self.project.id,
+            endpoint=self.endpoint.id,
+        )
+        self.assertIsNone(result)
+
+    def test_remove_project_endpoint_with_options(self):
+        arglist = [
+            self.endpoint.id,
+            self.project.id,
+            '--project-domain', self.domain.id,
+        ]
+        verifylist = [
+            ('endpoint', self.endpoint.id),
+            ('project', self.project.id),
+            ('project_domain', self.domain.id),
+        ]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        result = self.cmd.take_action(parsed_args)
+
+        self.ep_filter_mock.delete_endpoint_from_project.assert_called_with(
+            project=self.project.id,
+            endpoint=self.endpoint.id,
+        )
+        self.assertIsNone(result)