diff options
author | Jay Mundrawala <jdmundrawala@gmail.com> | 2015-09-14 07:54:57 -0700 |
---|---|---|
committer | Jay Mundrawala <jdmundrawala@gmail.com> | 2015-09-14 08:28:45 -0700 |
commit | e24de71409a861f9bc6318755debb2c18d27f2a3 (patch) | |
tree | 6a5ed0af063131e142d1f5f025a8ba2dc57fc11a | |
parent | 5d4dd15ced791b49ec571cfd21ad3ae2afe6b132 (diff) | |
download | chef-e24de71409a861f9bc6318755debb2c18d27f2a3.tar.gz |
Fix failing specs on Windows
The verify tests were not correctly mocking things
out.
-rw-r--r-- | spec/support/shared/unit/provider/file.rb | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/spec/support/shared/unit/provider/file.rb b/spec/support/shared/unit/provider/file.rb index 7de9698451..f552fe0a2a 100644 --- a/spec/support/shared/unit/provider/file.rb +++ b/spec/support/shared/unit/provider/file.rb @@ -466,10 +466,12 @@ shared_examples_for Chef::Provider::File do } let(:verification) { double("Verification") } + let(:verification_fail) { double("Verification Fail") } context "with user-supplied verifications" do it "calls #verify on each verification with tempfile path" do - allow(Chef::Resource::File::Verification).to receive(:new).and_return(verification) + allow(Chef::Resource::File::Verification).to( + receive(:new).with(anything(), "true", anything()).and_return(verification)) provider.new_resource.verify "true" provider.new_resource.verify "true" expect(verification).to receive(:verify).with(tempfile.path).twice.and_return(true) @@ -477,10 +479,14 @@ shared_examples_for Chef::Provider::File do end it "raises an exception if any verification fails" do + allow(Chef::Resource::File::Verification).to( + receive(:new).with(anything(), "true", anything()).and_return(verification)) + allow(Chef::Resource::File::Verification).to( + receive(:new).with(anything(), "false", anything()).and_return(verification_fail)) provider.new_resource.verify "true" provider.new_resource.verify "false" - allow(verification).to receive(:verify).with("true").and_return(true) - allow(verification).to receive(:verify).with("false").and_return(false) + expect(verification).to receive(:verify).with(tempfile.path).and_return(true) + expect(verification_fail).to receive(:verify).with(tempfile.path).and_return(false) expect{provider.send(:do_validate_content)}.to raise_error(Chef::Exceptions::ValidationFailed) end end |