diff options
author | yugui <yugui@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2011-01-16 06:30:28 +0000 |
---|---|---|
committer | yugui <yugui@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2011-01-16 06:30:28 +0000 |
commit | 27f15990add9196eb04e3a6b4da951e7e4280beb (patch) | |
tree | a64573ed36a7a9fc922dee04b0ea5a1c11eaba03 /test/ripper | |
parent | 85e99f1d432544c9963f29307da1bf227f0933e3 (diff) |
merges r30381 from trunk into ruby_1_9_2.
--
* parse.y (stmt): missing ripper rule. i.e., `a::B ||= c 1'.
http://twitter.com/#!/wannabe53/status/18797576396472321
http://twitter.com/#!/wannabe53/status/18798416150663168
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_1_9_2@30558 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'test/ripper')
-rw-r--r-- | test/ripper/dummyparser.rb | 14 | ||||
-rw-r--r-- | test/ripper/test_parser_events.rb | 17 |
2 files changed, 21 insertions, 10 deletions
diff --git a/test/ripper/dummyparser.rb b/test/ripper/dummyparser.rb index ffc1f78ac6..07a7eda023 100644 --- a/test/ripper/dummyparser.rb +++ b/test/ripper/dummyparser.rb @@ -10,7 +10,7 @@ class Node @children = nodes end - attr_reader :children + attr_reader :name, :children def to_s "#{@name}(#{Node.trim_nil(@children).map {|n| n.to_s }.join(',')})" @@ -47,12 +47,14 @@ class NodeList end class DummyParser < Ripper - def hook(name) + def hook(*names) class << self; self; end.class_eval do - define_method(name) do |*a, &b| - result = super(*a, &b) - yield(*a) - result + names.each do |name| + define_method(name) do |*a, &b| + result = super(*a, &b) + yield(name, *a) + result + end end end self diff --git a/test/ripper/test_parser_events.rb b/test/ripper/test_parser_events.rb index 2ac623f76d..15a5d2b3a9 100644 --- a/test/ripper/test_parser_events.rb +++ b/test/ripper/test_parser_events.rb @@ -20,7 +20,7 @@ class TestRipper::ParserEvents < Test::Unit::TestCase def parse(str, nm = nil, &bl) dp = DummyParser.new(str) - dp.hook(nm, &bl) if nm + dp.hook(*nm, &bl) if nm dp.parse.to_s end @@ -347,10 +347,10 @@ class TestRipper::ParserEvents < Test::Unit::TestCase assert_equal true, thru_heredoc_beg assert_match(/string_content\(\),heredoc\n/, tree, bug1921) heredoc = nil - parse("<<EOS\nheredoc1\nheredoc2\nEOS\n", :on_string_add) {|n, s| heredoc = s} + parse("<<EOS\nheredoc1\nheredoc2\nEOS\n", :on_string_add) {|e, n, s| heredoc = s} assert_equal("heredoc1\nheredoc2\n", heredoc, bug1921) heredoc = nil - parse("<<-EOS\nheredoc1\nheredoc2\n\tEOS\n", :on_string_add) {|n, s| heredoc = s} + parse("<<-EOS\nheredoc1\nheredoc2\n\tEOS\n", :on_string_add) {|e, n, s| heredoc = s} assert_equal("heredoc1\nheredoc2\n", heredoc, bug1921) end @@ -676,6 +676,15 @@ class TestRipper::ParserEvents < Test::Unit::TestCase assert_equal true, thru_opassign end + def test_opassign_error + thru_opassign = [] + events = [:on_opassign, :on_assign_error] + parse('a::X ||= c 1', events) {|a,*b| + thru_opassign << a + } + assert_equal events, thru_opassign + end + def test_param_error thru_param_error = false parse('def foo(A) end', :on_param_error) {thru_param_error = true} @@ -1104,7 +1113,7 @@ class TestRipper::ParserEvents < Test::Unit::TestCase def test_unterminated_regexp compile_error = false - parse('/', :compile_error) {|msg| compile_error = msg} + parse('/', :compile_error) {|e, msg| compile_error = msg} assert_equal("unterminated regexp meets end of file", compile_error) end end if ripper_test |