From a2d7c97a9187f3d88230e273756ed4836fa8ac19 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?David=20Rodr=C3=ADguez?= <deivid.rodriguez@riseup.net>
Date: Thu, 15 Aug 2019 18:58:07 +0200
Subject: [PATCH] [bundler/bundler] Extract yet another helper method

https://github.com/bundler/bundler/commit/948a863bd8
---
 spec/bundler/quality_spec.rb    | 10 +---------
 spec/bundler/support/helpers.rb | 29 ++++++++++++++++++-----------
 2 files changed, 19 insertions(+), 20 deletions(-)

diff --git a/spec/bundler/quality_spec.rb b/spec/bundler/quality_spec.rb
index 5f00d4b6a0..40c0b0366e 100644
--- a/spec/bundler/quality_spec.rb
+++ b/spec/bundler/quality_spec.rb
@@ -225,16 +225,8 @@ RSpec.describe "The library itself" do
   end
 
   it "can still be built" do
-    with_root_gemspec do |gemspec|
-      Dir.chdir(root) { gem_command! :build, gemspec }
-    end
-
-    bundler_path = root.join("bundler-#{Bundler::VERSION}.gem")
-
-    begin
+    with_built_bundler do |_gem_path|
       expect(err).to be_empty, "bundler should build as a gem without warnings, but\n#{err}"
-    ensure
-      bundler_path.rmtree
     end
   end
 
diff --git a/spec/bundler/support/helpers.rb b/spec/bundler/support/helpers.rb
index 6f982b4f4c..9cd468dfa1 100644
--- a/spec/bundler/support/helpers.rb
+++ b/spec/bundler/support/helpers.rb
@@ -297,20 +297,13 @@ module Spec
       options = gems.last.is_a?(Hash) ? gems.pop : {}
       gem_repo = options.fetch(:gem_repo) { gem_repo1 }
       gems.each do |g|
-        path = if g == :bundler
-          with_root_gemspec do |gemspec|
-            Dir.chdir(root) { gem_command! :build, gemspec }
-          end
-          bundler_path = root.join("bundler-#{Bundler::VERSION}.gem")
+        if g == :bundler
+          with_built_bundler {|gem_path| install_gem(gem_path) }
         elsif g.to_s =~ %r{\A(?:[A-Z]:)?/.*\.gem\z}
-          g
+          install_gem(g)
         else
-          "#{gem_repo}/gems/#{g}.gem"
+          install_gem("#{gem_repo}/gems/#{g}.gem")
         end
-
-        install_gem(path)
-
-        bundler_path && bundler_path.rmtree
       end
     end
 
@@ -320,6 +313,20 @@ module Spec
       gem_command! :install, "--no-document --ignore-dependencies '#{path}'"
     end
 
+    def with_built_bundler
+      with_root_gemspec do |gemspec|
+        Dir.chdir(root) { gem_command! :build, gemspec.to_s }
+      end
+
+      bundler_path = root + "bundler-#{Bundler::VERSION}.gem"
+
+      begin
+        yield(bundler_path)
+      ensure
+        bundler_path.rmtree
+      end
+    end
+
     def with_gem_path_as(path)
       backup = ENV.to_hash
       ENV["GEM_HOME"] = path.to_s