Add tests for FIPs port forwarding API's new policy rules

Related-blueprint: bp/secure-rbac-roles
Change-Id: Ic3ec1e6fa296248f8db08edcfb3120c85d2c872a

neutron/tests/unit/conf/policies/test_floatingip_port_forwarding.py

@@ -0,0 +1,244 @@
+# Copyright (c) 2021 Red Hat Inc.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#    http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
+# implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+from unittest import mock
+
+from oslo_policy import policy as base_policy
+from oslo_utils import uuidutils
+
+from neutron import policy
+from neutron.tests.unit.conf.policies import base
+
+
+class FloatingipPortForwardingAPITestCase(base.PolicyBaseTestCase):
+
+    def setUp(self):
+        super(FloatingipPortForwardingAPITestCase, self).setUp()
+        self.fip = {
+            'id': uuidutils.generate_uuid(),
+            'project_id': self.project_id}
+
+        self.target = {
+            'project_id': self.project_id,
+            'floatingip_id': self.fip['id'],
+            'ext_parent_floatingip_id': self.fip['id']}
+        self.alt_target = {
+            'project_id': self.alt_project_id,
+            'floatingip_id': self.fip['id'],
+            'ext_parent_floatingip_id': self.fip['id']}
+
+        self.plugin_mock = mock.Mock()
+        self.plugin_mock.get_floatingip.return_value = self.fip
+        mock.patch(
+            'neutron_lib.plugins.directory.get_plugin',
+            return_value=self.plugin_mock).start()
+
+
+class SystemAdminTests(FloatingipPortForwardingAPITestCase):
+
+    def setUp(self):
+        super(SystemAdminTests, self).setUp()
+        self.context = self.system_admin_ctx
+
+    def test_create_fip_pf(self):
+        self.assertTrue(
+            policy.enforce(self.context,
+                           'create_floatingip_port_forwarding',
+                           self.target))
+        self.assertTrue(
+            policy.enforce(self.context,
+                           'create_floatingip_port_forwarding',
+                           self.alt_target))
+
+    def test_get_fip_pf(self):
+        self.assertTrue(
+            policy.enforce(self.context,
+                           'get_floatingip_port_forwarding',
+                           self.target))
+        self.assertTrue(
+            policy.enforce(self.context,
+                           'get_floatingip_port_forwarding',
+                           self.alt_target))
+
+    def test_update_fip_pf(self):
+        self.assertTrue(
+            policy.enforce(self.context,
+                           'update_floatingip_port_forwarding',
+                           self.target))
+        self.assertTrue(
+            policy.enforce(self.context,
+                           'update_floatingip_port_forwarding',
+                           self.alt_target))
+
+    def test_delete_fip_pf(self):
+        self.assertTrue(
+            policy.enforce(self.context,
+                           'delete_floatingip_port_forwarding',
+                           self.target))
+        self.assertTrue(
+            policy.enforce(self.context,
+                           'delete_floatingip_port_forwarding',
+                           self.alt_target))
+
+
+class SystemMemberTests(SystemAdminTests):
+
+    def setUp(self):
+        super(SystemMemberTests, self).setUp()
+        self.context = self.system_member_ctx
+
+    def test_create_fip_pf(self):
+        self.assertRaises(
+            base_policy.PolicyNotAuthorized,
+            policy.enforce,
+            self.context, 'create_floatingip_port_forwarding',
+            self.target)
+        self.assertRaises(
+            base_policy.PolicyNotAuthorized,
+            policy.enforce,
+            self.context, 'create_floatingip_port_forwarding',
+            self.alt_target)
+
+    def test_update_fip_pf(self):
+        self.assertRaises(
+            base_policy.PolicyNotAuthorized,
+            policy.enforce,
+            self.context, 'update_floatingip_port_forwarding',
+            self.target)
+        self.assertRaises(
+            base_policy.PolicyNotAuthorized,
+            policy.enforce,
+            self.context, 'update_floatingip_port_forwarding',
+            self.alt_target)
+
+    def test_delete_fip_pf(self):
+        self.assertRaises(
+            base_policy.PolicyNotAuthorized,
+            policy.enforce,
+            self.context, 'delete_floatingip_port_forwarding',
+            self.target)
+        self.assertRaises(
+            base_policy.PolicyNotAuthorized,
+            policy.enforce,
+            self.context, 'delete_floatingip_port_forwarding',
+            self.alt_target)
+
+
+class SystemReaderTests(SystemMemberTests):
+
+    def setUp(self):
+        super(SystemReaderTests, self).setUp()
+        self.context = self.system_reader_ctx
+
+
+class ProjectAdminTests(FloatingipPortForwardingAPITestCase):
+
+    def setUp(self):
+        super(ProjectAdminTests, self).setUp()
+        self.context = self.project_admin_ctx
+
+    def test_create_fip_pf(self):
+        self.assertTrue(
+            policy.enforce(self.context,
+                           'create_floatingip_port_forwarding',
+                           self.target))
+        self.assertRaises(
+            base_policy.PolicyNotAuthorized,
+            policy.enforce,
+            self.context, 'create_floatingip_port_forwarding',
+            self.alt_target)
+
+    def test_get_fip_pf(self):
+        self.assertTrue(
+            policy.enforce(self.context,
+                           'get_floatingip_port_forwarding',
+                           self.target))
+        self.assertRaises(
+            base_policy.PolicyNotAuthorized,
+            policy.enforce,
+            self.context, 'get_floatingip_port_forwarding',
+            self.alt_target)
+
+    def test_update_fip_pf(self):
+        self.assertTrue(
+            policy.enforce(self.context,
+                           'update_floatingip_port_forwarding',
+                           self.target))
+        self.assertRaises(
+            base_policy.PolicyNotAuthorized,
+            policy.enforce,
+            self.context, 'update_floatingip_port_forwarding',
+            self.alt_target)
+
+    def test_delete_fip_pf(self):
+        self.assertTrue(
+            policy.enforce(self.context,
+                           'delete_floatingip_port_forwarding',
+                           self.target))
+        self.assertRaises(
+            base_policy.PolicyNotAuthorized,
+            policy.enforce,
+            self.context, 'delete_floatingip_port_forwarding',
+            self.alt_target)
+
+
+class ProjectMemberTests(ProjectAdminTests):
+
+    def setUp(self):
+        super(ProjectMemberTests, self).setUp()
+        self.context = self.project_member_ctx
+
+
+class ProjectReaderTests(ProjectMemberTests):
+
+    def setUp(self):
+        super(ProjectReaderTests, self).setUp()
+        self.context = self.project_reader_ctx
+
+    def test_create_fip_pf(self):
+        self.assertRaises(
+            base_policy.PolicyNotAuthorized,
+            policy.enforce,
+            self.context, 'create_floatingip_port_forwarding',
+            self.target)
+        self.assertRaises(
+            base_policy.PolicyNotAuthorized,
+            policy.enforce,
+            self.context, 'create_floatingip_port_forwarding',
+            self.alt_target)
+
+    def test_update_fip_pf(self):
+        self.assertRaises(
+            base_policy.PolicyNotAuthorized,
+            policy.enforce,
+            self.context, 'update_floatingip_port_forwarding',
+            self.target)
+        self.assertRaises(
+            base_policy.PolicyNotAuthorized,
+            policy.enforce,
+            self.context, 'update_floatingip_port_forwarding',
+            self.alt_target)
+
+    def test_delete_fip_pf(self):
+        self.assertRaises(
+            base_policy.PolicyNotAuthorized,
+            policy.enforce,
+            self.context, 'delete_floatingip_port_forwarding',
+            self.target)
+        self.assertRaises(
+            base_policy.PolicyNotAuthorized,
+            policy.enforce,
+            self.context, 'delete_floatingip_port_forwarding',
+            self.alt_target)