Merge pull request #2227 from reitermarkus/spec-formula_installer_bottle

Convert `formula_installer_bottle` test to spec.

Library/Homebrew/test/formula_installer_bottle_test.rb → Library/Homebrew/test/formula_installer_bottle_spec.rb

-require "testing_env"
 require "formula"
 require "formula_installer"
 require "keg"
@@ -6,22 +5,28 @@ require "tab"
 require "test/support/fixtures/testball"
 require "test/support/fixtures/testball_bottle"
 
-class InstallBottleTests < Homebrew::TestCase
-  def temporary_bottle_install(formula)
-    refute_predicate formula, :installed?
-    assert_predicate formula, :bottled?
-    assert_predicate formula, :pour_bottle?
+RSpec::Matchers.alias_matcher :pour_bottle, :be_pour_bottle
 
-    installer = FormulaInstaller.new(formula)
+describe FormulaInstaller do
+  matcher :be_poured_from_bottle do
+    match(&:poured_from_bottle)
+  end
+
+  def temporarily_install_bottle(formula)
+    expect(formula).not_to be_installed
+    expect(formula).to be_bottled
+    expect(formula).to pour_bottle
 
-    shutup { installer.install }
+    shutup do
+      described_class.new(formula).install
+    end
 
     keg = Keg.new(formula.prefix)
 
-    assert_predicate formula, :installed?
+    expect(formula).to be_installed
 
     begin
-      assert_predicate Tab.for_keg(keg), :poured_from_bottle
+      expect(Tab.for_keg(keg)).to be_poured_from_bottle
 
       yield formula
     ensure
@@ -31,48 +36,46 @@ class InstallBottleTests < Homebrew::TestCase
       formula.bottle.clear_cache
     end
 
-    refute_predicate keg, :exist?
-    refute_predicate formula, :installed?
+    expect(keg).not_to exist
+    expect(formula).not_to be_installed
   end
 
-  def test_a_basic_bottle_install
-    DevelopmentTools.stubs(:installed?).returns(false)
+  specify "basic bottle install" do
+    allow(DevelopmentTools).to receive(:installed?).and_return(false)
 
-    temporary_bottle_install(TestballBottle.new) do |f|
-      # Copied directly from test_formula_installer.rb as we expect
-      # the same behavior
+    temporarily_install_bottle(TestballBottle.new) do |f|
+      # Copied directly from formula_installer_spec.rb
+      # as we expect the same behavior.
 
       # Test that things made it into the Keg
-      assert_predicate f.bin, :directory?
+      expect(f.bin).to be_a_directory
 
-      assert_predicate f.libexec, :directory?
+      expect(f.libexec).to be_a_directory
 
-      refute_predicate f.prefix+"main.c", :exist?
+      expect(f.prefix/"main.c").not_to exist
 
-      # Test that things make it into the Cellar
+      # Test that things made it into the Cellar
       keg = Keg.new f.prefix
       keg.link
 
-      bin = HOMEBREW_PREFIX+"bin"
-      assert_predicate bin, :directory?
+      bin = HOMEBREW_PREFIX/"bin"
+      expect(bin).to be_a_directory
     end
   end
 
-  def test_build_tools_error
-    DevelopmentTools.stubs(:installed?).returns(false)
+  specify "build tools error" do
+    allow(DevelopmentTools).to receive(:installed?).and_return(false)
 
     # Testball doesn't have a bottle block, so use it to test this behavior
     formula = Testball.new
 
-    refute_predicate formula, :installed?
-    refute_predicate formula, :bottled?
+    expect(formula).not_to be_installed
+    expect(formula).not_to be_bottled
 
-    installer = FormulaInstaller.new(formula)
-
-    assert_raises(BuildToolsError) do
-      installer.install
-    end
+    expect {
+      FormulaInstaller.new(formula).install
+    }.to raise_error(BuildToolsError)
 
-    refute_predicate formula, :installed?
+    expect(formula).not_to be_installed
   end
 end