codeclimate 0.2.12 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 9cbb3a16763441f73ecd126b74ac0cd7e086315f
-  data.tar.gz: 17dd70c727bb8ad8496e65aabcdf3fd6d0ba2254
+  metadata.gz: ae822f93038f3d619b712789705ffae7c208c4c4
+  data.tar.gz: b0e1790279fb9722a45a1d732aaeb742c6c31103
 SHA512:
-  metadata.gz: 4ff9002f80834f1bccfa64838fbd88730b9f4246f03bbe24a3b7603caeb1cb551075b70fb64c02de968875ace8fd959444004d0865614ba8ca53308e13b8dcb0
-  data.tar.gz: ad2cf0955f53a1b2c2085bada281958735a8afaef4865abc6e262702215a96374e7ff5cb99798c9671afaa6df4bb4c8171d5e9e3e203561491c43e4d9787e327
+  metadata.gz: 7211c83e5b25b001e3ffe13e9cbc10d75a4261807d02fb9778cf2bb443738073eefb5171797654634b8553cbfa77483ae687720109444d0d8d6b7101980e5711
+  data.tar.gz: b292451406d827dd80bc3429e474c869a452fe1e4f48c9bee7efe2007a88a2adab5dcc41991f40f5fd85239611672711185ad0604cf6ba5d0717d41798109643

data/lib/cc/analyzer/formatters.rb

@@ -12,7 +12,7 @@ module CC
       }.freeze
 
       def self.resolve(name)
-        FORMATTERS[name.to_sym].new or raise InvalidFormatterError, "'#{name}' is not a valid formatter. Valid options are: #{FORMATTERS.keys.join(", ")}"
+        FORMATTERS[name.to_sym].new(ENV['FILESYSTEM_DIR']) or raise InvalidFormatterError, "'#{name}' is not a valid formatter. Valid options are: #{FORMATTERS.keys.join(", ")}"
       end
     end
   end

data/lib/cc/analyzer/formatters/formatter.rb

@@ -2,7 +2,8 @@ module CC
   module Analyzer
     module Formatters
       class Formatter
-        def initialize(output = $stdout)
+        def initialize(filesystem, output = $stdout)
+          @filesystem = filesystem
           @output = output
         end
 

data/lib/cc/analyzer/formatters/json_formatter.rb

@@ -3,7 +3,8 @@ module CC
     module Formatters
       class JSONFormatter < Formatter
 
-        def initialize
+        def initialize(filesystem)
+          @filesystem = filesystem
           @has_begun = false
         end
 

data/lib/cc/analyzer/formatters/plain_text_formatter.rb

@@ -35,7 +35,8 @@ module CC
 
             IssueSorter.new(file_issues).by_location.each do |issue|
               if location = issue["location"]
-                print(colorize(LocationDescription.new(location, ": "), :cyan))
+                source_buffer = @filesystem.source_buffer_for(location["path"])
+                print(colorize(LocationDescription.new(source_buffer, location, ": "), :cyan))
               end
 
               print(issue["description"])

data/lib/cc/analyzer/location_description.rb

@@ -1,26 +1,23 @@
 module CC
   module Analyzer
     class LocationDescription
-      def initialize(location, suffix = "")
+      def initialize(source_buffer, location, suffix = "")
+        @source_buffer = source_buffer
         @location = location
         @suffix = suffix
       end
 
       def to_s
-        str = ""
         if location["lines"]
-          str << render_lines
-        elsif positions = location["positions"]
-          str << render_position(positions["begin"])
-
-          if positions["end"]
-            str << "-"
-            str << render_position(positions["end"])
-          end
+          begin_line = location["lines"]["begin"]
+          end_line = location["lines"]["end"]
+        elsif location["positions"]
+          begin_line = position_to_line(location["positions"]["begin"])
+          end_line = position_to_line(location["positions"]["end"])
         end
 
+        str = render_lines(begin_line, end_line)
         str << suffix unless str.blank?
-
         str
       end
 
@@ -28,23 +25,20 @@ module CC
 
       attr_reader :location, :suffix
 
-      def render_lines
-        str = location["lines"]["begin"].to_s
-        str << "-#{location["lines"]["end"]}" if location["lines"]["end"]
-        str
+      def render_lines(begin_line, end_line)
+        if end_line == begin_line
+          begin_line.to_s
+        else
+          "#{begin_line}-#{end_line}"
+        end
       end
 
-      def render_position(position)
-        str = ""
-
+      def position_to_line(position)
         if position["line"]
-          str << position["line"].to_s
-          str << ":#{position["column"]}" if position["column"]
-        elsif position["offset"]
-          str << position["offset"].to_s
+          position["line"]
+        else
+          @source_buffer.decompose_position(position["offset"]).first
         end
-
-        str
       end
     end
   end

data/lib/cc/cli/analyze.rb

@@ -43,7 +43,7 @@ module CC
       end
 
       def formatter
-        @formatter ||= Formatters::PlainTextFormatter.new
+        @formatter ||= Formatters::PlainTextFormatter.new(filesystem)
       end
 
       def source_dir

data/lib/cc/cli/init.rb

@@ -11,6 +11,7 @@ module CC
         else
           create_codeclimate_yaml
           say "Config file .codeclimate.yml successfully generated.\nEdit and then try running 'validate-config' to check configuration."
+          create_default_configs
         end
       end
 
@@ -33,6 +34,27 @@ module CC
         filesystem.write_path(CODECLIMATE_YAML, config.to_yaml)
       end
 
+      def create_default_configs
+        available_configs.each do |config_path|
+          file_name = File.basename(config_path)
+          if filesystem.exist?(file_name)
+            say "Skipping generating #{file_name} file (already exists)."
+          else
+            filesystem.write_path(file_name, File.read(config_path))
+            say "Config file #{file_name} successfully generated."
+          end
+        end
+      end
+
+      def available_configs
+        all_paths = eligible_engines.flat_map do |engine_name, _|
+          engine_directory = File.expand_path("../../../../config/#{engine_name}", __FILE__)
+          Dir.glob("#{engine_directory}/*", File::FNM_DOTMATCH)
+        end
+
+        all_paths.reject { |path| ['.', '..'].include?(File.basename(path)) }
+      end
+
       def exclude_paths(paths)
         expanded_paths = []
         paths.each do |dir|
@@ -52,7 +74,7 @@ module CC
       end
 
       def eligible_engines
-        engine_registry.list.each_with_object({}) do |(engine_name, config), result|
+        @eligible_engines ||= engine_registry.list.each_with_object({}) do |(engine_name, config), result|
           if engine_eligible?(config)
             result[engine_name] = config
           end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: codeclimate
 version: !ruby/object:Gem::Version
-  version: 0.2.12
+  version: 0.3.0
 platform: ruby
 authors:
 - Code Climate
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-08-23 00:00:00.000000000 Z
+date: 2015-09-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport