rails_best_practices 1.20.0 → 1.20.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7b66ea6e0e742d6db68f867fed12f8242aa0d767b7b3fb4728043c97984f455a
-  data.tar.gz: 6f8ff9290459f8436908396fec8612d6050c7f5b278697475c4601e8931c0c19
+  metadata.gz: e7a2b67944c27b7a11e48f2c490a4ea1080fc4df39ad1d59b20cdeaedfb4b083
+  data.tar.gz: d96f6d7da9909bac895162f70ae1247a98a8df39da7fd2ce17365e959568c4f6
 SHA512:
-  metadata.gz: '03850b2873726ec5f3928bc1a846320927c56793c71221c715bbbc630cd1f10333b52ac9a713a78db3cbf1db6e2f25a34493d06bcce76d184f2447f5d888deb1'
-  data.tar.gz: 72586f6144df24dd9fd11861c443554f3903bc0cd8401bd08b1e6a89ff356d545b475b8e465b7e95e1803dd198517695fb91d81d6a746df9654e51525e638365
+  metadata.gz: 244acdd4067af5f87364236d4fe5bc40d5a4d68b98d1c6786aaf0e53c7766626ef670c4aaa61d793a47bba643fddcd2cbd545a4697dff8d0a3d6f7238912c2b5
+  data.tar.gz: 9b4ca4c95172317d9017691399fcb9211295bad98959477fa8fa24a054a7a6283dc7cdbabd926e925ccf0f3ea9bfe1dde3e594cc42a2db7bd3b37758cb7bdc6e

data/Gemfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 gemspec
 
@@ -6,4 +8,4 @@ gem 'coveralls', require: false
 gem 'guard'
 gem 'guard-rspec'
 gem 'pry'
-gem 'rspec', '~> 2.14.0'
+gem 'rspec'

data/Gemfile.lock

@@ -1,9 +1,9 @@
 PATH
   remote: .
   specs:
-    rails_best_practices (1.20.0)
+    rails_best_practices (1.20.1)
       activesupport
-      code_analyzer (>= 0.5.1)
+      code_analyzer (>= 0.5.2)
       erubis
       i18n
       json
@@ -13,17 +13,17 @@ PATH
 GEM
   remote: https://rubygems.org/
   specs:
-    activesupport (6.0.2.1)
+    activesupport (6.1.3.2)
       concurrent-ruby (~> 1.0, >= 1.0.2)
-      i18n (>= 0.7, < 2)
-      minitest (~> 5.1)
-      tzinfo (~> 1.1)
-      zeitwerk (~> 2.2)
+      i18n (>= 1.6, < 2)
+      minitest (>= 5.1)
+      tzinfo (~> 2.0)
+      zeitwerk (~> 2.3)
     awesome_print (1.8.0)
-    code_analyzer (0.5.1)
+    code_analyzer (0.5.2)
       sexp_processor
     coderay (1.1.2)
-    concurrent-ruby (1.1.5)
+    concurrent-ruby (1.1.8)
     coveralls (0.8.23)
       json (>= 1.8, < 3)
       simplecov (~> 0.16.1)
@@ -50,7 +50,7 @@ GEM
     haml (5.1.2)
       temple (>= 0.8.0)
       tilt
-    i18n (1.7.0)
+    i18n (1.8.10)
       concurrent-ruby (~> 1.0)
     json (2.3.0)
     listen (3.2.1)
@@ -58,7 +58,7 @@ GEM
       rb-inotify (~> 0.9, >= 0.9.10)
     lumberjack (1.0.13)
     method_source (0.9.2)
-    minitest (5.13.0)
+    minitest (5.14.4)
     nenv (0.3.0)
     notiffany (0.1.3)
       nenv (~> 0.1)
@@ -71,16 +71,21 @@ GEM
     rb-inotify (0.10.1)
       ffi (~> 1.0)
     require_all (3.0.0)
-    rspec (2.14.1)
-      rspec-core (~> 2.14.0)
-      rspec-expectations (~> 2.14.0)
-      rspec-mocks (~> 2.14.0)
-    rspec-core (2.14.8)
-    rspec-expectations (2.14.5)
-      diff-lcs (>= 1.1.3, < 2.0)
-    rspec-mocks (2.14.6)
-    ruby-progressbar (1.10.1)
-    sexp_processor (4.13.0)
+    rspec (3.9.0)
+      rspec-core (~> 3.9.0)
+      rspec-expectations (~> 3.9.0)
+      rspec-mocks (~> 3.9.0)
+    rspec-core (3.9.1)
+      rspec-support (~> 3.9.1)
+    rspec-expectations (3.9.1)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.9.0)
+    rspec-mocks (3.9.1)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.9.0)
+    rspec-support (3.9.2)
+    ruby-progressbar (1.11.0)
+    sexp_processor (4.15.2)
     shellany (0.0.1)
     simplecov (0.16.1)
       docile (~> 1.1)
@@ -94,12 +99,11 @@ GEM
     term-ansicolor (1.7.1)
       tins (~> 1.0)
     thor (1.0.1)
-    thread_safe (0.3.6)
     tilt (2.0.10)
     tins (1.22.2)
-    tzinfo (1.2.6)
-      thread_safe (~> 0.1)
-    zeitwerk (2.2.2)
+    tzinfo (2.0.4)
+      concurrent-ruby (~> 1.0)
+    zeitwerk (2.4.2)
 
 PLATFORMS
   ruby
@@ -114,7 +118,7 @@ DEPENDENCIES
   pry
   rails_best_practices!
   rake
-  rspec (~> 2.14.0)
+  rspec
   slim
 
 BUNDLED WITH

data/Guardfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # A sample Guardfile
 # More info at https://github.com/guard/guard#readme
 

data/Rakefile

@@ -1 +1,3 @@
+# frozen_string_literal: true
+
 require 'bundler/gem_tasks'

data/lib/rails_best_practices.rb

@@ -11,5 +11,4 @@ require 'rails_best_practices/reviews'
 require 'rails_best_practices/option_parser'
 require 'rails_best_practices/cli'
 
-module RailsBestPractices
-end
+module RailsBestPractices; end

data/lib/rails_best_practices/analyzer.rb

@@ -104,30 +104,31 @@ module RailsBestPractices
     #
     # @return [Array] all files for parsing
     def parse_files
-      @parse_files ||= begin
-        files = expand_dirs_to_files(@path)
-        files = file_sort(files)
+      @parse_files ||=
+        begin
+          files = expand_dirs_to_files(@path)
+          files = file_sort(files)
 
-        if @options['only'].present?
-          files = file_accept(files, @options['only'])
-        end
+          if @options['only'].present?
+            files = file_accept(files, @options['only'])
+          end
 
-        # By default, tmp, vender, spec, test, features are ignored.
-        %w[vendor spec test features tmp].each do |dir|
-          files = file_ignore(files, File.join(@path, dir)) unless @options[dir]
-        end
+          # By default, tmp, vender, spec, test, features are ignored.
+          %w[vendor spec test features tmp].each do |dir|
+            files = file_ignore(files, File.join(@path, dir)) unless @options[dir]
+          end
 
-        # Exclude files based on exclude regexes if the option is set.
-        @options['exclude'].each do |pattern|
-          files = file_ignore(files, pattern)
-        end
+          # Exclude files based on exclude regexes if the option is set.
+          @options['exclude'].each do |pattern|
+            files = file_ignore(files, pattern)
+          end
 
-        %w[Capfile Gemfile Gemfile.lock].each do |file|
-          files.unshift File.join(@path, file)
-        end
+          %w[Capfile Gemfile Gemfile.lock].each do |file|
+            files.unshift File.join(@path, file)
+          end
 
-        files.compact
-      end
+          files.compact
+        end
     end
 
     # expand all files with extenstion rb, erb, haml, slim, builder and rxml under the dirs
@@ -158,7 +159,10 @@ module RailsBestPractices
       models = files.find_all { |file| file =~ Core::Check::MODEL_FILES }
       mailers = files.find_all { |file| file =~ Core::Check::MAILER_FILES }
       helpers = files.find_all { |file| file =~ Core::Check::HELPER_FILES }
-      others = files.find_all { |file| file !~ Core::Check::MAILER_FILES && file !~ Core::Check::MODEL_FILES && file !~ Core::Check::HELPER_FILES }
+      others =
+        files.find_all do |file|
+          file !~ Core::Check::MAILER_FILES && file !~ Core::Check::MODEL_FILES && file !~ Core::Check::HELPER_FILES
+        end
       models + mailers + helpers + others
     end
 
@@ -194,7 +198,10 @@ module RailsBestPractices
     def load_hg_info
       hg_progressbar = ProgressBar.create(title: 'Hg Info', total: errors.size) if display_bar?
       errors.each do |error|
-        hg_info = `cd #{@runner.class.base_path} && hg blame -lvcu #{error.filename[@runner.class.base_path.size..-1].gsub(/^\//, '')} | sed -n /:#{error.line_number.split(',').first}:/p`
+        info_command = "cd #{@runner.class.base_path}"
+        info_command += " && hg blame -lvcu #{error.filename[@runner.class.base_path.size..-1].gsub(%r{^/}, '')}"
+        info_command += " | sed -n /:#{error.line_number.split(',').first}:/p"
+        hg_info = system(info_command)
         unless hg_info == ''
           hg_commit_username = hg_info.split(':')[0].strip
           error.hg_username = hg_commit_username.split(/\ /)[0..-2].join(' ')
@@ -208,9 +215,11 @@ module RailsBestPractices
     # load git commit and git username info.
     def load_git_info
       git_progressbar = ProgressBar.create(title: 'Git Info', total: errors.size) if display_bar?
-      start = @runner.class.base_path =~ /\/$/ ? @runner.class.base_path.size : @runner.class.base_path.size + 1
+      start = @runner.class.base_path =~ %r{/$} ? @runner.class.base_path.size : @runner.class.base_path.size + 1
       errors.each do |error|
-        git_info = `cd #{@runner.class.base_path} && git blame -L #{error.line_number.split(',').first},+1 #{error.filename[start..-1]}`
+        info_command = "cd #{@runner.class.base_path}"
+        info_command += " && git blame -L #{error.line_number.split(',').first},+1 #{error.filename[start..-1]}"
+        git_info = system(info_command)
         unless git_info == ''
           git_commit, git_username = git_info.split(/\d{4}-\d{2}-\d{2}/).first.split('(')
           error.git_commit = git_commit.split(' ').first.strip
@@ -224,7 +233,10 @@ module RailsBestPractices
     # output errors with html format.
     def output_html_errors
       require 'erubis'
-      template = @options['template'] ? File.read(File.expand_path(@options['template'])) : File.read(File.join(File.dirname(__FILE__), '..', '..', 'assets', 'result.html.erb'))
+      template =
+        @options['template'] ?
+          File.read(File.expand_path(@options['template'])) :
+          File.read(File.join(File.dirname(__FILE__), '..', '..', 'assets', 'result.html.erb'))
 
       if @options['with-github']
         last_commit_id = @options['last-commit-id'] || `cd #{@runner.class.base_path} && git rev-parse HEAD`.chomp
@@ -235,27 +247,28 @@ module RailsBestPractices
       File.open(@options['output-file'], 'w+') do |file|
         eruby = Erubis::Eruby.new(template)
         file.puts eruby.evaluate(
-          errors: errors,
-          error_types: error_types,
-          textmate: @options['with-textmate'],
-          vscode: @options['with-vscode'],
-          sublime: @options['with-sublime'],
-          mvim: @options['with-mvim'],
-          github: @options['with-github'],
-          github_name: @options['github-name'],
-          last_commit_id: last_commit_id,
-          git: @options['with-git'],
-          hg: @options['with-hg']
-        )
+                    errors: errors,
+                    error_types: error_types,
+                    textmate: @options['with-textmate'],
+                    vscode: @options['with-vscode'],
+                    sublime: @options['with-sublime'],
+                    mvim: @options['with-mvim'],
+                    github: @options['with-github'],
+                    github_name: @options['github-name'],
+                    last_commit_id: last_commit_id,
+                    git: @options['with-git'],
+                    hg: @options['with-hg']
+                  )
       end
     end
 
     def output_xml_errors
       require 'rexml/document'
 
-      document = REXML::Document.new.tap do |d|
-        d << REXML::XMLDecl.new
-      end
+      document =
+        REXML::Document.new.tap do |d|
+          d << REXML::XMLDecl.new
+        end
 
       checkstyle = REXML::Element.new('checkstyle', document)
 
@@ -289,13 +302,10 @@ module RailsBestPractices
 
     # output errors with json format.
     def output_json_errors
-      errors_as_hashes = errors.map do |err|
-        {
-          filename:    err.filename,
-          line_number: err.line_number,
-          message:     err.message
-        }
-      end
+      errors_as_hashes =
+        errors.map do |err|
+          { filename: err.filename, line_number: err.line_number, message: err.message }
+        end
 
       File.open(@options['output-file'], 'w+') do |file|
         file.write JSON.dump(errors_as_hashes)

data/lib/rails_best_practices/core/check.rb

@@ -3,24 +3,25 @@
 module RailsBestPractices
   module Core
     # A Check class that takes charge of checking the sexp.
+
     class Check < CodeAnalyzer::Checker
-      ALL_FILES = /.*/
-      CONTROLLER_FILES = /app\/(controllers|cells)\/.*\.rb$/
-      MIGRATION_FILES = /db\/migrate\/.*\.rb$/
-      MODEL_FILES = /app\/models\/.*\.rb$/
-      MAILER_FILES = /app\/models\/.*mailer\.rb$|app\/mailers\/.*\.rb/
-      VIEW_FILES = /app\/(views|cells)\/.*\.(erb|haml|slim|builder|rxml)$/
-      PARTIAL_VIEW_FILES = /app\/(views|cells)\/.*\/_.*\.(erb|haml|slim|builder|rxml)$/
-      ROUTE_FILES = /config\/routes.*\.rb/
-      SCHEMA_FILE = /db\/schema\.rb/
-      HELPER_FILES = /app\/helpers\/.*\.rb$/
-      DEPLOY_FILES = /config\/deploy.*\.rb/
-      CONFIG_FILES = /config\/(application|environment|environments\/.*)\.rb/
-      INITIALIZER_FILES = /config\/initializers\/.*\.rb/
-      CAPFILE = /Capfile/
-      GEMFILE_LOCK = /Gemfile\.lock/
-
-      SKIP_FILES = /db\/schema.rb/
+      ALL_FILES = /.*/.freeze
+      CONTROLLER_FILES = %r{app/(controllers|cells)/.*\.rb$}.freeze
+      MIGRATION_FILES = %r{db/migrate/.*\.rb$}.freeze
+      MODEL_FILES = %r{app/models/.*\.rb$}.freeze
+      MAILER_FILES = %r{app/models/.*mailer\.rb$|app/mailers/.*\.rb}.freeze
+      VIEW_FILES = %r{app/(views|cells)/.*\.(erb|haml|slim|builder|rxml)$}.freeze
+      PARTIAL_VIEW_FILES = %r{app/(views|cells)/.*/_.*\.(erb|haml|slim|builder|rxml)$}.freeze
+      ROUTE_FILES = %r{config/routes.*\.rb}.freeze
+      SCHEMA_FILE = %r{db/schema\.rb}.freeze
+      HELPER_FILES = %r{app/helpers/.*\.rb$}.freeze
+      DEPLOY_FILES = %r{config/deploy.*\.rb}.freeze
+      CONFIG_FILES = %r{config/(application|environment|environments/.*)\.rb}.freeze
+      INITIALIZER_FILES = %r{config/initializers/.*\.rb}.freeze
+      CAPFILE = /Capfile/.freeze
+      GEMFILE_LOCK = /Gemfile\.lock/.freeze
+
+      SKIP_FILES = %r{db/schema.rb}.freeze
 
       def initialize(options = {})
         options.each do |key, value|
@@ -33,7 +34,7 @@ module RailsBestPractices
       # @param [String] the file name of node.
       # @return [Boolean] true if the check will need to parse the file.
       def parse_file?(node_file)
-        is_interesting_file?(node_file) and !is_ignored?(node_file)
+        node_file.is_a?(String) && is_interesting_file?(node_file) && !is_ignored?(node_file)
       end
 
       def is_interesting_file?(node_file)
@@ -60,13 +61,10 @@ module RailsBestPractices
       # @param [String] filename, is the filename of source code
       # @param [Integer] line_number, is the line number of the source code which is reviewing
       def add_error(message, filename = @node.file, line_number = @node.line_number)
-        errors << RailsBestPractices::Core::Error.new(
-          filename: filename,
-          line_number: line_number,
-          message: message,
-          type: self.class.to_s,
-          url: url
-        )
+        errors <<
+          RailsBestPractices::Core::Error.new(
+            filename: filename, line_number: line_number, message: message, type: self.class.to_s, url: url
+          )
       end
 
       # errors that violate the rails best practices.
@@ -99,9 +97,9 @@ module RailsBestPractices
         end
       end
 
-      class <<self
+      class << self
         def url(url = nil)
-          url ?  @url = url : @url
+          url ? @url = url : @url
         end
 
         def debug?
@@ -114,6 +112,7 @@ module RailsBestPractices
       end
 
       # Helper to parse the class name.
+
       module Classable
         def self.included(base)
           base.class_eval do
@@ -165,6 +164,7 @@ module RailsBestPractices
       end
 
       # Helper to parse the module name.
+
       module Moduleable
         def self.included(base)
           base.class_eval do
@@ -197,7 +197,15 @@ module RailsBestPractices
       module Callable
         def self.included(base)
           base.class_eval do
-            interesting_nodes :call, :fcall, :var_ref, :vcall, :command_call, :command, :alias, :bare_assoc_hash, :method_add_arg
+            interesting_nodes :call,
+                              :fcall,
+                              :var_ref,
+                              :vcall,
+                              :command_call,
+                              :command,
+                              :alias,
+                              :bare_assoc_hash,
+                              :method_add_arg
 
             # remembe the message of call node.
             add_callback :start_call do |node|
@@ -371,10 +379,8 @@ module RailsBestPractices
             class_name = '.*' if class_name == '*'
             class_expression = Regexp.new class_name
 
-            class_names = Prepares.klasses
-                                  .select { |klass| klass.class_name == method.class_name }
-                                  .map(&:extend_class_name)
-                                  .compact
+            class_names =
+              Prepares.klasses.select { |klass| klass.class_name == method.class_name }.map(&:extend_class_name).compact
 
             class_names.unshift method.class_name
             matched = class_names.any? { |name| name =~ class_expression }
@@ -385,6 +391,7 @@ module RailsBestPractices
       end
 
       # Helper to parse the access control.
+
       module Accessable
         def self.included(base)
           base.class_eval do

data/lib/rails_best_practices/core/checks_loader.rb

@@ -14,17 +14,19 @@ module RailsBestPractices
 
       # load all reviews according to configuration.
       def load_reviews
-        load_checks_from_config { |check_name| RailsBestPractices::Reviews.const_get(check_name.gsub(/Check$/, 'Review')) }
+        load_checks_from_config do |check_name|
+          RailsBestPractices::Reviews.const_get(check_name.gsub(/Check$/, 'Review'))
+        end
       end
 
       private
 
-        # read the checks from yaml config.
+      # read the checks from yaml config.
       def checks_from_config
         @checks ||= YAML.load_file @config
       end
 
-        # load all checks from the configuration
+      # load all checks from the configuration
       def load_checks_from_config(&block)
         checks_from_config.each_with_object([]) do |check, active_checks|
           check_instance = instantiate_check(block, *check)
@@ -32,13 +34,13 @@ module RailsBestPractices
         end
       end
 
-        # instantiates a check
+      # instantiates a check
       def instantiate_check(block, check_name, options)
         check_class = load_check_class(check_name, &block)
-        check_class.new(options || {}) unless check_class.nil?
+        check_class&.new(options || {})
       end
 
-        # loads the class for a check by calling the given block
+      # loads the class for a check by calling the given block
       def load_check_class(check_name)
         yield(check_name)
       rescue NameError