kotovalexarian-likes-github/ruby--ruby
1
0
Fork 0
mirror of https://github.com/ruby/ruby.git synced 2022-11-09 12:17:21 -05:00
Code Releases Activity

variable.c: fix receiver on private constant

Browse source 
* variable.c (rb_const_search): fix NameError :receiver attribute
  on private constant, should raise with the included module, not
  the ICLASS.

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63696 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
This commit is contained in:
nobu 2018-06-19 08:15:52 +00:00
parent b3f56c6be0
commit 1b474b869c
2 changed files with 39 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

42
test/ruby/test_module.rb
View file

@ -1349,21 +1349,55 @@ class TestModule < Test::Unit::TestCase
assert_raise(ArgumentError, bug8540) { c.new.send :foo= } assert_raise(ArgumentError, bug8540) { c.new.send :foo= }
end end
def test_private_constant def test_private_constant_in_class
c = Class.new c = Class.new
c.const_set(:FOO, "foo") c.const_set(:FOO, "foo")
assert_equal("foo", c::FOO) assert_equal("foo", c::FOO)
c.private_constant(:FOO) c.private_constant(:FOO)
assert_raise(NameError) { c::FOO } e = assert_raise(NameError) {c::FOO}
assert_equal(c, e.receiver)
assert_equal(:FOO, e.name)
assert_equal("foo", c.class_eval("FOO")) assert_equal("foo", c.class_eval("FOO"))
assert_equal("foo", c.const_get("FOO")) assert_equal("foo", c.const_get("FOO"))
$VERBOSE, verbose = nil, $VERBOSE $VERBOSE, verbose = nil, $VERBOSE
c.const_set(:FOO, "foo") c.const_set(:FOO, "foo")
$VERBOSE = verbose $VERBOSE = verbose
assert_raise(NameError) { c::FOO } e = assert_raise(NameError) {c::FOO}
assert_raise_with_message(NameError, /#{c}::FOO/) do assert_equal(c, e.receiver)
assert_equal(:FOO, e.name)
e = assert_raise_with_message(NameError, /#{c}::FOO/) do
Class.new(c)::FOO Class.new(c)::FOO
end end
assert_equal(c, e.receiver)
assert_equal(:FOO, e.name)
end
def test_private_constant_in_module
m = Module.new
m.const_set(:FOO, "foo")
assert_equal("foo", m::FOO)
m.private_constant(:FOO)
e = assert_raise(NameError) {m::FOO}
assert_equal(m, e.receiver)
assert_equal(:FOO, e.name)
assert_equal("foo", m.class_eval("FOO"))
assert_equal("foo", m.const_get("FOO"))
$VERBOSE, verbose = nil, $VERBOSE
m.const_set(:FOO, "foo")
$VERBOSE = verbose
e = assert_raise(NameError) {m::FOO}
assert_equal(m, e.receiver)
assert_equal(:FOO, e.name)
e = assert_raise(NameError, /#{m}::FOO/) do
Module.new {include m}::FOO
end
assert_equal(m, e.receiver)
assert_equal(:FOO, e.name)
e = assert_raise(NameError, /#{m}::FOO/) do
Class.new {include m}::FOO
end
assert_equal(m, e.receiver)
assert_equal(:FOO, e.name)
end end
def test_private_constant2 def test_private_constant2

1
variable.c
View file

@ -2362,6 +2362,7 @@ rb_const_search(VALUE klass, ID id, int exclude, int recurse, int visibility)
while ((ce = rb_const_lookup(tmp, id))) { while ((ce = rb_const_lookup(tmp, id))) {
if (visibility && RB_CONST_PRIVATE_P(ce)) { if (visibility && RB_CONST_PRIVATE_P(ce)) {
if (BUILTIN_TYPE(tmp) == T_ICLASS) tmp = RBASIC(tmp)->klass;
rb_name_err_raise("private constant %2$s::%1$s referenced", rb_name_err_raise("private constant %2$s::%1$s referenced",
tmp, ID2SYM(id)); tmp, ID2SYM(id));
} }