Merge pull request #18570 from fdupont-redhat/v2v_throttling_use_migr…

…ate_state

[V2V] Fix ConversionHost active_tasks method to use state == 'migrate'

(cherry picked from commit 7407f63)

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

app/models/conversion_host.rb

@@ -5,7 +5,7 @@ class ConversionHost < ApplicationRecord
 
   belongs_to :resource, :polymorphic => true
   has_many :service_template_transformation_plan_tasks, :dependent => :nullify
-  has_many :active_tasks, -> { where(:state => 'active') }, :class_name => ServiceTemplateTransformationPlanTask, :inverse_of => :conversion_host
+  has_many :active_tasks, -> { where(:state => ['active', 'migrate']) }, :class_name => ServiceTemplateTransformationPlanTask, :inverse_of => :conversion_host
   delegate :ext_management_system, :hostname, :ems_ref, :to => :resource, :allow_nil => true
 
   include_concern 'Configurations'

spec/models/conversion_host_spec.rb

@@ -10,7 +10,7 @@
     let(:conversion_host_2) { FactoryGirl.create(:conversion_host, :resource => vm) }
     let(:task_1) { FactoryGirl.create(:service_template_transformation_plan_task, :state => 'active', :conversion_host => conversion_host_1) }
     let(:task_2) { FactoryGirl.create(:service_template_transformation_plan_task, :conversion_host => conversion_host_1) }
-    let(:task_3) { FactoryGirl.create(:service_template_transformation_plan_task, :state => 'active', :conversion_host => conversion_host_2) }
+    let(:task_3) { FactoryBot.create(:service_template_transformation_plan_task, :state => 'migrate', :conversion_host => conversion_host_2) }
 
     before do
       allow(conversion_host_1).to receive(:active_tasks).and_return([task_1])