metric_fu 2.0.0 → 2.0.1

data/lib/generators/reek.rb

@@ -1,8 +1,8 @@
 module MetricFu
-  
+
   class Reek < Generator
     REEK_REGEX = /^(\S+) (.*) \((.*)\)$/
-    
+
     def emit
       files_to_reek = MetricFu.reek[:dirs_to_reek].map{|dir| Dir[File.join(dir, "**/*.rb")] }
       files = remove_excluded_files(files_to_reek.flatten)

data/lib/generators/saikuro.rb

@@ -1,34 +1,34 @@
 module MetricFu
- 
+
   class Saikuro < Generator
 
-    def emit 
-      options_string = MetricFu.saikuro.inject("") do |options, option| 
-        options + "--#{option.join(' ')} " unless option == :input_directory 
-      end 
-      
-      MetricFu.saikuro[:input_directory].each do |input_dir| 
-        options_string += "--input_directory #{input_dir} " 
-      end 
-      sh %{saikuro #{options_string}} do |ok, response| 
-        unless ok 
-          puts "Saikuro failed with exit status: #{response.exitstatus}" 
-          exit 1 
-        end 
-      end 
-    end 
+    def emit
+      options_string = MetricFu.saikuro.inject("") do |options, option|
+        options + "--#{option.join(' ')} " unless option == :input_directory
+      end
+
+      MetricFu.saikuro[:input_directory].each do |input_dir|
+        options_string += "--input_directory #{input_dir} "
+      end
+      sh %{saikuro #{options_string}} do |ok, response|
+        unless ok
+          puts "Saikuro failed with exit status: #{response.exitstatus}"
+          exit 1
+        end
+      end
+    end
 
     def format_directories
       dirs = MetricFu.saikuro[:input_directory].join(" | ")
       "\"#{dirs}\""
     end
- 
+
     def analyze
       @files = sort_files(assemble_files)
       @classes = sort_classes(assemble_classes(@files))
       @meths = sort_methods(assemble_methods(@files))
     end
- 
+
     def to_h
       files = @files.map do |file|
         my_file = file.to_h
@@ -41,12 +41,12 @@ module MetricFu
                    }
       }
     end
-    
+
     private
     def sort_methods(methods)
       methods.sort_by {|method| method.complexity.to_i}.reverse
     end
-    
+
     def assemble_methods(files)
       methods = []
       files.each do |file|
@@ -59,15 +59,15 @@ module MetricFu
       end
       methods
     end
-    
+
     def sort_classes(classes)
       classes.sort_by {|k| k.complexity.to_i}.reverse
     end
-    
+
     def assemble_classes(files)
       files.map {|f| f.elements}.flatten
     end
-    
+
     def sort_files(files)
       files.sort_by do |file|
         file.elements.
@@ -75,7 +75,7 @@ module MetricFu
              complexity.to_i
       end.reverse
     end
-    
+
     def assemble_files
       files = []
       Dir.glob("#{metric_directory}/**/*.html").each do |path|
@@ -88,11 +88,11 @@ module MetricFu
       end
       files
     end
-    
+
   end
- 
+
   class Saikuro::SFile
- 
+
     attr_reader :elements
 
     def initialize(path)
@@ -103,7 +103,7 @@ module MetricFu
     ensure
       @file_handle.close if @file_handle
     end
- 
+
     def self.is_valid_text_file?(path)
       File.open(path, "r") do |f|
         if f.eof? || !f.readline.match(/--/)
@@ -113,16 +113,16 @@ module MetricFu
         end
       end
     end
- 
+
     def filename
       File.basename(@path, '_cyclo.html')
     end
- 
+
     def to_h
       merge_classes
       {:classes => @elements}
     end
- 
+
     def get_elements
       begin
         while (line = @file_handle.readline) do
@@ -146,8 +146,8 @@ module MetricFu
         nil
       end
     end
- 
- 
+
+
     def merge_classes
       new_elements = []
       get_class_names.each do |target_class|
@@ -160,7 +160,7 @@ module MetricFu
           lines += el.lines.to_i
           defns << el.defs
         end
- 
+
         new_element = {:class_name => target_class,
                        :complexity => complexity,
                        :lines => lines,
@@ -168,12 +168,12 @@ module MetricFu
         new_element[:methods] = new_element[:methods].
                                 sort_by {|x| x[:complexity] }.
                                 reverse
- 
+
         new_elements << new_element
       end
       @elements = new_elements if new_elements
     end
- 
+
     def get_class_names
       class_names = []
       @elements.each do |element|
@@ -183,18 +183,18 @@ module MetricFu
       end
       class_names
     end
- 
+
   end
- 
+
   class Saikuro::ParsingElement
     TYPE_REGEX=/Type:(.*) Name/
     NAME_REGEX=/Name:(.*) Complexity/
     COMPLEXITY_REGEX=/Complexity:(.*) Lines/
     LINES_REGEX=/Lines:(.*)/
- 
+
     attr_reader :complexity, :lines, :defs, :element_type
     attr_accessor :name
- 
+
     def initialize(line)
       @line = line
       @element_type = line.match(TYPE_REGEX)[1].strip
@@ -203,11 +203,11 @@ module MetricFu
       @lines = line.match(LINES_REGEX)[1].strip
       @defs = []
     end
- 
+
     def <<(line)
       @defs << Saikuro::ParsingElement.new(line)
     end
- 
+
     def to_h
       base = {:name => @name, :complexity => @complexity.to_i, :lines => @lines.to_i}
       unless @defs.empty?

data/lib/generators/stats.rb

@@ -11,7 +11,7 @@ module MetricFu
       lines = remove_noise(output)
 
       @stats = {}
-      
+
       set_global_stats(lines.pop)
       set_granular_stats(lines)
 
@@ -21,9 +21,9 @@ module MetricFu
     def to_h
       {:stats => @stats}
     end
-    
+
     private
-    
+
     def remove_noise(output)
       lines = output.split("\n")
       lines = lines.find_all {|line| line[0].chr != "+" }
@@ -31,14 +31,14 @@ module MetricFu
       lines.shift
       lines
     end
-    
+
     def set_global_stats(totals)
       totals = totals.split("  ").find_all {|el| ! el.empty? }
       @stats[:codeLOC] = totals[0].match(/\d.*/)[0].to_i
       @stats[:testLOC] = totals[1].match(/\d.*/)[0].to_i
       @stats[:code_to_test_ratio] = totals[2].match(/1\:(\d.*)/)[1].to_f
     end
-    
+
     def set_granular_stats(lines)
       @stats[:lines] = lines.map do |line|
         elements = line.split("|")
@@ -47,7 +47,7 @@ module MetricFu
         info_line = {}
         info_line[:name] = elements.shift
         elements.map! {|el| el.to_i }
-        [:lines, :loc, :classes, :methods, 
+        [:lines, :loc, :classes, :methods,
          :methods_per_class, :loc_per_method].each do |sym|
           info_line[sym] = elements.shift
         end

data/lib/graphs/engines/bluff.rb

@@ -12,7 +12,7 @@ module MetricFu
       g.marker_font_size = "10px"
     EOS
   end
-  
+
   class FlayBluffGrapher < FlayGrapher
     def graph!
       content = <<-EOS

data/lib/graphs/engines/gchart.rb

@@ -1,8 +1,8 @@
 module MetricFu
   module GchartGrapher
-    COLORS = %w{009999 FF7400 A60000 008500 E6399B 344AD7 00B860 D5CCB9}    
+    COLORS = %w{009999 FF7400 A60000 008500 E6399B 344AD7 00B860 D5CCB9}
     GCHART_GRAPH_SIZE = "945x317" # maximum permitted image size is 300000 pixels
-    
+
     NUMBER_OF_TICKS = 6
     def determine_y_axis_scale(values)
       values.collect! {|val| val || 0.0 }
@@ -18,10 +18,10 @@ module MetricFu
       end
     end
   end
-  
+
   class Grapher
     include MetricFu::GchartGrapher
-    
+
     def self.require_graphing_gem
       require 'gchart' if MetricFu.graph_engine == :gchart
     rescue LoadError
@@ -64,7 +64,7 @@ module MetricFu
         :filename => File.join(MetricFu.output_directory, 'flog.png'))
     end
   end
-  
+
   class RcovGchartGrapher < RcovGrapher
     def graph!
       url = Gchart.line(
@@ -86,7 +86,7 @@ module MetricFu
       values = []
       legend = @reek_count.keys.sort
       legend.collect {|k| values << @reek_count[k]}
-    
+
       url = Gchart.line(
         :size => GCHART_GRAPH_SIZE,
         :title => URI.escape("Reek: code smells"),
@@ -102,7 +102,7 @@ module MetricFu
         :filename => File.join(MetricFu.output_directory, 'reek.png'))
     end
   end
-  
+
   class RoodiGchartGrapher < RoodiGrapher
     def graph!
       determine_y_axis_scale(@roodi_count)

data/lib/graphs/flog_grapher.rb

@@ -21,8 +21,8 @@ module MetricFu
 
     def calc_top_five_percent_average(metrics)
       return calc_top_five_percent_average_legacy(metrics) if metrics[:flog][:pages]
-      
-      method_scores = metrics[:flog][:method_containers].inject([]) do |method_scores, container| 
+
+      method_scores = metrics[:flog][:method_containers].inject([]) do |method_scores, container|
         method_scores += container[:methods].values.map {|v| v[:score]}
       end
       method_scores.sort!.reverse!
@@ -37,7 +37,7 @@ module MetricFu
         total_for_five_percent / number_of_methods_that_is_five_percent.to_f
       end
     end
-    
+
     def calc_top_five_percent_average_legacy(metrics)
       methods = metrics[:flog][:pages].inject([]) {|methods, page| methods << page[:scanned_methods]}
       methods.flatten!

data/lib/graphs/grapher.rb

@@ -3,7 +3,7 @@ module MetricFu
     def initialize
       self.class.require_graphing_gem
     end
-    
+
     def self.require_graphing_gem
       # to be overridden by charting engines
     end

data/lib/metric_fu.rb

@@ -3,7 +3,7 @@ require 'yaml'
 begin
   require 'active_support/core_ext/object/to_json'
   require 'active_support/core_ext/object/blank'
-rescue LoadError  
+rescue LoadError
   require 'activesupport'
 end
 
@@ -18,7 +18,7 @@ graph_dir        = File.join(MetricFu::LIB_ROOT, 'graphs')
 
 # We need to require these two things first because our other classes
 # depend on them.
-require File.join(base_dir, 'report') 
+require File.join(base_dir, 'report')
 require File.join(base_dir, 'generator')
 require File.join(base_dir, 'graph')
 require File.join(base_dir, 'scoring_strategies')

data/lib/templates/awesome/churn.html.erb

@@ -55,4 +55,4 @@
 
 
 
-<p>Generated on <%= Time.now.localtime %></p>	
+<p>Generated on <%= Time.now.localtime %></p>

data/lib/templates/awesome/css/integrity.css

@@ -332,4 +332,3 @@ a.failed {
   text-align: right; }
   #footer strong {
     font-weight: bold; }
-    

data/lib/templates/awesome/flay.html.erb

@@ -16,7 +16,7 @@
 <table>
   <tr>
     <th>Files</th>
-    <th>Matches</th>		  
+    <th>Matches</th>
   </tr>
   <% count = 0 %>
   <% @flay[:matches].each do |match| %>
@@ -31,4 +31,4 @@
     <% count += 1 %>
   <% end %>
 </table>
-<p>Generated on <%= Time.now.localtime %></p>	
+<p>Generated on <%= Time.now.localtime %></p>

data/lib/templates/awesome/flog.html.erb

@@ -33,7 +33,7 @@
 
 <% @flog[:method_containers].each do |method_container| %>
   <h2 id="<%= method_container[:path].gsub(/[^a-z]+/, '_') %>"><%= link_to_filename(method_container[:path]) %></h2>
-  
+
     <% method_container[:methods].each do |full_method_name, method_info| %>
       <% path, line = method_info[:path].split(":") if method_info[:path] %>
       <p><%= link_to_filename(path, line, full_method_name) %></p>
@@ -52,4 +52,4 @@
       </table>
     <% end %>
 <% end %>
-<p>Generated on <%= Time.now.localtime %></p>	
+<p>Generated on <%= Time.now.localtime %></p>

data/lib/templates/awesome/hotspots.html.erb

@@ -9,8 +9,8 @@ No Hotspots were found.
 <% granularities = [:files, :classes, :methods] %>
 <table>
 <tr valign="top">
-<% granularities.each do |granularity| %>  
-   <th width='33%'> 
+<% granularities.each do |granularity| %>
+   <th width='33%'>
 <%= granularity.to_s.capitalize %></th>
 <% end %>
 </tr>
@@ -24,7 +24,7 @@ No Hotspots were found.
 <% items_length = 0 %>
 <% columns = [0, 1, 2] %>
 <% while (items_length < items[0].length || items_length < items[1].length || items_length < items[2].length) do %>
-    <tr  valign="top"> 
+    <tr  valign="top">
        <% columns.each do |column| %>
        <% item = items[column].length >= items_length ? items[column][items_length] : nil %>
        <% if item %>
@@ -51,4 +51,4 @@ No Hotspots were found.
 <% end %>
 
 
-<p>Generated on <%= Time.now.localtime %></p>	
+<p>Generated on <%= Time.now.localtime %></p>

data/lib/templates/awesome/index.html.erb

@@ -23,7 +23,7 @@
 <% end %>
 <% if @stats %>
   <li class='even failure'><a href="stats.html">Stats</a></li>
-<% end %>  
+<% end %>
 <% if @rails_best_practices %>
   <li class='even failure'><a href="rails_best_practices.html">Rails Best Practices report</a></li>
 <% end %>
@@ -31,4 +31,4 @@
   <li class='even failure'><a href="hotspots.html">Hotspots</a></li>
 <% end %>
 </ul>
-<p>Generated on <%= Time.now.localtime %></p>	
+<p>Generated on <%= Time.now.localtime %></p>