Merge pull request #9393 from joshcooper/uniquefile_9385

Check for nil before closing Uniquefile

Author: mhashizume

Diff for: lib/puppet/util/execution.rb

@@ -323,7 +323,7 @@ def self.execute(command, options = NoOptionsSpecified)
       unless options[:squelch]
         # if we opened a pipe, we need to clean it up.
         reader.close if reader
-        stdout.close! if Puppet::Util::Platform.windows?
+        stdout.close! if stdout && Puppet::Util::Platform.windows?
       end
     end
 

Diff for: spec/unit/util/execution_spec.rb

@@ -834,6 +834,15 @@ def expect_cwd_to_be(cwd)
 
           Puppet::Util::Execution.execute('test command')
         end
+
+        it "should raise if it fails to create a Uniquefile for stdout" do
+          allow(Puppet::FileSystem::Uniquefile).to receive(:new)
+            .and_raise(Errno::ENOENT, 'C:\Users\ADMINI~1\AppData\Local\Temp\doesnotexist')
+
+          expect {
+            Puppet::Util::Execution.execute('test command')
+          }.to raise_error(Errno::ENOENT, 'No such file or directory - C:\Users\ADMINI~1\AppData\Local\Temp\doesnotexist')
+        end
       end
 
       it "should raise an error if failonfail is true and the child failed" do