From c6eb1e687fced5eb083919c52725c69e959af6e4 Mon Sep 17 00:00:00 2001
From: Martin Hradil <mhradil@redhat.com>
Date: Fri, 16 Nov 2018 14:08:10 +0000
Subject: [PATCH] Fix compliance travis - save records after changing
 compliances

ManageIQ/manageiq#17475 changed how last_compliance_* methods work,
now, the changes to .compliances need to be saved in the db for these methods to works
---
 spec/helpers/compliance_summary_helper_spec.rb | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/spec/helpers/compliance_summary_helper_spec.rb b/spec/helpers/compliance_summary_helper_spec.rb
index 097b7c49649..ea968fafe29 100644
--- a/spec/helpers/compliance_summary_helper_spec.rb
+++ b/spec/helpers/compliance_summary_helper_spec.rb
@@ -18,23 +18,25 @@
 
     it "#textual_compliance_status" do
       @record.compliances = [@compliance1]
+      @record.save
       date = @compliance1.timestamp
       expect(helper.textual_compliance_status).to eq(:label    => "Status",
                                                      :icon     => "pficon pficon-ok",
                                                      :value    => "Compliant as of #{time_ago_in_words(date.in_time_zone(Time.zone)).titleize} Ago",
                                                      :title    => "Show Details of Compliance Check on #{format_timezone(date)}",
                                                      :explorer => true,
-                                                     :link     => "/vm_infra/show?count=1&display=compliance_history")
+                                                     :link     => "/vm_infra/show/#{@record.id}?count=1&display=compliance_history")
     end
 
     it "#textual_compliance_history" do
       @record.compliances = [@compliance1, @compliance2]
+      @record.save
       expect(helper.textual_compliance_history).to eq(:label    => "History",
                                                       :icon     => "pficon pficon-history",
                                                       :value    => "Available",
                                                       :explorer => true,
                                                       :title    => "Show Compliance History of this VM or Template (Last 10 Checks)",
-                                                      :link     => "/vm_infra/show?display=compliance_history")
+                                                      :link     => "/vm_infra/show/#{@record.id}?display=compliance_history")
     end
   end
 
@@ -46,21 +48,23 @@
 
     it "#textual_compliance_status" do
       @record.compliances = [@compliance1]
+      @record.save
       date = @compliance1.timestamp
       expect(helper.textual_compliance_status).to eq(:label => "Status",
                                                      :icon  => "pficon pficon-ok",
                                                      :value => "Compliant as of #{time_ago_in_words(date.in_time_zone(Time.zone)).titleize} Ago",
                                                      :title => "Show Details of Compliance Check on #{format_timezone(date)}",
-                                                     :link  => "/host/show?count=1&display=compliance_history")
+                                                     :link  => "/host/show/#{@record.id}?count=1&display=compliance_history")
     end
 
     it "#textual_compliance_history" do
       @record.compliances = [@compliance1, @compliance2]
+      @record.save
       expect(helper.textual_compliance_history).to eq(:label => "History",
                                                       :icon  => "pficon pficon-history",
                                                       :value => "Available",
                                                       :title => "Show Compliance History of this Host / Node (Last 10 Checks)",
-                                                      :link  => "/host/show?display=compliance_history")
+                                                      :link  => "/host/show/#{@record.id}?display=compliance_history")
     end
   end
 end