-
Notifications
You must be signed in to change notification settings - Fork 209
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
Fix bundler related test for test-bundled-gems #778
Conversation
command_line = if defined?(Bundler) | ||
[ruby, "-rbundler/setup", "-rrbs/test/setup", "sample.rb"] | ||
else | ||
command_line = if ENV['RUBY'] |
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.
Seems like we can omit the ruby
var, but it's up to you. (If you have another fix...)
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.
Thanks, I removed the ruby
var. 🚮
@@ -333,6 +333,7 @@ def test_collection_install | |||
|
|||
path: #{dir.join('gem_rbs_collection')} | |||
YAML | |||
dir.join('Gemfile').write('') |
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.
🤹
c79a4e7
to
1bf2784
Compare
1bf2784
to
5cebaf8
Compare
* Bundle RBS 1.6.0 * Specify bundled rbs commit hash to fix test ref ruby/rbs#778 * Bundle RBS 1.6.1 Co-authored-by: Masataka Pocke Kuwabara <kuwabara@pocke.me>
This PR fixes bundler related tests fro test-bundled-gems.
The RBS's tests fail on ruby/ruby's CI. https://github.com/ruby/ruby/pull/4809/checks?check_run_id=3516979076
The following three tests fail
require 'bundler'
always, soif defined?(Bundler)
is not appropriate solution to detect availability ofbunder/setup
.Bundler::LockfileParser
needsGemfile
in the current directory to detectBundler.root
, but the CLI test doesn't putGemfile
.BUNDLE_GEMFILE
env var.bundle install
as a dependency of goodcheck gem.rbs-amber
.