Merge pull request #864 from vladshablinsky/skip-upgrade

Skip upgrade of pinned dependency if it's outdated

3 files changed, 79 insertions, 0 deletions

diff --git a/Library/Homebrew/test/test_formula.rb b/Library/Homebrew/test/test_formula.rb
index 7ebb17884..0e6650a95 100644
--- a/Library/Homebrew/test/test_formula.rb
+++ b/Library/Homebrew/test/test_formula.rb
@@ -491,6 +491,31 @@ class FormulaTests < Homebrew::TestCase
     f3.rack.rmtree
   end
 
+  def test_eligible_kegs_for_cleanup_keg_pinned
+    f1 = Class.new(Testball) { version "0.1" }.new
+    f2 = Class.new(Testball) { version "0.2" }.new
+    f3 = Class.new(Testball) { version "0.3" }.new
+
+    shutup do
+      f1.brew { f1.install }
+      f1.pin
+      f2.brew { f2.install }
+      f3.brew { f3.install }
+    end
+
+    assert_equal HOMEBREW_LIBRARY.join("PinnedKegs/#{f1.name}").resolved_path, f1.prefix
+
+    assert_predicate f1, :installed?
+    assert_predicate f2, :installed?
+    assert_predicate f3, :installed?
+
+    assert_equal [Keg.new(f2.prefix)], shutup { f3.eligible_kegs_for_cleanup }
+  ensure
+    f1.unpin
+    [f1, f2, f3].each(&:clear_cache)
+    f3.rack.rmtree
+  end
+
   def test_eligible_kegs_for_cleanup_head_installed
     f = formula do
       version "0.1"
diff --git a/Library/Homebrew/test/test_formula_installer.rb b/Library/Homebrew/test/test_formula_installer.rb
index 55b456fba..f519d0715 100644
--- a/Library/Homebrew/test/test_formula_installer.rb
+++ b/Library/Homebrew/test/test_formula_installer.rb
@@ -92,3 +92,45 @@ class InstallTests < Homebrew::TestCase
     end
   end
 end
+
+class FormulaInstallerTests < Homebrew::TestCase
+  def test_check_install_sanity_pinned_dep
+    dep_name = "dependency"
+    dep_path = CoreTap.new.formula_dir/"#{dep_name}.rb"
+    dep_path.write <<-EOS.undent
+      class #{Formulary.class_s(dep_name)} < Formula
+        url "foo"
+        version "0.2"
+      end
+    EOS
+
+    Formulary::FORMULAE.delete(dep_path)
+    dependency = Formulary.factory(dep_name)
+
+    dependent = formula do
+      url "foo"
+      version "0.5"
+      depends_on "#{dependency.name}"
+    end
+
+    dependency.prefix("0.1").join("bin/a").mkpath
+    HOMEBREW_LIBRARY.join("PinnedKegs").mkpath
+    FileUtils.ln_s dependency.prefix("0.1"), HOMEBREW_LIBRARY.join("PinnedKegs/#{dep_name}")
+
+    dependency_keg = Keg.new(dependency.prefix("0.1"))
+    dependency_keg.link
+
+    assert_predicate dependency_keg, :linked?
+    assert_predicate dependency, :pinned?
+
+    fi = FormulaInstaller.new(dependent)
+    assert_raises(CannotInstallFormulaError) { fi.check_install_sanity }
+  ensure
+    dependency.unpin
+    dependency_keg.unlink
+    dependency_keg.uninstall
+    dependency.clear_cache
+    dep_path.unlink
+    Formulary::FORMULAE.delete(dep_path)
+  end
+end
diff --git a/Library/Homebrew/test/test_integration_cmds.rb b/Library/Homebrew/test/test_integration_cmds.rb
index 9ea305344..ead87c8ab 100644
--- a/Library/Homebrew/test/test_integration_cmds.rb
+++ b/Library/Homebrew/test/test_integration_cmds.rb
@@ -529,6 +529,18 @@ class IntegrationCommandTests < Homebrew::TestCase
     assert foo_dir.exist?
   end
 
+  def test_reinstall_pinned
+    setup_test_formula "testball"
+
+    HOMEBREW_CELLAR.join("testball/0.1").mkpath
+    HOMEBREW_LIBRARY.join("PinnedKegs").mkpath
+    FileUtils.ln_s HOMEBREW_CELLAR.join("testball/0.1"), HOMEBREW_LIBRARY.join("PinnedKegs/testball")
+
+    assert_match "testball is pinned. You must unpin it to reinstall.", cmd("reinstall", "testball")
+
+    HOMEBREW_LIBRARY.join("PinnedKegs").rmtree
+  end
+
   def test_home
     setup_test_formula "testball"