fukuzatsu 0.9.12 → 0.9.13

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f3435a15f15b81c3a57c8b8f325f1c9acdd21447
-  data.tar.gz: c1803f26411ef439239dab446743ae4231162ef3
+  metadata.gz: 33b9be43725bfc69fd427bc78857cb173db66586
+  data.tar.gz: 93eb38e086bcf10b4e38cccc9cec53e3a10bcd7b
 SHA512:
-  metadata.gz: 672355b3c4cf948b6d4222242de5be9aa0bbb7cb35757a6adff631e4c3467a11917a1349f813e65c948628c87cb4e13d1c07d02971f57b00b524c9fcffbbfc7a
-  data.tar.gz: 2f36f1ba855de4eb02e86b26ca72c21b3bf088723860fb26a7d9441f839e40eada29796c1a54f1c77a404e38f7efc11c13c20b516f65f64ab7b18ce86acf9b70
+  metadata.gz: 0f42bf703830a6e88d468b0fc64c96f3de69c9b69a79557ebe59c6a0913b3858ed68c498685daddcdf454df3bbf1fa622a042b7632e568b09602dec6828299d6
+  data.tar.gz: 38fa1c7d5ecbd1be0725af017d162f58ef23f5c0b2cbd43014ac224461b56ead7df38dc08c596549e4b83c33782afa300a61dddf71bce645118f8bc27a246522

data/lib/fukuzatsu/cli.rb

@@ -12,43 +12,62 @@ module Fukuzatsu
     method_option :format, :type => :string, :default => 'text', :aliases => "-f"
     method_option :threshold, :type => :numeric, :default => 0, :aliases => "-t"
 
-    def check(path)
+    def check(path="./")
 
       file_summary = []
       file_complexities = []
+      last_file = {}
 
       file_list(path).each do |path_to_file|
         file = ParsedFile.new(path_to_file: path_to_file)
-        case options['format']
-        when 'html'
-          Formatters::Html.new(file).export
-        when 'csv'
-          Formatters::Csv.new(file).export
-        else
-          Formatters::Text.new(file).export
-        end
-        file_summary << {path_to_file: path_to_file, class_name: "#{file.class_name}", complexity: file.complexity}
+        parser = formatter(options).new(file)
+        parser.export
+        file_summary << {
+          results_file: "#{parser.output_path}/#{parser.filename}",
+          path_to_file: path_to_file,
+          class_name: "#{file.class_name}",
+          complexity: file.complexity
+        }
         file_complexities << file.complexity
       end
 
-      Formatters::HtmlIndex.new(file_summary).export if options['format'] == 'html'
-
-      handle_complexity(file_complexities.sort.last, options['threshold'])
+      last_file = handle_index(file_summary) if options['format'] == 'html'
+      report(last_file[:results_file], file_summary.map{|s| s[:results_file]})
+      handle_complexity(options, file_complexities.sort.last, options['threshold'])
 
     end
 
+    default_task :check
+
     private
 
-    def write_index(file_list)
-      Formatters::Html.write_index(file_list)
+    def formatter(options)
+      formatter = case options['format']
+      when 'html'
+        Formatters::Html
+      when 'csv'
+        Formatters::Csv
+      else
+        Formatters::Text
+      end
+    end
+
+    def handle_index(file_summary)
+      index = Formatters::HtmlIndex.new(file_summary)
+      index.export
+      {results_file: "#{index.output_path}/#{index.filename}"}
+    end
+
+    def report(last_file, file_list)
+     return if options['format'] == "text"
+     puts "Results written to:"
+     puts last_file.present? && "#{last_file}" || file_list.join("\r\n")
     end
 
-    def handle_complexity(max, threshold)
-      return if options['threshold'] == 0
+    def handle_complexity(options, highest_complexity, threshold)
+      return if options['threshold'].to_i == 0
       return if highest_complexity <= options['threshold']
-      puts "=" * 40
       puts "Maximum complexity is #{highest_complexity}, which is greater than the threshold of #{options['threshold']}."
-      puts "=" * 40
       exit 1
     end
 
@@ -62,4 +81,4 @@ module Fukuzatsu
 
   end
 
-end
+end

data/lib/fukuzatsu/formatters/base.rb

@@ -18,8 +18,12 @@ module Formatters
       ["class", "method", "complexity"]
     end
 
+    def root_path
+      "doc/fukuzatsu"
+    end
+
     def output_path
-      output_path = "doc/fukuzatsu/#{self.file.path_to_file.split('/')[0..-2].join("/")}"
+      output_path = "#{root_path}/#{self.file.path_to_file.split('/')[0..-2].join("/")}"
       FileUtils.mkpath(output_path)
       output_path
     end
@@ -28,6 +32,10 @@ module Formatters
       self.file.path_to_file.split('/')[-1] + file_extension
     end
 
+    def file_extension
+      ""
+    end
+
     def export
       begin
         outfile = File.open("#{output_path}/#{filename}", 'w')

data/lib/fukuzatsu/formatters/html.rb

@@ -4,17 +4,6 @@ module Formatters
 
     include Formatters::Base
 
-    def self.write_index(file_list)
-      index_template = File.read(File.dirname(__FILE__) + "/templates/index.html.haml")
-      Haml::Engine.new(index_template).render(
-        Object.new, {
-          file_list: file_list,
-          date: Time.now.strftime("%Y/%m/%d"),
-          time: Time.now.strftime("%l:%M %P")
-        }
-      )
-    end
-
     def header
       columns.map{|col| "<th>#{col.titleize}</th>"}.join("\r\n")
     end

data/lib/fukuzatsu/formatters/html_index.rb

@@ -15,9 +15,8 @@ module Formatters
     end
 
     def output_path
-      output_path = "doc/fukuzatsu/"
-      FileUtils.mkpath(output_path)
-      output_path
+      FileUtils.mkpath(root_path)
+      root_path
     end
 
     def content

data/lib/fukuzatsu/formatters/templates/index.html.haml

@@ -85,7 +85,8 @@
       $(document).ready(function(){
         $('.output-table').dataTable({
           bLengthChange: false,
-          iDisplayLength: 25000
+          iDisplayLength: 25000,
+          "order": [[2, "desc"]]
         });
       });
 

data/lib/fukuzatsu/formatters/templates/output.html.haml

@@ -72,7 +72,8 @@
       $(document).ready(function(){
         $('.output-table').dataTable({
           bLengthChange: false,
-          iDisplayLength: 25000
+          iDisplayLength: 25000,
+          "order": [[2, "desc"]]
         });
       });
 

data/lib/fukuzatsu/version.rb

@@ -1,3 +1,3 @@
 module Fukuzatsu
-  VERSION = "0.9.12"
+  VERSION = "0.9.13"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fukuzatsu
 version: !ruby/object:Gem::Version
-  version: 0.9.12
+  version: 0.9.13
 platform: ruby
 authors:
 - Bantik
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-07-04 00:00:00.000000000 Z
+date: 2014-07-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ephemeral