p8-metric_fu 0.8.4.16 → 0.8.9

data/HISTORY

@@ -1,3 +1,15 @@
+=== MetricFu 0.8.9 / 2009-1-20
+
+* Thanks to Andre Arko and Petrik de Heus for adding the following features:
+* The source control type is auto-detected for Churn
+* Moved all presentation to templates
+* Wrote specs for all classes
+* Added flay, Reek and Roodi metrics
+* There's now a configuration class (see README for details)
+* Unification of metrics reports
+* Metrics can be generated using one command
+* Adding new metrics reports has been standardized
+
 === MetricFu 0.8.0 / 2008-10-06
 
 * Source Control Churn now supports git (thanks to Erik St Martin)

data/MIT-LICENSE

@@ -1,4 +1,4 @@
-Copyright (c) 2008 Jake Scruggs
+Copyright (c) 2008,2009 Jake Scruggs
 
 Permission is hereby granted, free of charge, to any person
 obtaining a copy of this software and associated documentation

data/README

@@ -1,4 +1,4 @@
-Version 0.8.0
+Version 0.8.9
 http://github.com/jscruggs/metric_fu
 
 Metric_fu began its life as a plugin for Rails that generated code metrics reports.  As of version 0.7.0, metric_fu is a gem owing to the excellent work done by Sean Soper.  
@@ -12,7 +12,7 @@ Then in your Rakefile:
 require 'metric_fu'
 
 If you like to vendor gems, you can unpack metric_fu into vendor/gems and require it like so:
-require 'vendor/gems/jscruggs-metric_fu-0.8.0/lib/metric_fu'
+require 'vendor/gems/jscruggs-metric_fu-0.8.9/lib/metric_fu'
 
 then you don't have to install it on every box you run it on.
 
@@ -144,4 +144,4 @@ You can also change the minimum churn count like so:
 
 ****Thanks****
 
-I'd like to thank the authors of Saikuro, Flog, Rcov, CruiseControl.rb, and Rails for creating such excellent open source products.  Also Sean Soper, Michael Schubert, Kurtis Seebaldt, Toby Tripp, Paul Gross, and Chirdeep Shetty for their help and advice.
+I'd like to thank the authors of Saikuro, Flog, Rcov, CruiseControl.rb, Flay, Reek, Roodi and Rails for creating such excellent open source products.  Also Andre Arko, Petrik de Heus, Sean Soper, Erik St Martin, Andy Gregorowicz, Bastien, Michael Schubert, Kurtis Seebaldt, Toby Tripp, Paul Gross, and Chirdeep Shetty for their help and advice.

data/lib/metric_fu/base.rb

@@ -70,9 +70,9 @@ module MetricFu
       def link_to_filename(name, line = nil)
         filename = File.expand_path(name)
         if PLATFORM['darwin']
-          %{<a href="txmt://open/?url=file://#{filename}&line=#{line}">#{name}</a>}
+          %{<a href="txmt://open/?url=file://#{filename}&line=#{line}">#{name}:#{line}</a>}
         else
-          %{<a href="file://#{filename}">#{name}</a>}
+          %{<a href="file://#{filename}">#{name}:#{line}</a>}
         end
       end
       

data/lib/metric_fu/flog.rb

@@ -45,15 +45,8 @@ module MetricFu
     SCORE_FORMAT = "%0.2f"
 
     class Base
-      MODULE_NAME = "([A-Za-z]+)+"
-      METHOD_NAME = "#([a-z0-9]+_?)+\\??\\!?"
-      SCORE = "\\d+\\.\\d+"
-
-      METHOD_NAME_RE = Regexp.new("#{MODULE_NAME}#{METHOD_NAME}")
-      SCORE_RE = Regexp.new(SCORE)
-
-      METHOD_LINE_RE = Regexp.new("#{MODULE_NAME}#{METHOD_NAME}:\\s\\(#{SCORE}\\)")
-      OPERATOR_LINE_RE = Regexp.new("\\s+(#{SCORE}):\\s(.*)$")
+      METHOD_LINE_REGEX = /([A-Za-z]+#.*):\s\((\d+\.\d+)\)/
+      OPERATOR_LINE_REGEX = /\s+(\d+\.\d+):\s(.*)$/
 
       class << self
 
@@ -63,21 +56,18 @@ module MetricFu
           page = Page.new(score)
 
           text.each_line do |method_line|
-            if METHOD_LINE_RE =~ method_line and
-               method_name = method_line[METHOD_NAME_RE] and
-               score = method_line[SCORE_RE]
-               page.scanned_methods << ScannedMethod.new(method_name, score)
+           if match = method_line.match(METHOD_LINE_REGEX)
+              page.scanned_methods << ScannedMethod.new(match[1], match[2])
             end
-
-            if OPERATOR_LINE_RE =~ method_line and
-               operator = method_line[OPERATOR_LINE_RE, 2] and
-               score = method_line[SCORE_RE]
-               return if page.scanned_methods.empty?
-               page.scanned_methods.last.operators << Operator.new(score, operator)
+            
+            if match = method_line.match(OPERATOR_LINE_REGEX)
+              return if page.scanned_methods.empty?
+              page.scanned_methods.last.operators << Operator.new(match[1], match[2])
             end
           end
           page
         end
+        
       end
     end 
   

data/lib/tasks/railroad.rake

@@ -1,7 +1,7 @@
 namespace :metrics do
   
   RAILROAD_DIR = File.join(MetricFu::BASE_DIRECTORY, 'railroad')
-  RAILROAD_INDEX = File.join(RAILROAD_DIR, 'index.html')
+  RAILROAD_FILE = File.join(RAILROAD_DIR, 'index.html')
   
   task :railroad => ['railroad:all'] do
   end
@@ -15,20 +15,23 @@ namespace :metrics do
   
     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(RAILROAD_DIR,'models.png')}`
+      #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(RAILROAD_DIR,'controllers.png')}`
+      #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(RAILROAD_DIR,'aasm.png')}`
+      #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

data/spec/flog_spec.rb

@@ -45,6 +45,67 @@ BM
          1.1: entryRelationship
 IM
 
+    @assignment_method = <<-MTHD
+    Total Flog = 21.6 (5.4 +/- 3.3 flog / method)
+
+    ActivityReport#existing_measure_attributes=: (8.5)
+         4.1: assignment
+         1.8: id
+         1.6: to_s
+         1.4: []
+         1.4: activity_report_measures
+         1.2: each
+         1.2: branch
+    MTHD
+
+    @class_methods_grouped_together = <<-MTHD
+    Total Flog = 61.8 (7.7 +/- 95.3 flog / method)
+
+    User#none: (32.8)
+         7.2: include
+         3.6: validates_length_of
+         3.6: validates_format_of
+         2.4: validates_presence_of
+         2.4: validates_uniqueness_of
+         1.4: bad_login_message
+         1.4: name_regex
+         1.4: bad_email_message
+         1.4: bad_name_message
+         1.4: login_regex
+         1.4: email_regex
+         1.2: private
+         1.2: has_and_belongs_to_many
+         1.2: before_create
+         1.2: attr_accessible
+         0.4: lit_fixnum
+    MTHD
+  end
+  
+  it "should be able to parse class_methods_grouped_together" do
+    page = Base.parse(@class_methods_grouped_together)
+    page.should_not be_nil
+    page.score.should == 61.8
+    page.scanned_methods.size.should == 1
+    sm = page.scanned_methods.first
+    sm.name.should == 'User#none'
+    sm.score.should == 32.8
+    
+    sm.operators.size.should == 16
+    sm.operators.first.score.should == 7.2
+    sm.operators.first.operator.should == "include"
+    
+    sm.operators.last.score.should == 0.4
+    sm.operators.last.operator.should == "lit_fixnum"
+  end
+  
+  it "should be able to parse an assignment method" do
+    page = Base.parse(@assignment_method)
+    page.should_not be_nil
+    page.score.should == 21.6
+    page.scanned_methods.size.should == 1
+    sm = page.scanned_methods.first
+    sm.name.should == 'ActivityReport#existing_measure_attributes='
+    sm.score.should == 8.5
   end
 
   it "should be able to parse an alpha only method" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: p8-metric_fu
 version: !ruby/object:Gem::Version 
-  version: 0.8.4.16
+  version: 0.8.9
 platform: ruby
 authors: 
 - Jake Scruggs
@@ -69,7 +69,7 @@ dependencies:
       - !ruby/object:Gem::Version 
         version: 0.0.0
     version: 
-description: Code metrics from Flog, Flay, RCov, Saikuro, Churn, and Rails' stats task
+description: Code metrics from Flog, Flay, RCov, Saikuro, Churn, Reek, Roodi and Rails' stats task
 email: jake.scruggs@gmail.com
 executables: []