Is bundler a dependency of turbo tests?

[deivid-rodriguez]

I'm trying to add turbo tests to dependabot-core at dependabot/dependabot-core#8092, but I'm getting an issue due to the dependency on Bundler.

So this is a question-PR. Does turbo tests actually depend on Bundler?

[ilyazub]

Thanks for your contribution, @deivid-rodriguez!

I introduced bundle exec in 09f2d37. From what I remember, the reason was the failing turbo_tests execution outside of the turbo_tests repo. (This might be not valid now.)

Also, Bundler is used in the GitHub workflow (I left ${{matrix.env}} here by mistake):

turbo_tests/.github/workflows/tests.yml

Line 29 in ce2ea87

${{matrix.env}} bundle exec turbo_tests -n4

[deivid-rodriguez]

Oh, I see, I totally missed that. Thanks!

I believe if we are already in a bundle exec context, we already have the proper ENV setup, so just rspec would use the rspec version in your app context. While if you're not in a bundle exec context, rspec should look for a globally installed rspec version.

So maybe rspec without bundle exec just works.

[ilyazub]

So maybe rspec without bundle exec just works.

It worked for me locally and on CI (https://github.com/serpapi/turbo_tests/actions/runs/6336870000)

I bumped the version and removed unnecessary matrix.env from the GitHub workflow in 2ae707e.

[deivid-rodriguez]

Thanks, I just noticed that the hard dependency on bundler was removed by d0d84ac, so I think it's indeed fine to remove the dependency since it's now optional.

[ilyazub]

I just released this PR in turbo_tests v2.2.0

• https://rubygems.org/gems/turbo_tests
• https://github.com/serpapi/turbo_tests/releases/tag/v2.2.0

@deivid-rodriguez Thank you for introducing turbo_tests to dependabot! Let's see if helps to fix the ci-test in dependabot/dependabot-core#8092.

[deivid-rodriguez]

Thank you!