(#9440) exec resource: set path param to path fact

puppetlabs · Aug 7, 2024 · 8a1f94c · 8a1f94c
1 parent fc698a2
commit 8a1f94c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/lib/puppet/type/exec.rb b/lib/puppet/type/exec.rb
@@ -223,7 +223,9 @@ def sync
  newparam(:path) do
  desc "The search path used for command execution.
  Commands must be fully qualified if no path is specified. Paths
- can be specified as an array or as a '#{File::PATH_SEPARATOR}' separated list."
+ can be specified as an array or as a '#{File::PATH_SEPARATOR}' separated list. Defaults to the `path` fact."
+
+ defaultto Puppet.runtime[:facter].value('path')
 
  # Support both arrays and colon-separated fields.
  def value=(*values)

diff --git a/spec/unit/type/exec_spec.rb b/spec/unit/type/exec_spec.rb
@@ -910,6 +910,8 @@ def instance(path)
  let :abs do make_absolute('/bin/echo') end
  let :path do make_absolute('/bin') end
 
+ allow(Facter).to receive(:value).with('path').and_return('/usr/bin:/bin')
+
  it "should fail with relative command and no path" do
  expect { type.new(:command => rel) }.
  to raise_error Puppet::Error, /no path was specified/