flyerhzm-metric_fu 1.0.0

data/spec/generators/reek_spec.rb

@@ -0,0 +1,126 @@
+require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
+
+describe Reek do
+  describe "analyze method" do
+    before :each do
+      MetricFu::Reek.stub!(:verify_dependencies!).and_return(true)
+      @lines = <<-HERE
+"app/controllers/activity_reports_controller.rb" -- 4 warnings:
+ActivityReportsController#authorize_user calls current_user.primary_site_ids multiple times (Duplication)
+ActivityReportsController#authorize_user calls params[id] multiple times (Duplication)
+ActivityReportsController#authorize_user calls params[primary_site_id] multiple times (Duplication)
+ActivityReportsController#authorize_user has approx 6 statements (Long Method)
+
+"app/controllers/application.rb" -- 1 warnings:
+ApplicationController#start_background_task/block/block is nested (Nested Iterators)
+
+"app/controllers/link_targets_controller.rb" -- 1 warnings:
+LinkTargetsController#authorize_user calls current_user.role multiple times (Duplication)
+
+"app/controllers/newline_controller.rb" -- 1 warnings:
+NewlineController#some_method calls current_user.<< "new line\n" multiple times (Duplication)
+      HERE
+      MetricFu::Configuration.run {}
+      File.stub!(:directory?).and_return(true)
+      reek = MetricFu::Reek.new
+      reek.instance_variable_set(:@output, @lines)
+      @matches = reek.analyze
+    end
+      
+    it "should find the code smell's method name" do
+      smell = @matches.first[:code_smells].first
+      smell[:method].should == "ActivityReportsController#authorize_user"
+    end
+    
+    it "should find the code smell's type" do
+      smell = @matches[1][:code_smells].first
+      smell[:type].should == "Nested Iterators"
+    end
+    
+    it "should find the code smell's message" do
+      smell = @matches[1][:code_smells].first
+      smell[:message].should == "is nested"
+    end
+    
+    it "should find the code smell's type" do
+      smell = @matches.first
+      smell[:file_path].should == "app/controllers/activity_reports_controller.rb"
+    end
+    
+    it "should NOT insert nil smells into the array when there's a newline in the method call" do
+      @matches.last[:code_smells].should == @matches.last[:code_smells].compact
+      @matches.last.should == {:file_path=>"app/controllers/newline_controller.rb", 
+                                :code_smells=>[{:type=>"Duplication", 
+                                                  :method=>"\"", 
+                                                  :message=>"multiple times"}]}
+      # Note: hopefully a temporary solution until I figure out how to deal with newlines in the method call more effectively -Jake 5/11/2009
+    end
+  end
+  
+end
+
+describe Reek do
+  before :each do
+    @reek = MetricFu::Reek.new
+    @lines11 = <<-HERE
+"app/controllers/activity_reports_controller.rb" -- 4 warnings:
+ActivityReportsController#authorize_user calls current_user.primary_site_ids multiple times (Duplication)
+ActivityReportsController#authorize_user calls params[id] multiple times (Duplication)
+ActivityReportsController#authorize_user calls params[primary_site_id] multiple times (Duplication)
+ActivityReportsController#authorize_user has approx 6 statements (Long Method)
+
+"app/controllers/application.rb" -- 1 warnings:
+ApplicationController#start_background_task/block/block is nested (Nested Iterators)
+
+"app/controllers/link_targets_controller.rb" -- 1 warnings:
+LinkTargetsController#authorize_user calls current_user.role multiple times (Duplication)
+
+"app/controllers/newline_controller.rb" -- 1 warnings:
+NewlineController#some_method calls current_user.<< "new line\n" multiple times (Duplication)
+      HERE
+    @lines12 = <<-HERE
+app/controllers/activity_reports_controller.rb -- 4 warnings (+3 masked):
+  ActivityReportsController#authorize_user calls current_user.primary_site_ids multiple times (Duplication)
+  ActivityReportsController#authorize_user calls params[id] multiple times (Duplication)
+  ActivityReportsController#authorize_user calls params[primary_site_id] multiple times (Duplication)
+  ActivityReportsController#authorize_user has approx 6 statements (Long Method)
+app/controllers/application.rb -- 1 warnings:
+  ApplicationController#start_background_task/block/block is nested (Nested Iterators)
+app/controllers/link_targets_controller.rb -- 1 warnings (+1 masked):
+  LinkTargetsController#authorize_user calls current_user.role multiple times (Duplication)
+app/controllers/newline_controller.rb -- 1 warnings:
+  NewlineController#some_method calls current_user.<< "new line\n" multiple times (Duplication)
+      HERE
+  end
+
+  context 'with Reek 1.1 output format' do
+    it 'reports 1.1 style when the output is empty' do
+      @reek.instance_variable_set(:@output, "")
+      @reek.should_not be_reek_12
+    end
+    it 'detects 1.1 format output' do
+      @reek.instance_variable_set(:@output, @lines11)
+      @reek.should_not be_reek_12
+    end
+
+    it 'massages empty output to be unchanged' do
+      @reek.instance_variable_set(:@output, "")
+      @reek.massage_for_reek_12.should be_empty
+    end
+  end
+
+  context 'with Reek 1.2 output format' do
+    before :each do
+      @reek = MetricFu::Reek.new
+    end
+    it 'detects 1.2 format output' do
+      @reek.instance_variable_set(:@output, @lines12)
+      @reek.should be_reek_12
+    end
+
+    it 'correctly massages 1.2 output' do
+      @reek.instance_variable_set(:@output, @lines12)
+      @reek.massage_for_reek_12.should == @lines11
+    end
+  end
+end

data/spec/generators/saikuro_spec.rb

@@ -0,0 +1,58 @@
+require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
+ 
+describe Saikuro do
+  describe "to_h method" do
+    before :all do
+      MetricFu::Configuration.run {}
+      File.stub!(:directory?).and_return(true)
+      saikuro = MetricFu::Saikuro.new
+      saikuro.stub!(:metric_directory).and_return(File.join(File.dirname(__FILE__), "..", "resources", "saikuro"))
+      saikuro.analyze
+      @output = saikuro.to_h
+    end
+
+    it "should find the filename of a file" do
+      @output[:saikuro][:files].first[:filename].should == 'users_controller.rb'
+    end
+ 
+    it "should find the name of the classes" do
+      @output[:saikuro][:classes].first[:name].should == "UsersController"
+      @output[:saikuro][:classes][1][:name].should == "SessionsController"
+    end
+
+    it "should put the most complex method first" do
+      @output[:saikuro][:methods].first[:name].should == "UsersController#create"
+      @output[:saikuro][:methods].first[:complexity].should == 4
+    end
+ 
+    it "should find the complexity of a method" do
+      @output[:saikuro][:methods].first[:complexity].should == 4
+    end
+ 
+    it "should find the lines of a method" do
+      @output[:saikuro][:methods].first[:lines].should == 15
+    end
+  end
+
+  describe "format_directories method" do
+    it "should format the directories" do
+      MetricFu::Configuration.run {}
+      File.stub!(:directory?).and_return(true)
+      saikuro = MetricFu::Saikuro.new
+ 
+      MetricFu.saikuro[:input_directory] = ["app", "lib"]
+  
+      saikuro.format_directories.should == "\"app | lib\""
+    end
+  end
+ 
+  describe Saikuro::SFile do
+    describe "getting elements from a Saikuro result file" do
+     it "should parse nested START/END sections" do
+       path = File.join(File.dirname(__FILE__), "..", "resources", "saikuro_sfiles", "thing.rb_cyclo.html")
+        sfile = Saikuro::SFile.new path
+       sfile.elements.map { |e| e.complexity }.sort.should eql(["0","0","2"])
+      end
+    end
+  end
+end

data/spec/generators/stats_spec.rb

@@ -0,0 +1,74 @@
+require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
+
+describe Stats do
+  describe "emit method" do
+    it "should gather the raw data" do
+      MetricFu::Configuration.run {}
+      File.stub!(:directory?).and_return(true)
+      stats = MetricFu::Stats.new
+      stats.should_receive(:`).with("rake stats > tmp/metric_fu/scratch/stats/stats.txt")
+      stats.emit
+    end
+  end
+  
+  describe "analyze method" do
+    before :each do
+      @lines =  <<-HERE.gsub(/^\s*/, "")
+      +----------------------+-------+-------+---------+---------+-----+-------+
+      | Name                 | Lines |   LOC | Classes | Methods | M/C | LOC/M |
+      +----------------------+-------+-------+---------+---------+-----+-------+
+      | Controllers          |   470 |   382 |       7 |      53 |   7 |     5 |
+      | Helpers              |   128 |    65 |       0 |       6 |   0 |     8 |
+      | Models               |   351 |   285 |       9 |      31 |   3 |     7 |
+      | Libraries            |   305 |   183 |       2 |      30 |  15 |     4 |
+      | Model specs          |   860 |   719 |       0 |       2 |   0 |   357 |
+      | View specs           |     0 |     0 |       0 |       0 |   0 |     0 |
+      | Controller specs     |  1570 |  1308 |       1 |      10 |  10 |   128 |
+      | Helper specs         |   191 |   172 |       0 |       0 |   0 |     0 |
+      | Library specs        |    31 |    27 |       0 |       0 |   0 |     0 |
+      +----------------------+-------+-------+---------+---------+-----+-------+
+      | Total                |  3906 |  3141 |      19 |     132 |   6 |    21 |
+      +----------------------+-------+-------+---------+---------+-----+-------+
+        Code LOC: 915     Test LOC: 2226     Code to Test Ratio: 1:2.4
+
+      HERE
+      MetricFu::Configuration.run {}
+      File.stub!(:directory?).and_return(true)
+      stats = MetricFu::Stats.new
+      File.should_receive(:open).and_return(mock("file", :read => @lines))
+      @results = stats.analyze
+    end
+    
+    it "should get code Lines Of Code" do
+      @results[:codeLOC].should == 915
+    end
+    
+    it "should get test Lines Of Code" do
+      @results[:testLOC].should == 2226
+    end
+    
+    it "should get code to test ratio" do
+      @results[:code_to_test_ratio].should == 2.4
+    end
+    
+    it "should get data on models" do
+      model_data = @results[:lines].find {|line| line[:name] == "Models"}
+      model_data[:classes].should == 9
+      model_data[:methods].should == 31
+      model_data[:loc].should == 285
+      model_data[:lines].should == 351
+      model_data[:methods_per_class].should == 3
+      model_data[:loc_per_method].should == 7
+    end
+  end
+  
+  describe "to_h method" do
+    it "should put things into a hash" do
+      MetricFu::Configuration.run {}
+      File.stub!(:directory?).and_return(true)
+      stats = MetricFu::Stats.new
+      stats.instance_variable_set(:@stats, "the_stats")
+      stats.to_h[:stats].should == "the_stats"
+    end
+  end
+end

data/spec/graphs/engines/bluff_spec.rb

@@ -0,0 +1,15 @@
+require File.expand_path(File.dirname(__FILE__) + "/../../spec_helper")
+
+describe "Bluff graphers responding to #graph!" do
+  it "should write chart file" do
+    MetricFu.configuration
+    graphs = {}
+    MetricFu::AVAILABLE_GRAPHS.each do |graph|
+      graphs[graph] = MetricFu.const_get("#{graph.to_s.capitalize}BluffGrapher").new
+    end
+    graphs.each do |key, val|
+      val.graph!
+      lambda{ File.open(File.join(MetricFu.output_directory, "#{key.to_s.downcase}.js")) }.should_not raise_error
+    end
+  end
+end

data/spec/graphs/engines/gchart_spec.rb

@@ -0,0 +1,15 @@
+require File.expand_path(File.dirname(__FILE__) + "/../../spec_helper")
+
+describe "Gchart graphers responding to #graph!" do
+  it "should write chart file" do
+    MetricFu.configuration
+    graphs = {}
+    MetricFu::AVAILABLE_GRAPHS.each do |graph|
+      graphs[graph] = MetricFu.const_get("#{graph.to_s.capitalize}GchartGrapher").new
+    end
+    graphs.each do |key, val|
+      val.graph!
+      lambda{ File.open(File.join(MetricFu.output_directory, "#{key.to_s.downcase}.png")) }.should_not raise_error
+    end
+  end
+end

data/spec/graphs/flay_grapher_spec.rb

@@ -0,0 +1,37 @@
+require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
+
+describe FlayGrapher do 
+  before :each do
+    @flay_grapher = MetricFu::FlayGrapher.new
+    MetricFu.configuration
+  end
+  
+  it "should respond to flay_score and labels" do
+    @flay_grapher.should respond_to(:flay_score)
+    @flay_grapher.should respond_to(:labels)
+  end
+  
+  describe "responding to #initialize" do
+    it "should initialise flay_score and labels" do
+      @flay_grapher.flay_score.should == []
+      @flay_grapher.labels.should == {}
+    end
+  end
+  
+  describe "responding to #get_metrics" do
+    before(:each) do
+      @metrics = YAML::load(File.open(File.join(File.dirname(__FILE__), "..", "resources", "yml", "20090630.yml")))
+      @date = "1/2"
+    end
+    
+    it "should push 476 to flay_score" do
+      @flay_grapher.flay_score.should_receive(:push).with(476)      
+      @flay_grapher.get_metrics(@metrics, @date)
+    end
+    
+    it "should update labels with the date" do
+      @flay_grapher.labels.should_receive(:update).with({ 0 => "1/2" })
+      @flay_grapher.get_metrics(@metrics, @date)
+    end
+  end
+end

data/spec/graphs/flog_grapher_spec.rb

@@ -0,0 +1,45 @@
+require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
+
+describe MetricFu::FlogGrapher do
+  before :each do
+    @flog_grapher = MetricFu::FlogGrapher.new
+    MetricFu.configuration
+  end
+  
+  it "should respond to flog_total, flog_average and labels" do
+    @flog_grapher.should respond_to(:flog_average)
+    @flog_grapher.should respond_to(:labels)
+    @flog_grapher.should respond_to(:top_five_percent_average)
+  end
+  
+  describe "responding to #initialize" do
+    it "should initialize top_five_percent_average, flog_average and labels" do
+      @flog_grapher.flog_average.should == []
+      @flog_grapher.labels.should == {}
+      @flog_grapher.top_five_percent_average.should == []
+    end
+  end
+  
+  describe "responding to #get_metrics" do
+    before(:each) do
+      @metrics = YAML::load(File.open(File.join(File.dirname(__FILE__), "..", "resources", "yml", "20090630.yml")))
+      @date = "1/2"
+    end
+    
+    it "should push to top_five_percent_average" do
+      average = (73.6 + 68.5 + 66.1 + 46.6 + 44.8 + 44.1 + 41.2 + 36.0) / 8.0
+      @flog_grapher.top_five_percent_average.should_receive(:push).with(average)      
+      @flog_grapher.get_metrics(@metrics, @date)
+    end
+    
+    it "should push 9.9 to flog_average" do
+      @flog_grapher.flog_average.should_receive(:push).with(9.9)
+      @flog_grapher.get_metrics(@metrics, @date)
+    end
+    
+    it "should update labels with the date" do
+      @flog_grapher.labels.should_receive(:update).with({ 0 => "1/2" })
+      @flog_grapher.get_metrics(@metrics, @date)
+    end
+  end
+end

data/spec/graphs/rcov_grapher_spec.rb

@@ -0,0 +1,37 @@
+require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
+
+describe RcovGrapher do 
+  before :each do
+    @rcov_grapher = MetricFu::RcovGrapher.new
+    MetricFu.configuration
+  end
+  
+  it "should respond to rcov_percent and labels" do
+    @rcov_grapher.should respond_to(:rcov_percent)
+    @rcov_grapher.should respond_to(:labels)
+  end
+  
+  describe "responding to #initialize" do
+    it "should initialise rcov_percent and labels" do
+      @rcov_grapher.rcov_percent.should == []
+      @rcov_grapher.labels.should == {}
+    end
+  end
+  
+  describe "responding to #get_metrics" do
+    before(:each) do
+      @metrics = YAML::load(File.open(File.join(File.dirname(__FILE__), "..", "resources", "yml", "20090630.yml")))
+      @date = "1/2"
+    end
+    
+    it "should push 49.6 to rcov_percent" do
+      @rcov_grapher.rcov_percent.should_receive(:push).with(49.6)      
+      @rcov_grapher.get_metrics(@metrics, @date)
+    end
+    
+    it "should update labels with the date" do
+      @rcov_grapher.labels.should_receive(:update).with({ 0 => "1/2" })
+      @rcov_grapher.get_metrics(@metrics, @date)
+    end
+  end
+end

data/spec/graphs/reek_grapher_spec.rb

@@ -0,0 +1,46 @@
+require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
+
+describe ReekGrapher do 
+  before :each do
+    @reek_grapher = MetricFu::ReekGrapher.new
+    MetricFu.configuration
+  end
+  
+  it "should respond to reek_count and labels" do
+    @reek_grapher.should respond_to(:reek_count)
+    @reek_grapher.should respond_to(:labels)
+  end
+  
+  describe "responding to #initialize" do
+    it "should initialise reek_count and labels" do
+      @reek_grapher.reek_count.should == {}
+      @reek_grapher.labels.should == {}
+    end
+  end
+  
+  describe "responding to #get_metrics" do
+    before(:each) do
+      @metrics = YAML::load(File.open(File.join(File.dirname(__FILE__), "..", "resources", "yml", "20090630.yml")))
+      @date = "1/2"
+    end
+    
+    it "should set a hash of code smells to reek_count" do
+      @reek_grapher.get_metrics(@metrics, @date)
+      @reek_grapher.reek_count.should == {
+        "Uncommunicative Name" => [27],
+        "Feature Envy" => [20],
+        "Utility Function" => [15],
+        "Long Method" => [26],
+        "Nested Iterators" => [12],
+        "Control Couple" => [4],
+        "Duplication" => [48],
+        "Large Class" => [1] 
+      }
+    end
+    
+    it "should update labels with the date" do
+      @reek_grapher.labels.should_receive(:update).with({ 0 => "1/2" })
+      @reek_grapher.get_metrics(@metrics, @date)
+    end
+  end
+end