diff options
author | Zuul <zuul@review.opendev.org> | 2023-05-11 10:12:49 +0000 |
---|---|---|
committer | Gerrit Code Review <review@openstack.org> | 2023-05-11 10:12:49 +0000 |
commit | 456b6399bece9a9ac6274e56130cff6680d99096 (patch) | |
tree | 5905f0a283cfdfc3246858b5bf036dd66a1e9cc0 /cinder/tests/unit/policies/test_volume_actions.py | |
parent | 9e254521021cef0031537bc24d19dc4f08d2cb3c (diff) | |
parent | 6df1839bdf288107c600b3e53dff7593a6d4c161 (diff) | |
download | cinder-456b6399bece9a9ac6274e56130cff6680d99096.tar.gz |
Diffstat (limited to 'cinder/tests/unit/policies/test_volume_actions.py')
-rw-r--r-- | cinder/tests/unit/policies/test_volume_actions.py | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/cinder/tests/unit/policies/test_volume_actions.py b/cinder/tests/unit/policies/test_volume_actions.py index d9b2edc8e..17dacd5ef 100644 --- a/cinder/tests/unit/policies/test_volume_actions.py +++ b/cinder/tests/unit/policies/test_volume_actions.py @@ -89,6 +89,9 @@ class VolumeActionsPolicyTest(base.BasePolicyTest): self._initialize_connection) self.api_path = '/v3/%s/volumes' % (self.project_id) self.api_version = mv.BASE_VERSION + self.mock_is_service = self.patch( + 'cinder.volume.api.API.is_service_request', + return_value=True) def _initialize_connection(self, volume, connector): return {'data': connector} @@ -946,6 +949,7 @@ class VolumeProtectionTests(test_base.CinderPolicyTests): self.assertEqual(HTTPStatus.ACCEPTED, response.status_int) body = {"os-detach": {}} + # Detach for user call succeeds because the volume has no attachments response = self._get_request_response(admin_context, path, 'POST', body=body) self.assertEqual(HTTPStatus.ACCEPTED, response.status_int) @@ -966,6 +970,7 @@ class VolumeProtectionTests(test_base.CinderPolicyTests): body=body) self.assertEqual(HTTPStatus.ACCEPTED, response.status_int) + # Succeeds for a user call because there are no attachments body = {"os-detach": {}} response = self._get_request_response(user_context, path, 'POST', body=body) @@ -1062,6 +1067,7 @@ class VolumeProtectionTests(test_base.CinderPolicyTests): 'terminate_connection') def test_admin_can_initialize_terminate_conn(self, mock_t, mock_i): admin_context = self.admin_context + admin_context.service_roles = ['service'] volume = self._create_fake_volume(admin_context) path = '/v3/%(project_id)s/volumes/%(volume_id)s/action' % { @@ -1084,6 +1090,7 @@ class VolumeProtectionTests(test_base.CinderPolicyTests): 'terminate_connection') def test_owner_can_initialize_terminate_conn(self, mock_t, mock_i): user_context = self.user_context + user_context.service_roles = ['service'] volume = self._create_fake_volume(user_context) path = '/v3/%(project_id)s/volumes/%(volume_id)s/action' % { |