summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authornobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2017-03-25 03:23:43 +0000
committernobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2017-03-25 03:23:43 +0000
commit217f599a1e1c09618b0ebeffb34be790f70b8c68 (patch)
tree1cf6c97d0fff0928365e6d4cae5fc4e535569238
parent19a1f70364ccabaace321d492d35a99ad9053423 (diff)
class.c: prohibit refinement module
* class.c (ensure_includable): cannot include refinement module, or the type and the class do not match. [ruby-core:79632] [Bug #13236] git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@58083 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r--class.c3
-rw-r--r--test/ruby/test_refinement.rb14
2 files changed, 17 insertions, 0 deletions
diff --git a/class.c b/class.c
index fc99ce4513..d73724bd60 100644
--- a/class.c
+++ b/class.c
@@ -854,6 +854,9 @@ ensure_includable(VALUE klass, VALUE module)
{
rb_frozen_class_p(klass);
Check_Type(module, T_MODULE);
+ if (!NIL_P(rb_refinement_module_get_refined_class(module))) {
+ rb_raise(rb_eArgError, "refinement module is not allowed");
+ }
OBJ_INFECT(klass, module);
}
diff --git a/test/ruby/test_refinement.rb b/test/ruby/test_refinement.rb
index f6e08729d7..82bdb49659 100644
--- a/test/ruby/test_refinement.rb
+++ b/test/ruby/test_refinement.rb
@@ -1886,6 +1886,20 @@ class TestRefinement < Test::Unit::TestCase
assert_equal("Parent -> Child", SuperToModule::Child.test, bug)
end
+ def test_include_refinement
+ bug = '[ruby-core:79632] [Bug #13236] cannot include refinement module'
+ r = nil
+ m = Module.new do
+ r = refine(String) {def test;:ok end}
+ end
+ assert_raise_with_message(ArgumentError, /refinement/, bug) do
+ m.module_eval {include r}
+ end
+ assert_raise_with_message(ArgumentError, /refinement/, bug) do
+ m.module_eval {prepend r}
+ end
+ end
+
private
def eval_using(mod, s)