awesome_print 0.3.0 → 0.4.0

data/lib/ap/mixin/active_support.rb

@@ -1,4 +1,4 @@
-# Copyright (c) 2010 Michael Dvorkin
+# Copyright (c) 2010-2011 Michael Dvorkin
 #
 # Awesome Print is freely distributable under the terms of MIT license.
 # See LICENSE file or http://www.opensource.org/licenses/mit-license.php
@@ -14,6 +14,8 @@ module AwesomePrintActiveSupport
   #------------------------------------------------------------------------------
   def printable_with_active_support(object)
     printable = printable_without_active_support(object)
+    return printable if !defined?(ActiveSupport::TimeWithZone) || !defined?(HashWithIndifferentAccess)
+
     if printable == :self
       if object.is_a?(ActiveSupport::TimeWithZone)
         printable = :active_support_time

data/lib/ap/mixin/mongo_mapper.rb

@@ -0,0 +1,54 @@
+# Copyright (c) 2010-2011 Michael Dvorkin
+#
+# Awesome Print is freely distributable under the terms of MIT license.
+# See LICENSE file or http://www.opensource.org/licenses/mit-license.php
+#------------------------------------------------------------------------------
+module AwesomePrintMongoMapper
+
+  def self.included(base)
+    base.send :alias_method, :printable_without_mongo_mapper, :printable
+    base.send :alias_method, :printable, :printable_with_mongo_mapper
+  end
+
+  # Add MongoMapper class names to the dispatcher pipeline.
+  #------------------------------------------------------------------------------
+  def printable_with_mongo_mapper(object)
+    printable = printable_without_mongo_mapper(object)
+    return printable if !defined?(MongoMapper::Document)
+
+    if printable == :self
+      if object.is_a?(MongoMapper::Document) || object.is_a?(MongoMapper::EmbeddedDocument)
+        printable = :mongo_mapper_instance
+      end
+    elsif printable == :class && (object.ancestors & [MongoMapper::Document, MongoMapper::EmbeddedDocument]).size > 0
+      printable = :mongo_mapper_class
+    end
+    printable
+  end
+
+  # Format MongoMapper instance object.
+  #------------------------------------------------------------------------------
+  def awesome_mongo_mapper_instance(object)
+    return object.inspect if !defined?(ActiveSupport::OrderedHash)
+
+    data = object.keys.keys.sort_by{|k| k}.inject(ActiveSupport::OrderedHash.new) do |hash, name|
+      hash[name] = object[name]
+      hash
+    end
+    "#{object} " + awesome_hash(data)
+  end
+
+  # Format MongoMapper class object.
+  #------------------------------------------------------------------------------
+  def awesome_mongo_mapper_class(object)
+    return object.inspect if !defined?(ActiveSupport::OrderedHash) || !object.respond_to?(:keys)
+
+    data = object.keys.sort_by{|k| k}.inject(ActiveSupport::OrderedHash.new) do |hash, c|
+      hash[c.first] = (c.last.type || "undefined").to_s.underscore.intern
+      hash
+    end
+    "class #{object} < #{object.superclass} " << awesome_hash(data)
+  end
+end
+
+AwesomePrint.send(:include, AwesomePrintMongoMapper)

data/lib/ap.rb

@@ -1,18 +1,24 @@
-# Copyright (c) 2010 Michael Dvorkin
+# Copyright (c) 2010-2011 Michael Dvorkin
 #
 # Awesome Print is freely distributable under the terms of MIT license.
 # See LICENSE file or http://www.opensource.org/licenses/mit-license.php
 #------------------------------------------------------------------------------
-require File.dirname(__FILE__) + "/ap/core_ext/array"
-require File.dirname(__FILE__) + "/ap/core_ext/string"
-require File.dirname(__FILE__) + "/ap/core_ext/object"
-require File.dirname(__FILE__) + "/ap/core_ext/class"
-require File.dirname(__FILE__) + "/ap/core_ext/kernel"
-require File.dirname(__FILE__) + "/ap/awesome_print"
-
-require File.dirname(__FILE__) + "/ap/core_ext/logger" if defined?(::Logger) or defined?(::ActiveSupport::BufferedLogger)
+#
+# AwesomePrint might be loaded implicitly through ~/.irbrc so do nothing
+# for subsequent requires.
+#
+unless defined?(AwesomePrint)
+  %w(array string method object class kernel).each do |file|
+    require File.dirname(__FILE__) + "/ap/core_ext/#{file}"
+  end
 
-require File.dirname(__FILE__) + "/ap/mixin/action_view" if defined?(::ActionView)
-require File.dirname(__FILE__) + "/ap/mixin/active_record" if defined?(::ActiveRecord)
-require File.dirname(__FILE__) + "/ap/mixin/active_support" if defined?(::ActiveSupport)
+  require File.dirname(__FILE__) + "/ap/awesome_print"
+  require File.dirname(__FILE__) + "/ap/core_ext/logger"   if defined?(Logger)
+  require File.dirname(__FILE__) + "/ap/mixin/action_view" if defined?(ActionView)
 
+  # Load the following under normal circumstatnces as well as in Rails
+  # console when required from ~/.irbrc.
+  require File.dirname(__FILE__) + "/ap/mixin/active_record"  if defined?(ActiveRecord)  || (defined?(IRB) && ENV['RAILS_ENV'])
+  require File.dirname(__FILE__) + "/ap/mixin/active_support" if defined?(ActiveSupport) || (defined?(IRB) && ENV['RAILS_ENV'])
+  require File.dirname(__FILE__) + "/ap/mixin/mongo_mapper"   if defined?(MongoMapper)
+end

data/lib/awesome_print.rb

@@ -1,4 +1,4 @@
-# Copyright (c) 2010 Michael Dvorkin
+# Copyright (c) 2010-2011 Michael Dvorkin
 #
 # Awesome Print is freely distributable under the terms of MIT license.
 # See LICENSE file or http://www.opensource.org/licenses/mit-license.php
@@ -7,19 +7,20 @@
 # This is the copy of original 'ap.rb' file that matches the gem name. It makes
 # it possible to omit the :require part in bundler's Gemfile:
 #
-# gem 'awesome_print', '>= 0.2.1', :require => 'ap'
-# gem 'awesome_print', '>= 3.0.0'
+# gem 'awesome_print', '>= 0.4.0'
 #
-require File.dirname(__FILE__) + "/ap/core_ext/array"
-require File.dirname(__FILE__) + "/ap/core_ext/string"
-require File.dirname(__FILE__) + "/ap/core_ext/object"
-require File.dirname(__FILE__) + "/ap/core_ext/class"
-require File.dirname(__FILE__) + "/ap/core_ext/kernel"
-require File.dirname(__FILE__) + "/ap/awesome_print"
+unless defined?(AwesomePrint)
+  %w(array string method object class kernel).each do |file|
+    require File.dirname(__FILE__) + "/ap/core_ext/#{file}"
+  end
 
-require File.dirname(__FILE__) + "/ap/core_ext/logger" if defined?(::Logger) or defined?(::ActiveSupport::BufferedLogger)
-
-require File.dirname(__FILE__) + "/ap/mixin/action_view" if defined?(::ActionView)
-require File.dirname(__FILE__) + "/ap/mixin/active_record" if defined?(::ActiveRecord)
-require File.dirname(__FILE__) + "/ap/mixin/active_support" if defined?(::ActiveSupport)
+  require File.dirname(__FILE__) + "/ap/awesome_print"
+  require File.dirname(__FILE__) + "/ap/core_ext/logger"   if defined?(Logger)
+  require File.dirname(__FILE__) + "/ap/mixin/action_view" if defined?(ActionView)
 
+  # Load the following under normal circumstatnces as well as in Rails
+  # console when required from ~/.irbrc.
+  require File.dirname(__FILE__) + "/ap/mixin/active_record"  if defined?(ActiveRecord)  || (defined?(IRB) && ENV['RAILS_ENV'])
+  require File.dirname(__FILE__) + "/ap/mixin/active_support" if defined?(ActiveSupport) || (defined?(IRB) && ENV['RAILS_ENV'])
+  require File.dirname(__FILE__) + "/ap/mixin/mongo_mapper"   if defined?(MongoMapper)
+end

data/rails/init.rb

@@ -1 +1,9 @@
-require File.join(File.dirname(__FILE__), "..", "init")
+# Copyright (c) 2010-2011 Michael Dvorkin
+#
+# Awesome Print is freely distributable under the terms of MIT license.
+# See LICENSE file or http://www.opensource.org/licenses/mit-license.php
+#------------------------------------------------------------------------------
+#
+# Load awesome_print when installed as Rails 2.3.x plugin.
+#
+require File.join(File.dirname(__FILE__), "..", "init") unless defined?(AwesomePrint)

data/spec/action_view_spec.rb

@@ -1,35 +1,25 @@
 require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
 
-require 'action_view'
-require 'ap/mixin/action_view'
+begin
+  require 'action_view'
+  require 'ap/mixin/action_view'
 
-describe "AwesomePrint ActionView extensions" do
-  before(:each) do
-    @view = ActionView::Base.new
-  end
+  describe "AwesomePrint ActionView extension" do
+    before(:each) do
+      @view = ActionView::Base.new
+    end
     
-  it "should wrap ap output with <pre> tag" do
-    obj = 42
-    @view.ap(obj, :plain => true).should == '<pre class="debug_dump">42</pre>'
-  end
-
-  it "should encode HTML entities" do
-    obj = "  &<hello>"
-    @view.ap(obj, :plain => true).should == '<pre class="debug_dump">&quot;  &amp;&lt;hello&gt;&quot;</pre>'
-  end
-
-  it "should convert primary ANSI colors to HTML" do
-    obj = 42
-    [ :gray, :red, :green, :yellow, :blue, :purple, :cyan, :white ].each do |color|
-      @view.ap(obj, :color => { :fixnum => color }).should == %Q|<pre class="debug_dump"><font color="#{color}">42</font></pre>|
+    it "uses HTML and adds 'debug_dump' class to plain <pre> tag" do
+      markup = rand
+      @view.ap(markup, :plain => true).should == %Q|<pre class="debug_dump">#{markup}</pre>|
     end
-  end
 
-  it "should convert mixed ANSI colors to HTML" do
-    obj = 42
-    [ :grayish, :redish, :greenish, :yellowish, :blueish, :purpleish, :cyanish, :whiteish, :black, :pale ].zip(
-    [ :black, :darkred, :darkgreen, :brown, :navy, :darkmagenta, :darkcyan, :slategray, :black, :slategray ]) do |ansi, html|
-      @view.ap(obj, :color => { :fixnum => ansi }).should == %Q|<pre class="debug_dump"><font color="#{html}">42</font></pre>|
+    it "uses HTML and adds 'debug_dump' class to colorized <pre> tag" do
+      markup = ' &<hello>'
+      @view.ap(markup).should == '<pre class="debug_dump" style="color:brown">&quot; &amp;&lt;hello&gt;&quot;</pre>'
     end
   end
+
+rescue LoadError
+  puts "Skipping ActionView specs..."
 end

data/spec/active_record_spec.rb

@@ -1,56 +1,56 @@
 require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
 
-require 'active_record'
-require 'ap/mixin/active_record'
+begin
+  require 'active_record'
+  require 'ap/mixin/active_record'
 
+  if defined?(::ActiveRecord)
 
-if defined?(::ActiveRecord)
-
-  # Create tableless ActiveRecord model.
-  #------------------------------------------------------------------------------
-  class User < ActiveRecord::Base
-    def self.columns()
-      @columns ||= []
-    end
+    # Create tableless ActiveRecord model.
+    #------------------------------------------------------------------------------
+    class User < ActiveRecord::Base
+      def self.columns()
+        @columns ||= []
+      end
 
-    def self.column(name, sql_type = nil, default = nil, null = true)
-      columns << ActiveRecord::ConnectionAdapters::Column.new(name.to_s, default, sql_type.to_s, null)
-    end
+      def self.column(name, sql_type = nil, default = nil, null = true)
+        columns << ActiveRecord::ConnectionAdapters::Column.new(name.to_s, default, sql_type.to_s, null)
+      end
 
-    column :id, :integer
-    column :name, :string
-    column :rank, :integer
-    column :admin, :boolean
-    column :created_at, :datetime
+      column :id, :integer
+      column :name, :string
+      column :rank, :integer
+      column :admin, :boolean
+      column :created_at, :datetime
 
-    def self.table_exists?
-      true
+      def self.table_exists?
+        true
+      end
     end
-  end
   
-  class SubUser < User
-    def self.columns
-      User.columns
+    class SubUser < User
+      def self.columns
+        User.columns
+      end
     end
-  end
   
-  describe "AwesomePrint/ActiveRecord" do
-    before(:each) do
-      stub_dotfile!
-    end
-
-    #------------------------------------------------------------------------------
-    describe "ActiveRecord instance" do
+    describe "AwesomePrint/ActiveRecord" do
       before(:each) do
-        ActiveRecord::Base.default_timezone = :utc
-        @diana = User.new(:name => "Diana", :rank => 1, :admin => false, :created_at => "1992-10-10 12:30:00")
-        @laura = User.new(:name => "Laura", :rank => 2, :admin => true,  :created_at => "2003-05-26 14:15:00")
-        @ap = AwesomePrint.new(:plain => true)
+        stub_dotfile!
       end
 
-      it "display single record" do
-        out = @ap.send(:awesome, @diana)
-        str = <<-EOS.strip
+      #------------------------------------------------------------------------------
+      describe "ActiveRecord instance" do
+        before(:each) do
+          ActiveRecord::Base.default_timezone = :utc
+          @diana = User.new(:name => "Diana", :rank => 1, :admin => false, :created_at => "1992-10-10 12:30:00")
+          @laura = User.new(:name => "Laura", :rank => 2, :admin => true,  :created_at => "2003-05-26 14:15:00")
+          @ap = AwesomePrint.new(:plain => true)
+        end
+
+        it "display single record" do
+          out = @ap.send(:awesome, @diana)
+          str = <<-EOS.strip
 #<User:0x01234567> {
             :id => nil,
           :name => "Diana",
@@ -59,18 +59,18 @@ if defined?(::ActiveRecord)
     :created_at => ?
 }
 EOS
-        if RUBY_VERSION.to_f < 1.9
-          str.sub!('?', 'Sat Oct 10 12:30:00 UTC 1992')
-        else
-          str.sub!('?', '1992-10-10 12:30:00 UTC')
-        end
+          if RUBY_VERSION.to_f < 1.9
+            str.sub!('?', 'Sat Oct 10 12:30:00 UTC 1992')
+          else
+            str.sub!('?', '1992-10-10 12:30:00 UTC')
+          end
 
-        out.gsub(/0x([a-f\d]+)/, "0x01234567").should == str
-      end
+          out.gsub(/0x([a-f\d]+)/, "0x01234567").should == str
+        end
 
-      it "display multiple records" do
-        out = @ap.send(:awesome, [ @diana, @laura ])
-        str = <<-EOS.strip
+        it "display multiple records" do
+          out = @ap.send(:awesome, [ @diana, @laura ])
+          str = <<-EOS.strip
 [
     [0] #<User:0x01234567> {
                 :id => nil,
@@ -88,23 +88,23 @@ EOS
     }
 ]
 EOS
-        if RUBY_VERSION.to_f < 1.9
-          str.sub!('?', 'Sat Oct 10 12:30:00 UTC 1992')
-          str.sub!('!', 'Mon May 26 14:15:00 UTC 2003')
-        else
-          str.sub!('?', '1992-10-10 12:30:00 UTC')
-          str.sub!('!', '2003-05-26 14:15:00 UTC')
-        end
+          if RUBY_VERSION.to_f < 1.9
+            str.sub!('?', 'Sat Oct 10 12:30:00 UTC 1992')
+            str.sub!('!', 'Mon May 26 14:15:00 UTC 2003')
+          else
+            str.sub!('?', '1992-10-10 12:30:00 UTC')
+            str.sub!('!', '2003-05-26 14:15:00 UTC')
+          end
 
-        out.gsub(/0x([a-f\d]+)/, "0x01234567").should == str
+          out.gsub(/0x([a-f\d]+)/, "0x01234567").should == str
+        end
       end
-    end
 
-    #------------------------------------------------------------------------------
-    describe "ActiveRecord class" do
-      it "should print the class" do
-        @ap = AwesomePrint.new(:plain => true)
-        @ap.send(:awesome, User).should == <<-EOS.strip
+      #------------------------------------------------------------------------------
+      describe "ActiveRecord class" do
+        it "should print the class" do
+          @ap = AwesomePrint.new(:plain => true)
+          @ap.send(:awesome, User).should == <<-EOS.strip
 class User < ActiveRecord::Base {
             :id => :integer,
           :name => :string,
@@ -112,13 +112,12 @@ class User < ActiveRecord::Base {
          :admin => :boolean,
     :created_at => :datetime
 }
-        EOS
-  
-end
+EOS
+        end
 
-it "should print the class for non-direct subclasses of AR::Base" do
-  @ap = AwesomePrint.new(:plain => true)
-  @ap.send(:awesome, SubUser).should == <<-EOS.strip
+        it "should print the class for non-direct subclasses of AR::Base" do
+          @ap = AwesomePrint.new(:plain => true)
+          @ap.send(:awesome, SubUser).should == <<-EOS.strip
 class SubUser < User {
             :id => :integer,
           :name => :string,
@@ -126,9 +125,12 @@ class SubUser < User {
          :admin => :boolean,
     :created_at => :datetime
 }
-  EOS
-  
+EOS
+        end
       end
     end
   end
+
+rescue LoadError
+  puts "Skipping ActiveRecord specs..."
 end

data/spec/awesome_print_spec.rb

@@ -263,6 +263,45 @@ EOS
     end
   end
 
+  #------------------------------------------------------------------------------
+  describe "Hash with several keys" do
+    before(:each) do
+      @hash = {"b" => "b", :a => "a", :z => "z", "alpha" => "alpha"}
+    end
+
+    it "plain multiline" do
+      out = @hash.ai(:plain => true)
+      if RUBY_VERSION.to_f < 1.9 # Order of @hash keys is not guaranteed.
+        out.should =~ /^\{[^\}]+\}/m
+        out.should =~ /        "b" => "b",?/
+        out.should =~ /         :a => "a",?/
+        out.should =~ /         :z => "z",?/
+        out.should =~ /    "alpha" => "alpha",?$/
+      else
+        out.should == <<-EOS.strip
+{
+        "b" => "b",
+         :a => "a",
+         :z => "z",
+    "alpha" => "alpha"
+}
+EOS
+      end
+    end
+    
+    it "plain multiline with sorted keys" do
+      @hash.ai(:plain => true, :sorted_hash_keys => true).should == <<-EOS.strip
+{
+         :a => "a",
+    "alpha" => "alpha",
+        "b" => "b",
+         :z => "z"
+}
+EOS
+    end
+
+  end
+
   #------------------------------------------------------------------------------
   describe "Negative options[:indent]" do
     before(:each) do
@@ -415,6 +454,47 @@ EOS
       end
       weird.new.ai(:plain => true).should == ''
     end
+
+    # See https://github.com/michaeldv/awesome_print/issues/35
+    it "handle array grep when pattern contains / chapacter" do
+      hash = { "1/x" => 1,  "2//x" => :"2" }
+      grepped = hash.keys.grep(/^(\d+)\//) { $1 }
+      grepped.ai(:plain => true, :multiline => false).should == '[ "1", "2" ]'
+    end
+
+    it "returns value passed as a parameter" do
+      object = rand
+      self.stub!(:puts)
+      (ap object).should == object
+    end
+
+    # Require different file name this time (lib/ap.rb vs. lib/awesome_print).
+    it "several require 'awesome_print' should do no harm" do
+      require File.expand_path(File.dirname(__FILE__) + '/../lib/ap')
+      lambda { rand.ai }.should_not raise_error
+    end
+  end
+
+  describe "HTML output" do
+    it "wraps ap output with plain <pre> tag" do
+      markup = rand
+      markup.ai(:html => true, :plain => true).should == "<pre>#{markup}</pre>"
+    end
+
+    it "wraps ap output with colorized <pre> tag" do
+      markup = rand
+      markup.ai(:html => true).should == %Q|<pre style="color:blue">#{markup}</pre>|
+    end
+
+    it "encodes HTML entities (plain)" do
+      markup = ' &<hello>'
+      markup.ai(:html => true, :plain => true).should == '<pre>&quot; &amp;&lt;hello&gt;&quot;</pre>'
+    end
+
+    it "encodes HTML entities (color)" do
+      markup = ' &<hello>'
+      markup.ai(:html => true).should == '<pre style="color:brown">&quot; &amp;&lt;hello&gt;&quot;</pre>'
+    end
   end
 
   #------------------------------------------------------------------------------
@@ -465,15 +545,36 @@ EOS
     it "inherited from File should be displayed as File" do
       class My < File; end
 
-      my = File.new('/dev/null')
+      my = File.new('/dev/null') rescue File.new('nul')
       my.ai(:plain => true).should == "#{my.inspect}\n" << `ls -alF #{my.path}`.chop
     end
 
     it "inherited from Dir should be displayed as Dir" do
       class My < Dir; end
 
-      my = My.new('/tmp')
+      require 'tmpdir'
+      my = My.new(Dir.tmpdir)
       my.ai(:plain => true).should == "#{my.inspect}\n" << `ls -alF #{my.path}`.chop
     end
+
+    it "should handle a class that defines its own #send method" do
+      class My
+        def send(arg1, arg2, arg3); end
+      end
+
+      my = My.new
+      my.methods.ai(:plain => true).should_not raise_error(ArgumentError)
+    end
+
+    it "should handle a class defines its own #method method (ex. request.method)" do
+      class My
+        def method
+          'POST'
+        end
+      end
+
+      my = My.new
+      my.methods.ai(:plain => true).should_not raise_error(ArgumentError)
+    end
   end
 end

data/spec/colorization_spec.rb

@@ -0,0 +1,84 @@
+require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+
+describe "AwesomePrint" do
+  before(:each) do
+    stub_dotfile!
+  end
+
+  describe "colorization" do
+    PLAIN = '[ 1, :two, "three", [ nil, [ true, false ] ] ]'
+    COLORIZED = "[ \e[1;34m1\e[0m, \e[0;36m:two\e[0m, \e[0;33m\"three\"\e[0m, [ \e[1;31mnil\e[0m, [ \e[1;32mtrue\e[0m, \e[1;31mfalse\e[0m ] ] ]"
+
+    before(:each) do
+      AwesomePrint.force_colors!(false)
+      ENV['TERM'] = "xterm-colors"
+      ENV.delete('ANSICON')
+      @arr = [ 1, :two, "three", [ nil, [ true, false] ] ]
+    end
+    
+    it "colorizes tty processes by default" do
+      stub_tty!(STDOUT, true)
+
+      @arr.ai(:multiline => false).should == COLORIZED
+    end
+
+    it "colorizes tty processes by default" do
+      stub_tty!(STDOUT, true)
+
+      @arr.ai(:multiline => false).should == COLORIZED
+    end
+
+    
+    it "colorizes processes with ENV['ANSICON'] by default" do
+      stub_tty!(STDOUT, true)
+      ENV['ANSICON'] = "1"
+
+      @arr.ai(:multiline => false).should == COLORIZED
+    end
+
+    it "does not colorize tty processes running in dumb terminals by default" do
+      stub_tty!(STDOUT, true)
+      ENV['TERM'] = "dumb"
+
+      @arr.ai(:multiline => false).should == PLAIN
+    end
+
+    it "does not colorize subprocesses by default" do
+      stub_tty!(STDOUT, false)
+
+      @arr.ai(:multiline => false).should == PLAIN
+    end
+    
+    describe "forced" do
+      before(:each) do
+        AwesomePrint.force_colors!
+      end
+      
+      it "still colorizes tty processes" do
+        stub_tty!(STDOUT, true)
+
+        @arr.ai(:multiline => false).should == COLORIZED
+      end
+      
+      it "colorizes dumb terminals" do
+        stub_tty!(STDOUT, true)
+        ENV["TERM"] = "dumb"
+
+        @arr.ai(:multiline => false).should == COLORIZED
+      end
+
+      it "colorizes subprocess" do
+        stub_tty!(STDOUT, true)
+        @arr.ai(:multiline => false).should == COLORIZED
+      end
+    end
+  end
+  
+  def stub_tty!(stream, value)
+    eval(%{class << stream
+      def tty?
+        #{value}
+      end
+    end})
+  end
+end

data/spec/logger_spec.rb

@@ -6,7 +6,7 @@ require 'ap/core_ext/logger'
 
 describe "AwesomePrint logging extensions" do
   before(:all) do
-    @logger = Logger.new('/dev/null')
+    @logger = Logger.new('/dev/null') rescue Logger.new('nul')
   end
 
   describe "ap method" do

data/spec/methods_spec.rb

@@ -406,4 +406,23 @@ describe "Methods arrays" do
     out = Hello.methods.grep(/^m(\d)$/) { %w(none one)[$1.to_i] }.ai(:plain => true)
     out.should == "[\n    [0] none() Hello\n    [1]  one() Hello\n]"
   end
+
+  # See https://github.com/michaeldv/awesome_print/issues/30 for details.
+  it "grepping methods and converting them to_sym should work as expected" do
+    class Hello
+      private
+      def him; end
+
+      def his
+        private_methods.grep(/^h..$/) { |n| n.to_sym }
+      end
+
+      def her
+        private_methods.grep(/^.e.$/) { |n| n.to_sym }
+      end
+    end
+
+    hello = Hello.new
+    (hello.send(:his) - hello.send(:her)).sort_by { |x| x.to_s }.should == [ :him, :his ]
+  end
 end