Merge pull request #2215 from opscode/tball/homebrew_fix

Fixing bug where  tried to use the homebrew provider on OSX and didn't correctly check for lack of homebrew-specific attribute on the resource

1 files changed, 9 insertions, 0 deletions

diff --git a/spec/unit/provider/package/homebrew_spec.rb b/spec/unit/provider/package/homebrew_spec.rb
index d38458546d..dccd8edf11 100644
--- a/spec/unit/provider/package/homebrew_spec.rb
+++ b/spec/unit/provider/package/homebrew_spec.rb
@@ -158,6 +158,15 @@ describe Chef::Provider::Package::Homebrew do
       allow(provider).to receive(:shell_out!).and_return(OpenStruct.new(:stdout => 'homestarrunner'))
       expect(provider.brew('info', 'opts', 'bananas')).to eql('homestarrunner')
     end
+
+    context "when new_resource is Package" do
+      let(:new_resource) { Chef::Resource::Package.new('emacs') }
+
+      it "does not try to read homebrew_user from Package, which does not have it" do
+        allow(provider).to receive(:shell_out!).and_return(OpenStruct.new(:stdout => 'zombo'))
+        expect(provider.brew).to eql('zombo')
+      end
+    end
   end
 
   context 'when testing actions' do