| commit | 6da23e69300207a8ede4bf476f971557e597216f | [log] [tgz] |
|---|---|---|
| author | Felipe Monteiro <felipe.monteiro@att.com> | Sun Mar 18 05:22:01 2018 +0000 |
| committer | Felipe Monteiro <felipe.monteiro@att.com> | Tue Mar 20 03:22:01 2018 +0000 |
| tree | 29092831e304e7d6c2da32507dc5b96fe5a8eff3 | |
| parent | f918e08ce1380cd3d0dab87850cd6e1d4c9f1e52 [diff] |
Fix tests for snapshots extended_snapshot_attributes policy
This fixes the Patrole test for
"volume_extension:extended_snapshot_attributes" which is
incorrectly being executed by a create test which is wrong according
to [0]. Instead, the test should be executed for
GET /snapshots/{snapshot_id} and GET /snapshots/details. Further, the
test should use soft or non-fatal authorization, which was recently
fixed in Cinder [1].
Finally, the tests should live in test_volumes_snapshots_rbac
for the same reason the policy is listed under snapshots family
of policies in cinder [0].
The expected attributes for these tests can be found here: [2].
[0] https://github.com/openstack/cinder/blob/831665f59a0d87f2dbf7fa4f7496305977a98b0f/cinder/policies/snapshots.py#L48
[1] https://review.openstack.org/#/c/553657/1
[2] https://github.com/openstack/cinder/blob/831665f59a0d87f2dbf7fa4f7496305977a98b0f/cinder/api/contrib/extended_snapshot_attributes.py#L25
Change-Id: I7770148584e10a08b3e133b1793a1533f1728fc1