nucop 0.1.2 → 0.3.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f9fc165f5dbcc9624bfd5ed3c1c372f3f81ff325ca44d59990c4db36a2f0da39
-  data.tar.gz: a10769b0dc3d5de55c23442b4d27ea6bedecde3db859b3e679b5583c2b3e9846
+  metadata.gz: 40f334562fcb3d2ff95256e19d78f2f33ab300a155814d11dae6d5aa7d2c6f3e
+  data.tar.gz: 61b561150890b4f2cd0ec877ebaaf11468af19a1087e5a0ebbb7791a7bec9c26
 SHA512:
-  metadata.gz: 43218b9441df727be126479a83813cbfc048af1a472f1fc1a5dc365871eaad0cf95bab8b091abf34238243dd3f3843fe0d0408a57e59a17193de288f0e2ed697
-  data.tar.gz: 986be2c74320253d2746d25229fd9ad724f61029e1200d7b447b7373aa83ec813921af6bd3745eff5d0cd92e5f51c69a7176881bde42a2f95526cfc1311bd6ab
+  metadata.gz: c5d7935525a7f3e71779314e40de44e5a65efeed0a878a4340ec88cb030d54bc40c34e7dc2d00923d0ee272aa9be9ac7d52d9ea641b85e8f007068dfe0d067d7
+  data.tar.gz: 327247f6c795b46fda02787e460d6a4a38e8e8464f5c4ae73dad4709a1694a398d6926fe86d25465b8ed768a73d5b24734031928260fb3777e83a2a59f2a96fc

data/bin/nucop

@@ -5,4 +5,4 @@ $LOAD_PATH.unshift("#{__dir__}/../lib")
 require "nucop"
 require "nucop/cli"
 
-Nucop::CLI.start(ARGV)
+Nucop::Cli.start(ARGV)

data/lib/nucop.rb

@@ -3,9 +3,9 @@ require "nucop/version"
 require "rubocop"
 require "git_diff_parser"
 
-Dir[File.join(__dir__, "nucop/helpers/**/*.rb")].each { |f| require f }
-Dir[File.join(__dir__, "nucop/formatters/**/*.rb")].each { |f| require f }
-Dir[File.join(__dir__, "nucop/cops/**/*.rb")].each { |f| require f }
+Dir[File.join(__dir__, "nucop/helpers/**/*.rb")].sort.each { |f| require f }
+Dir[File.join(__dir__, "nucop/formatters/**/*.rb")].sort.each { |f| require f }
+Dir[File.join(__dir__, "nucop/cops/**/*.rb")].sort.each { |f| require f }
 
 module Nucop
 end

data/lib/nucop/cli.rb

@@ -5,11 +5,12 @@ RUBOCOP_DEFAULT_CONFIG_FILE = ".rubocop.yml"
 CONFIGURATION_FILEPATH = ".nucop.yml"
 
 module Nucop
-  class CLI < Thor
+  class Cli < Thor
     desc "diff_enforced", "run RuboCop on the current diff using only the enforced cops"
     method_option "commit-spec", default: "origin/master", desc: "the commit used to determine the diff."
     method_option "auto-correct", type: :boolean, default: false, desc: "runs RuboCop with auto-correct option"
-    method_option "junit_report", type: :string, default: "", desc: "runs RuboCop with junit formatter option"
+    method_option "junit_report", type: :string, default: nil, desc: "runs RuboCop with junit formatter option"
+    method_option "json", type: :string, default: nil, desc: "Output results as JSON format to the provided file"
     def diff_enforced
       invoke :diff, nil, options.merge(only: cops_to_enforce.join(","))
     end
@@ -66,16 +67,30 @@ module Nucop
     def rubocop(files = nil)
       print_cops_being_run(options[:only])
       config_file = options[:"exclude-backlog"] ? RUBOCOP_DEFAULT_CONFIG_FILE : options[:rubocop_todo_config_file]
-      junit_report_path = options[:"junit_report"]
-      junit_report_options = junit_report_path.to_s.empty? ? "" : "--format Nucop::Formatters::JUnitFormatter --out #{junit_report_path} --format progress"
-
       rubocop_requires = [
         "--require rubocop-rspec",
         "--require rubocop-performance",
         "--require rubocop-rails"
       ]
 
-      system("bundle exec rubocop --parallel #{rubocop_requires.join(' ')} #{junit_report_options} --force-exclusion --config #{config_file} #{pass_through_option(options, 'auto-correct')} #{pass_through_flag(options, 'only')} #{files}")
+      formatters = []
+      formatters << "--format Nucop::Formatters::JUnitFormatter --out #{options[:junit_report]}" if options[:junit_report]
+      formatters << "--format json --out #{options[:json]}" if options[:json]
+      formatters << "--format progress" if formatters.any?
+
+      command = [
+        "bundle exec rubocop",
+        "--parallel",
+        rubocop_requires.join(" "),
+        formatters.join(" "),
+        "--force-exclusion",
+        "--config", config_file,
+        pass_through_option(options, "auto-correct"),
+        pass_through_flag(options, "only"),
+        files
+      ].join(" ")
+
+      system(command)
     end
 
     desc "regen_backlog", "update the RuboCop backlog, disabling offending files and excluding all cops with over 500 violating files."
@@ -235,7 +250,9 @@ module Nucop
     end
 
     def enabled_cops
-      YAML.load(`bundle exec rubocop --parallel --show-cops`).select { |_, config| config["Enabled"] }.map(&:first)
+      YAML.load(`bundle exec rubocop --parallel --show-cops`) # rubocop:disable Security/YAMLLoad
+        .select { |_, config| config["Enabled"] }
+        .map(&:first)
     end
 
     # Override Thor's options method to include Nucop's options

data/lib/nucop/formatters/junit_formatter.rb

@@ -21,16 +21,17 @@ module Nucop
         # One test case per cop per file
         COPS.each do |cop|
           cop_offences = offences.select { |offence| offence.cop_name == cop.cop_name }
-          unless cop_offences.empty?
-            REXML::Element.new("testcase", @testsuite).tap do |f|
-              f.attributes["classname"] = file.gsub(/\.rb\Z/, "").gsub("#{Dir.pwd}/", "").tr("/", ".")
-              f.attributes["name"]      = "Rubocop: #{cop.cop_name}"
-              f.attributes["file"]      = cop.cop_name
-              cop_offences.each do |offence|
-                REXML::Element.new("failure", f).tap do |e|
-                  e.add_text("#{offence.message}\n\n")
-                  e.add_text(offence.location.to_s.sub("/usr/src/app/", ""))
-                end
+
+          next if cop_offences.empty?
+
+          REXML::Element.new("testcase", @testsuite).tap do |f|
+            f.attributes["classname"] = file.gsub(/\.rb\Z/, "").gsub("#{Dir.pwd}/", "").tr("/", ".")
+            f.attributes["name"] = "Rubocop: #{cop.cop_name}"
+            f.attributes["file"] = cop.cop_name
+            cop_offences.each do |offence|
+              REXML::Element.new("failure", f).tap do |e|
+                e.add_text("#{offence.message}\n\n")
+                e.add_text(offence.location.to_s.sub("/usr/src/app/", ""))
               end
             end
           end

data/lib/nucop/helpers/factory_bot_helper.rb

@@ -1,7 +1,7 @@
 module Nucop
   module Helpers
     module FactoryBotHelper
-      extend self
+      module_function
 
       FACTORY_BOT_METHODS = [
         :build,

data/lib/nucop/helpers/next_cop_for_promotion.rb

@@ -30,7 +30,7 @@ module Nucop
             data << count_match.captures.first
           end
 
-          if (name_match = line.match(/^(\w+[\/]\w+):$/))
+          if (name_match = line.match(/^(\w+\/\w+):$/))
             data << name_match.captures.first
           end
         end

data/lib/nucop/version.rb

@@ -1,3 +1,3 @@
 module Nucop
-  VERSION = "0.1.2"
+  VERSION = "0.3.2"
 end

metadata

@@ -1,127 +1,127 @@
 --- !ruby/object:Gem::Specification
 name: nucop
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.3.2
 platform: ruby
 authors:
 - Jason Schweier
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-11-01 00:00:00.000000000 Z
+date: 2020-07-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: git_diff_parser
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 13.0.0
-  type: :development
+        version: '3.2'
+  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 13.0.0
+        version: '3.2'
 - !ruby/object:Gem::Dependency
-  name: rspec
+  name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 3.9.0
-  type: :development
+        version: 0.88.0
+  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 3.9.0
+        version: 0.88.0
 - !ruby/object:Gem::Dependency
-  name: git_diff_parser
+  name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: '3.2'
+        version: 1.7.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: '3.2'
+        version: 1.7.0
 - !ruby/object:Gem::Dependency
-  name: rubocop
+  name: rubocop-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.75.1
+        version: 2.6.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.75.1
+        version: 2.6.0
 - !ruby/object:Gem::Dependency
-  name: rubocop-performance
+  name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.5.0
+        version: 1.42.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.5.0
+        version: 1.42.0
 - !ruby/object:Gem::Dependency
-  name: rubocop-rails
+  name: ruby-progressbar
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.3.2
+        version: '1.10'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.3.2
+        version: '1.10'
 - !ruby/object:Gem::Dependency
-  name: rubocop-rspec
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.36.0
-  type: :runtime
+        version: 13.0.1
+  type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.36.0
+        version: 13.0.1
 - !ruby/object:Gem::Dependency
-  name: ruby-progressbar
+  name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.10'
-  type: :runtime
+        version: 3.9.0
+  type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.10'
+        version: 3.9.0
 description: 
 email:
 - jasons@nulogy.com
@@ -148,9 +148,14 @@ files:
 - lib/nucop/helpers/next_cop_for_promotion.rb
 - lib/nucop/version.rb
 - spec/spec_helper.rb
-homepage: 
-licenses: []
-metadata: {}
+homepage: https://rubygems.org/gems/nucop
+licenses:
+- MIT
+metadata:
+  homepage_uri: https://github.com/nulogy/nucop
+  changelog_uri: https://github.com/nulogy/nucop/blob/master/CHANGELOG.md
+  source_code_uri: https://github.com/nulogy/nucop
+  bug_tracker_uri: https://github.com/nulogy/nucop/issues
 post_install_message: 
 rdoc_options: []
 require_paths:
@@ -166,8 +171,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6
+rubygems_version: 3.1.4
 signing_key: 
 specification_version: 4
 summary: Nulogy's implementation of RuboCop, including custom cops and additional