Improve approach to get type name

Author: adisonlampert

lib/debug/server_dap.rb

@@ -1010,16 +1010,6 @@ def evaluate_result r
       variable nil, r
     end
 
-    def type_name obj
-      klass = M_CLASS.bind_call(obj)
-
-      begin
-        klass.name || klass.to_s
-      rescue Exception => e
-        "<Error: #{e.message} (#{e.backtrace.first}>"
-      end
-    end
-
     def variable_ name, obj, indexedVariables: 0, namedVariables: 0
       if indexedVariables > 0 || namedVariables > 0
         vid = @var_map.size + 1
@@ -1037,17 +1027,20 @@ def variable_ name, obj, indexedVariables: 0, namedVariables: 0
         str = value_inspect(obj)
       end
 
+      klass = M_CLASS.bind_call(obj)
+      type_name = M_NAME.bind_call(klass)
+
       if name
         { name: name,
           value: str,
-          type: type_name(obj),
+          type: type_name,
           variablesReference: vid,
           indexedVariables: indexedVariables,
           namedVariables: namedVariables,
         }
       else
         { result: str,
-          type: type_name(obj),
+          type: type_name,
           variablesReference: vid,
           indexedVariables: indexedVariables,
           namedVariables: namedVariables,

lib/debug/thread_client.rb

@@ -14,6 +14,7 @@ module DEBUGGER__
   M_RESPOND_TO_P = method(:respond_to?).unbind
   M_METHOD = method(:method).unbind
   M_OBJECT_ID = method(:object_id).unbind
+  M_NAME = method(:name).unbind
 
   module SkipPathHelper
     def skip_path?(path)

test/protocol/variables_test.rb

@@ -102,35 +102,7 @@ def test_overwritten_name_method
         variable_info = locals.find { |local| local[:name] == "f" }
 
         assert_match /#<Foo:.*>/, variable_info[:value]
-        assert_match /<Error: wrong number of arguments \(given 0, expected 1\) /, variable_info[:type]
-
-        req_terminate_debuggee
-      end
-    end
-  end
-
-  class DAPOverwrittenToSMethod < ProtocolTestCase
-    PROGRAM = <<~RUBY
-      1| class Foo
-      2|   def self.name
-      3|     nil
-      4|   end
-      5|   def self.to_s(value) end
-      6| end
-      7| f = Foo.new
-      8| __LINE__
-    RUBY
-
-    def test_overwritten_to_s_method
-      run_protocol_scenario PROGRAM, cdp: false do
-        req_add_breakpoint 8
-        req_continue
-
-        locals = gather_variables
-
-        variable_info = locals.find { |local| local[:name] == "f" }
-        assert_match /#<Foo:.*>/, variable_info[:value]
-        assert_match /<Error: wrong number of arguments \(given 0, expected 1\) /, variable_info[:type]
+        assert_equal "Foo", variable_info[:type]
 
         req_terminate_debuggee
       end