leftovers 0.11.0 → 0.11.2

data/lib/leftovers/processor_builders.rb

@@ -1,20 +1,7 @@
-# frozen-string-literal: true
+# frozen_string_literal: true
 
 module Leftovers
   module ProcessorBuilders
-    autoload(:Action, "#{__dir__}/processor_builders/action")
-    autoload(:AddPrefix, "#{__dir__}/processor_builders/add_prefix")
-    autoload(:AddSuffix, "#{__dir__}/processor_builders/add_suffix")
-    autoload(:Argument, "#{__dir__}/processor_builders/argument")
-    autoload(:Dynamic, "#{__dir__}/processor_builders/dynamic")
-    autoload(:Each, "#{__dir__}/processor_builders/each")
-    autoload(:Itself, "#{__dir__}/processor_builders/itself")
-    autoload(:Keyword, "#{__dir__}/processor_builders/keyword")
-    autoload(:KeywordArgument, "#{__dir__}/processor_builders/keyword_argument")
-    autoload(:Receiver, "#{__dir__}/processor_builders/receiver")
-    autoload(:TransformChain, "#{__dir__}/processor_builders/transform_chain")
-    autoload(:TransformSet, "#{__dir__}/processor_builders/transform_set")
-    autoload(:Transform, "#{__dir__}/processor_builders/transform")
-    autoload(:Value, "#{__dir__}/processor_builders/value")
+    include Autoloader
   end
 end

data/lib/leftovers/processors/add_definition_node.rb

@@ -1,4 +1,4 @@
-# frozen-string-literal: true
+# frozen_string_literal: true
 
 module Leftovers
   module Processors
@@ -7,7 +7,7 @@ module Leftovers
         return unless str
         return if str.empty?
 
-        acc.add_definition_node Leftovers::DefinitionNode.new(current_node, name: str.to_sym)
+        acc.add_definition_node DefinitionNode.new(current_node, name: str.to_sym)
       end
 
       freeze

data/lib/leftovers/processors/camelize.rb

@@ -15,9 +15,9 @@ module Leftovers
         return unless str
 
         @then_processor.process(str.camelize, current_node, matched_node, acc)
-      rescue NoMethodError
-        Leftovers.error <<~MESSAGE
-          Tried using the String#camelize method, but the activesupport gem was not available and/or not required
+      rescue ::NoMethodError
+        ::Leftovers.error <<~MESSAGE
+          Tried using the ::String#camelize method, but the activesupport gem was not available and/or not required
           `gem install activesupport`, and/or add `requires: ['active_support', 'active_support/core_ext/string']` to your .leftovers.yml
         MESSAGE
       end

data/lib/leftovers/processors/deconstantize.rb

@@ -15,9 +15,9 @@ module Leftovers
         return unless str
 
         @then_processor.process(str.deconstantize, current_node, matched_node, acc)
-      rescue NoMethodError
-        Leftovers.error <<~MESSAGE
-          Tried using the String#deconstantize method, but the activesupport gem was not available and/or not required
+      rescue ::NoMethodError
+        ::Leftovers.error <<~MESSAGE
+          Tried using the ::String#deconstantize method, but the activesupport gem was not available and/or not required
           `gem install activesupport`, and/or add `requires: ['active_support', 'active_support/core_ext/string']` to your .leftovers.yml
         MESSAGE
       end

data/lib/leftovers/processors/demodulize.rb

@@ -15,9 +15,9 @@ module Leftovers
         return unless str
 
         @then_processor.process(str.demodulize, current_node, matched_node, acc)
-      rescue NoMethodError
-        Leftovers.error <<~MESSAGE
-          Tried using the String#demodulize method, but the activesupport gem was not available and/or not required
+      rescue ::NoMethodError
+        ::Leftovers.error <<~MESSAGE
+          Tried using the ::String#demodulize method, but the activesupport gem was not available and/or not required
           `gem install activesupport`, and/or add `requires: ['active_support', 'active_support/core_ext/string']` to your .leftovers.yml
         MESSAGE
       end

data/lib/leftovers/processors/each_for_definition_set.rb

@@ -1,4 +1,4 @@
-# frozen-string-literal: true
+# frozen_string_literal: true
 
 module Leftovers
   module Processors
@@ -14,7 +14,7 @@ module Leftovers
       end
 
       def process(str, current_node, matched_node, acc)
-        set = ::Leftovers::DefinitionNodeSet.new
+        set = DefinitionNodeSet.new
 
         @processors.each do |processor|
           processor.process(str, current_node, matched_node, set)

data/lib/leftovers/processors/eval.rb

@@ -1,4 +1,4 @@
-# frozen-string-literal: true
+# frozen_string_literal: true
 
 module Leftovers
   module Processors

data/lib/leftovers/processors/parameterize.rb

@@ -15,9 +15,9 @@ module Leftovers
         return unless str
 
         @then_processor.process(str.parameterize, current_node, matched_node, acc)
-      rescue NoMethodError
-        Leftovers.error <<~MESSAGE
-          Tried using the String#parameterize method, but the activesupport gem was not available and/or not required
+      rescue ::NoMethodError
+        ::Leftovers.error <<~MESSAGE
+          Tried using the ::String#parameterize method, but the activesupport gem was not available and/or not required
           `gem install activesupport`, and/or add `requires: ['active_support', 'active_support/core_ext/string']` to your .leftovers.yml
         MESSAGE
       end

data/lib/leftovers/processors/placeholder.rb

@@ -1,4 +1,4 @@
-# frozen-string-literal: true
+# frozen_string_literal: true
 
 module Leftovers
   module Processors

data/lib/leftovers/processors/pluralize.rb

@@ -15,9 +15,9 @@ module Leftovers
         return unless str
 
         @then_processor.process(str.pluralize, current_node, matched_node, acc)
-      rescue NoMethodError
-        Leftovers.error <<~MESSAGE
-          Tried using the String#pluralize method, but the activesupport gem was not available and/or not required
+      rescue ::NoMethodError
+        ::Leftovers.error <<~MESSAGE
+          Tried using the ::String#pluralize method, but the activesupport gem was not available and/or not required
           `gem install activesupport`, and/or add `requires: ['active_support', 'active_support/core_ext/string']` to your .leftovers.yml
         MESSAGE
       end

data/lib/leftovers/processors/set_default_privacy.rb

@@ -1,4 +1,4 @@
-# frozen-string-literal: true
+# frozen_string_literal: true
 
 module Leftovers
   module Processors

data/lib/leftovers/processors/set_privacy.rb

@@ -1,4 +1,4 @@
-# frozen-string-literal: true
+# frozen_string_literal: true
 
 module Leftovers
   module Processors

data/lib/leftovers/processors/singularize.rb

@@ -15,9 +15,9 @@ module Leftovers
         return unless str
 
         @then_processor.process(str.singularize, current_node, matched_node, acc)
-      rescue NoMethodError
-        Leftovers.error <<~MESSAGE
-          Tried using the String#singularize method, but the activesupport gem was not available and/or not required
+      rescue ::NoMethodError
+        ::Leftovers.error <<~MESSAGE
+          Tried using the ::String#singularize method, but the activesupport gem was not available and/or not required
           `gem install activesupport`, and/or add `requires: ['active_support', 'active_support/core_ext/string']` to your .leftovers.yml
         MESSAGE
       end

data/lib/leftovers/processors/titleize.rb

@@ -15,9 +15,9 @@ module Leftovers
         return unless str
 
         @then_processor.process(str.titleize, current_node, matched_node, acc)
-      rescue NoMethodError
-        Leftovers.error <<~MESSAGE
-          Tried using the String#titleize method, but the activesupport gem was not available and/or not required
+      rescue ::NoMethodError
+        ::Leftovers.error <<~MESSAGE
+          Tried using the ::String#titleize method, but the activesupport gem was not available and/or not required
           `gem install activesupport`, and/or add `requires: ['active_support', 'active_support/core_ext/string']` to your .leftovers.yml
         MESSAGE
       end

data/lib/leftovers/processors/underscore.rb

@@ -15,9 +15,9 @@ module Leftovers
         return unless str
 
         @then_processor.process(str.underscore, current_node, matched_node, acc)
-      rescue NoMethodError
-        Leftovers.error <<~MESSAGE
-          Tried using the String#underscore method, but the activesupport gem was not available and/or not required
+      rescue ::NoMethodError
+        ::Leftovers.error <<~MESSAGE
+          Tried using the ::String#underscore method, but the activesupport gem was not available and/or not required
           `gem install activesupport`, and/or add `requires: ['active_support', 'active_support/core_ext/string']` to your .leftovers.yml
         MESSAGE
       end

data/lib/leftovers/processors.rb

@@ -1,49 +1,7 @@
-# frozen-string-literal: true
+# frozen_string_literal: true
 
 module Leftovers
   module Processors
-    autoload(:AddCall, "#{__dir__}/processors/add_call")
-    autoload(:AddDefinitionNode, "#{__dir__}/processors/add_definition_node")
-    autoload(:AddDynamicPrefix, "#{__dir__}/processors/add_dynamic_prefix")
-    autoload(:AddDynamicSuffix, "#{__dir__}/processors/add_dynamic_suffix")
-    autoload(:AddPrefix, "#{__dir__}/processors/add_prefix")
-    autoload(:AddSuffix, "#{__dir__}/processors/add_suffix")
-    autoload(:AppendSym, "#{__dir__}/processors/append_sym")
-    autoload(:Camelize, "#{__dir__}/processors/camelize")
-    autoload(:Capitalize, "#{__dir__}/processors/capitalize")
-    autoload(:Deconstantize, "#{__dir__}/processors/deconstantize")
-    autoload(:DeleteAfter, "#{__dir__}/processors/delete_after")
-    autoload(:DeleteAfterLast, "#{__dir__}/processors/delete_after_last")
-    autoload(:DeleteBefore, "#{__dir__}/processors/delete_before")
-    autoload(:DeleteBeforeLast, "#{__dir__}/processors/delete_before_last")
-    autoload(:DeletePrefix, "#{__dir__}/processors/delete_prefix")
-    autoload(:DeleteSuffix, "#{__dir__}/processors/delete_suffix")
-    autoload(:Demodulize, "#{__dir__}/processors/demodulize")
-    autoload(:Downcase, "#{__dir__}/processors/downcase")
-    autoload(:Each, "#{__dir__}/processors/each")
-    autoload(:EachForDefinitionSet, "#{__dir__}/processors/each_for_definition_set")
-    autoload(:EachKeyword, "#{__dir__}/processors/each_keyword")
-    autoload(:EachKeywordArgument, "#{__dir__}/processors/each_keyword_argument")
-    autoload(:EachPositionalArgument, "#{__dir__}/processors/each_positional_argument")
-    autoload(:EachPositionalArgumentFrom, "#{__dir__}/processors/each_positional_argument_from")
-    autoload(:Eval, "#{__dir__}/processors/eval")
-    autoload(:Itself, "#{__dir__}/processors/itself")
-    autoload(:KeywordArgument, "#{__dir__}/processors/keyword_argument")
-    autoload(:MatchCurrentNode, "#{__dir__}/processors/match_current_node")
-    autoload(:MatchMatchedNode, "#{__dir__}/processors/match_matched_node")
-    autoload(:Parameterize, "#{__dir__}/processors/parameterize")
-    autoload(:Placeholder, "#{__dir__}/processors/placeholder")
-    autoload(:Pluralize, "#{__dir__}/processors/pluralize")
-    autoload(:PositionalArgument, "#{__dir__}/processors/positional_argument")
-    autoload(:Receiver, "#{__dir__}/processors/receiver")
-    autoload(:ReplaceValue, "#{__dir__}/processors/replace_value")
-    autoload(:SetDefaultPrivacy, "#{__dir__}/processors/set_default_privacy")
-    autoload(:SetPrivacy, "#{__dir__}/processors/set_privacy")
-    autoload(:Singularize, "#{__dir__}/processors/singularize")
-    autoload(:Split, "#{__dir__}/processors/split")
-    autoload(:Swapcase, "#{__dir__}/processors/swapcase")
-    autoload(:Titleize, "#{__dir__}/processors/titleize")
-    autoload(:Underscore, "#{__dir__}/processors/underscore")
-    autoload(:Upcase, "#{__dir__}/processors/upcase")
+    include Autoloader
   end
 end

data/lib/leftovers/rake_task.rb

@@ -27,7 +27,7 @@ module Leftovers
     def escaped_argv(argv = @default_argv)
       return if argv.empty?
 
-      Shellwords.shelljoin(argv)
+      ::Shellwords.shelljoin(argv)
     end
 
     def describe_task
@@ -49,7 +49,7 @@ module Leftovers
     end
 
     def run(argv)
-      exitstatus = Leftovers::CLI.new(argv: argv).run
+      exitstatus = CLI.new(argv: argv).run
 
       exit exitstatus unless exitstatus == 0
     end

data/lib/leftovers/reporter.rb

@@ -1,63 +1,65 @@
 # frozen_string_literal: true
 
 module Leftovers
-  class Reporter
-    def prepare; end
+  module Reporter
+    class << self
+      def prepare; end
 
-    def report(only_test:, none:)
-      report_list('Only directly called in tests:', only_test)
-      report_list('Not directly called at all:', none)
-      report_instructions
+      def report(collection)
+        report_list('Only directly called in tests:', collection.with_tests)
+        report_list('Not directly called at all:', collection.without_tests)
+        report_instructions
 
-      1
-    end
+        1
+      end
 
-    def report_success
-      puts green('Everything is used')
+      def report_success
+        puts green('Everything is used')
 
-      0
-    end
+        0
+      end
 
-    private
+      private
 
-    def report_instructions
-      puts <<~HELP
+      def report_instructions
+        puts <<~HELP
 
-        how to resolve: #{green Leftovers.resolution_instructions_link}
-      HELP
-    end
+          how to resolve: #{green ::Leftovers.resolution_instructions_link}
+        HELP
+      end
 
-    def report_list(title, list)
-      return if list.empty?
+      def report_list(title, list)
+        return if list.empty?
 
-      puts red(title)
-      list.each { |d| print_definition(d) }
-    end
+        puts red(title)
+        list.each { |d| print_definition(d) }
+      end
 
-    def print_definition(definition)
-      puts "#{aqua definition.location_s} " \
-        "#{definition} " \
-        "#{grey definition.highlighted_source("\e[33m", "\e[0;2m")}"
-    end
+      def print_definition(definition)
+        puts "#{aqua definition.location_s} " \
+          "#{definition} " \
+          "#{grey definition.highlighted_source("\e[33m", "\e[0;2m")}"
+      end
 
-    def puts(string)
-      Leftovers.puts(string)
-    end
+      def puts(string)
+        ::Leftovers.puts(string)
+      end
 
-    def red(string)
-      "\e[31m#{string}\e[0m"
-    end
+      def red(string)
+        "\e[31m#{string}\e[0m"
+      end
 
-    def green(string)
-      "\e[32m#{string}\e[0m"
-    end
+      def green(string)
+        "\e[32m#{string}\e[0m"
+      end
 
-    def aqua(string)
-      "\e[36m#{string}\e[0m"
-    end
+      def aqua(string)
+        "\e[36m#{string}\e[0m"
+      end
 
-    def grey(string)
-      "\e[2m#{string}\e[0m"
+      def grey(string)
+        "\e[2m#{string}\e[0m"
+      end
     end
   end
 end

data/lib/leftovers/runner.rb

@@ -0,0 +1,40 @@
+# frozen_string_literal: true
+
+module Leftovers
+  class Runner
+    attr_writer :reporter
+
+    def run
+      reporter.prepare
+      return reporter.report_success if collection.empty?
+
+      reporter.report(collection)
+    end
+
+    def parallel=(value)
+      collector.parallel = value
+    end
+
+    def progress=(value)
+      collector.progress = value
+    end
+
+    def collection
+      @collection ||= begin
+        collector.collect
+
+        collector.collection
+      end
+    end
+
+    private
+
+    def reporter
+      @reporter ||= Reporter
+    end
+
+    def collector
+      @collector ||= Collector.new
+    end
+  end
+end

data/lib/leftovers/todo_reporter.rb

@@ -3,100 +3,102 @@
 require 'pathname'
 
 module Leftovers
-  class TodoReporter
-    def prepare
-      return unless path.exist?
-
-      puts "Removing previous #{path.basename} file"
-      puts ''
-      path.delete
-    end
-
-    def report(only_test:, none:)
-      path.write(generate_file_body(only_test, none))
-      report_instructions
-
-      0
-    end
-
-    def report_success
-      puts "No #{path.basename} file generated, everything is used"
-
-      0
-    end
-
-    private
-
-    def report_instructions
-      puts <<~MESSAGE
-        generated #{path.basename}.
-        running leftovers again will read this file and not alert you to any unused items mentioned in it.
-
-        commit this file so you/your team can gradually address these items while still having leftovers alert you to any newly unused items.
-      MESSAGE
-    end
-
-    def path
-      Leftovers.pwd.join('.leftovers_todo.yml')
-    end
-
-    def generate_file_body(only_test, none)
-      <<~YML.chomp
-        #{generation_message.chomp}
-        #
-        #{resolution_instructions}
-        #{todo_data(only_test, none).chomp}
-      YML
-    end
-
-    def generation_message
-      <<~YML
-        # This file was generated by `leftovers --write-todo`
-        # Generated at: #{Time.now.utc.strftime('%F %T')} UTC
-      YML
-    end
-
-    def resolution_instructions
-      <<~YML
-        # for instructions on how to address these
-        # see #{Leftovers.resolution_instructions_link}
-      YML
-    end
-
-    def todo_data(only_test, none)
-      [
-        list_data(
-          :test_only, 'Only directly called in tests', only_test
-        ),
-        list_data(
-          :keep, 'Not directly called at all', none
-        )
-      ].compact.join
-    end
-
-    def list_data(key, title, list)
-      return if list.empty?
-
-      <<~YML
-        #{key}:
-          # #{title}:
-        #{print_definition_list(list)}
-
-      YML
-    end
-
-    def print_definition_list(definition_list)
-      definition_list.map { |definition| print_definition(definition) }.sort.join("\n")
-    end
-
-    def print_definition(definition)
-      return print_definition_list(definition.definitions) if definition.is_a?(DefinitionSet)
-
-      "  - #{definition.to_s.inspect} # #{definition.location_s} #{definition.source_line.strip}"
-    end
-
-    def puts(string)
-      Leftovers.puts(string)
+  module TodoReporter
+    class << self
+      def prepare
+        return unless path.exist?
+
+        puts "Removing previous #{path.basename} file"
+        puts ''
+        path.delete
+      end
+
+      def report(collection)
+        path.write(generate_file_body(collection))
+        report_instructions
+
+        0
+      end
+
+      def report_success
+        puts "No #{path.basename} file generated, everything is used"
+
+        0
+      end
+
+      private
+
+      def report_instructions
+        puts <<~MESSAGE
+          generated #{path.basename}.
+          running leftovers again will read this file and not alert you to any unused items mentioned in it.
+
+          commit this file so you/your team can gradually address these items while still having leftovers alert you to any newly unused items.
+        MESSAGE
+      end
+
+      def path
+        ::Leftovers.pwd.join('.leftovers_todo.yml')
+      end
+
+      def generate_file_body(collection)
+        <<~YML.chomp
+          #{generation_message.chomp}
+          #
+          #{resolution_instructions}
+          #{todo_data(collection).chomp}
+        YML
+      end
+
+      def generation_message
+        <<~YML
+          # This file was generated by `leftovers --write-todo`
+          # Generated at: #{Time.now.utc.strftime('%F %T')} UTC
+        YML
+      end
+
+      def resolution_instructions
+        <<~YML
+          # for instructions on how to address these
+          # see #{::Leftovers.resolution_instructions_link}
+        YML
+      end
+
+      def todo_data(collection)
+        [
+          list_data(
+            :test_only, 'Only directly called in tests', collection.with_tests
+          ),
+          list_data(
+            :keep, 'Not directly called at all', collection.without_tests
+          )
+        ].compact.join
+      end
+
+      def list_data(key, title, list)
+        return if list.empty?
+
+        <<~YML
+          #{key}:
+            # #{title}:
+          #{print_definition_list(list)}
+
+        YML
+      end
+
+      def print_definition_list(definition_list)
+        definition_list.map { |definition| print_definition(definition) }.sort.join("\n")
+      end
+
+      def print_definition(definition)
+        return print_definition_list(definition.definitions) if definition.is_a?(DefinitionSet)
+
+        "  - #{definition.to_s.inspect} # #{definition.location_s} #{definition.source_line.strip}"
+      end
+
+      def puts(string)
+        ::Leftovers.puts(string)
+      end
     end
   end
 end

data/lib/leftovers/unexpected_case.rb

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+# :nocov:
+module Leftovers
+  class UnexpectedCase < Error
+  end
+end
+# :nocov:

data/lib/leftovers/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Leftovers
-  VERSION = '0.11.0'
+  VERSION = '0.11.2'
 end