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

Author: bastelfreak

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)

spec/integration/application/agent_spec.rb

@@ -649,31 +649,28 @@ def with_another_agent_running(&block)
        )).to_stdout
        .and output(/No more routes to fileserver/).to_stderr
     end
-
     it 'preserves the old cached catalog if validation fails with the old one' do
       catalog_handler = -> (req, res) {
-        catalog = compile_to_catalog(<<-MANIFEST, node)
-          exec { 'unqualified_command': }
-        MANIFEST
-
-        res.body = formatter.render(catalog)
-        res['Content-Type'] = formatter.mime
-      }
-
-      server.start_server(mounts: {catalog: catalog_handler}) do |port|
-        Puppet[:serverport] = port
-        expect {
-          agent.command_line.args << '--test'
-          agent.run
-        }.to exit_with(1)
-         .and output(%r{Retrieving plugin}).to_stdout
-         .and output(%r{Validation of Exec\[unqualified_command\] failed: 'unqualified_command' is not qualified and no path was specified}).to_stderr
-      end
-
-      # cached catalog should not be updated
-      cached_catalog = "#{File.join(Puppet[:client_datadir], 'catalog', Puppet[:certname])}.json"
-      expect(File).to_not be_exist(cached_catalog)
-    end
+        catalog = compile_to_catalog("'unqualified_command': path => ''}", node)
+
+        res.body = formatter.render(catalog)
+        res['Content-Type'] = formatter.mime
+      }
+
+      server.start_server(mounts: {catalog: catalog_handler}) do |port|
+        Puppet[:serverport] = port
+        expect {
+          agent.command_line.args << '--test'
+          agent.run
+        }.to exit_with(1)
+         .and output(%r{Retrieving plugin}).to_stdout
+         .and output(%r{Validation of Exec\[unqualified_command\] failed: 'unqualified_command' is not qualified and no path was specified}).to_stderr
+      end
+
+      # cached catalog should not be updated
+      cached_catalog = "#{File.join(Puppet[:client_datadir], 'catalog', Puppet[:certname])}.json"
+      expect(File).to_not be_exist(cached_catalog)
+    end
   end
 
   context "reporting" do

spec/unit/provider/exec/posix_spec.rb

@@ -15,13 +15,6 @@ def make_exe
   let(:provider) { described_class.new(resource) }
 
   describe "#validatecmd" do
-    it "should fail if no path is specified and the command is not fully qualified" do
-      expect { provider.validatecmd("foo") }.to raise_error(
-        Puppet::Error,
-        "'foo' is not qualified and no path was specified. Please qualify the command or specify a path."
-      )
-    end
-
     it "should pass if a path is given" do
       provider.resource[:path] = ['/bogus/bin']
       provider.validatecmd("../foo")

spec/unit/provider/exec/windows_spec.rb

@@ -88,8 +88,8 @@
   end
 
   describe "#validatecmd" do
-    it "should fail if the command isn't absolute and there is no path" do
-      expect { provider.validatecmd('foo') }.to raise_error(Puppet::Error, /'foo' is not qualified and no path was specified/)
+    it "should not fail if the command isn't absolute and there is no path" do
+      expect(provider.validatecmd('foo')).to eq(nil)
     end
 
     it "should not fail if the command is absolute and there is no path" do

spec/unit/provider/exec_spec.rb

@@ -73,20 +73,6 @@ def echo_from_ruby_exit_1(message)
     end
 
     context "when validating the command" do
-      it "redacts the arguments if the command is relative" do
-        expect {
-          apply_compiled_manifest(<<-MANIFEST)
-            exec { 'echo':
-              command => Sensitive.new('echo #{supersecret}')
-            }
-          MANIFEST
-        }.to raise_error do |err|
-          expect(err).to be_a(Puppet::Error)
-          expect(err.message).to match(/'echo' is not qualified and no path was specified. Please qualify the command or specify a path./)
-          expect(err.message).to_not match(/#{supersecret}/)
-        end
-      end
-
       it "redacts the arguments if the command is a directory" do
         dir = tmpdir('exec')
         apply_compiled_manifest(<<-MANIFEST)

spec/unit/type/exec_spec.rb

@@ -910,11 +910,6 @@ def instance(path)
     let :abs  do make_absolute('/bin/echo') end
     let :path do make_absolute('/bin') end
 
-    it "should fail with relative command and no path" do
-      expect { type.new(:command => rel) }.
-        to raise_error Puppet::Error, /no path was specified/
-    end
-
     it "should accept a relative command with a path" do
       expect(type.new(:command => rel, :path => path)).to be
     end