edouard-metric_fu 1.0.2 → 1.0.3

data/README

@@ -1 +1 @@
-See http://metric-fu.rubyforge.org/ for documentation, or the HISTORY file for a change log.
+See http://metric-fu.rubyforge.org/ for documentation, or the HISTORY file for a change log.

data/Rakefile

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

data/spec/base/configuration_spec.rb

@@ -129,7 +129,7 @@ describe MetricFu::Configuration do
     end
     
     it 'should set @template_class to StandardTemplate' do
-      template_class.should == AwesomeTemplate
+      template_class.should == StandardTemplate
     end
 
     it 'should set @flay to {:dirs_to_flay => @code_dirs}' do

data/spec/base/generator_spec.rb

@@ -154,28 +154,6 @@ 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/generators/churn_spec.rb

@@ -8,20 +8,20 @@ describe Churn do
     end
     
     it "should setup git if .git exits" do
-      MetricFu::Churn.should_receive(:system).and_return(true)
+      File.should_receive(:exist?).with(".git").and_return(true)
       Churn::Git.should_receive(:new)
       MetricFu::Churn.new
     end
     
     it "should setup git if .svn exits" do
-      MetricFu::Churn.should_receive(:system).and_return(false)
+      File.should_receive(:exist?).with(".git").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
-      MetricFu::Churn.should_receive(:system).and_return(false)
+      File.stub!(:exist?).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)
-      MetricFu::Churn.should_receive(:system).and_return(true)
+      File.should_receive(:exist?).with(".git").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)
-      MetricFu::Churn.should_receive(:system).and_return(false)
+      File.should_receive(:exist?).with(".git").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)
-      MetricFu::Churn.should_receive(:system).and_return(true)
+      File.should_receive(:exist?).with(".git").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)
-      MetricFu::Churn.should_receive(:system).and_return(true)
+      File.should_receive(:exist?).with(".git").and_return(true)
     end
     
     it "should put the changes into a hash" do

data/spec/generators/flay_spec.rb

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

@@ -123,7 +123,6 @@ describe Flog do
       1.2: assignment
       1.2: all
     HERE
-    MetricFu::Flog.stub!(:verify_dependencies!).and_return(true)
   end
   
   describe "parse method" do
@@ -169,13 +168,6 @@ 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
@@ -205,4 +197,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

@@ -3,7 +3,6 @@ require 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'
- 
+
 describe Saikuro do
   describe "to_h method" do
     before :all do
@@ -10,54 +10,44 @@ 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
     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
- 
-  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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: edouard-metric_fu
 version: !ruby/object:Gem::Version 
-  version: 1.0.2
+  version: 1.0.3
 platform: ruby
 authors: 
 - Jake Scruggs
@@ -10,7 +10,6 @@ authors:
 - Petrik de Heus
 - Grant McInnes
 - Nick Quaranto
-- "\xC3\x89douard Bri\xC3\xA8re"
 autorequire: 
 bindir: bin
 cert_chain: []
@@ -39,34 +38,44 @@ dependencies:
         version: 2.1.0
     version: 
 - !ruby/object:Gem::Dependency 
-  name: relevance-rcov
+  name: rcov
+  type: :runtime
+  version_requirement: 
+  version_requirements: !ruby/object:Gem::Requirement 
+    requirements: 
+    - - ">"
+      - !ruby/object:Gem::Version 
+        version: 0.8.3
+    version: 
+- !ruby/object:Gem::Dependency 
+  name: reek
   type: :runtime
   version_requirement: 
   version_requirements: !ruby/object:Gem::Requirement 
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        version: 0.8.3.3
+        version: 1.0.0
     version: 
 - !ruby/object:Gem::Dependency 
-  name: mojombo-chronic
+  name: roodi
   type: :runtime
   version_requirement: 
   version_requirements: !ruby/object:Gem::Requirement 
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        version: 0.3.0
+        version: 1.3.5
     version: 
 - !ruby/object:Gem::Dependency 
-  name: topfunky-gruff
+  name: chronic
   type: :runtime
   version_requirement: 
   version_requirements: !ruby/object:Gem::Requirement 
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        version: 0.3.5
+        version: 0.2.3
     version: 
 description: Code metrics from Flog, Flay, RCov, Saikuro, Churn, Reek, Roodi and Rails' stats task
 email: jake.scruggs@gmail.com
@@ -84,53 +93,6 @@ files:
 - TODO
 - MIT-LICENSE
 - Rakefile
-- lib/base/base_template.rb
-- lib/base/configuration.rb
-- lib/base/generator.rb
-- lib/base/md5_tracker.rb
-- lib/base/report.rb
-- lib/generators/churn.rb
-- lib/generators/flay.rb
-- lib/generators/flog.rb
-- lib/generators/rcov.rb
-- lib/generators/reek.rb
-- lib/generators/roodi.rb
-- lib/generators/saikuro.rb
-- lib/generators/stats.rb
-- lib/metric_fu.rb
-- lib/templates/standard/churn.html.erb
-- lib/templates/standard/default.css
-- lib/templates/standard/flay.html.erb
-- lib/templates/standard/flog.html.erb
-- lib/templates/standard/index.html.erb
-- lib/templates/standard/rcov.html.erb
-- lib/templates/standard/reek.html.erb
-- lib/templates/standard/roodi.html.erb
-- lib/templates/standard/saikuro.html.erb
-- lib/templates/standard/standard_template.rb
-- lib/templates/standard/stats.html.erb
-- 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/rcov.html.erb
-- lib/templates/awesome/reek.html.erb
-- lib/templates/awesome/roodi.html.erb
-- lib/templates/awesome/saikuro.html.erb
-- lib/templates/awesome/awesome_template.rb
-- lib/templates/awesome/stats.html.erb
-- lib/templates/awesome/layout.html.erb
-- lib/base/graph.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
-- tasks/metric_fu.rake
-- vendor/_fonts/monaco.ttf
-- tasks/railroad.rake
-- vendor/saikuro/saikuro.rb
 - Manifest.txt
 has_rdoc: true
 homepage: http://metric-fu.rubyforge.org/
@@ -158,7 +120,7 @@ rubyforge_project:
 rubygems_version: 1.2.0
 signing_key: 
 specification_version: 2
-summary: A fistful of code metrics, with awesome templates and graphs
+summary: A fistful of code metrics
 test_files: 
 - spec/base/base_template_spec.rb
 - spec/base/configuration_spec.rb

data/lib/base/base_template.rb

@@ -1,134 +0,0 @@
-module MetricFu
-
-  # The Template class is intended as an abstract class for concrete
-  # template classes to subclass.  It provides a variety of utility
-  # methods to make templating a bit easier.  However, classes do not
-  # have to inherit from here in order to provide a template.  The only
-  # requirement for a template class is that it provides a #write method
-  # to actually write out the template.  See StandardTemplate for an
-  # example.
-  class Template
-    attr_accessor :report
-    
-    private
-    # Creates a new erb evaluated result from the passed in section.
-    #
-    # @param section String
-    #   The section name of 
-    #
-    # @return String
-    #   The erb evaluated string
-    def erbify(section)
-      require 'erb'
-      erb_doc = File.read(template(section))
-      ERB.new(erb_doc).result(binding)
-    end
-
-    # Determines whether a template file exists for a given section
-    # of the full template.
-    #
-    # @param section String
-    #   The section of the template to check against
-    #
-    # @return Boolean
-    #   Does a template file exist for this section or not?
-    def template_exists?(section)
-      File.exist?(template(section))
-    end
-    
-    # Copies an instance variable mimicing the name of the section
-    # we are trying to render, with a value equal to the passed in 
-    # constant.  Allows the concrete template classes to refer to 
-    # that instance variable from their ERB rendering
-    #
-    # @param section String
-    #  The name of the instance variable to create
-    #
-    # @param contents Object
-    #   The value to set as the value of the created instance
-    #   variable
-    def create_instance_var(section, contents)
-      instance_variable_set("@#{section}", contents)        
-    end
-
-    # Generates the filename of the template file to load and 
-    # evaluate.  In this case, the path to the template directory +
-    # the section name + .html.erb
-    #
-    # @param section String
-    #   A section of the template to render
-    #
-    # @return String
-    #   A file path
-    def template(section)
-      File.join(this_directory,  section.to_s + ".html.erb")
-    end
-
-    # Returns the filename that the template will render into for
-    # a given section.  In this case, the section name + '.html'
-    #
-    # @param section String
-    #   A section of the template to render
-    #
-    # @return String
-    #   The output filename
-    def output_filename(section)
-      section.to_s + ".html"
-    end
-
-    # Returns the contents of a given css file in order to 
-    # render it inline into a template.
-    #
-    # @param css String
-    #   The name of a css file to open
-    #
-    # @return String
-    #   The contents of the css file
-    def inline_css(css)
-      open(File.join(this_directory, css)) { |f| f.read }      
-    end
-  
-    # Provides a link to open a file through the textmate protocol
-    # on Darwin, or otherwise, a simple file link.
-    #
-    # @param name String
-    #   
-    # @param line Integer
-    #   The line number to link to, if textmate is available.  Defaults
-    #   to nil
-    #
-    # @return String
-    #   An anchor link to a textmate reference or a file reference
-    def link_to_filename(name, line = nil)
-      filename = File.expand_path(name)
-      if MetricFu.configuration.platform.include?('darwin')
-        "<a href='txmt://open/?url=file://" \
-        +"#{filename}&line=#{line}'>#{name}:#{line}</a>"
-      else 
-       "<a href='file://#{filename}'>#{name}:#{line}</a>"
-      end
-    end
-  
-
-    # Provides a brain dead way to cycle between two values during
-    # an iteration of some sort.  Pass in the first_value, the second_value,
-    # and the cardinality of the iteration.
-    #
-    # @param first_value Object
-    #
-    # @param second_value Object
-    #
-    # @param iteration Integer
-    #   The number of times through the iteration.
-    #
-    # @return Object
-    #   The first_value if iteration is even.  The second_value if
-    #   iteration is odd.
-    def cycle(first_value, second_value, iteration)
-      return first_value if iteration % 2 == 0
-      return second_value
-    end      
-    
-
-  end
-end