Merge pull request #1029 from lgalis/fix_textual_provider_svg_and_con…

…troller_for_ansible_job

Fix the controller and svg in textual provider for Ansible Tower Jobs

app/helpers/configuration_job_helper/textual_summary.rb

@@ -43,14 +43,14 @@ def textual_service
   end
 
   def textual_provider
-    h = {:label => _("Provider"), :image => "svg/vendor-automation_manager_configuration.svg"}
+    h = {:label => _("Provider"), :image => "svg/vendor-ansible.svg"}
     provider = @record.ext_management_system
     if provider.nil?
       h[:value] = _("None")
     else
       h[:value] = provider.name
       h[:title] = _("Show this Parent Provider")
-      h[:link]  = url_for_only_path(:controller => 'provider_foreman', :action => 'explorer', :id => "at-#{to_cid(provider.id)}")
+      h[:link]  = url_for_only_path(:controller => 'automation_manager', :action => 'explorer', :id => "at-#{to_cid(provider.id)}")
     end
     h
   end

spec/helpers/configuration_job_helper/textual_summary_spec.rb

@@ -0,0 +1,14 @@
+describe ConfigurationJobHelper::TextualSummary do
+  include CompressedIds
+  include ApplicationHelper
+
+  let(:zone) { EvmSpecHelper.local_miq_server.zone }
+  let(:automation_provider) { FactoryGirl.create(:provider_ansible_tower, :name => "ansibletest", :url => "test", :zone => zone) }
+
+  it "#textual_provider" do
+    manager = ManageIQ::Providers::AnsibleTower::AutomationManager.find_by(:provider_id => automation_provider.id)
+    @record = FactoryGirl.create(:ansible_tower_job, :ext_management_system => manager)
+    expect(textual_provider[:image]).to eq("svg/vendor-ansible.svg")
+    expect(textual_provider[:link]).to eq("/automation_manager/explorer/at-#{to_cid(manager.id)}")
+  end
+end