churn 1.0.5 → 1.0.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6b9e47efbf25dd33432e6a632a22547482c1823786f9fcfe5cc0ddc8867a3d9f
-  data.tar.gz: 7aa8ef62039ced894b3ca96934223fe7c231565c49c07a8fcc261896d6c1c28c
+  metadata.gz: 60626635b7c857bc6a5129f45d42aff3eba37caa4b478492316a2030a0bdebcc
+  data.tar.gz: 321cfa0f67c7c1a5b0e0ca2b6da68b925d5f598a73af631f13f293a454bcb48c
 SHA512:
-  metadata.gz: fb77031df8cc5a6a864e9c13ffa30ed32282094f38ec126af32f026605a232e647557fd74d54bf03fcb18aaa580e77835889dfa5ef9487a2daefb17feee506ca
-  data.tar.gz: c307331df0e19c3cb6031314ff154e5c1158c41021f0dd9ca4ce965bbe6a7c1840c80a29c4f9edc5945d0c986324085224e76cfb2dd97b0d7e67dbda6511b42a
+  metadata.gz: '0584e9330499a66eeabea707a4aa586ec891d6f5a71acd0d6017d5e745e50f4f9950b4feab97651599a7a279a6cfd8bad4ba1981c21bbc0ba5e564f8ad63ca6e'
+  data.tar.gz: 6a6a562a668f501d69a51f5bd35a59e6da48aee865431ce17b6ee934c1e2622fe84312e861b1c0d1d805172089398e16107c4201f0ba5227a64ff8d7aff04448

data/churn.gemspec

@@ -27,7 +27,19 @@ Gem::Specification.new do |s|
   s.add_runtime_dependency "ruby_parser", "~> 3.0"
   # TODO: Just have clean output and drop hirb
   s.add_runtime_dependency "hirb", ">= 0"
-  # TODO: Make this an optional depency and eventually
-  # replace dependency with straight net::http
-  s.add_runtime_dependency "rest-client", ">= 1.6.0"
+
+  s.add_development_dependency 'pry-byebug'
+  s.add_development_dependency "minitest", "~> 5.3"
+  s.add_development_dependency "shoulda", ">= 0"
+  s.add_development_dependency "test_construct", "~> 2.0.0"
+  s.add_development_dependency "rake", ">= 0"
+  s.add_development_dependency "mocha", "~> 1.1.0"
+  s.add_development_dependency "simplecov", ">= 0"
+  s.add_development_dependency "yard", "~> 0.9.20"
+  # TODO: what is using this ronn
+  s.add_development_dependency "ronn"
+  #s.add_development_dependency "kramdown", platform: :jruby
+  s.add_development_dependency "redcarpet"
+  # TODO: what is using this? Tests I think, remove
+  s.add_development_dependency 'activesupport', '>= 4.0'
 end

data/lib/churn/calculator.rb

@@ -4,8 +4,6 @@ require 'ruby_parser'
 require 'json'
 require 'hirb'
 require 'fileutils'
-require 'rest_client'
-
 
 require_relative 'scm/source_control'
 require_relative 'scm/git_analyzer'
@@ -54,21 +52,10 @@ module Churn
       else
         emit
         analyze
-        remote_report
         print ? self.to_s : self.to_h
       end
     end
 
-    def remote_report
-      if @churn_options.report_host
-        puts "posting churn results to #{@churn_options.report_host}"
-        data = {:name => @churn_options.name, :revision => @revisions.first, :data => self.to_h}.to_json
-        RestClient.post @churn_options.report_host, {"results" => data}, :content_type => :json, :accept => :json
-      end
-    rescue Errno::ECONNREFUSED
-      puts "error posting churn results connection refused to host: #{@churn_options.report_host}"
-    end
-
     # this method generates the past history of a churn project from first
     # commit to current running the report for oldest commits first so they
     # are built up correctly
@@ -284,7 +271,16 @@ module Churn
     end
 
     def reject_ignored_files(files)
-      files.reject{ |file, _| @ignores.any?{ |ignore| /#{ignore}/ =~ file } }
+       files.reject do |file, _|
+         @ignores.any? do |ignore|
+           begin
+             /#{ignore}/ =~ file
+           rescue RegexpError => e
+             puts "churn: ignoring invalid regex: #{e}"
+             false
+           end
+         end
+       end
     end
 
   end

data/lib/churn/location_mapping.rb

@@ -6,6 +6,7 @@ module Churn
   # this is based off https://github.com/seattlerb/ruby_parser which seems to have some known line number bugs
   # perhaps look at and move more to the style of line numbers from metric_fu
   # https://github.com/metricfu/metric_fu/blob/master/lib/data_structures/line_numbers.rb
+  # NOTE: showing more line number issues with Ruby 2.6
   class LocationMapping < SexpProcessor
 
     attr_reader :klasses_collection, :methods_collection
@@ -20,7 +21,7 @@ module Churn
     end
 
     def get_info(file)
-      ast = @parser.process(File.read(file), file)
+      ast = @parser.process(File.read(file).chomp, file)
       process ast
     end
 
@@ -39,7 +40,7 @@ module Churn
 
     def deep_last_line(exp)
       lines = []
-      exp.deep_each{|x| lines << x.line }
+      exp.deep_each{|x| lines << x.line; }
       lines.max + 1
     end
 

data/lib/churn/options.rb

@@ -7,9 +7,8 @@ module Churn
     DEFAULT_CHURN_DIRECTORY = "tmp/churn"
     DEFAULT_MINIMUM_CHURN_COUNT = 5
     DEFAULT_START_TIME = '3 months ago'
-    DEFAULT_REPORT_HOST = 'http://churn.picoappz.com'
 
-    attr_accessor :data_directory, :minimum_churn_count, :ignores, :start_date, :history, :report_host, :name, :file_extension, :file_prefix
+    attr_accessor :data_directory, :minimum_churn_count, :ignores, :start_date, :history, :name, :file_extension, :file_prefix
 
     def initialize()
       @data_directory      = DEFAULT_CHURN_DIRECTORY
@@ -17,7 +16,6 @@ module Churn
       @ignores             = '/dev/null'
       @start_date          = DEFAULT_START_TIME
       @history             = nil
-      @report_host         = nil
       @name                = nil
       @file_extension      = nil
       @file_prefix         = nil
@@ -35,17 +33,6 @@ module Churn
       if @history=='true'
         @history = DEFAULT_START_TIME
       end
-      if !options[:report].nil? && options[:report]!=''
-        @report_host         = options[:report]
-        if @report_host=='true'
-          @report_host = DEFAULT_REPORT_HOST
-        end
-      end
-
-      @name = options[:name] if !options[:name].nil? && options[:name]!=''
-      if !@report_host.nil? && @name.nil?
-        raise ArgumentError.new "If you are reporting churn results you must pass a valid github project name in the form of username/project_name"
-      end
 
       self
     end

data/lib/churn/version.rb

@@ -1,3 +1,3 @@
 module Churn
-  VERSION = "1.0.5"
+  VERSION = "1.0.6"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: churn
 version: !ruby/object:Gem::Version
-  version: 1.0.5
+  version: 1.0.6
 platform: ruby
 authors:
 - Dan Mayer
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-11-13 00:00:00.000000000 Z
+date: 2020-05-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: main
@@ -81,19 +81,159 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rest-client
+  name: pry-byebug
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.6.0
-  type: :runtime
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: minitest
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '5.3'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '5.3'
+- !ruby/object:Gem::Dependency
+  name: shoulda
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: test_construct
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.0.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.0.0
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: mocha
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.1.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.1.0
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: yard
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.9.20
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.9.20
+- !ruby/object:Gem::Dependency
+  name: ronn
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: redcarpet
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: activesupport
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '4.0'
+  type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.6.0
+        version: '4.0'
 description: 'High method and class churn has been shown to have increased bug and
   error rates. This gem helps you know what is changing a lot so you can do additional
   testing, code review, or refactoring to try to tame the volatile code. '