metric_fu 4.11.3 → 4.11.4

data/spec/metric_fu/metrics/churn/configuration_spec.rb

@@ -1,15 +1,13 @@
-require 'spec_helper'
-require 'shared/configured'
-
-describe MetricFu::Configuration, 'for churn' do
-  it_behaves_like 'configured' do
+require "spec_helper"
+require "shared/configured"
 
+describe MetricFu::Configuration, "for churn" do
+  it_behaves_like "configured" do
     it "should set @churn to {}" do
-      load_metric 'churn'
+      load_metric "churn"
       expect(MetricFu::Metric.get_metric(:churn).run_options).to eq(
-              { :start_date => %q("1 year ago"), :minimum_churn_count => 10, :ignore_files=>[], :data_directory=> MetricFu::Io::FileSystem.scratch_directory('churn')}
+               start_date: '"1 year ago"', minimum_churn_count: 10, ignore_files: [], data_directory: MetricFu::Io::FileSystem.scratch_directory("churn")
       )
     end
-
   end
 end

data/spec/metric_fu/metrics/churn/generator_spec.rb

@@ -1,39 +1,36 @@
 require "spec_helper"
-MetricFu.metrics_require { 'churn/generator' }
+MetricFu.metrics_require { "churn/generator" }
 
 describe MetricFu::ChurnGenerator do
-
   # TODO extract yaml
   let(:churn_hash) { YAML::load("--- \n:churn: \n  :changed_files: \n  - spec/graphs/flog_grapher_spec.rb\n  - spec/base/graph_spec.rb\n  - lib/templates/awesome/layout.html.erb\n  - lib/graphs/rcov_grapher.rb\n  - lib/base/base_template.rb\n  - spec/graphs/grapher_spec.rb\n  - lib/templates/awesome/flog.html.erb\n  - lib/templates/awesome/flay.html.erb\n  - lib/graphs/roodi_grapher.rb\n  - lib/graphs/reek_grapher.rb\n  - HISTORY\n  - spec/graphs/roodi_grapher_spec.rb\n  - lib/generators/rcov.rb\n  - spec/graphs/engines/gchart_spec.rb\n  - spec/graphs/rcov_grapher_spec.rb\n  - lib/templates/javascripts/excanvas.js\n  - lib/templates/javascripts/bluff-min.js\n  - spec/graphs/reek_grapher_spec.rb\n") }
 
   let(:config_setup) {
-    ENV['CC_BUILD_ARTIFACTS'] = nil
+    ENV["CC_BUILD_ARTIFACTS"] = nil
     MetricFu.configure.reset
   }
 
   describe "analyze method" do
     before :each do
       config_setup
-      @changes = {"lib/generators/flog.rb" => 2, "lib/metric_fu.rb" => 3}
+      @changes = { "lib/generators/flog.rb" => 2, "lib/metric_fu.rb" => 3 }
     end
 
     it "should be empty on error no output captured" do
       churn = MetricFu::ChurnGenerator.new
       churn.instance_variable_set(:@output, nil)
       result = churn.analyze
-      expect(result).to eq({:churn => {}})
+      expect(result).to eq(churn: {})
     end
 
     it "should return yaml results" do
       churn = MetricFu::ChurnGenerator.new
       churn.instance_variable_set(:@output, churn_hash)
       result = churn.analyze
-      expect(result).to eq({:churn => {:changed_files => ["spec/graphs/flog_grapher_spec.rb", "spec/base/graph_spec.rb", "lib/templates/awesome/layout.html.erb", "lib/graphs/rcov_grapher.rb", "lib/base/base_template.rb", "spec/graphs/grapher_spec.rb", "lib/templates/awesome/flog.html.erb", "lib/templates/awesome/flay.html.erb", "lib/graphs/roodi_grapher.rb", "lib/graphs/reek_grapher.rb", "HISTORY", "spec/graphs/roodi_grapher_spec.rb", "lib/generators/rcov.rb", "spec/graphs/engines/gchart_spec.rb", "spec/graphs/rcov_grapher_spec.rb", "lib/templates/javascripts/excanvas.js", "lib/templates/javascripts/bluff-min.js", "spec/graphs/reek_grapher_spec.rb"]}})
+      expect(result).to eq(churn: { changed_files: ["spec/graphs/flog_grapher_spec.rb", "spec/base/graph_spec.rb", "lib/templates/awesome/layout.html.erb", "lib/graphs/rcov_grapher.rb", "lib/base/base_template.rb", "spec/graphs/grapher_spec.rb", "lib/templates/awesome/flog.html.erb", "lib/templates/awesome/flay.html.erb", "lib/graphs/roodi_grapher.rb", "lib/graphs/reek_grapher.rb", "HISTORY", "spec/graphs/roodi_grapher_spec.rb", "lib/generators/rcov.rb", "spec/graphs/engines/gchart_spec.rb", "spec/graphs/rcov_grapher_spec.rb", "lib/templates/javascripts/excanvas.js", "lib/templates/javascripts/bluff-min.js", "spec/graphs/reek_grapher_spec.rb"] })
     end
-
   end
 
-
   describe "to_h method" do
     before :each do
       config_setup
@@ -41,12 +38,11 @@ describe MetricFu::ChurnGenerator do
 
     it "should put the changes into a hash" do
       churn = MetricFu::ChurnGenerator.new
-      churn.instance_variable_set(:@churn, {:churn => 'results'})
+      churn.instance_variable_set(:@churn, churn: "results")
       expect(churn.to_h[:churn]).to eq("results")
     end
   end
 
-
   describe "emit method" do
     before :each do
       config_setup
@@ -66,4 +62,3 @@ describe MetricFu::ChurnGenerator do
     end
   end
 end
-

data/spec/metric_fu/metrics/flay/configuration_spec.rb

@@ -1,15 +1,13 @@
-require 'spec_helper'
-require 'shared/configured'
-
-describe MetricFu::Configuration, 'for flay' do
-  it_behaves_like 'configured' do
+require "spec_helper"
+require "shared/configured"
 
+describe MetricFu::Configuration, "for flay" do
+  it_behaves_like "configured" do
     it "should set @flay to {:dirs_to_flay => @code_dirs}" do
-      load_metric 'flay'
+      load_metric "flay"
       expect(MetricFu::Metric.get_metric(:flay).run_options).to eq(
-              {:dirs_to_flay => ['lib'], :minimum_score=>nil}
+              dirs_to_flay: ["lib"], minimum_score: nil
       )
     end
-
   end
 end

data/spec/metric_fu/metrics/flay/generator_spec.rb

@@ -1,10 +1,10 @@
 require "spec_helper"
-MetricFu.metrics_require { 'flay/generator' }
+MetricFu.metrics_require { "flay/generator" }
 
 describe MetricFu::FlayGenerator do
   describe "emit method" do
     it "should look at the dirs" do
-      options = { :dirs_to_flay => ['app', 'lib'] }
+      options = { dirs_to_flay: ["app", "lib"] }
       allow(File).to receive(:directory?).and_return(true)
       @flay = MetricFu::FlayGenerator.new(options)
 
@@ -13,7 +13,7 @@ describe MetricFu::FlayGenerator do
     end
 
     it "should limit flay scores by the minimum_score" do
-      options = { :dirs_to_flay => [], :minimum_score => 99 }
+      options = { dirs_to_flay: [], minimum_score: 99 }
       allow(File).to receive(:directory?).and_return(true)
       @flay = MetricFu::FlayGenerator.new(options)
 
@@ -40,57 +40,56 @@ Total score (lower is better) = 246
       HERE
       MetricFu::Configuration.run {}
       allow(File).to receive(:directory?).and_return(true)
-      @flay = MetricFu::FlayGenerator.new('base_dir')
+      @flay = MetricFu::FlayGenerator.new("base_dir")
       @flay.instance_variable_set(:@output, lines)
     end
 
     it "should analyze and return matches" do
-      expect(@flay.analyze).to eq([ ["Total score (lower is better) = 246"],
-                                ["\n1) IDENTICAL code found in :or (mass*2 = 68)",
-                                  "app/controllers/link_targets_controller.rb:57",
-                                  "app/controllers/primary_sites_controller.rb:138"],
-                                ["2) Similar code found in :if (mass = 64)",
-                                  "app/controllers/primary_sites_controller.rb:75",
-                                  "app/controllers/primary_sites_controller.rb:76",
-                                  "app/controllers/primary_sites_controller.rb:88",
-                                  "app/controllers/primary_sites_controller.rb:89"] ])
+      expect(@flay.analyze).to eq([["Total score (lower is better) = 246"],
+                                   ["\n1) IDENTICAL code found in :or (mass*2 = 68)",
+                                    "app/controllers/link_targets_controller.rb:57",
+                                    "app/controllers/primary_sites_controller.rb:138"],
+                                   ["2) Similar code found in :if (mass = 64)",
+                                    "app/controllers/primary_sites_controller.rb:75",
+                                    "app/controllers/primary_sites_controller.rb:76",
+                                    "app/controllers/primary_sites_controller.rb:88",
+                                    "app/controllers/primary_sites_controller.rb:89"]])
     end
   end
 
   describe "to_h method" do
-
     before :each do
-      lines = [ ["Total score (lower is better) = 284"],
-                  ["\n1) IDENTICAL code found in :or (mass*2 = 68)",
-                    "app/controllers/link_targets_controller.rb:57",
-                    "app/controllers/primary_sites_controller.rb:138"],
-                  ["2) Similar code found in :if (mass = 64)",
-                    "app/controllers/primary_sites_controller.rb:75",
-                    "app/controllers/primary_sites_controller.rb:76",
-                    "app/controllers/primary_sites_controller.rb:88",
-                    "app/controllers/primary_sites_controller.rb:89"],
-                  ["3) Similar code found in :defn (mass = 40)",
-                    "app/controllers/link_targets_controller.rb:40",
-                    "app/controllers/primary_sites_controller.rb:98"],
-                  ["4) Similar code found in :defn (mass = 38)",
-                    "app/controllers/link_targets_controller.rb:13",
-                    "app/controllers/primary_sites_controller.rb:50"],
-                  ["5) Similar code found in :defn (mass = 38)",
-                    "app/models/primary_site.rb:104",
-                    "app/models/primary_site.rb:109"],
-                  ["6) Similar code found in :call (mass = 36)",
-                    "app/controllers/bookmarklet_integration_controller.rb:6",
-                    "app/controllers/bookmarklet_integration_controller.rb:17"]]
+      lines = [["Total score (lower is better) = 284"],
+               ["\n1) IDENTICAL code found in :or (mass*2 = 68)",
+                "app/controllers/link_targets_controller.rb:57",
+                "app/controllers/primary_sites_controller.rb:138"],
+               ["2) Similar code found in :if (mass = 64)",
+                "app/controllers/primary_sites_controller.rb:75",
+                "app/controllers/primary_sites_controller.rb:76",
+                "app/controllers/primary_sites_controller.rb:88",
+                "app/controllers/primary_sites_controller.rb:89"],
+               ["3) Similar code found in :defn (mass = 40)",
+                "app/controllers/link_targets_controller.rb:40",
+                "app/controllers/primary_sites_controller.rb:98"],
+               ["4) Similar code found in :defn (mass = 38)",
+                "app/controllers/link_targets_controller.rb:13",
+                "app/controllers/primary_sites_controller.rb:50"],
+               ["5) Similar code found in :defn (mass = 38)",
+                "app/models/primary_site.rb:104",
+                "app/models/primary_site.rb:109"],
+               ["6) Similar code found in :call (mass = 36)",
+                "app/controllers/bookmarklet_integration_controller.rb:6",
+                "app/controllers/bookmarklet_integration_controller.rb:17"]]
 
       MetricFu::Configuration.run {}
       allow(File).to receive(:directory?).and_return(true)
-      flay = MetricFu::FlayGenerator.new('base_dir')
+      flay = MetricFu::FlayGenerator.new("base_dir")
       flay.instance_variable_set(:@matches, lines)
       @results = flay.to_h
     end
 
     it "should find the total_score" do
-      expect(@results[:flay][:total_score]).to eq('284')
+      expect(@results[:flay][:total_score]).to eq("284")
     end
 
     it "should have 6 matches" do

data/spec/metric_fu/metrics/flay/grapher_spec.rb

@@ -1,5 +1,5 @@
 require "spec_helper"
-MetricFu.metrics_require { 'flay/grapher' }
+MetricFu.metrics_require { "flay/grapher" }
 
 describe FlayGrapher do
   before :each do
@@ -49,7 +49,7 @@ describe FlayGrapher do
       end
 
       it "should update labels with the date" do
-        expect(@flay_grapher.labels).to receive(:update).with({ 0 => "1/2" })
+        expect(@flay_grapher.labels).to receive(:update).with(0 => "1/2")
         @flay_grapher.get_metrics(@metrics, @date)
       end
     end

data/spec/metric_fu/metrics/flog/configuration_spec.rb

@@ -1,20 +1,18 @@
-require 'spec_helper'
-require 'shared/configured'
-
-describe MetricFu::Configuration, 'for flog' do
-  it_behaves_like 'configured' do
+require "spec_helper"
+require "shared/configured"
 
+describe MetricFu::Configuration, "for flog" do
+  it_behaves_like "configured" do
     if MetricFu.configuration.mri?
       it "should set @flog to {:dirs_to_flog => @code_dirs}" do
-        load_metric 'flog'
-        expect(MetricFu::Metric.get_metric(:flog).run_options).to eq({
-          :all => true,
-         :continue => true,
-         :dirs_to_flog => ["lib"],
-         :quiet => true
-         })
+        load_metric "flog"
+        expect(MetricFu::Metric.get_metric(:flog).run_options).to eq(
+          all: true,
+          continue: true,
+          dirs_to_flog: ["lib"],
+          quiet: true
+         )
       end
     end
-
   end
 end

data/spec/metric_fu/metrics/flog/generator_spec.rb

@@ -1,8 +1,7 @@
 require "spec_helper"
-MetricFu.metrics_require { 'flog/generator' }
+MetricFu.metrics_require { "flog/generator" }
 
 describe MetricFu::FlogGenerator do
-
   break if metric_not_activated?(:flog)
 
   before :each do
@@ -13,8 +12,8 @@ describe MetricFu::FlogGenerator do
 
   describe "emit method" do
     it "should look for files and flog them" do
-      expect(FlogCLI).to receive(:parse_options).with(["--all","--continue"]).and_return("options")
-      expect(FlogCLI).to receive(:new).with("options").and_return(flogger = double('flogger'))
+      expect(FlogCLI).to receive(:parse_options).with(["--all", "--continue"]).and_return("options")
+      expect(FlogCLI).to receive(:new).with("options").and_return(flogger = double("flogger"))
       expect(flogger).to receive(:flog).with("lib")
       @flog.emit
     end
@@ -25,34 +24,34 @@ describe MetricFu::FlogGenerator do
       first_full_method_name = "ClassName#first_method_name"
       second_full_method_name = "ClassName#second_method_name"
 
-      flogger = double('flogger', :method_locations => {first_full_method_name => '/file/location.rb:11',
-                                                      second_full_method_name => '/file/location.rb:22'},
-                                :totals => {first_full_method_name => 11.11,
-                                            second_full_method_name => 22.22})
+      flogger = double("flogger", method_locations: { first_full_method_name => "/file/location.rb:11",
+                                                      second_full_method_name => "/file/location.rb:22" },
+                                  totals: { first_full_method_name => 11.11,
+                                            second_full_method_name => 22.22 })
       expect(flogger).to receive(:calculate)
       expect(flogger).to receive(:each_by_score).and_yield(
-        first_full_method_name, 11.11, {:branch => 11.1, :puts => 1.1}
+        first_full_method_name, 11.11, branch: 11.1, puts: 1.1
       ).and_yield(
-        second_full_method_name, 22.22, {:branch => 22.2, :puts => 2.2}
+        second_full_method_name, 22.22, branch: 22.2, puts: 2.2
       )
       @flog.instance_variable_set(:@flogger, flogger)
       @flog.analyze
       method_containers = @flog.instance_variable_get(:@method_containers)
       expect(method_containers.size).to eq(1)
 
-      expected={:methods=>{"ClassName#first_method_name" => { :path=>"/file/location.rb:11",
-                                                              :score=>11.11,
-                                                              :operators=>{ :branch=>11.1,
-                                                                            :puts=>1.1}},
-                           "ClassName#second_method_name" => {:path=>"/file/location.rb:22",
-                                                              :score=>22.22,
-                                                              :operators=>{ :branch=>22.2,
-                                                                            :puts=>2.2}}},
-                :path=>"/file/location.rb",
-                :average_score=>((11.11 + 22.22) / 2.0),
-                :total_score=>33.33,
-                :highest_score=>22.22,
-                :name=>"ClassName"}
+      expected = { methods: { "ClassName#first_method_name" => { path: "/file/location.rb:11",
+                                                                 score: 11.11,
+                                                                 operators: { branch: 11.1,
+                                                                              puts: 1.1 } },
+                              "ClassName#second_method_name" => { path: "/file/location.rb:22",
+                                                                  score: 22.22,
+                                                                  operators: { branch: 22.2,
+                                                                               puts: 2.2 } } },
+                   path: "/file/location.rb",
+                   average_score: ((11.11 + 22.22) / 2.0),
+                   total_score: 33.33,
+                   highest_score: 22.22,
+                   name: "ClassName" }
 
       expect(method_containers["ClassName"].to_h).to eq(expected)
     end
@@ -60,19 +59,18 @@ describe MetricFu::FlogGenerator do
 
   describe "to_h method" do
     it "should make-a nice hash" do
-      flogger = double('flogger', :total_score => 111.1, :average => 7.3)
+      flogger = double("flogger", total_score: 111.1, average: 7.3)
       @flog.instance_variable_set(:@flogger, flogger)
-      method_containers = {:ignore_me_1 =>  double('container_1', :highest_score => 11.1, :to_h => 'container_1'),
-                           :ignore_me_2 =>  double('container_2', :highest_score => 33.3, :to_h => 'container_2'),
-                           :ignore_me_3 =>  double('container_3', :highest_score => 22.2, :to_h => 'container_3')}
+      method_containers = { ignore_me_1: double("container_1", highest_score: 11.1, to_h: "container_1"),
+                            ignore_me_2: double("container_2", highest_score: 33.3, to_h: "container_2"),
+                            ignore_me_3: double("container_3", highest_score: 22.2, to_h: "container_3") }
       @flog.instance_variable_set(:@method_containers, method_containers)
 
-      expected = {:flog => { :total => 111.1,
-                  :average => 7.3,
-                  :method_containers => ['container_2', 'container_3', 'container_1']}}
+      expected = { flog: { total: 111.1,
+                           average: 7.3,
+                           method_containers: ["container_2", "container_3", "container_1"] } }
 
       expect(@flog.to_h).to eq(expected)
     end
-
   end
 end

data/spec/metric_fu/metrics/flog/grapher_spec.rb

@@ -1,11 +1,10 @@
 require "spec_helper"
-MetricFu.metrics_require { 'flog/grapher' }
+MetricFu.metrics_require { "flog/grapher" }
 
 describe MetricFu::FlogGrapher do
   before :each do
     MetricFu.configuration
     @flog_grapher = MetricFu::FlogGrapher.new
-
   end
 
   it "should respond to flog_total, flog_average and labels" do
@@ -26,12 +25,12 @@ describe MetricFu::FlogGrapher do
     before(:each) do
       methods = {}
       100.times do |i|
-        methods["method_name_#{i}"] = {:score => i.to_f}
+        methods["method_name_#{i}"] = { score: i.to_f }
       end
 
-      @metrics = {:flog => {:total => 111.1,
-                            :average => 7.7,
-                            :method_containers => [ {:methods => methods } ] } }
+      @metrics = { flog: { total: 111.1,
+                           average: 7.7,
+                           method_containers: [{ methods: methods }] } }
       @date = "1/2"
     end
 
@@ -48,7 +47,7 @@ describe MetricFu::FlogGrapher do
 
     context "when metrics were not generated" do
       before(:each) do
-        @metrics = FIXTURE.load_metric('metric_missing.yml')
+        @metrics = FIXTURE.load_metric("metric_missing.yml")
         @date = "1/2"
       end
 
@@ -70,7 +69,7 @@ describe MetricFu::FlogGrapher do
 
     context "when metrics have been generated" do
       before(:each) do
-        @metrics = FIXTURE.load_metric('20090630.yml')
+        @metrics = FIXTURE.load_metric("20090630.yml")
         @date = "1/2"
       end
 
@@ -86,7 +85,7 @@ describe MetricFu::FlogGrapher do
       end
 
       it "should update labels with the date" do
-        expect(@flog_grapher.labels).to receive(:update).with({ 0 => "1/2" })
+        expect(@flog_grapher.labels).to receive(:update).with(0 => "1/2")
         @flog_grapher.get_metrics(@metrics, @date)
       end
     end
@@ -105,5 +104,4 @@ describe MetricFu::FlogGrapher do
       @flog_grapher.get_metrics(@metrics, @date)
     end
   end
-
 end

data/spec/metric_fu/metrics/hotspots/analysis/analyzed_problems_spec.rb

@@ -1,8 +1,7 @@
-require 'spec_helper'
-MetricFu.metrics_require { 'hotspots/analysis/analyzed_problems' }
+require "spec_helper"
+MetricFu.metrics_require { "hotspots/analysis/analyzed_problems" }
 
 describe MetricFu::HotspotAnalyzedProblems do
-
   before do
     enable_hotspots
   end
@@ -13,7 +12,7 @@ describe MetricFu::HotspotAnalyzedProblems do
       common_columns = %w{metric}
       granularities =  %w{file_path class_name method_name}
       tool_analyzers = MetricFu::Hotspot.analyzers
-      analyzer_columns = common_columns + granularities + tool_analyzers.map{|analyzer| analyzer.columns}.flatten
+      analyzer_columns = common_columns + granularities + tool_analyzers.map(&:columns).flatten
 
       analyzer_tables = MetricFu::AnalyzerTables.new(analyzer_columns)
       tool_analyzers.each do |analyzer|
@@ -29,7 +28,6 @@ describe MetricFu::HotspotAnalyzedProblems do
   end
 
   context "with several types of data" do
-
     before do
       @result_hash = HOTSPOT_DATA["several_metrics.yml"]
       @analyzed_problems = analyzed_problems(@result_hash)
@@ -38,8 +36,8 @@ describe MetricFu::HotspotAnalyzedProblems do
 
     it "gives all issues for a class" do
       expected = {
-        :reek => "found 2 code smells",
-        :flog => "complexity is 37.9"
+        reek: "found 2 code smells",
+        flog: "complexity is 37.9"
       }
       # TODO Unsure if we want to make problems_with and location public or private at this point
       expect(@worst_items[:classes].first.problems).to eq(expected)
@@ -47,44 +45,42 @@ describe MetricFu::HotspotAnalyzedProblems do
 
     it "gives all issues for a method" do
       expected = {
-        :reek => "found 1 code smells",
-        :flog => "complexity is 37.9"}
+        reek: "found 1 code smells",
+        flog: "complexity is 37.9" }
       expect(@worst_items[:methods].first.problems).to eq(expected)
     end
 
     it "gives all issues for a file" do
       expected = {
-        :reek => "found 2 code smells" ,
-        :flog => "complexity is 37.9",
-        :churn => "detected high level of churn (changed 54 times)"}
+        reek: "found 2 code smells",
+        flog: "complexity is 37.9",
+        churn: "detected high level of churn (changed 54 times)" }
       expect(@worst_items[:files].first.problems).to eq(expected)
     end
 
     it "provide location for a method" do
       expected = MetricFu::Location.new("lib/client/client.rb",
-                              "Client",
-                              "Client#client_requested_sync")
+                                        "Client",
+                                        "Client#client_requested_sync")
       expect(@worst_items[:methods].first.location).to eq(expected)
     end
 
     it "provides location for a class" do
       expected = MetricFu::Location.new("lib/client/client.rb",
-                              "Client",
-                              nil)
+                                        "Client",
+                                        nil)
       expect(@worst_items[:classes].first.location).to eq(expected)
     end
 
     it "provides location for a file" do
       expected = MetricFu::Location.new("lib/client/client.rb",
-                              nil,
-                              nil)
+                                        nil,
+                                        nil)
       expect(@worst_items[:files].first.location).to eq(expected)
     end
-
   end
 
   context "with Saikuro data" do
-
     before do
       @result_hash = HOTSPOT_DATA["saikuro.yml"]
       @analyzed_problems = analyzed_problems(@result_hash)
@@ -93,18 +89,16 @@ describe MetricFu::HotspotAnalyzedProblems do
 
     it "gives complexity for method" do
       expected = {
-        :saikuro => "complexity is 1.0"
+        saikuro: "complexity is 1.0"
       }
       expect(@worst_items[:methods].last.problems).to eq(expected)
     end
 
     it "gives average complexity for class" do
       expected = {
-        :saikuro => "average complexity is 5.0"
+        saikuro: "average complexity is 5.0"
       }
       expect(@worst_items[:classes].last.problems).to eq(expected)
     end
-
   end
-
 end