Fixing provider commands and Util#execute so they always include the command output when possible, as mentioned on the list

git-svn-id: https://reductivelabs.com/svn/puppet/trunk@1635 980ebf18-57e1-0310-9a29-db15c13687c0

1 files changed, 39 insertions, 0 deletions

diff --git a/test/providers/provider.rb b/test/providers/provider.rb
index e540f29fe..28d92e3d6 100644
--- a/test/providers/provider.rb
+++ b/test/providers/provider.rb
@@ -142,6 +142,45 @@ class TestProvider < Test::Unit::TestCase
 
         assert(! provider.default?, "Was considered default")
     end
+
+    # Make sure that failed commands get their output in the error.
+    def test_outputonfailure
+        provider = newprovider
+
+        dir = tstdir()
+        file = File.join(dir, "mycmd")
+        sh = Puppet::Util.binary("sh")
+        File.open(file, "w") { |f|
+            f.puts %{#!#{sh}
+            echo A Failure >&2
+            exit 2
+            }
+        }
+        File.chmod(0755, file)
+
+        provider.commands :cmd => file
+
+        inst = provider.new(nil)
+
+        assert_raise(Puppet::ExecutionFailure) do
+            inst.cmd "some arguments"
+        end
+
+        out = nil
+        begin
+            inst.cmd "some arguments"
+        rescue Puppet::ExecutionFailure => detail
+            out = detail.to_s
+        end
+
+        assert(out =~ /A Failure/,
+               "Did not receive command output on failure")
+
+        assert(out =~ /Could not execute/,
+               "Did not receive info wrapper on failure")
+
+
+    end
 end
 
 # $Id$