discard inactive VMs from the master list right off the bat

That being done, we don't need the `vm.active?` check in `vm_migration_status` again

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

app/models/transformation_mapping/vm_migration_validator.rb

@@ -47,7 +47,12 @@ def identify_vms
         next
       end
 
-      vms = vm_objects.includes(:lans, :storages, :host, :ext_management_system).select { |vm| mapped_clusters.include?(vm.ems_cluster) }
+      if vm_objects.select { |vm| vm.name == vm_name && !vm.active? }.any?
+        invalid_list << VmMigrateStruct.new(vm_name, nil, VM_INVALID, VM_INACTIVE)
+        next
+      end
+
+      vms = vm_objects.select { |vm| mapped_clusters.include?(vm.ems_cluster) }
       vms = vms.select { |vm| vm.name == vm_name }
       vms = vms.select { |vm| vm.uid_ems == row['uid_ems'] } if row['uid_ems'].present?
       vms = vms.select { |vm| vm.host.name == row['host'] } if row['host'].present?
@@ -90,8 +95,6 @@ def validate_vm(vm, quick = true)
   end
 
   def vm_migration_status(vm)
-    return VM_INACTIVE unless vm.active?
-
     vm_as_resources = ServiceResource.joins(:service_template).where(:resource => vm, :service_templates => {:type => 'ServiceTemplateTransformationPlan'})
 
     # VM has not been migrated before