reek 6.0.2 → 6.5.0

data/lib/reek/smell_detectors/control_parameter_helpers/control_parameter_finder.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require_relative './call_in_condition_finder'
+require_relative 'call_in_condition_finder'
 require_relative '../../ast/node'
 
 module Reek

data/lib/reek/smell_detectors/data_clump.rb

@@ -84,7 +84,7 @@ module Reek
       end
 
       def candidate_clumps
-        candidate_methods.each_cons(max_copies + 1).map do |methods|
+        candidate_methods.combination(max_copies + 1).map do |methods|
           common_argument_names_for(methods)
         end.select do |clump|
           clump.length >= min_clump_size

data/lib/reek/smell_detectors/duplicate_method_call.rb

@@ -67,11 +67,11 @@ module Reek
       class FoundCall
         def initialize(call_node)
           @call_node = call_node
-          @occurences = []
+          @occurrences = []
         end
 
         def record(occurence)
-          occurences.push occurence
+          occurrences.push occurence
         end
 
         def call
@@ -79,16 +79,16 @@ module Reek
         end
 
         def occurs
-          occurences.length
+          occurrences.length
         end
 
         def lines
-          occurences.map(&:line)
+          occurrences.map(&:line)
         end
 
         private
 
-        attr_reader :call_node, :occurences
+        attr_reader :call_node, :occurrences
       end
 
       private_constant :FoundCall

data/lib/reek/smell_detectors/instance_variable_assumption.rb

@@ -20,7 +20,7 @@ module Reek
       # @return [Array<SmellWarning>]
       #
       def sniff
-        assumptions = (variables_from_context - variables_from_initialize).uniq
+        assumptions = (variables_from_context - variables_from_initializers).uniq
 
         assumptions.map do |assumption|
           build_smell_warning(assumption)
@@ -42,14 +42,14 @@ module Reek
           parameters: { assumption: assumption.to_s })
       end
 
-      def variables_from_initialize
-        initialize_exp = method_expressions.detect do |method|
-          method.name == :initialize
-        end
-
-        return [] unless initialize_exp
+      def variables_from_initializers
+        variables_from_initialize.map do |method|
+          method.each_node(:ivasgn).map(&:name)
+        end.flatten
+      end
 
-        initialize_exp.each_node(:ivasgn).map(&:name)
+      def variables_from_initialize
+        method_expressions.select { |method| method.name == :initialize }
       end
 
       def variables_from_context

data/lib/reek/smell_detectors/nested_iterators.rb

@@ -26,7 +26,7 @@ module Reek
       # The name of the config field that sets the names of any
       # methods for which nesting should not be considered
       IGNORE_ITERATORS_KEY = 'ignore_iterators'
-      DEFAULT_IGNORE_ITERATORS = ['tap'].freeze
+      DEFAULT_IGNORE_ITERATORS = ['tap', 'Tempfile.create'].freeze
 
       def self.default_config
         super.merge(
@@ -128,8 +128,9 @@ module Reek
 
       # @quality :reek:FeatureEnvy
       def ignored_iterator?(exp)
-        ignore_iterators.any? { |pattern| /#{pattern}/ =~ exp.call.name } ||
-          exp.without_block_arguments?
+        ignore_iterators.any? do |pattern|
+          /#{pattern}/ =~ exp.call.format_to_ruby
+        end || exp.without_block_arguments?
       end
     end
   end

data/lib/reek/smell_detectors/uncommunicative_module_name.rb

@@ -31,7 +31,7 @@ module Reek
       DEFAULT_ACCEPT_PATTERNS = [].freeze
 
       def self.default_config
-        super().merge(
+        super.merge(
           REJECT_KEY => DEFAULT_REJECT_PATTERNS,
           ACCEPT_KEY => DEFAULT_ACCEPT_PATTERNS)
       end

data/lib/reek/smell_detectors/unused_private_method.rb

@@ -50,9 +50,9 @@ module Reek
       # @return [Array<Hit>]
       #
       def hits
-        unused_private_methods.map do |defined_method|
+        unused_private_methods.filter_map do |defined_method|
           Hit.new(defined_method) unless ignore_method?(defined_method)
-        end.compact
+        end
       end
 
       #
@@ -61,6 +61,7 @@ module Reek
       def unused_private_methods
         defined_private_methods = context.defined_instance_methods(visibility: :private)
         called_method_names     = context.instance_method_calls.map(&:name)
+        called_method_names.concat(context.instance_method_names_via_to_call)
 
         defined_private_methods.reject do |defined_method|
           called_method_names.include?(defined_method.name)

data/lib/reek/smell_warning.rb

@@ -59,7 +59,7 @@ module Reek
 
     # @public
     def to_hash
-      stringified_params = Hash[parameters.map { |key, val| [key.to_s, val] }]
+      stringified_params = parameters.transform_keys(&:to_s)
       base_hash.merge(stringified_params)
     end
 

data/lib/reek/source/source_code.rb

@@ -75,11 +75,17 @@ module Reek
 
       def code
         @code ||=
-          case source
-          when File, Pathname then source.read
-          when IO             then source.readlines.join
-          when String         then source
-          end.force_encoding(Encoding::UTF_8)
+          begin
+            str =
+              case source
+              when File, Pathname then source.read
+              when IO             then source.readlines.join
+              when String         then source
+              end
+
+            str = str.dup if str.frozen?
+            str.force_encoding(Encoding::UTF_8)
+          end
       end
 
       attr_reader :parser, :source

data/lib/reek/source/source_locator.rb

@@ -62,10 +62,8 @@ module Reek
 
             false
           end
-        elsif path_excluded?(path)
-          true
         else
-          false
+          path_excluded?(path)
         end
       end
 

data/lib/reek/spec/should_reek_of.rb

@@ -69,19 +69,21 @@ module Reek
       end
 
       def set_failure_messages_for_smell_type
-        self.failure_message = "Expected #{origin} to reek of #{smell_type}, "\
-          'but it didn\'t'
-        self.failure_message_when_negated = "Expected #{origin} not to reek "\
-          "of #{smell_type}, but it did"
+        self.failure_message = "Expected #{origin} to reek of #{smell_type}, " \
+                               'but it didn\'t'
+        self.failure_message_when_negated = "Expected #{origin} not to reek " \
+                                            "of #{smell_type}, but it did"
       end
 
       def set_failure_messages_for_smell_details
-        self.failure_message = "Expected #{origin} to reek of #{smell_type} "\
-          "(which it did) with smell details #{smell_details}, which it didn't.\n"\
-          "The number of smell details I had to compare with the given one was #{matching_smell_types.count} "\
-          "and here they are:\n"\
+        self.failure_message =
+          "Expected #{origin} to reek of #{smell_type} " \
+          "(which it did) with smell details #{smell_details}, which it didn't.\n" \
+          "The number of smell details I had to compare with the given one was #{matching_smell_types.count} " \
+          "and here they are:\n" \
           "#{all_relevant_smell_details_formatted}"
-        self.failure_message_when_negated = "Expected #{origin} not to reek of "\
+        self.failure_message_when_negated =
+          "Expected #{origin} not to reek of " \
           "#{smell_type} with smell details #{smell_details}, but it did"
       end
 

data/lib/reek/spec.rb

@@ -92,7 +92,7 @@ module Reek
     end
 
     #
-    # See the documentaton for "reek_of".
+    # See the documentation for "reek_of".
     #
     # Notable differences to reek_of:
     #   1.) "reek_of" doesn't mind if there are other smells of a different type.

data/lib/reek/version.rb

@@ -3,11 +3,11 @@
 # @public
 module Reek
   #
-  # This module holds the Reek version informations
+  # This module holds the Reek version information
   #
   # @public
   module Version
     # @public
-    STRING = '6.0.2'
+    STRING = '6.5.0'
   end
 end

data/reek.gemspec

@@ -1,34 +1,42 @@
-require File.join(File.dirname(__FILE__), 'lib/reek/version')
+# frozen_string_literal: true
 
-Gem::Specification.new do |s|
-  s.name = 'reek'
-  s.version = Reek::Version::STRING
+require_relative 'lib/reek/version'
 
-  s.authors = ['Kevin Rutherford', 'Timo Roessner', 'Matijs van Zuijlen', 'Piotr Szotkowski']
-  s.description =
+Gem::Specification.new do |spec|
+  spec.name = 'reek'
+  spec.version = Reek::Version::STRING
+  spec.authors = ['Kevin Rutherford', 'Timo Roessner', 'Matijs van Zuijlen', 'Piotr Szotkowski']
+  spec.email = ['timo.roessner@googlemail.com']
+
+  spec.summary = 'Code smell detector for Ruby'
+  spec.description =
     'Reek is a tool that examines Ruby classes, modules and methods and reports ' \
     'any code smells it finds.'
+  spec.homepage = 'https://github.com/troessner/reek'
+  spec.license = 'MIT'
+
+  spec.extra_rdoc_files = ['CHANGELOG.md', 'License.txt']
+  spec.files = Dir.chdir(File.expand_path(__dir__)) do
+    `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features|samples|docs|tasks)/}) }
+  end
+  spec.files << Dir['docs/yard_plugin.rb']
 
-  s.license = 'MIT'
-  s.email = ['timo.roessner@googlemail.com']
-  s.extra_rdoc_files = ['CHANGELOG.md', 'License.txt']
-  s.files = `git ls-files -z`.split("\0")
-  s.executables = s.files.grep(%r{^bin/}).map { |path| File.basename(path) }
-  s.homepage = 'https://github.com/troessner/reek'
-  s.rdoc_options = %w(--main README.md -x assets/|bin/|config/|features/|spec/|tasks/)
-  s.required_ruby_version = '>= 2.4.0'
-  s.summary = 'Code smell detector for Ruby'
+  spec.executables = spec.files.grep(%r{^bin/}).map { |path| File.basename(path) }
+  spec.rdoc_options = %w(--main README.md -x assets/|bin/|config/|features/|spec/|tasks/)
+  spec.required_ruby_version = '>= 3.1.0'
 
-  s.metadata = {
-    'homepage_uri'      => 'https://github.com/troessner/reek',
-    'source_code_uri'   => 'https://github.com/troessner/reek',
-    'bug_tracker_uri'   => 'https://github.com/troessner/reek/issues',
-    'changelog_uri'     => 'https://github.com/troessner/reek/CHANGELOG.md',
-    'documentation_uri' => 'https://www.rubydoc.info/gems/reek'
+  spec.metadata = {
+    'homepage_uri'          => 'https://github.com/troessner/reek',
+    'source_code_uri'       => 'https://github.com/troessner/reek',
+    'bug_tracker_uri'       => 'https://github.com/troessner/reek/issues',
+    'changelog_uri'         => 'https://github.com/troessner/reek/CHANGELOG.md',
+    'documentation_uri'     => 'https://www.rubydoc.info/gems/reek',
+    'rubygems_mfa_required' => 'true'
   }
 
-  s.add_runtime_dependency 'kwalify', '~> 0.7.0'
-  s.add_runtime_dependency 'parser',  '< 2.8', '>= 2.5.0.0', '!= 2.5.1.1'
-  s.add_runtime_dependency 'psych',   '~> 3.1'
-  s.add_runtime_dependency 'rainbow', '>= 2.0', '< 4.0'
+  spec.add_dependency 'dry-schema', '~> 1.13'
+  spec.add_dependency 'logger',  '~> 1.6'
+  spec.add_dependency 'parser',  '~> 3.3.0'
+  spec.add_dependency 'rainbow', '>= 2.0', '< 4.0'
+  spec.add_dependency 'rexml',   '~> 3.1'
 end