Merge pull request #16444 from yrudman/name-of-refresh-task-should-fi…

…t-255

Truncate name of refresh task to 255

app/models/ems_refresh.rb

@@ -200,7 +200,7 @@ def self.queue_merge(targets, ems, create_task = false)
 
   def self.create_refresh_task(ems, targets)
     task_options = {
-      :action => "EmsRefresh(#{ems.name}) [#{targets}]",
+      :action => "EmsRefresh(#{ems.name}) [#{targets}]".truncate(255),
       :userid => "system"
     }
 

spec/models/ems_refresh_spec.rb

@@ -89,6 +89,16 @@
         expect(task_ids.length).to eq(2)
       end
     end
+
+    describe ".create_refresh_task" do
+      it "create refresh task and trancates task name to 255 symbols" do
+        vm = FactoryGirl.create(:vm_vmware, :name => "vm_vmware1", :ext_management_system => @ems)
+        targets = Array.new(500) { vm }
+        task_name = described_class.send(:create_refresh_task, @ems, targets).name
+        expect(task_name.include?(@ems.name)).to eq true
+        expect(task_name.length).to eq 255
+      end
+    end
   end
 
   def queue_refresh_and_assert_queue_item(target, expected_targets)