diff options
| author | Earlopain <14981592+Earlopain@users.noreply.github.com> | 2026-01-16 10:47:18 +0100 |
|---|---|---|
| committer | git <svn-admin@ruby-lang.org> | 2026-01-16 20:19:26 +0000 |
| commit | 87147ba5e983f554fb56f859af0d858d5eff0896 (patch) | |
| tree | 092aced5e58639bd9d1240a91c4a322e697caeb2 /test | |
| parent | 1c7e19f961a3e9051cf728deeea196d4d72a89cd (diff) | |
[ruby/prism] Make the ripper shim work with rdoc
The filter class is a 1:1 copy of ruby.
rdoc has 32 test failures. It seems to expect `on_sp` in some cases to render code as written.
https://github.com/ruby/prism/commit/74bb12c825
Diffstat (limited to 'test')
| -rw-r--r-- | test/prism/ruby/ripper_test.rb | 32 |
1 files changed, 22 insertions, 10 deletions
diff --git a/test/prism/ruby/ripper_test.rb b/test/prism/ruby/ripper_test.rb index cac20a073d..2a0504c19f 100644 --- a/test/prism/ruby/ripper_test.rb +++ b/test/prism/ruby/ripper_test.rb @@ -59,7 +59,7 @@ module Prism "whitequark/slash_newline_in_heredocs.txt" ] - omitted_lexer_parse = [ + omitted_lex = [ "comments.txt", "heredoc_percent_q_newline_delimiter.txt", "heredoc_with_escaped_newline_at_start.txt", @@ -80,8 +80,20 @@ module Prism define_method("#{fixture.test_name}_sexp_raw") { assert_ripper_sexp_raw(fixture.read) } end - Fixture.each_for_current_ruby(except: incorrect | omitted_lexer_parse) do |fixture| - define_method("#{fixture.test_name}_lexer_parse") { assert_ripper_lexer_parse(fixture.read) } + Fixture.each_for_current_ruby(except: incorrect | omitted_lex) do |fixture| + define_method("#{fixture.test_name}_lex") { assert_ripper_lex(fixture.read) } + end + + def test_lexer + lexer = Translation::Ripper::Lexer.new("foo") + expected = [[1, 0], :on_ident, "foo", Translation::Ripper::EXPR_CMDARG] + + assert_equal([expected], lexer.lex) + assert_equal(expected, lexer.parse[0].to_a) + assert_equal(lexer.parse[0].to_a, lexer.scan[0].to_a) + + assert_equal(%i[on_int on_op], Translation::Ripper::Lexer.new("1 +").lex.map(&:event)) + assert_raise(SyntaxError) { Translation::Ripper::Lexer.new("1 +").lex(raise_errors: true) } end def test_tokenize @@ -106,15 +118,15 @@ module Prism assert_equal Ripper.sexp_raw(source), Prism::Translation::Ripper.sexp_raw(source) end - def assert_ripper_lexer_parse(source) - prism = Translation::Ripper::Lexer.new(source).parse - ripper = Ripper::Lexer.new(source).parse - ripper.reject! { |elem| elem.event == :on_sp } # Prism doesn't emit on_sp - ripper.sort_by!(&:pos) # Prism emits tokens by their order in the code, not in parse order + def assert_ripper_lex(source) + prism = Translation::Ripper.lex(source) + ripper = Ripper.lex(source) + ripper.reject! { |elem| elem[1] == :on_sp } # Prism doesn't emit on_sp + ripper.sort_by! { |elem| elem[0] } # Prism emits tokens by their order in the code, not in parse order [prism.size, ripper.size].max.times do |i| - expected = ripper[i].to_a - actual = prism[i].to_a + expected = ripper[i] + actual = prism[i] # Since tokens related to heredocs are not emitted in the same order, # the state also doesn't line up. if expected[1] == :on_heredoc_end && actual[1] == :on_heredoc_end |
