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
run specs on 3.1 #1867
run specs on 3.1 #1867
Conversation
Signed-off-by: kasif <kadnan@progress.com>
Signed-off-by: kasif <kadnan@progress.com>
spec/kitchen/util_spec.rb
Outdated
# ].map { |f| File.join(@root, f) } | ||
# (listed - expected).must_equal [] | ||
# (expected - listed).must_equal [] | ||
# end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we need to optionally include "bar/."
if the Ruby version is below 3.1. I can't see anything in the Ruby 3.1 release notes. But I've confirmed locally that between 3.0 and 3.1 Dir.glob no longer returns the trailing .
directories. i.e. this is no longer returned in 3.1
"./.vagrant/bundler/."
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right. I found the bug in the tracker here https://bugs.ruby-lang.org/issues/18436
So it looks like the trailing .
was in fact a bug
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@damacus , wondering what would be the best way to handle the spec in this case for ruby 3.0 and lower. For now, i have added a check for version >=3.1
Signed-off-by: kasif <kadnan@progress.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🥳
Signed-off-by: kasif kadnan@progress.com
Description
Describe what this change achieves
Issues Resolved
List any existing issues this PR resolves
Check List