Merge pull request #16436 from pkliczewski/master

Refresh new target do not run post_refresh
ManageIQ · Nov 17, 2017 · 013c7b4 · 013c7b4
2 parents 9282ecd + 4284854
commit 013c7b4
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/app/models/ems_refresh.rb b/app/models/ems_refresh.rb
@@ -109,7 +109,7 @@ def self.refresh_new_target(ems_id, target_hash, target_class, target_find)
     end
 
     ems.refresher.refresh(get_target_objects(target))
-
+    target.post_create_actions_queue if target.respond_to?(:post_create_actions_queue)
     target
   end
 

diff --git a/spec/models/ems_refresh_spec.rb b/spec/models/ems_refresh_spec.rb
@@ -184,7 +184,10 @@ def assert_queue_item(expected_targets)
   end
 
   context '.refresh_new_target' do
-    let(:ems) { FactoryGirl.create(:ems_vmware) }
+    let(:ems) do
+      _, _, zone = EvmSpecHelper.create_guid_miq_server_zone
+      FactoryGirl.create(:ems_vmware, :zone => zone)
+    end
 
     context 'targeting a new vm' do
       let(:vm_hash) do
-Original file line number
+Diff line change
@@ Expand Up @@
         end
         ems.refresher.refresh(get_target_objects(target))
+        target.post_create_actions_queue if target.respond_to?(:post_create_actions_queue)
         target
       end
@@ Expand Down @@