Merge pull request #7351 from josh-barker/bug/why-run-windows-package-local-file

Add whyrun message when installing a local file on Windows

1 files changed, 10 insertions, 3 deletions

diff --git a/spec/unit/provider/package/windows_spec.rb b/spec/unit/provider/package/windows_spec.rb
index 0f4fd465c8..aed0ca88be 100644
--- a/spec/unit/provider/package/windows_spec.rb
+++ b/spec/unit/provider/package/windows_spec.rb
@@ -398,13 +398,20 @@ describe Chef::Provider::Package::Windows, :windows_only do
     context "a missing local file is given" do
       let(:resource_source) { "C:/a_missing_file.exe" }
       let(:installer_type) { nil }
-
-      it "raises a Package error" do
+      before do
         allow(::File).to receive(:exist?).with(provider.new_resource.source).and_return(false)
-
         provider.load_current_resource
+      end
+
+      it "raises a Package error" do
         expect { provider.run_action(:install) }.to raise_error(Chef::Exceptions::Package)
       end
+
+      it "why_run mode doesn't raise an error" do
+        Chef::Config[:why_run] = true
+        expect { provider.run_action(:install) }.not_to raise_error
+        Chef::Config[:why_run] = false
+      end
     end
   end