From 9e28354705e2aa312e5dee775e3bd08095fc7e9c Mon Sep 17 00:00:00 2001 From: Nobuyoshi Nakada Date: Sun, 19 May 2024 19:42:57 +0900 Subject: ripper: Fix excess `compile_error` at simple backref op_asgn Fix up 89cfc1520717257073012ec07105c551e4b8af7c. --- test/ripper/test_parser_events.rb | 24 ++++++++++++------------ 1 file changed, 12 insertions(+), 12 deletions(-) (limited to 'test') diff --git a/test/ripper/test_parser_events.rb b/test/ripper/test_parser_events.rb index a5c7401968..5434acb523 100644 --- a/test/ripper/test_parser_events.rb +++ b/test/ripper/test_parser_events.rb @@ -267,28 +267,28 @@ class TestRipper::ParserEvents < Test::Unit::TestCase end def test_assign_error_backref - thru_assign_error = false + errors = [] result = - parse('$& = 1', :on_assign_error) {thru_assign_error = true} - assert_equal true, thru_assign_error + parse('$& = 1', %i[on_assign_error compile_error]) {|e, *| errors << e} + assert_equal %i[on_assign_error], errors assert_equal '[assign(assign_error(var_field($&)),1)]', result - thru_assign_error = false + errors = [] result = - parse('$&, _ = 1', :on_assign_error) {thru_assign_error = true} - assert_equal true, thru_assign_error + parse('$&, _ = 1', %i[on_assign_error compile_error]) {|e, *| errors << e} + assert_equal %i[on_assign_error], errors assert_equal '[massign([assign_error(var_field($&)),var_field(_)],1)]', result - thru_assign_error = false + errors = [] result = - parse('$& += 1', :on_assign_error) {thru_assign_error = true} - assert_equal true, thru_assign_error + parse('$& += 1', %i[on_assign_error compile_error]) {|e, *| errors << e} + assert_equal %i[on_assign_error], errors assert_equal '[assign_error(opassign(var_field($&),+=,1))]', result - thru_assign_error = false + errors = [] result = - parse('$& += cmd 1, 2', :on_assign_error) {thru_assign_error = true} - assert_equal true, thru_assign_error + parse('$& += cmd 1, 2', %i[on_assign_error compile_error]) {|e, *| errors << e} + assert_equal %i[on_assign_error], errors assert_equal '[assign_error(opassign(var_field($&),+=,command(cmd,[1,2])))]', result end -- cgit v1.2.3