Call tag_details with the :no_cache option. Updated spec to test for it.

ManageIQ · May 31, 2018 · 3b6f0bb · 3b6f0bb
1 parent fbd89d2
commit 3b6f0bb
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/app/views/layouts/exp_atom/_edit_tags.html.haml b/app/views/layouts/exp_atom/_edit_tags.html.haml
@@ -5,7 +5,7 @@
     exp_model       Model in use for this expression
 
 - opts = ["<#{_('Choose')}>"]
-- opts += MiqExpression.tag_details(nil, {})
+- opts += MiqExpression.tag_details(nil, {:no_cache => true})
 
 .spacer
 

diff --git a/spec/controllers/ops_controller/ops_rbac_spec.rb b/spec/controllers/ops_controller/ops_rbac_spec.rb
@@ -379,7 +379,7 @@
       expect(@edit[:filter_expression][:exp_typ]).to eq('tags')
     end
 
-    it "calls MiqExpression.tag_details to get only the My Company type tag categories" do
+    it "calls MiqExpression.tag_details with :no_cache to get only the My Company type tag categories" do
       new = {:use_filter_expression => true,
              :name                  => 'Name',
              :description           => "Test",
@@ -408,7 +408,7 @@
       allow(controller).to receive(:replace_right_cell)
 
       post :tree_select, :params => { :id => 'root', :format => :js }
-      expect(MiqExpression).to receive(:tag_details)
+      expect(MiqExpression).to receive(:tag_details).with(nil, :no_cache => true)
       post :rbac_group_field_changed, :params => { :id => 'new', :use_filter_expression => "true"}
     end