Support detailed_message for MixinClassError

satoryu · Mar 20, 2023 · d15b41b · d15b41b
1 parent 1c503b6
commit d15b41b
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 0 deletions.
diff --git a/lib/rbs/errors.rb b/lib/rbs/errors.rb
@@ -426,6 +426,8 @@ def location
  end
 
  class MixinClassError < DefinitionError
+ include DetailedMessageable
+
  attr_reader :type_name
  attr_reader :member
 

diff --git a/test/rbs/ancestor_builder_test.rb b/test/rbs/ancestor_builder_test.rb
@@ -700,6 +700,13 @@ class Qux
 
  assert_raises MixinClassError do
  builder.instance_ancestors(type_name("::Qux"))
+ end.tap do |error|
+ assert_equal <<~DETAILED_MESSAGE, error.detailed_message if Exception.method_defined?(:detailed_message)
+ #{error.message} (RBS::MixinClassError)
+
+ include Foo
+ ^^^^^^^^^^^
+ DETAILED_MESSAGE
  end
  end
  end
@@ -723,6 +730,13 @@ class Qux
 
  assert_raises MixinClassError do
  builder.instance_ancestors(type_name("::Qux"))
+ end.tap do |error|
+ assert_equal <<~DETAILED_MESSAGE, error.detailed_message if Exception.method_defined?(:detailed_message)
+ #{error.message} (RBS::MixinClassError)
+
+ prepend Foo
+ ^^^^^^^^^^^
+ DETAILED_MESSAGE
  end
  end
  end
@@ -747,6 +761,13 @@ class Qux
 
  assert_raises MixinClassError do
  builder.one_singleton_ancestors(type_name("::Qux"))
+ end.tap do |error|
+ assert_equal <<~DETAILED_MESSAGE, error.detailed_message if Exception.method_defined?(:detailed_message)
+ #{error.message} (RBS::MixinClassError)
+
+ extend Foo
+ ^^^^^^^^^^
+ DETAILED_MESSAGE
  end
  end
  end