khall-metric_fu 1.0.2.1

data/spec/generators/reek_spec.rb

@@ -0,0 +1,59 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe Reek do
+  describe "analyze method" do
+    before :each do
+      @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

data/spec/generators/saikuro_spec.rb

@@ -0,0 +1,53 @@
+require 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 "emit 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"]
+      
+      File.stub!(:dirname).and_return('..')
+      File.stub!(:expand_path)
+      
+      saikuro.should_receive(:sh).with(/"app \| lib"/)
+      
+      saikuro.emit
+    end
+  end
+end

data/spec/generators/stats_spec.rb

@@ -0,0 +1,74 @@
+require 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/spec_helper.rb

@@ -0,0 +1,28 @@
+require 'rubygems'
+require 'spec'
+require 'date'
+
+require File.join(File.dirname(__FILE__), '/../lib/metric_fu.rb')
+include MetricFu
+
+Mystat = <<-EOF
+(in /Users/gmcinnes/Documents/projects/NeerBeer/src/Web)
++----------------------+-------+-------+---------+---------+-----+-------+
+| Name                 | Lines |   LOC | Classes | Methods | M/C | LOC/M |
++----------------------+-------+-------+---------+---------+-----+-------+
+| Controllers          |   893 |   405 |      16 |      41 |   2 |     7 |
+| Helpers              |   569 |   352 |       0 |      52 |   0 |     4 |
+| Models               |  1758 |   453 |      26 |      48 |   1 |     7 |
+| Libraries            |  2507 |  1320 |      19 |     175 |   9 |     5 |
+| Model specs          |  2285 |   965 |       0 |       1 |   0 |   963 |
+| View specs           |   821 |   654 |       0 |       2 |   0 |   325 |
+| Controller specs     |  1144 |   871 |       0 |       9 |   0 |    94 |
+| Helper specs         |   652 |   465 |       0 |       1 |   0 |   463 |
+| Library specs        |  1456 |  1141 |       8 |      14 |   1 |    79 |
++----------------------+-------+-------+---------+---------+-----+-------+
+| Total                | 12085 |  6626 |      69 |     343 |   4 |    17 |
++----------------------+-------+-------+---------+---------+-----+-------+
+  Code LOC: 2530     Test LOC: 4096     Code to Test Ratio: 1:1.6
+
+EOF
+

data/tasks/metric_fu.rake

@@ -0,0 +1,15 @@
+require 'rake'
+namespace :metrics do
+  desc "Generate all metrics reports"
+  task :all do
+    MetricFu::Configuration.run {}
+    MetricFu.metrics.each {|metric| MetricFu.report.add(metric) }
+    MetricFu.report.save_output(MetricFu.report.to_yaml,
+                                MetricFu.base_directory, 
+                                'report.yml')
+    MetricFu.report.save_templatized_report
+    if MetricFu.report.open_in_browser?
+      MetricFu.report.show_in_browser(MetricFu.output_directory)
+    end
+  end
+end

data/tasks/railroad.rake

@@ -0,0 +1,39 @@
+namespace :metrics do
+  
+  RAILROAD_DIR = File.join(MetricFu::BASE_DIRECTORY, 'railroad')
+  RAILROAD_FILE = File.join(RAILROAD_DIR, 'index.html')
+  
+  task :railroad => ['railroad:all'] do
+  end
+  
+  namespace :railroad do
+  
+    desc "Create all railroad reports"
+    task :all => [:models, :controllers, :aasm] do
+      #system("open #{RAILROAD_INDEX}") if PLATFORM['darwin']
+    end
+  
+    desc "Create a railroad models report"
+    task :models do
+      #mkdir_p(RAILROAD_DIR) unless File.directory?(RAILROAD_DIR)
+      `railroad -M -a -m -l -v | neato -Tpng > #{File.join(MetricFu::BASE_DIRECTORY,'model-diagram.png')}`
+      #`echo "<a href=\"railroad/models.png\">Model diagram</a><br />" >> #{RAILROAD_FILE}`
+    end
+  
+    desc "Create a railroad controllers report"
+    task :controllers do
+      #mkdir_p(RAILROAD_DIR) unless File.directory?(RAILROAD_DIR)
+      `railroad -C -l -v | neato -Tpng > #{File.join(MetricFu::BASE_DIRECTORY,'controller-diagram.png')}`
+      #`echo "<a href=\"railroad/controllers.png\">Controller diagram</a><br />" >> #{RAILROAD_FILE}`
+    end
+  
+    desc "Create a railroad acts_as_state_machine report"
+    task :aasm do
+      #mkdir_p(RAILROAD_DIR) unless File.directory?(RAILROAD_DIR)
+      `railroad -A -l -v | neato -Tpng > #{File.join(MetricFu::BASE_DIRECTORY,'aasm-diagram.png')}`
+      #`echo "<a href=\"railroad/aasm.png\">State machine diagram</a><br />" >> #{RAILROAD_FILE}`
+    end
+    
+  end
+
+end