diff options
author | David RodrÃguez <deivid.rodriguez@riseup.net> | 2019-07-23 19:19:31 +0200 |
---|---|---|
committer | Hiroshi SHIBATA <hsbt@ruby-lang.org> | 2019-08-03 09:30:00 +0900 |
commit | 267a72be800da0ec47f441d7d8210dc6aec137af (patch) | |
tree | 800fc3da598bbe32a9a5644193134e4a2355a036 /spec | |
parent | e84e63230bd024b652451cb7d1a44b9d9e0449ae (diff) | |
download | ruby-267a72be800da0ec47f441d7d8210dc6aec137af.tar.gz |
[bundler/bundler] Enable `Style/UnneededInterpolation` cop
https://github.com/bundler/bundler/commit/4c6a3c3cf7
Diffstat (limited to 'spec')
-rw-r--r-- | spec/bundler/bundler/definition_spec.rb | 2 | ||||
-rw-r--r-- | spec/bundler/bundler/plugin/installer_spec.rb | 6 | ||||
-rw-r--r-- | spec/bundler/bundler/shared_helpers_spec.rb | 2 | ||||
-rw-r--r-- | spec/bundler/cache/git_spec.rb | 18 | ||||
-rw-r--r-- | spec/bundler/commands/exec_spec.rb | 2 | ||||
-rw-r--r-- | spec/bundler/install/gemfile/sources_spec.rb | 4 |
6 files changed, 17 insertions, 17 deletions
diff --git a/spec/bundler/bundler/definition_spec.rb b/spec/bundler/bundler/definition_spec.rb index 57222f8af5..92f836299d 100644 --- a/spec/bundler/bundler/definition_spec.rb +++ b/spec/bundler/bundler/definition_spec.rb @@ -230,7 +230,7 @@ RSpec.describe Bundler::Definition do context "eager unlock" do let(:source_list) do Bundler::SourceList.new.tap do |source_list| - source_list.global_rubygems_source = "#{file_uri_for(gem_repo4)}" + source_list.global_rubygems_source = file_uri_for(gem_repo4) end end diff --git a/spec/bundler/bundler/plugin/installer_spec.rb b/spec/bundler/bundler/plugin/installer_spec.rb index c7e0d06746..e89720f6f7 100644 --- a/spec/bundler/bundler/plugin/installer_spec.rb +++ b/spec/bundler/bundler/plugin/installer_spec.rb @@ -58,7 +58,7 @@ RSpec.describe Bundler::Plugin::Installer do end let(:result) do - installer.install(["ga-plugin"], :git => "#{file_uri_for(lib_path("ga-plugin"))}") + installer.install(["ga-plugin"], :git => file_uri_for(lib_path("ga-plugin"))) end it "returns the installed spec after installing" do @@ -98,7 +98,7 @@ RSpec.describe Bundler::Plugin::Installer do context "rubygems plugins" do let(:result) do - installer.install(["re-plugin"], :source => "#{file_uri_for(gem_repo2)}") + installer.install(["re-plugin"], :source => file_uri_for(gem_repo2)) end it "returns the installed spec after installing " do @@ -113,7 +113,7 @@ RSpec.describe Bundler::Plugin::Installer do context "multiple plugins" do let(:result) do - installer.install(["re-plugin", "ma-plugin"], :source => "#{file_uri_for(gem_repo2)}") + installer.install(["re-plugin", "ma-plugin"], :source => file_uri_for(gem_repo2)) end it "returns the installed spec after installing " do diff --git a/spec/bundler/bundler/shared_helpers_spec.rb b/spec/bundler/bundler/shared_helpers_spec.rb index 0fb728099e..910854fe46 100644 --- a/spec/bundler/bundler/shared_helpers_spec.rb +++ b/spec/bundler/bundler/shared_helpers_spec.rb @@ -74,7 +74,7 @@ RSpec.describe Bundler::SharedHelpers do end context ".bundle is global .bundle" do - let(:global_rubygems_dir) { Pathname.new("#{bundled_app}") } + let(:global_rubygems_dir) { Pathname.new(bundled_app) } before do Dir.mkdir ".bundle" diff --git a/spec/bundler/cache/git_spec.rb b/spec/bundler/cache/git_spec.rb index d31d607d25..1348d033e2 100644 --- a/spec/bundler/cache/git_spec.rb +++ b/spec/bundler/cache/git_spec.rb @@ -22,7 +22,7 @@ end gem "foo", :git => '#{lib_path("foo-1.0")}' G - bundle "#{cmd}", forgotten_command_line_options([:all, :cache_all] => true) + bundle cmd, forgotten_command_line_options([:all, :cache_all] => true) expect(bundled_app("vendor/cache/foo-1.0-#{ref}")).to exist expect(bundled_app("vendor/cache/foo-1.0-#{ref}/.git")).not_to exist expect(bundled_app("vendor/cache/foo-1.0-#{ref}/.bundlecache")).to be_file @@ -40,7 +40,7 @@ end G bundle "install --path vendor/bundle" - bundle "#{cmd}", forgotten_command_line_options([:all, :cache_all] => true) + bundle cmd, forgotten_command_line_options([:all, :cache_all] => true) expect(bundled_app("vendor/cache/foo-1.0-#{ref}")).to exist expect(bundled_app("vendor/cache/foo-1.0-#{ref}/.git")).not_to exist @@ -56,8 +56,8 @@ end gem "foo", :git => '#{lib_path("foo-1.0")}' G - bundle! "#{cmd}", forgotten_command_line_options([:all, :cache_all] => true) - bundle! "#{cmd}", forgotten_command_line_options([:all, :cache_all] => true) + bundle! cmd, forgotten_command_line_options([:all, :cache_all] => true) + bundle! cmd, forgotten_command_line_options([:all, :cache_all] => true) expect(out).to include "Updating files in vendor/cache" FileUtils.rm_rf lib_path("foo-1.0") @@ -72,7 +72,7 @@ end gem "foo", :git => '#{lib_path("foo-1.0")}' G - bundle "#{cmd}", forgotten_command_line_options([:all, :cache_all] => true) + bundle cmd, forgotten_command_line_options([:all, :cache_all] => true) update_git "foo" do |s| s.write "lib/foo.rb", "puts :CACHE" @@ -82,7 +82,7 @@ end expect(ref).not_to eq(old_ref) bundle! "update", :all => true - bundle! "#{cmd}", forgotten_command_line_options([:all, :cache_all] => true) + bundle! cmd, forgotten_command_line_options([:all, :cache_all] => true) expect(bundled_app("vendor/cache/foo-1.0-#{ref}")).to exist expect(bundled_app("vendor/cache/foo-1.0-#{old_ref}")).not_to exist @@ -129,7 +129,7 @@ end bundle %(config set local.foo #{lib_path("foo-1.0")}) bundle "install" - bundle "#{cmd}", forgotten_command_line_options([:all, :cache_all] => true) + bundle cmd, forgotten_command_line_options([:all, :cache_all] => true) expect(bundled_app("vendor/cache/foo-invalid-#{ref}")).to exist @@ -161,7 +161,7 @@ end G ref = git.ref_for("master", 11) - bundle "#{cmd}", forgotten_command_line_options([:all, :cache_all] => true) + bundle cmd, forgotten_command_line_options([:all, :cache_all] => true) expect(bundled_app("vendor/cache/has_submodule-1.0-#{ref}")).to exist expect(bundled_app("vendor/cache/has_submodule-1.0-#{ref}/submodule-1.0")).to exist @@ -175,7 +175,7 @@ end gem "foo", :git => '#{lib_path("foo-1.0")}' G - bundle "#{cmd}" + bundle cmd expect(err).to include("Your Gemfile contains path and git dependencies.") end diff --git a/spec/bundler/commands/exec_spec.rb b/spec/bundler/commands/exec_spec.rb index b01a31fb50..752d213994 100644 --- a/spec/bundler/commands/exec_spec.rb +++ b/spec/bundler/commands/exec_spec.rb @@ -299,7 +299,7 @@ RSpec.describe "bundle exec" do G rubylib = ENV["RUBYLIB"] - rubylib = "#{rubylib}".split(File::PATH_SEPARATOR).unshift "#{bundler_path}" + rubylib = rubylib.to_s.split(File::PATH_SEPARATOR).unshift bundler_path rubylib = rubylib.uniq.join(File::PATH_SEPARATOR) bundle "exec 'echo $RUBYLIB'" diff --git a/spec/bundler/install/gemfile/sources_spec.rb b/spec/bundler/install/gemfile/sources_spec.rb index 7af97958c1..b2861e1c00 100644 --- a/spec/bundler/install/gemfile/sources_spec.rb +++ b/spec/bundler/install/gemfile/sources_spec.rb @@ -550,8 +550,8 @@ RSpec.describe "bundle install with gems on multiple sources" do context "and the gemfile changes" do it "is still able to find that gem from remote sources" do - source_uri = "#{file_uri_for(gem_repo1)}" - second_uri = "#{file_uri_for(gem_repo4)}" + source_uri = file_uri_for(gem_repo1) + second_uri = file_uri_for(gem_repo4) build_repo4 do build_gem "rack", "2.0.1.1.forked" |