jscruggs-metric_fu 1.0.2 → 1.1.0

data/spec/base/generator_spec.rb

@@ -1,4 +1,4 @@
-require File.dirname(__FILE__) + "/../spec_helper.rb"
+require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
 
 describe MetricFu::Generator do
 
@@ -154,6 +154,28 @@ describe MetricFu::Generator do
       @concrete_class.should_receive(:to_h)
       @concrete_class.generate_report
     end
+    
+    it "should raise error if the concrete class is missing a required dependency" do
+      concrete_class_with_missing_gem = Class.new(MetricFu::Generator) do
+        def self.verify_dependencies!
+          raise 'gem install something # if you want these tasks'
+        end
+      end
+      lambda { concrete_class_with_missing_gem.generate_report }.should raise_error("gem install something # if you want these tasks")
+    end
 
   end
+  
+  describe "instantiation" do
+    it "should fail is dependencies not verified" do
+      ConcreteClass.should_receive(:verify_dependencies!).and_raise("Missing a required gem. Please 'gem install something'")      
+      lambda { ConcreteClass.new() }.should raise_error("Missing a required gem. Please 'gem install something'")
+    end
+
+    it "should succeed when dependencies verified" do
+      ConcreteClass.should_receive(:verify_dependencies!).and_return(true)
+      ConcreteClass.new()
+    end
+  end
+  
 end

data/spec/base/md5_tracker_spec.rb

@@ -1,4 +1,4 @@
-require File.dirname(__FILE__) + '/../spec_helper.rb'
+require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
 
 describe MetricFu::MD5Tracker do
   before do

data/spec/base/report_spec.rb

@@ -1,4 +1,4 @@
-require File.dirname(__FILE__) + "/../spec_helper.rb"
+require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
 
 describe MetricFu do
   

data/spec/generators/churn_spec.rb

@@ -1,4 +1,4 @@
-require File.dirname(__FILE__) + '/../spec_helper'
+require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
 
 describe Churn do
   describe "initialize" do
@@ -8,20 +8,20 @@ describe Churn do
     end
     
     it "should setup git if .git exits" do
-      File.should_receive(:exist?).with(".git").and_return(true)
+      MetricFu::Churn.should_receive(:system).and_return(true)
       Churn::Git.should_receive(:new)
       MetricFu::Churn.new
     end
     
     it "should setup git if .svn exits" do
-      File.should_receive(:exist?).with(".git").and_return(false)
+      MetricFu::Churn.should_receive(:system).and_return(false)
       File.should_receive(:exist?).with(".svn").and_return(true)
       Churn::Svn.should_receive(:new)
       MetricFu::Churn.new()
     end
     
     it "should raise an error if not .svn or .git" do
-      File.stub!(:exist?).and_return(false)
+      MetricFu::Churn.should_receive(:system).and_return(false)
       lambda{MetricFu::Churn.new()}.should raise_error(Exception)
     end
   end
@@ -30,7 +30,7 @@ describe Churn do
     before :each do
       MetricFu::Configuration.run {|config| config.churn = {:minimum_churn_count => 2} }
       File.stub!(:directory?).and_return(true)
-      File.should_receive(:exist?).with(".git").and_return(true)
+      MetricFu::Churn.should_receive(:system).and_return(true)
       @git = Churn::Git.new
       Churn::Git.should_receive(:new).and_return(@git)
       @lines = <<-HERE.gsub(/^\s*/, "")
@@ -73,7 +73,7 @@ describe Churn do
     before :each do
       MetricFu::Configuration.run{|config| config.churn = {:minimum_churn_count => 2} }
       File.stub!(:directory?).and_return(true)
-      File.should_receive(:exist?).with(".git").and_return(false)
+      MetricFu::Churn.should_receive(:system).and_return(false)
       File.should_receive(:exist?).with(".svn").and_return(true)
       @svn = Churn::Svn.new
       Churn::Svn.should_receive(:new).and_return(@svn)
@@ -120,7 +120,7 @@ describe Churn do
     before :each do
       MetricFu::Configuration.run {}
       File.stub!(:directory?).and_return(true)
-      File.should_receive(:exist?).with(".git").and_return(true)
+      MetricFu::Churn.should_receive(:system).and_return(true)
       @changes = {"lib/generators/flog.rb"=>2, "lib/metric_fu.rb"=>3}
     end
     
@@ -139,7 +139,7 @@ describe Churn do
     before :each do
       MetricFu::Configuration.run {}
       File.stub!(:directory?).and_return(true)
-      File.should_receive(:exist?).with(".git").and_return(true)
+      MetricFu::Churn.should_receive(:system).and_return(true)
     end
     
     it "should put the changes into a hash" do

data/spec/generators/flay_spec.rb

@@ -1,6 +1,9 @@
-require File.dirname(__FILE__) + '/../spec_helper.rb'
+require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
 
 describe Flay do 
+  before :each do
+    MetricFu::Flay.stub!(:verify_dependencies!).and_return(true)
+  end
   describe "emit method" do
     before :each do
       MetricFu::Configuration.run {|config| config.flay = { :dirs_to_flay => ['app', 'lib']  } }

data/spec/generators/flog_spec.rb

@@ -1,4 +1,4 @@
-require File.dirname(__FILE__) + '/../spec_helper'
+require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
 describe Flog do
   before :each do
     @text = <<-HERE
@@ -123,6 +123,7 @@ describe Flog do
       1.2: assignment
       1.2: all
     HERE
+    MetricFu::Flog.stub!(:verify_dependencies!).and_return(true)
   end
   
   describe "parse method" do
@@ -168,6 +169,13 @@ describe Flog do
         flog_page = flog.parse(text)
         flog_page.scanned_methods.first.name.should == "SomeNamespace::UsersController#create"
     end
+
+    it "should parse empty flog files" do
+      text = ""
+      flog = MetricFu::Flog.new('base_dir')
+      flog_page = flog.parse(text)
+      flog_page.should be_nil
+    end
   end
   
   describe "to_h function" do
@@ -197,4 +205,4 @@ describe Flog do
       @flog_hash[:flog][:pages].first[:path].should == "/app/controllers/user_controller.rb"
     end
   end
-end
+end

data/spec/generators/reek_spec.rb

@@ -1,8 +1,9 @@
-require File.dirname(__FILE__) + '/../spec_helper'
+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)

data/spec/generators/saikuro_spec.rb

@@ -1,5 +1,5 @@
-require File.dirname(__FILE__) + '/../spec_helper'
-
+require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
+ 
 describe Saikuro do
   describe "to_h method" do
     before :all do
@@ -10,44 +10,49 @@ describe Saikuro do
       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
+
+  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"]
-      
-      File.stub!(:dirname).and_return('..')
-      File.stub!(:expand_path)
-      
-      saikuro.should_receive(:sh).with(/"app \| lib"/)
-      
-      saikuro.emit
+  
+      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

@@ -1,4 +1,4 @@
-require File.dirname(__FILE__) + '/../spec_helper'
+require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
 
 describe Stats do
   describe "emit method" do

data/tasks/metric_fu.rake

@@ -6,8 +6,15 @@ namespace :metrics do
     MetricFu.metrics.each {|metric| MetricFu.report.add(metric) }
     MetricFu.report.save_output(MetricFu.report.to_yaml,
                                 MetricFu.base_directory, 
-                                'report.yml')
+                                "report.yml")
+    MetricFu.report.save_output(MetricFu.report.to_yaml,
+                                MetricFu.data_directory, 
+                                "#{Time.now.strftime("%Y%m%d")}.yml")
     MetricFu.report.save_templatized_report
+    
+    MetricFu.graphs.each {|graph| MetricFu.graph.add(graph) }
+    MetricFu.graph.generate
+    
     if MetricFu.report.open_in_browser?
       MetricFu.report.show_in_browser(MetricFu.output_directory)
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: jscruggs-metric_fu
 version: !ruby/object:Gem::Version 
-  version: 1.0.2
+  version: 1.1.0
 platform: ruby
 authors: 
 - Jake Scruggs
@@ -9,11 +9,13 @@ authors:
 - Andre Arko
 - Petrik de Heus
 - Grant McInnes
+- Nick Quaranto
+- "\xC3\x89douard Bri\xC3\xA8re"
 autorequire: 
 bindir: bin
 cert_chain: []
 
-date: 2009-04-08 00:00:00 -07:00
+date: 2009-05-16 00:00:00 -07:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -37,44 +39,34 @@ dependencies:
         version: 2.1.0
     version: 
 - !ruby/object:Gem::Dependency 
-  name: rcov
-  type: :runtime
-  version_requirement: 
-  version_requirements: !ruby/object:Gem::Requirement 
-    requirements: 
-    - - ">"
-      - !ruby/object:Gem::Version 
-        version: 0.8.1
-    version: 
-- !ruby/object:Gem::Dependency 
-  name: reek
+  name: relevance-rcov
   type: :runtime
   version_requirement: 
   version_requirements: !ruby/object:Gem::Requirement 
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        version: 1.0.0
+        version: 0.8.3.3
     version: 
 - !ruby/object:Gem::Dependency 
-  name: roodi
+  name: mojombo-chronic
   type: :runtime
   version_requirement: 
   version_requirements: !ruby/object:Gem::Requirement 
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        version: 1.3.5
+        version: 0.3.0
     version: 
 - !ruby/object:Gem::Dependency 
-  name: chronic
+  name: topfunky-gruff
   type: :runtime
   version_requirement: 
   version_requirements: !ruby/object:Gem::Requirement 
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        version: 0.2.3
+        version: 0.3.5
     version: 
 description: Code metrics from Flog, Flay, RCov, Saikuro, Churn, Reek, Roodi and Rails' stats task
 email: jake.scruggs@gmail.com
@@ -95,6 +87,7 @@ files:
 - lib/base/base_template.rb
 - lib/base/configuration.rb
 - lib/base/generator.rb
+- lib/base/graph.rb
 - lib/base/md5_tracker.rb
 - lib/base/report.rb
 - lib/generators/churn.rb
@@ -105,7 +98,24 @@ files:
 - lib/generators/roodi.rb
 - lib/generators/saikuro.rb
 - lib/generators/stats.rb
+- lib/graphs/flay_grapher.rb
+- lib/graphs/flog_grapher.rb
+- lib/graphs/rcov_grapher.rb
+- lib/graphs/reek_grapher.rb
+- lib/graphs/roodi_grapher.rb
 - lib/metric_fu.rb
+- lib/templates/awesome/awesome_template.rb
+- lib/templates/awesome/churn.html.erb
+- lib/templates/awesome/default.css
+- lib/templates/awesome/flay.html.erb
+- lib/templates/awesome/flog.html.erb
+- lib/templates/awesome/index.html.erb
+- lib/templates/awesome/layout.html.erb
+- lib/templates/awesome/rcov.html.erb
+- lib/templates/awesome/reek.html.erb
+- lib/templates/awesome/roodi.html.erb
+- lib/templates/awesome/saikuro.html.erb
+- lib/templates/awesome/stats.html.erb
 - lib/templates/standard/churn.html.erb
 - lib/templates/standard/default.css
 - lib/templates/standard/flay.html.erb
@@ -118,7 +128,7 @@ files:
 - lib/templates/standard/standard_template.rb
 - lib/templates/standard/stats.html.erb
 - tasks/metric_fu.rake
-- tasks/railroad.rake
+- vendor/_fonts/monaco.ttf
 - vendor/saikuro/saikuro.rb
 - Manifest.txt
 has_rdoc: true
@@ -147,7 +157,7 @@ rubyforge_project:
 rubygems_version: 1.2.0
 signing_key: 
 specification_version: 2
-summary: A fistful of code metrics
+summary: A fistful of code metrics, with awesome templates and graphs
 test_files: 
 - spec/base/base_template_spec.rb
 - spec/base/configuration_spec.rb

data/tasks/railroad.rake

@@ -1,39 +0,0 @@
-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