Merge pull request #17465 from lgalis/add_option_to_clear_classificat…

…ions

Add option to clear classifications for tag_details
(cherry picked from commit e5adfd2)

https://bugzilla.redhat.com/show_bug.cgi?id=1591495

lib/miq_expression.rb

@@ -899,6 +899,9 @@ def self._model_details(relats, opts)
 
   def self.tag_details(path, opts)
     result = []
+    if opts[:no_cache]
+      @classifications = nil
+    end
     @classifications ||= categories
     @classifications.each do |name, cat|
       prefix = path.nil? ? "managed" : [path, "managed"].join(".")

spec/lib/miq_expression_spec.rb

@@ -3356,6 +3356,23 @@
       actual = described_class.tag_details(nil, {})
       expect(actual).to eq([["My Company Tags : Environment", "managed-env"]])
     end
+
+    it "returns the added classification when no_cache option is used" do
+      Tenant.seed
+      FactoryGirl.create(:classification,
+                         :name        => "first_classification",
+                         :description => "First Classification",
+                         :children    => [FactoryGirl.create(:classification)])
+      actual = described_class.tag_details(nil, {})
+      expect(actual).to eq([["My Company Tags : First Classification", "managed-first_classification"]])
+
+      FactoryGirl.create(:classification,
+                         :name        => "second_classification",
+                         :description => "Second Classification",
+                         :children    => [FactoryGirl.create(:classification)])
+      actual = described_class.tag_details(nil, :no_cache => true)
+      expect(actual).to eq([["My Company Tags : First Classification", "managed-first_classification"], ["My Company Tags : Second Classification", "managed-second_classification"]])
+    end
   end
 
   describe "miq_adv_search_lists" do