Merge pull request ManageIQ#14338 from isimluk/check-charts-headers

Specs: check that counts match for headers and col_order for charts

spec/product/reports_spec.rb

@@ -1,10 +1,10 @@
 describe 'YAML reports' do
   let(:report_dirs) { [REPORTS_FOLDER, "#{TIMELINES_FOLDER}/miq_reports"] }
   let(:report_yamls) { report_dirs.collect { |dir| Dir.glob(File.join(dir, "**", "*.yaml")) }.flatten }
+  let(:chart_dirs) { [CHARTS_REPORTS_FOLDER] }
+  let(:chart_yamls) { chart_dirs.collect { |dir| Dir.glob(File.join(dir, "**", "*.yaml")) }.flatten }
   let!(:user) { FactoryGirl.create(:user_with_group) }
 
-  # TODO: CHARTS_REPORTS_FOLDER
-
   before :each do
     EvmSpecHelper.local_miq_server
     @user = FactoryGirl.create(:user_with_group)
@@ -15,6 +15,7 @@
   end
 
   it 'can be build even though without data' do
+    # TODO: CHARTS_REPORTS_FOLDER
     report_yamls.each do |yaml|
       report_data = YAML.load(File.open(yaml))
       report_data.delete('menu_name')
@@ -26,7 +27,7 @@
   end
 
   it 'defines headers that match col_order' do
-    report_yamls.each do |yaml|
+    (chart_yamls + report_yamls).each do |yaml|
       report_data = YAML.load(File.open(yaml))
       col_order = report_data['col_order'].length
       headers = report_data['headers'].length