From 079f7d367655970e2ab5deca54f549e1d98d2842 Mon Sep 17 00:00:00 2001 From: Gregg Tanzillo Date: Fri, 11 Oct 2019 09:33:15 -0400 Subject: [PATCH] Merge pull request #19356 from jdeubel/add-cloud_volume_snapshot-to-rbac Adding Volume Snapshot (CloudVolumeSnapshot) to RBAC (cherry picked from commit 68a06a3d9ed7ba8f2a122450657e2e1636053d36) Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1773630 --- lib/rbac/filterer.rb | 1 + spec/lib/rbac/filterer_spec.rb | 18 ++++++++++++++++++ 2 files changed, 19 insertions(+) diff --git a/lib/rbac/filterer.rb b/lib/rbac/filterer.rb index 30b3aaf451d..b1635a88ae9 100644 --- a/lib/rbac/filterer.rb +++ b/lib/rbac/filterer.rb @@ -11,6 +11,7 @@ class Filterer CloudSubnet CloudTenant CloudVolume + CloudVolumeSnapshot ConfigurationProfile ConfigurationScriptBase ConfigurationScriptSource diff --git a/spec/lib/rbac/filterer_spec.rb b/spec/lib/rbac/filterer_spec.rb index 3168d31ef8a..59871ced470 100644 --- a/spec/lib/rbac/filterer_spec.rb +++ b/spec/lib/rbac/filterer_spec.rb @@ -310,6 +310,24 @@ def combine_filtered_ids(user_filtered_ids, belongsto_filtered_ids, managed_filt expect(results).to match_array [owner_tenant] end end + + context 'searching for instances of CloudVolumeSnapshot' do + let!(:csv) { FactoryBot.create_list(:cloud_volume_snapshot, 2).first } + + before do + csv.tag_with('/managed/environment/prod', :ns => '*') + end + + it 'lists only tagged CloudVolumeSnapshot' do + results = described_class.search(:class => CloudVolumeSnapshot, :user => user).first + expect(results).to match_array [csv] + end + + it 'lists only all CloudVolumeSnapshot' do + results = described_class.search(:class => CloudVolumeSnapshot, :user => admin_user).first + expect(results).to match_array CloudVolumeSnapshot.all + end + end end context 'with virtual custom attributes' do