diff --git a/app/models/manageiq/providers/redhat/infra_manager/vm_import.rb b/app/models/manageiq/providers/redhat/infra_manager/vm_import.rb index 84ff83016fa..739561e93ed 100644 --- a/app/models/manageiq/providers/redhat/infra_manager/vm_import.rb +++ b/app/models/manageiq/providers/redhat/infra_manager/vm_import.rb @@ -31,7 +31,7 @@ def import_vm(source_vm_id, target_params) end def validate_import_vm - api_version >= '4.0' + highest_supported_api_version && highest_supported_api_version >= '4' end private diff --git a/spec/models/manageiq/providers/redhat/infra_manager/vm_import_spec.rb b/spec/models/manageiq/providers/redhat/infra_manager/vm_import_spec.rb index 6ccf8754cba..b073ba4a94f 100644 --- a/spec/models/manageiq/providers/redhat/infra_manager/vm_import_spec.rb +++ b/spec/models/manageiq/providers/redhat/infra_manager/vm_import_spec.rb @@ -54,4 +54,18 @@ ) end end + + context 'checks version during validation' do + let(:ems) { FactoryGirl.create(:ems_redhat) } + + it 'validates successfully' do + allow(ems).to receive(:highest_supported_api_version).and_return('4') + expect(ems.validate_import_vm).to be_truthy + end + + it 'validates before connecting' do + allow(ems).to receive(:highest_supported_api_version).and_return(nil) + expect(ems.validate_import_vm).to be_falsey + end + end end