Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

sinatra-contrib tests fails on truffleruby 24.0.0 #2007

Open
dentarg opened this issue Mar 27, 2024 · 6 comments
Open

sinatra-contrib tests fails on truffleruby 24.0.0 #2007

dentarg opened this issue Mar 27, 2024 · 6 comments

Comments

@dentarg
Copy link
Member

dentarg commented Mar 27, 2024

We had passing tests with truffleruby 23.1.2, like ruby 3.2.2, Oracle GraalVM Native [x86_64-linux] on Jan 19.

We now have failing tests with truffleruby 24.0.0, like ruby 3.2.2, Oracle GraalVM Native [x86_64-linux].

Saving the log here

Failures:

  1) Sinatra::Capture hamlit behaves like a template language allows nested captures
     Failure/Error: expect(render(engine, "nested_#{lang}")).to eq("Say Hello World!")

       expected: "Say Hello World!"
            got: "Say Hello \#{::Hamlit::Utils.escape_html((a.strip))}"

       (compared using ==)
     Shared Example Group: "a template language" called from ./spec/capture_spec.rb:41
     # ./spec/capture_spec.rb:36:in `block (3 levels) in <top (required)>'

  2) Sinatra::Capture hamlit behaves like a template language captures content
     Failure/Error: expect(render(engine, "simple_#{lang}")).to eq("Say Hello World!")

       expected: "Say Hello World!"
            got: "Say Hello \#{::Hamlit::Utils.escape_html((a.strip))}!"

       (compared using ==)
     Shared Example Group: "a template language" called from ./spec/capture_spec.rb:41
     # ./spec/capture_spec.rb:32:in `block (3 levels) in <top (required)>'

  3) Sinatra::Capture haml behaves like a template language allows nested captures
     Failure/Error: expect(render(engine, "nested_#{lang}")).to eq("Say Hello World!")

       expected: "Say Hello World!"
            got: "Say Hello \#{(::Haml::Util.escape_html((a.strip))).to_s}"

       (compared using ==)
     Shared Example Group: "a template language" called from ./spec/capture_spec.rb:40
     # ./spec/capture_spec.rb:36:in `block (3 levels) in <top (required)>'

  4) Sinatra::Capture haml behaves like a template language captures content
     Failure/Error: expect(render(engine, "simple_#{lang}")).to eq("Say Hello World!")

       expected: "Say Hello World!"
            got: "Say Hello \#{(::Haml::Util.escape_html((a.strip))).to_s}!"

       (compared using ==)
     Shared Example Group: "a template language" called from ./spec/capture_spec.rb:40
     # ./spec/capture_spec.rb:32:in `block (3 levels) in <top (required)>'

cc @eregon

dentarg added a commit that referenced this issue Mar 27, 2024
dentarg added a commit that referenced this issue Mar 27, 2024
@eregon
Copy link
Contributor

eregon commented Mar 27, 2024

@andrykonchin Could you investigate?
From the output it looks like #{} interpolation is not done, but that surely works fine in regular Ruby code.
Maybe some Regexp issue or so since this is likely interpolation in some template or so?

@dentarg
Copy link
Member Author

dentarg commented May 31, 2024

Still fails on truffleruby 24.0.1 (but I guess it will be fixed eventually when truffleruby vendors the fixed prism?)

@eregon
Copy link
Contributor

eregon commented May 31, 2024

@dentarg What about truffleruby-head? Does it work there? We update Prism frequently on master, but we don't update it for patch releases.

@dentarg
Copy link
Member Author

dentarg commented May 31, 2024

@eregon
Copy link
Contributor

eregon commented May 31, 2024

Nice, then that fix will be in 24.1.0, and in the meantime I guess either allow to fail for that job or pin to 23.1 or only test truffleruby-head.

@dentarg
Copy link
Member Author

dentarg commented May 31, 2024

It is allowed to fail. This was just me checking in and making a note after seeing the failure in the CI summary :-)

Thanks for the info about 24.1.0. I will leave this open until we have that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants