[ruby/irb] refactoring an error handling in IRB::Inspector

* moved rescue clause to `#inspect_value` to catch all failures in inspectors
* test with all (currently five kind of) inspect modes
  - tweaked the input due to only `Marshal` can inspect(dump) a `BasicObject`

9d112fab8e

lib/irb/inspector.rb

@@ -100,21 +100,19 @@ module IRB # :nodoc:
     # Proc to call when the input is evaluated and output in irb.
     def inspect_value(v)
       @inspect.call(v)
+    rescue
+      puts "(Object doesn't support #inspect)"
+      ''
     end
   end
 
   Inspector.def_inspector([false, :to_s, :raw]){|v| v.to_s}
   Inspector.def_inspector([:p, :inspect]){|v|
-    begin
+    result = v.inspect
-      result = v.inspect
+    if IRB.conf[:MAIN_CONTEXT]&.use_colorize? && Color.inspect_colorable?(v)
-      if IRB.conf[:MAIN_CONTEXT]&.use_colorize? && Color.inspect_colorable?(v)
+      result = Color.colorize_code(result)
-        result = Color.colorize_code(result)
-      end
-      result
-    rescue NoMethodError
-      puts "(Object doesn't support #inspect)"
-      ''
     end
+    result
   }
   Inspector.def_inspector([true, :pp, :pretty_inspect], proc{require "irb/color_printer"}){|v|
     if IRB.conf[:MAIN_CONTEXT]&.use_colorize?

test/irb/test_context.rb

@@ -106,15 +106,22 @@ module TestIRB
 
     def test_eval_object_without_inspect_method
       verbose, $VERBOSE = $VERBOSE, nil
-      input = TestInputMethod.new([
+      all_assertions do |all|
-        "BasicObject.new\n",
+        IRB::Inspector::INSPECTORS.invert.each_value do |mode|
-      ])
+          all.for(mode) do
-      irb = IRB::Irb.new(IRB::WorkSpace.new(Object.new), input)
+            input = TestInputMethod.new([
-      out, err = capture_output do
+                "[BasicObject.new, Class.new]\n",
-        irb.eval_input
+              ])
+            irb = IRB::Irb.new(IRB::WorkSpace.new(Object.new), input)
+            irb.context.inspect_mode = mode
+            out, err = capture_output do
+              irb.eval_input
+            end
+            assert_empty err
+            assert_match(/\(Object doesn't support #inspect\)\n(=> )?\n/, out)
+          end
+        end
       end
-      assert_empty err
-      assert(/\(Object doesn't support #inspect\)\n(=> )?\n/, out)
     ensure
       $VERBOSE = verbose
     end