diff options
| author | David RodrÃguez <deivid.rodriguez@riseup.net> | 2020-01-10 09:56:42 +0100 |
|---|---|---|
| committer | David RodrÃguez <deivid.rodriguez@riseup.net> | 2020-01-12 20:46:33 +0100 |
| commit | 7dc38a75680f991bf32320cca850c32d1dcd90ca (patch) | |
| tree | e532f1f4eb42a0b9f438ef40380a1ab9ae810975 /spec/plugins/source_spec.rb | |
| parent | 55630c23f4e0683e820116af917888cf23569020 (diff) | |
| download | bundler-tests/less_flakyness.tar.gz | |
Remove global directory switching from specstests/less_flakyness
`Dir.chdir` is not thread safe, so it makes our parallel specs flaky.
Instead, use the following alternatives:
* Use `:chdir` parameter to `Open3` methods for specs that shell out.
* Stub `find_gemfile` or other relevant helpers for unit tests.
Diffstat (limited to 'spec/plugins/source_spec.rb')
| -rw-r--r-- | spec/plugins/source_spec.rb | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/spec/plugins/source_spec.rb b/spec/plugins/source_spec.rb index c8deee96b1..14643e5c81 100644 --- a/spec/plugins/source_spec.rb +++ b/spec/plugins/source_spec.rb @@ -21,6 +21,7 @@ RSpec.describe "bundler source plugin" do end G + allow(Bundler::SharedHelpers).to receive(:find_gemfile).and_return(bundled_app_gemfile) plugin_should_be_installed("bundler-source-psource") end @@ -75,6 +76,7 @@ RSpec.describe "bundler source plugin" do end it "installs the explicit one" do + allow(Bundler::SharedHelpers).to receive(:find_gemfile).and_return(bundled_app_gemfile) plugin_should_be_installed("another-psource") end @@ -100,6 +102,7 @@ RSpec.describe "bundler source plugin" do end it "installs the default one" do + allow(Bundler::SharedHelpers).to receive(:find_gemfile).and_return(bundled_app_gemfile) plugin_should_be_installed("bundler-source-psource") end end |
