tests: remove with_git_env method

A common git environment is now used in all tests, so this is no longer
required.

Library/Homebrew/dev-cmd/tests.rb

@@ -34,6 +34,7 @@ module Homebrew
       %w[AUTHOR COMMITTER].each do |role|
         ENV["GIT_#{role}_NAME"] = "brew tests"
         ENV["GIT_#{role}_EMAIL"] = "brew-tests@localhost"
+        ENV["GIT_#{role}_DATE"]  = "Sun Jan 22 19:59:13 2017 +0000"
       end
 
       Homebrew.install_gem_setup_path! "bundler"

Library/Homebrew/test/download_strategies_test.rb

@@ -158,15 +158,13 @@ class GitDownloadStrategyTests < Homebrew::TestCase
   end
 
   def setup_git_repo
-    using_git_env do
+    @cached_location.cd do
-      @cached_location.cd do
+      shutup do
-        shutup do
+        system "git", "init"
-          system "git", "init"
+        system "git", "remote", "add", "origin", "https://github.com/Homebrew/homebrew-foo"
-          system "git", "remote", "add", "origin", "https://github.com/Homebrew/homebrew-foo"
-        end
-        touch "README"
-        git_commit_all
       end
+      touch "README"
+      git_commit_all
     end
   end
 
@@ -185,11 +183,9 @@ class GitDownloadStrategyTests < Homebrew::TestCase
 
   def test_last_commit
     setup_git_repo
-    using_git_env do
+    @cached_location.cd do
-      @cached_location.cd do
+      touch "LICENSE"
-        touch "LICENSE"
+      git_commit_all
-        git_commit_all
-      end
     end
     assert_equal "f68266e", @strategy.last_commit
   end
@@ -202,17 +198,15 @@ class GitDownloadStrategyTests < Homebrew::TestCase
     resource.instance_variable_set(:@version, Version.create("HEAD"))
     @strategy = GitDownloadStrategy.new("baz", resource)
 
-    using_git_env do
+    remote_repo.cd do
-      remote_repo.cd do
+      shutup do
-        shutup do
+        system "git", "init"
-          system "git", "init"
+        system "git", "remote", "add", "origin", "https://github.com/Homebrew/homebrew-foo"
-          system "git", "remote", "add", "origin", "https://github.com/Homebrew/homebrew-foo"
-        end
-        touch "README"
-        git_commit_all
-        touch "LICENSE"
-        git_commit_all
       end
+      touch "README"
+      git_commit_all
+      touch "LICENSE"
+      git_commit_all
     end
 
     @strategy.shutup!

Library/Homebrew/test/formula_test.rb

@@ -533,14 +533,12 @@ class FormulaTests < Homebrew::TestCase
     cached_location = f.head.downloader.cached_location
     cached_location.mkpath
 
-    using_git_env do
+    cached_location.cd do
-      cached_location.cd do
+      FileUtils.touch "LICENSE"
-        FileUtils.touch "LICENSE"
+      shutup do
-        shutup do
+        system "git", "init"
-          system "git", "init"
+        system "git", "add", "--all"
-          system "git", "add", "--all"
+        system "git", "commit", "-m", "Initial commit"
-          system "git", "commit", "-m", "Initial commit"
-        end
       end
     end
 
@@ -1105,14 +1103,12 @@ class OutdatedVersionsTests < Homebrew::TestCase
       head "file://#{testball_repo}", using: :git
     end
 
-    using_git_env do
+    testball_repo.cd do
-      testball_repo.cd do
+      FileUtils.touch "LICENSE"
-        FileUtils.touch "LICENSE"
+      shutup do
-        shutup do
+        system "git", "init"
-          system "git", "init"
+        system "git", "add", "--all"
-          system "git", "add", "--all"
+        system "git", "commit", "-m", "Initial commit"
-          system "git", "commit", "-m", "Initial commit"
-        end
       end
     end
 

Library/Homebrew/test/install_test.rb

@@ -76,16 +76,14 @@ class IntegrationCommandTestInstall < IntegrationCommandTestCase
     repo_path = HOMEBREW_CACHE.join("repo")
     repo_path.join("bin").mkpath
 
-    using_git_env do
+    repo_path.cd do
-      repo_path.cd do
+      shutup do
-        shutup do
+        system "git", "init"
-          system "git", "init"
+        system "git", "remote", "add", "origin", "https://github.com/Homebrew/homebrew-foo"
-          system "git", "remote", "add", "origin", "https://github.com/Homebrew/homebrew-foo"
+        FileUtils.touch "bin/something.bin"
-          FileUtils.touch "bin/something.bin"
+        FileUtils.touch "README"
-          FileUtils.touch "README"
+        system "git", "add", "--all"
-          system "git", "add", "--all"
+        system "git", "commit", "-m", "Initial repo commit"
-          system "git", "commit", "-m", "Initial repo commit"
-        end
       end
     end
 

Library/Homebrew/test/support/helper/env.rb

@@ -16,18 +16,6 @@ module Test
       ensure
         restore_env old
       end
-
-      def using_git_env
-        git_env = ["AUTHOR", "COMMITTER"].each_with_object({}) do |role, env|
-          env["GIT_#{role}_NAME"]  = "brew tests"
-          env["GIT_#{role}_EMAIL"] = "brew-tests@localhost"
-          env["GIT_#{role}_DATE"]  = "Sun Jan 22 19:59:13 2017 +0000"
-        end
-
-        with_environment(git_env) do
-          yield
-        end
-      end
     end
   end
 end

Library/Homebrew/test/tap_test.rb

@@ -66,14 +66,12 @@ class TapTest < Homebrew::TestCase
   end
 
   def setup_git_repo
-    using_git_env do
+    @path.cd do
-      @path.cd do
+      shutup do
-        shutup do
+        system "git", "init"
-          system "git", "init"
+        system "git", "remote", "add", "origin", "https://github.com/Homebrew/homebrew-foo"
-          system "git", "remote", "add", "origin", "https://github.com/Homebrew/homebrew-foo"
+        system "git", "add", "--all"
-          system "git", "add", "--all"
+        system "git", "commit", "-m", "init"
-          system "git", "commit", "-m", "init"
-        end
       end
     end
   end