pry 0.12.2 → 0.13.0

data/lib/pry/commands/ls/local_vars.rb

@@ -1,36 +1,46 @@
+# frozen_string_literal: true
+
 class Pry
-  class Command::Ls < Pry::ClassCommand
-    class LocalVars < Pry::Command::Ls::Formatter
-      def initialize(opts, _pry_)
-        super(_pry_)
-        @default_switch = opts[:locals]
-        @sticky_locals = _pry_.sticky_locals
-      end
+  class Command
+    class Ls < Pry::ClassCommand
+      class LocalVars < Pry::Command::Ls::Formatter
+        def initialize(opts, pry_instance)
+          super(pry_instance)
+          @default_switch = opts[:locals]
+          @sticky_locals = pry_instance.sticky_locals
+        end
 
-      def output_self
-        name_value_pairs = @target.eval('local_variables').reject { |e|
-          @sticky_locals.keys.include?(e.to_sym)
-        }.map { |name|
-          [name, (@target.eval(name.to_s))]
-        }
-        format(name_value_pairs).join('')
-      end
+        def output_self
+          locals = @target.eval('local_variables').reject do |e|
+            @sticky_locals.key?(e.to_sym)
+          end
+          name_value_pairs = locals.map do |name|
+            [name, @target.eval(name.to_s)]
+          end
+          format(name_value_pairs).join('')
+        end
 
-      private
+        private
 
-      def format(name_value_pairs)
-        name_value_pairs.sort_by { |name, value|
-          value.to_s.size
-        }.reverse.map { |name, value|
-          colorized_assignment_style(name, format_value(value))
-        }
-      end
+        def format(name_value_pairs)
+          sorted = name_value_pairs.sort_by do |_name, value|
+            value.to_s.size
+          end
+          sorted.reverse.map do |name, value|
+            colorized_assignment_style(name, format_value(value))
+          end
+        end
 
-      def colorized_assignment_style(lhs, rhs, desired_width = 7)
-        colorized_lhs = color(:local_var, lhs)
-        color_escape_padding = colorized_lhs.size - lhs.size
-        pad = desired_width + color_escape_padding
-        "%-#{pad}s = %s" % [color(:local_var, colorized_lhs), rhs]
+        def colorized_assignment_style(lhs, rhs, desired_width = 7)
+          colorized_lhs = color(:local_var, lhs)
+          color_escape_padding = colorized_lhs.size - lhs.size
+          pad = desired_width + color_escape_padding
+          Kernel.format(
+            "%-#{pad}<name>s = %<value>s",
+            name: color(:local_var, colorized_lhs),
+            value: rhs
+          )
+        end
       end
     end
   end

data/lib/pry/commands/ls/ls_entity.rb

@@ -1,68 +1,64 @@
-require 'pry/commands/ls/grep'
-require 'pry/commands/ls/formatter'
-require 'pry/commands/ls/globals'
-require 'pry/commands/ls/constants'
-require 'pry/commands/ls/methods'
-require 'pry/commands/ls/self_methods'
-require 'pry/commands/ls/instance_vars'
-require 'pry/commands/ls/local_names'
-require 'pry/commands/ls/local_vars'
+# frozen_string_literal: true
 
 class Pry
-  class Command::Ls < Pry::ClassCommand
-    class LsEntity
-      attr_reader :_pry_
+  class Command
+    class Ls < Pry::ClassCommand
+      class LsEntity
+        attr_reader :pry_instance
 
-      def initialize(opts)
-        @interrogatee = opts[:interrogatee]
-        @no_user_opts = opts[:no_user_opts]
-        @opts = opts[:opts]
-        @args = opts[:args]
-        @grep = Grep.new(Regexp.new(opts[:opts][:G] || '.'))
-        @_pry_ = opts.delete(:_pry_)
-      end
+        def initialize(opts)
+          @interrogatee = opts[:interrogatee]
+          @no_user_opts = opts[:no_user_opts]
+          @opts = opts[:opts]
+          @args = opts[:args]
+          @grep = Grep.new(Regexp.new(opts[:opts][:G] || '.'))
+          @pry_instance = opts.delete(:pry_instance)
+        end
 
-      def entities_table
-        entities.map(&:write_out).reject { |o| !o }.join('')
-      end
+        def entities_table
+          entities.map(&:write_out).select { |o| o }.join('')
+        end
 
-      private
+        private
 
-      def grep(entity)
-        entity.tap { |o| o.grep = @grep }
-      end
+        def grep(entity)
+          entity.tap { |o| o.grep = @grep }
+        end
 
-      def globals
-        grep Globals.new(@opts, _pry_)
-      end
+        def globals
+          grep Globals.new(@opts, pry_instance)
+        end
 
-      def constants
-        grep Constants.new(@interrogatee, @no_user_opts, @opts, _pry_)
-      end
+        def constants
+          grep Constants.new(@interrogatee, @no_user_opts, @opts, pry_instance)
+        end
 
-      def methods
-        grep(Methods.new(@interrogatee, @no_user_opts, @opts, _pry_))
-      end
+        def methods
+          grep(Methods.new(@interrogatee, @no_user_opts, @opts, pry_instance))
+        end
 
-      def self_methods
-        grep SelfMethods.new(@interrogatee, @no_user_opts, @opts, _pry_)
-      end
+        def self_methods
+          grep SelfMethods.new(@interrogatee, @no_user_opts, @opts, pry_instance)
+        end
 
-      def instance_vars
-        grep InstanceVars.new(@interrogatee, @no_user_opts, @opts, _pry_)
-      end
+        def instance_vars
+          grep InstanceVars.new(@interrogatee, @no_user_opts, @opts, pry_instance)
+        end
 
-      def local_names
-        grep LocalNames.new(@no_user_opts, @args, _pry_)
-      end
+        def local_names
+          grep LocalNames.new(@no_user_opts, @args, pry_instance)
+        end
 
-      def local_vars
-        LocalVars.new(@opts, _pry_)
-      end
+        def local_vars
+          LocalVars.new(@opts, pry_instance)
+        end
 
-      def entities
-        [globals, constants, methods, self_methods, instance_vars, local_names,
-          local_vars]
+        def entities
+          [
+            globals, constants, methods, self_methods, instance_vars, local_names,
+            local_vars
+          ]
+        end
       end
     end
   end

data/lib/pry/commands/ls/methods.rb

@@ -1,53 +1,54 @@
-require 'pry/commands/ls/methods_helper'
-require 'pry/commands/ls/interrogatable'
+# frozen_string_literal: true
 
 class Pry
-  class Command::Ls < Pry::ClassCommand
-    class Methods < Pry::Command::Ls::Formatter
-      include Pry::Command::Ls::Interrogatable
-      include Pry::Command::Ls::MethodsHelper
+  class Command
+    class Ls < Pry::ClassCommand
+      class Methods < Pry::Command::Ls::Formatter
+        include Pry::Command::Ls::Interrogatable
+        include Pry::Command::Ls::MethodsHelper
 
-      def initialize(interrogatee, no_user_opts, opts, _pry_)
-        super(_pry_)
-        @interrogatee = interrogatee
-        @no_user_opts = no_user_opts
-        @default_switch = opts[:methods]
-        @instance_methods_switch = opts['instance-methods']
-        @ppp_switch = opts[:ppp]
-        @jruby_switch = opts['all-java']
-        @quiet_switch = opts[:quiet]
-        @verbose_switch = opts[:verbose]
-      end
+        def initialize(interrogatee, no_user_opts, opts, pry_instance)
+          super(pry_instance)
+          @interrogatee = interrogatee
+          @no_user_opts = no_user_opts
+          @default_switch = opts[:methods]
+          @instance_methods_switch = opts['instance-methods']
+          @ppp_switch = opts[:ppp]
+          @jruby_switch = opts['all-java']
+          @quiet_switch = opts[:quiet]
+          @verbose_switch = opts[:verbose]
+        end
 
-      def output_self
-        methods = all_methods.group_by(&:owner)
-        # Reverse the resolution order so that the most useful information
-        # appears right by the prompt.
-        resolution_order.take_while(&below_ceiling).reverse.map do |klass|
-          methods_here = (methods[klass] || []).select { |m| grep.regexp[m.name] }
-          heading = "#{ Pry::WrappedModule.new(klass).method_prefix }methods"
-          output_section(heading, format(methods_here))
-        end.join('')
-      end
+        def output_self
+          methods = all_methods.group_by(&:owner)
+          # Reverse the resolution order so that the most useful information
+          # appears right by the prompt.
+          resolution_order.take_while(&below_ceiling).reverse.map do |klass|
+            methods_here = (methods[klass] || []).select { |m| grep.regexp[m.name] }
+            heading = "#{Pry::WrappedModule.new(klass).method_prefix}methods"
+            output_section(heading, format(methods_here))
+          end.join('')
+        end
 
-      private
+        private
 
-      def correct_opts?
-        super || @instance_methods_switch || @ppp_switch || @no_user_opts
-      end
+        def correct_opts?
+          super || @instance_methods_switch || @ppp_switch || @no_user_opts
+        end
 
-      # Get a lambda that can be used with `take_while` to prevent over-eager
-      # traversal of the Object's ancestry graph.
-      def below_ceiling
-        ceiling = if @quiet_switch
-                    [Pry::Method.safe_send(interrogatee_mod, :ancestors)[1]] +
-                      _pry_.config.ls.ceiling
-                  elsif @verbose_switch
-                    []
-                  else
-                    _pry_.config.ls.ceiling.dup
-                  end
-        lambda { |klass| !ceiling.include?(klass) }
+        # Get a lambda that can be used with `take_while` to prevent over-eager
+        # traversal of the Object's ancestry graph.
+        def below_ceiling
+          ceiling = if @quiet_switch
+                      [Pry::Method.safe_send(interrogatee_mod, :ancestors)[1]] +
+                        pry_instance.config.ls.ceiling
+                    elsif @verbose_switch
+                      []
+                    else
+                      pry_instance.config.ls.ceiling.dup
+                    end
+          ->(klass) { !ceiling.include?(klass) }
+        end
       end
     end
   end

data/lib/pry/commands/ls/methods_helper.rb

@@ -1,46 +1,50 @@
-require 'pry/commands/ls/jruby_hacks'
-
-module Pry::Command::Ls::MethodsHelper
-
-  include Pry::Command::Ls::JRubyHacks
-
-  private
-
-  # Get all the methods that we'll want to output.
-  def all_methods(instance_methods = false)
-    methods = if instance_methods || @instance_methods_switch
-                Pry::Method.all_from_class(@interrogatee)
-              else
-                Pry::Method.all_from_obj(@interrogatee)
-              end
-
-    if Pry::Helpers::Platform.jruby? && !@jruby_switch
-      methods = trim_jruby_aliases(methods)
-    end
-
-    methods.select { |method| @ppp_switch || method.visibility == :public }
-  end
-
-  def resolution_order
-    if @instance_methods_switch
-      Pry::Method.instance_resolution_order(@interrogatee)
-    else
-      Pry::Method.resolution_order(@interrogatee)
-    end
-  end
-
-  def format(methods)
-    methods.sort_by(&:name).map do |method|
-      if method.name == 'method_missing'
-        color(:method_missing, 'method_missing')
-      elsif method.visibility == :private
-        color(:private_method, method.name)
-      elsif method.visibility == :protected
-        color(:protected_method, method.name)
-      else
-        color(:public_method, method.name)
+# frozen_string_literal: true
+
+class Pry
+  class Command
+    class Ls < Pry::ClassCommand
+      module MethodsHelper
+        include Pry::Command::Ls::JRubyHacks
+
+        private
+
+        # Get all the methods that we'll want to output.
+        def all_methods(instance_methods = false)
+          methods = if instance_methods || @instance_methods_switch
+                      Pry::Method.all_from_class(@interrogatee)
+                    else
+                      Pry::Method.all_from_obj(@interrogatee)
+                    end
+
+          if Pry::Helpers::Platform.jruby? && !@jruby_switch
+            methods = trim_jruby_aliases(methods)
+          end
+
+          methods.select { |method| @ppp_switch || method.visibility == :public }
+        end
+
+        def resolution_order
+          if @instance_methods_switch
+            Pry::Method.instance_resolution_order(@interrogatee)
+          else
+            Pry::Method.resolution_order(@interrogatee)
+          end
+        end
+
+        def format(methods)
+          methods.sort_by(&:name).map do |method|
+            if method.name == 'method_missing'
+              color(:method_missing, 'method_missing')
+            elsif method.visibility == :private
+              color(:private_method, method.name)
+            elsif method.visibility == :protected
+              color(:protected_method, method.name)
+            else
+              color(:public_method, method.name)
+            end
+          end
+        end
       end
     end
   end
-
 end

data/lib/pry/commands/ls/self_methods.rb

@@ -1,32 +1,33 @@
-require 'pry/commands/ls/interrogatable'
-require 'pry/commands/ls/methods_helper'
+# frozen_string_literal: true
 
 class Pry
-  class Command::Ls < Pry::ClassCommand
-    class SelfMethods < Pry::Command::Ls::Formatter
-      include Pry::Command::Ls::Interrogatable
-      include Pry::Command::Ls::MethodsHelper
+  class Command
+    class Ls < Pry::ClassCommand
+      class SelfMethods < Pry::Command::Ls::Formatter
+        include Pry::Command::Ls::Interrogatable
+        include Pry::Command::Ls::MethodsHelper
 
-      def initialize(interrogatee, no_user_opts, opts, _pry_)
-        super(_pry_)
-        @interrogatee = interrogatee
-        @no_user_opts = no_user_opts
-        @ppp_switch = opts[:ppp]
-        @jruby_switch = opts['all-java']
-      end
+        def initialize(interrogatee, no_user_opts, opts, pry_instance)
+          super(pry_instance)
+          @interrogatee = interrogatee
+          @no_user_opts = no_user_opts
+          @ppp_switch = opts[:ppp]
+          @jruby_switch = opts['all-java']
+        end
 
-      def output_self
-        methods = all_methods(true).select do |m|
-          m.owner == @interrogatee && grep.regexp[m.name]
+        def output_self
+          methods = all_methods(true).select do |m|
+            m.owner == @interrogatee && grep.regexp[m.name]
+          end
+          heading = "#{Pry::WrappedModule.new(@interrogatee).method_prefix}methods"
+          output_section(heading, format(methods))
         end
-        heading = "#{ Pry::WrappedModule.new(@interrogatee).method_prefix }methods"
-        output_section(heading, format(methods))
-      end
 
-      private
+        private
 
-      def correct_opts?
-        @no_user_opts && interrogating_a_module?
+        def correct_opts?
+          @no_user_opts && interrogating_a_module?
+        end
       end
     end
   end

data/lib/pry/commands/nesting.rb

@@ -1,25 +1,29 @@
+# frozen_string_literal: true
+
 class Pry
-  class Command::Nesting < Pry::ClassCommand
-    match 'nesting'
-    group 'Navigating Pry'
-    description 'Show nesting information.'
+  class Command
+    class Nesting < Pry::ClassCommand
+      match 'nesting'
+      group 'Navigating Pry'
+      description 'Show nesting information.'
 
-    banner <<-'BANNER'
-      Show nesting information.
-    BANNER
+      banner <<-'BANNER'
+        Show nesting information.
+      BANNER
 
-    def process
-      output.puts 'Nesting status:'
-      output.puts '--'
-      _pry_.binding_stack.each_with_index do |obj, level|
-        if level == 0
-          output.puts "#{level}. #{Pry.view_clip(obj.eval('self'))} (Pry top level)"
-        else
-          output.puts "#{level}. #{Pry.view_clip(obj.eval('self'))}"
+      def process
+        output.puts 'Nesting status:'
+        output.puts '--'
+        pry_instance.binding_stack.each_with_index do |obj, level|
+          if level == 0
+            output.puts "#{level}. #{Pry.view_clip(obj.eval('self'))} (Pry top level)"
+          else
+            output.puts "#{level}. #{Pry.view_clip(obj.eval('self'))}"
+          end
         end
       end
     end
-  end
 
-  Pry::Commands.add_command(Pry::Command::Nesting)
+    Pry::Commands.add_command(Pry::Command::Nesting)
+  end
 end