| commit | 087c010776a3f3a98d7d95c95d17645fa9d68fb7 | [log] [tgz] |
|---|---|---|
| author | Felipe Monteiro <felipe.monteiro@att.com> | Sat Dec 09 04:55:32 2017 +0000 |
| committer | Felipe Monteiro <felipe.monteiro@att.com> | Sat Dec 09 04:55:32 2017 +0000 |
| tree | d02aba0f89762512f23c59056a9072cd15b6d443 | |
| parent | 5718d26c32e6cdb2f2670ec5628fc3c79cd05140 [diff] |
Complete coverage for volume transfers policies This PS adds the last endpoint which enforces "volume:get_all_transfers": '/os-volume-transfer/detail' [0]. [0] https://github.com/openstack/cinder/blob/4ca4f644008f4c89edc472cacda19050367ba54a/cinder/policies/volume_transfer.py#L40 Change-Id: I82eb09c719583572573a490162383d15e78235c3
diff --git a/patrole_tempest_plugin/tests/api/volume/test_volume_transfers_rbac.py b/patrole_tempest_plugin/tests/api/volume/test_volume_transfers_rbac.py index 9640dc6..a33ebe0 100644 --- a/patrole_tempest_plugin/tests/api/volume/test_volume_transfers_rbac.py +++ b/patrole_tempest_plugin/tests/api/volume/test_volume_transfers_rbac.py
@@ -72,6 +72,13 @@ self.rbac_utils.switch_role(self, toggle_rbac_role=True) self.transfers_client.list_volume_transfers() + @decorators.idempotent_id('e84e45b0-9872-40bf-bf44-971266161a86') + @rbac_rule_validation.action(service="cinder", + rule="volume:get_all_transfers") + def test_list_volume_transfers_details(self): + with self.rbac_utils.override_role(self): + self.transfers_client.list_volume_transfers(detail=True) + @rbac_rule_validation.action(service="cinder", rule="volume:accept_transfer") @decorators.idempotent_id('987f2a11-d657-4984-a6c9-28f06c1cd014')