Skip to content

Commit

Permalink
Remove confusing warning on output error.
Browse files Browse the repository at this point in the history
  • Loading branch information
ConradIrwin committed Sep 14, 2011
1 parent 4db25f1 commit f0e8ebb
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 5 deletions.
1 change: 0 additions & 1 deletion lib/pry.rb
Expand Up @@ -29,7 +29,6 @@ class Pry
# Read the class name off of the singleton class to provide a default inspect.
klass = (class << value; self; end).ancestors.first
stringified = "#<#{klass}:0x#{value.__id__.to_s(16)}>"
Helpers::BaseHelpers.stagger_output("output error: #{ex.inspect}", output) if ex
end

Helpers::BaseHelpers.stagger_output("=> #{Helpers::BaseHelpers.colorize_code(stringified)}", output)
Expand Down
4 changes: 0 additions & 4 deletions test/test_pry_output.rb
Expand Up @@ -36,9 +36,5 @@
it "should not be phased by un-inspectable things" do
mock_pry("class NastyClass; undef pretty_inspect; end", "NastyClass.new").should =~ /#<NastyClass:0x[0-9a-f]+>/
end

it "should warn you about un-inspectable things" do
mock_pry("class NastyClass; undef pretty_inspect; end", "NastyClass.new").should =~ /output error: #<(NoMethodError|NameError): undefined method `pretty_inspect'/
end
end
end

0 comments on commit f0e8ebb

Please sign in to comment.