sinatra / sinatra

Classy web-development dressed in a DSL (official / canonical repo)

Home Page:https://sinatrarb.com

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Failing tests with ruby-head

dentarg opened this issue · comments

CI passed just a few days ago at https://github.com/dentarg/sinatra/actions/runs/7423722745/job/20201775664#step:3:23
ruby 3.4.0dev (2024-01-04T16:25:31Z :detached: 6c252912af) [x86_64-linux]

but now fails https://github.com/dentarg/sinatra/actions/runs/7423722745/job/20201775664
ruby 3.4.0dev (2024-01-09T19:02:26Z :detached: 80da9b1547) [x86_64-linux]

on this test https://github.com/sinatra/sinatra/blob/main/rack-protection/spec/lib/rack/protection/encrypted_cookie_spec.rb#L500-L504

I wonder why it started happening now? Changes in Ruby: ruby/ruby@6c25291...80da9b1

So d6294df didn't help, that's interesting

We have more fails, sinatra tests too: https://github.com/sinatra/sinatra/actions/runs/7476448903/job/20346834173#step:5:25

Maybe this one

get '/this/is/a/test/'
assert ok?
assert_equal 'right on', body
end

From @byroot on Slack

Clearly a bug IMO. That line doesn’t mutate that interpolated string.

So it should be reported to https://bugs.ruby-lang.org/, but it would be good to have a smaller reproducer for that.

I don't have ruby-head built locally and rubylang/ruby:master-nightly-bionic (docker image) isn't up-to-date

root@12277fb5a008:/app# ruby -v
ruby 3.3.0dev (2022-12-26T15:28:38Z master a968d783a3) [x86_64-linux]

I guess I could try to build that image myself though, not sure when I get to it though.

@byroot Oops, of course 🤦 didn't think that through. Thanks.

ruby-head no longer has any failing tests (example: https://github.com/sinatra/sinatra/actions/runs/8448241782)