Merge pull request #5 from lgalis/manager_name_not_updated_on_provide…

…r_edit

Manager name not updated on foreman provider edit

app/models/manageiq/providers/foreman/provider.rb

@@ -20,7 +20,7 @@ class ManageIQ::Providers::Foreman::Provider < ::Provider
 
   delegate :api_cached?, :ensure_api_cached, :to => :connect
 
-  before_create :ensure_managers
+  before_validation :ensure_managers
 
   validates :name, :presence => true, :uniqueness => true
   validates :url,  :presence => true

spec/models/manageiq/providers/foreman/provider_spec.rb

@@ -58,4 +58,15 @@
       expect(CustomizationScript.count).to   eq(0)
     end
   end
+
+  describe "#save" do
+    it "will update the name for the manager" do
+      provider = FactoryGirl.create(:provider_foreman, :zone => FactoryGirl.create(:zone), :name => 'Old Name')
+      expect(provider.configuration_manager.name).to eq('Old Name Configuration Manager')
+
+      provider.update(:name => 'New Name')
+      expect(provider.configuration_manager.name).to eq('New Name Configuration Manager')
+      expect(provider.provisioning_manager.name).to eq('New Name Provisioning Manager')
+    end
+  end
 end