Add a validation for conversion hosts

app/models/service_template_transformation_plan_request.rb

@@ -15,6 +15,16 @@ def source_vms
     vm_resources.where(:status => [ServiceResource::STATUS_QUEUED, ServiceResource::STATUS_FAILED]).pluck(:resource_id)
   end
 
+  def validate_conversion_hosts
+    transformation_mapping.transformation_mapping_items.each do |item|
+      next unless %w(EmsCluster CloudTenant).include?(item.source_type)
+      ems = item.destination_type.constantize.find(item.destination_id).ext_management_system
+      conversion_hosts = ConversionHost.all.select { |ch| ch.ext_management_system == ems }
+      return false if conversion_hosts.empty?
+    end
+    true
+  end
+
   def validate_vm(_vm_id)
     # TODO: enhance the logic to determine whether this VM can be included in this request
     true

spec/models/service_template_transformation_plan_request_spec.rb

@@ -5,6 +5,32 @@
       ServiceResource.new(:resource => vm, :status => status)
     end
   end
+
+  let(:dst_ems) { FactoryGirl.create(:ext_management_system) }
+  let(:src_cluster) { FactoryGirl.create(:ems_cluster) }
+  let(:dst_cluster) { FactoryGirl.create(:ems_cluster, :ext_management_system => dst_ems) }
+
+  let(:mapping) do
+    FactoryGirl.create(
+      :transformation_mapping,
+      :transformation_mapping_items => [TransformationMappingItem.new(:source => src_cluster, :destination => dst_cluster)]
+    )
+  end
+
+  let(:catalog_item_options) do
+    {
+      :name        => 'Transformation Plan',
+      :description => 'a description',
+      :config_info => {
+        :transformation_mapping_id => mapping.id,
+        :actions                   => [
+          {:vm_id => vms.first.id.to_s, :pre_service => false, :post_service => false},
+          {:vm_id => vms.last.id.to_s, :pre_service => false, :post_service => false},
+        ],
+      }
+    }
+  end
+
   let(:plan) { FactoryGirl.create(:service_template_transformation_plan, :service_resources => vm_requests) }
   let(:request) { FactoryGirl.create(:service_template_transformation_plan_request, :source => plan) }
 
@@ -28,6 +54,25 @@
     end
   end
 
+  describe '#validate_conversion_hosts' do
+    let(:plan) { ServiceTemplateTransformationPlan.create_catalog_item(catalog_item_options) }
+    let(:request) { FactoryGirl.create(:service_template_transformation_plan_request, :source => plan) }
+
+    context 'no conversion host exists in EMS' do
+      let(:host) { FactoryGirl.create(:host, :ext_management_system => FactoryGirl.create(:ext_management_system, :zone => FactoryGirl.create(:zone))) }
+      let(:conversion_host) { FactoryGirl.create(:conversion_host, :resource => host) }
+
+      it { expect(request.validate_conversion_hosts).to eq(false) }
+    end
+
+    context 'conversion host exists in EMS' do
+      let(:host) { FactoryGirl.create(:host, :ext_management_system => dst_ems) }
+      let(:conversion_host) { FactoryGirl.create(:conversion_host, :resource => host) }
+
+      it { expect(request.validate_conversion_hosts).to eq(false) }
+    end
+  end
+
   describe '#validate_vm' do
     it { expect(request.validate_vm(vms[0].id)).to be_truthy }
   end