summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPieter van de Bruggen <pieter@puppetlabs.com>2011-05-18 15:16:26 -0700
committerPieter van de Bruggen <pieter@puppetlabs.com>2011-05-18 15:16:26 -0700
commitcbc123c2641f1c51a71710ea5dac5f6114063d34 (patch)
tree74363cc166b49590547be5683ccbd77373dd3294
parent1f438da968e1583d900903407c2e5b17648fa937 (diff)
parent25eab1a5193762688cfa16d7bb750d9f962503c9 (diff)
downloadpuppet-cbc123c2641f1c51a71710ea5dac5f6114063d34.tar.gz
puppet-cbc123c2641f1c51a71710ea5dac5f6114063d34.tar.xz
puppet-cbc123c2641f1c51a71710ea5dac5f6114063d34.zip
Merge branch '2.6.next' into 2.6.x
-rw-r--r--lib/puppet/indirector/exec.rb6
-rwxr-xr-xspec/unit/indirector/exec_spec.rb8
-rwxr-xr-xspec/unit/indirector/node/exec_spec.rb2
3 files changed, 8 insertions, 8 deletions
diff --git a/lib/puppet/indirector/exec.rb b/lib/puppet/indirector/exec.rb
index 4683eda0f..8ea13ff95 100644
--- a/lib/puppet/indirector/exec.rb
+++ b/lib/puppet/indirector/exec.rb
@@ -16,8 +16,8 @@ class Puppet::Indirector::Exec < Puppet::Indirector::Terminus
private
# Proxy the execution, so it's easier to test.
- def execute(command)
- Puppet::Util.execute(command)
+ def execute(command, arguments)
+ Puppet::Util.execute(command,arguments)
end
# Call the external command and see if it returns our output.
@@ -33,7 +33,7 @@ class Puppet::Indirector::Exec < Puppet::Indirector::Terminus
# Add our name to it.
external_command << name
begin
- output = execute(external_command)
+ output = execute(external_command, :combine => false)
rescue Puppet::ExecutionFailure => detail
raise Puppet::Error, "Failed to find #{name} via exec: #{detail}"
end
diff --git a/spec/unit/indirector/exec_spec.rb b/spec/unit/indirector/exec_spec.rb
index de37f2775..89d91a3ee 100755
--- a/spec/unit/indirector/exec_spec.rb
+++ b/spec/unit/indirector/exec_spec.rb
@@ -33,22 +33,22 @@ describe Puppet::Indirector::Exec do
end
it "should execute the command with the object name as the only argument" do
- @searcher.expects(:execute).with(%w{/echo foo})
+ @searcher.expects(:execute).with(%w{/echo foo}, :combine => false)
@searcher.find(@request)
end
it "should return the output of the script" do
- @searcher.expects(:execute).with(%w{/echo foo}).returns("whatever")
+ @searcher.expects(:execute).with(%w{/echo foo}, :combine => false).returns("whatever")
@searcher.find(@request).should == "whatever"
end
it "should return nil when the command produces no output" do
- @searcher.expects(:execute).with(%w{/echo foo}).returns(nil)
+ @searcher.expects(:execute).with(%w{/echo foo}, :combine => false).returns(nil)
@searcher.find(@request).should be_nil
end
it "should raise an exception if there's an execution failure" do
- @searcher.expects(:execute).with(%w{/echo foo}).raises(Puppet::ExecutionFailure.new("message"))
+ @searcher.expects(:execute).with(%w{/echo foo}, :combine => false).raises(Puppet::ExecutionFailure.new("message"))
lambda {@searcher.find(@request)}.should raise_exception(Puppet::Error, 'Failed to find foo via exec: message')
end
diff --git a/spec/unit/indirector/node/exec_spec.rb b/spec/unit/indirector/node/exec_spec.rb
index d214a5ef6..ebf28bd9a 100755
--- a/spec/unit/indirector/node/exec_spec.rb
+++ b/spec/unit/indirector/node/exec_spec.rb
@@ -32,7 +32,7 @@ describe Puppet::Node::Exec do
@result = {}
# Use a local variable so the reference is usable in the execute definition.
result = @result
- @searcher.meta_def(:execute) do |command|
+ @searcher.meta_def(:execute) do |command, arguments|
return YAML.dump(result)
end