Merge pull request #20330 from agrare/fix_ems_foreman_specs

Fix foreman provider missing url in specs

spec/factories/ext_management_system.rb

@@ -127,7 +127,7 @@
           :aliases => ["manageiq/providers/configuration_manager"],
           :class   => "ManageIQ::Providers::Foreman::ConfigurationManager",
           :parent  => :ext_management_system do
-    provider :factory => :provider
+    provider :factory => :provider_foreman
   end
 
   # Automation managers
@@ -141,7 +141,7 @@
           :aliases => ["manageiq/providers/provisioning_manager"],
           :class   => "ManageIQ::Providers::Foreman::ProvisioningManager",
           :parent  => :ext_management_system do
-    provider :factory => :provider
+    provider :factory => :provider_foreman
   end
 
   # Leaf classes for ems_infra

spec/models/ext_management_system_spec.rb

@@ -140,9 +140,10 @@
     expect { ManageIQ::Providers::AutomationManager.new(:hostname => "abc", :name => "abc", :zone => FactoryBot.build(:zone)).validate! }.to raise_error(ActiveRecord::RecordInvalid)
     expect(ManageIQ::Providers::Vmware::InfraManager.new(:hostname => "abc", :name => "abc", :zone => FactoryBot.build(:zone)).validate!).to eq(true)
 
-    foreman_provider = ManageIQ::Providers::Foreman::Provider.new
-    expect(ManageIQ::Providers::Foreman::ConfigurationManager.new(:provider => foreman_provider, :hostname => "abc", :name => "abc", :zone => FactoryBot.build(:zone)).validate!).to eq(true)
-    expect(ManageIQ::Providers::Foreman::ProvisioningManager.new(:provider => foreman_provider, :hostname => "abc", :name => "abc", :zone => FactoryBot.build(:zone)).validate!).to eq(true)
+    zone = FactoryBot.create(:zone)
+    foreman_provider = ManageIQ::Providers::Foreman::Provider.new(:name => "abc", :zone => zone, :url => "https://abc")
+    expect(ManageIQ::Providers::Foreman::ConfigurationManager.new(:provider => foreman_provider, :name => "abc", :zone => zone).validate!).to eq(true)
+    expect(ManageIQ::Providers::Foreman::ProvisioningManager.new(:provider => foreman_provider, :name => "abc", :zone => zone).validate!).to eq(true)
   end
 
   context "#ipaddress / #ipaddress=" do