reek 4.6.0 → 5.0.0

data/lib/reek/report/formatter/progress_formatter.rb

@@ -1,80 +0,0 @@
-# frozen_string_literal: true
-module Reek
-  module Report
-    module Formatter
-      module ProgressFormatter
-        #
-        # Base class for progress formatters.
-        # Is responsible for formatting the progress emitted for each examiner
-        #
-        # @abstract Override {#header, #progress, #footer} to implement a progress formatter.
-        class Base
-          attr_reader :sources_count
-
-          def initialize(sources_count)
-            @sources_count = sources_count
-          end
-
-          def header
-            raise NotImplementedError
-          end
-
-          def progress(_examiner)
-            raise NotImplementedError
-          end
-
-          def footer
-            raise NotImplementedError
-          end
-        end
-
-        #
-        # Shows the status of each source as either a dot (.) or an S
-        #
-        class Dots < Base
-          NO_WARNINGS_COLOR = :green
-          WARNINGS_COLOR = :red
-
-          def header
-            "Inspecting #{sources_count} file(s):\n"
-          end
-
-          def progress(examiner)
-            examiner.smelly? ? display_smelly : display_clean
-          end
-
-          def footer
-            "\n\n"
-          end
-
-          private
-
-          def display_clean
-            Rainbow('.').color(NO_WARNINGS_COLOR)
-          end
-
-          def display_smelly
-            Rainbow('S').color(WARNINGS_COLOR)
-          end
-        end
-
-        #
-        # Does not show progress
-        #
-        class Quiet < Base
-          def header
-            ''
-          end
-
-          def progress(_examiner)
-            ''
-          end
-
-          def footer
-            ''
-          end
-        end
-      end
-    end
-  end
-end

data/lib/reek/report/formatter/simple_warning_formatter.rb

@@ -1,35 +0,0 @@
-# frozen_string_literal: true
-
-module Reek
-  module Report
-    module Formatter
-      #
-      # Basic formatter that just shows a simple message for each warning,
-      # prepended with the result of the passed-in location formatter.
-      #
-      class SimpleWarningFormatter
-        def initialize(location_formatter: BlankLocationFormatter)
-          @location_formatter = location_formatter
-        end
-
-        def format(warning)
-          "#{location_formatter.format(warning)}#{warning.base_message}"
-        end
-
-        # :reek:UtilityFunction
-        def format_hash(warning)
-          warning.yaml_hash
-        end
-
-        # :reek:UtilityFunction
-        def format_code_climate_hash(warning)
-          CodeClimateFormatter.new(warning).render
-        end
-
-        private
-
-        attr_reader :location_formatter
-      end
-    end
-  end
-end

data/lib/reek/report/formatter/wiki_link_warning_formatter.rb

@@ -1,35 +0,0 @@
-# frozen_string_literal: true
-require_relative 'simple_warning_formatter'
-
-module Reek
-  module Report
-    module Formatter
-      #
-      # Formatter that adds a link to the wiki to the basic message from
-      # SimpleWarningFormatter.
-      #
-      class WikiLinkWarningFormatter < SimpleWarningFormatter
-        BASE_URL_FOR_HELP_LINK = 'https://github.com/troessner/reek/blob/master/docs/'.freeze
-
-        def format(warning)
-          "#{super} [#{explanatory_link(warning)}]"
-        end
-
-        def format_hash(warning)
-          super.merge('wiki_link' => explanatory_link(warning))
-        end
-
-        private
-
-        def explanatory_link(warning)
-          "#{BASE_URL_FOR_HELP_LINK}#{class_name_to_param(warning.smell_type)}.md"
-        end
-
-        # :reek:UtilityFunction
-        def class_name_to_param(name)
-          name.split(/(?=[A-Z])/).join('-')
-        end
-      end
-    end
-  end
-end

data/lib/reek/report/formatter.rb

@@ -1,32 +0,0 @@
-# frozen_string_literal: true
-require_relative 'code_climate/code_climate_formatter'
-require_relative 'formatter/heading_formatter'
-require_relative 'formatter/location_formatter'
-require_relative 'formatter/progress_formatter'
-require_relative 'formatter/simple_warning_formatter'
-require_relative 'formatter/wiki_link_warning_formatter'
-
-module Reek
-  module Report
-    #
-    # Formatter handling the formatting of the report at large.
-    # Formatting of the individual warnings is handled by the
-    # passed-in warning formatter.
-    #
-    module Formatter
-      module_function
-
-      def format_list(warnings, formatter: SimpleWarningFormatter.new)
-        warnings.map { |warning| "  #{formatter.format(warning)}" }.join("\n")
-      end
-
-      def header(examiner)
-        count = examiner.smells_count
-        result = Rainbow("#{examiner.description} -- ").cyan +
-          Rainbow("#{count} warning").yellow
-        result += Rainbow('s').yellow unless count == 1
-        result
-      end
-    end
-  end
-end

data/samples/configuration/non_public_modifiers_mask.reek

@@ -1,3 +0,0 @@
----
-UtilityFunction:
-  public_methods_only: true

data/samples/smelly_with_inline_mask.rb

@@ -1,8 +0,0 @@
-# Smelly class
-# disables :reek:UncommunicativeVariableName
-class Smelly
-  # This will reek of UncommunicativeMethodName
-  def x
-    y = 10 # This will NOT reek of UncommunicativeVariableName
-  end
-end

data/samples/smelly_with_modifiers.rb

@@ -1,12 +0,0 @@
-# Smelly class for testing purposes
-#
-# Not necessary for the feature per se but for
-# removing distracting output. :reek:UnusedPrivateMethod
-#
-class Klass
-  def public_method(arg) arg.to_s; end
-  protected
-  def protected_method(arg) arg.to_s; end
-  private
-  def private_method(arg) arg.to_s; end
-end

data/samples/source_with_hidden_directories/.hidden/uncommunicative_method_name.rb

@@ -1,5 +0,0 @@
-# Klass comment.
-class Klass
-  def m
-  end
-end

data/samples/source_with_non_ruby_files/uncommunicative_parameter_name.rb

@@ -1,6 +0,0 @@
-# Klass comment
-class Klass
-  def method(a)
-    a + @x
-  end
-end