Merge pull request ManageIQ#4439 from mzazrivec/fix_ot_name

Correctly render orchestration template's name

app/controllers/orchestration_stack_controller.rb

@@ -20,7 +20,7 @@ def self.display_methods
   end
 
   def display_stack_orchestration_template
-    drop_breadcrumb(:name => "%{name} (Orchestration Template)" % {:name => @record.name},
+    drop_breadcrumb(:name => "%{name} (Orchestration Template)" % {:name => @record.orchestration_template.name},
                     :url  => show_link(@record, :display => @display))
   end
 

spec/controllers/orchestration_stack_controller_spec.rb

@@ -49,7 +49,8 @@
     end
 
     context "orchestration templates" do
-      let(:record) { FactoryGirl.create(:orchestration_stack_cloud_with_template) }
+      let(:ems) { FactoryGirl.create(:ems_cloud) }
+      let(:record) { FactoryGirl.create(:orchestration_stack_cloud_with_template, :ext_management_system => ems, :name => 'stack01') }
 
       before do
         session[:settings] = {
@@ -67,6 +68,12 @@
         expect(response.status).to eq(200)
         expect(response).to render_template(:partial => "orchestration_stack/_stack_orchestration_template")
       end
+
+      it "renders template name correctly" do
+        expect(response.status).to eq(200)
+        expect(response.body).to include("<h1>\ntemplate name")
+        expect(response.body).not_to include("<h1>\nstack01")
+      end
     end
   end