puppet 5.0.0 → 6.0.0

data/spec/unit/provider/service/openrc_spec.rb

@@ -2,20 +2,21 @@
 
 require 'spec_helper'
 
-describe Puppet::Type.type(:service).provider(:openrc) do
+describe 'Puppet::Type::Service::Provider::Openrc', unless: Puppet::Util::Platform.jruby? do
+  let(:provider_class) { Puppet::Type.type(:service).provider(:openrc) }
 
-  if Puppet.features.microsoft_windows?
+  if Puppet::Util::Platform.windows?
     # Get a pid for $CHILD_STATUS to latch on to
-    command = "cmd.exe /c \"exit 0\""
-    Puppet::Util::Execution.execute(command, {:failonfail => false})
+    cmd = "cmd.exe /c \"exit 0\""
+    Puppet::Util::Execution.execute(cmd, {:failonfail => false})
   end
 
   before :each do
-    Puppet::Type.type(:service).stubs(:defaultprovider).returns described_class
+    Puppet::Type.type(:service).stubs(:defaultprovider).returns provider_class
     ['/sbin/rc-service', '/bin/rc-status', '/sbin/rc-update'].each do |command|
       # Puppet::Util is both mixed in to providers and is also invoked directly
       # by Puppet::Provider::CommandDefiner, so we have to stub both out.
-      described_class.stubs(:which).with(command).returns(command)
+      provider_class.stubs(:which).with(command).returns(command)
       Puppet::Util.stubs(:which).with(command).returns(command)
     end
   end
@@ -23,12 +24,12 @@ describe Puppet::Type.type(:service).provider(:openrc) do
   describe ".instances" do
 
     it "should have an instances method" do
-      expect(described_class).to respond_to :instances
+      expect(provider_class).to respond_to :instances
     end
 
     it "should get a list of services from rc-service --list" do
-      described_class.expects(:rcservice).with('-C','--list').returns File.read(my_fixture('rcservice_list'))
-      expect(described_class.instances.map(&:name)).to eq([
+      provider_class.expects(:rcservice).with('-C','--list').returns File.read(my_fixture('rcservice_list'))
+      expect(provider_class.instances.map(&:name)).to eq([
         'alsasound',
         'consolefont',
         'lvm-monitoring',
@@ -43,12 +44,12 @@ describe Puppet::Type.type(:service).provider(:openrc) do
 
   describe "#start" do
     it "should use the supplied start command if specified" do
-      provider = described_class.new(Puppet::Type.type(:service).new(:name => 'sshd', :start => '/bin/foo'))
+      provider = provider_class.new(Puppet::Type.type(:service).new(:name => 'sshd', :start => '/bin/foo'))
       provider.expects(:execute).with(['/bin/foo'], :failonfail => true, :override_locale => false, :squelch => false, :combine => true)
       provider.start
     end
     it "should start the service with rc-service start otherwise" do
-      provider = described_class.new(Puppet::Type.type(:service).new(:name => 'sshd'))
+      provider = provider_class.new(Puppet::Type.type(:service).new(:name => 'sshd'))
       provider.expects(:execute).with(['/sbin/rc-service','sshd',:start], :failonfail => true, :override_locale => false, :squelch => false, :combine => true)
       provider.start
     end
@@ -56,25 +57,25 @@ describe Puppet::Type.type(:service).provider(:openrc) do
 
   describe "#stop" do
     it "should use the supplied stop command if specified" do
-      provider = described_class.new(Puppet::Type.type(:service).new(:name => 'sshd', :stop => '/bin/foo'))
+      provider = provider_class.new(Puppet::Type.type(:service).new(:name => 'sshd', :stop => '/bin/foo'))
       provider.expects(:execute).with(['/bin/foo'], :failonfail => true, :override_locale => false, :squelch => false, :combine => true)
       provider.stop
     end
     it "should stop the service with rc-service stop otherwise" do
-      provider = described_class.new(Puppet::Type.type(:service).new(:name => 'sshd'))
+      provider = provider_class.new(Puppet::Type.type(:service).new(:name => 'sshd'))
       provider.expects(:execute).with(['/sbin/rc-service','sshd',:stop], :failonfail => true, :override_locale => false, :squelch => false, :combine => true)
       provider.stop
     end
   end
 
   describe 'when invoking `rc-status`' do
-    subject { described_class.new(Puppet::Type.type(:service).new(:name => 'urandom')) }
+    subject { provider_class.new(Puppet::Type.type(:service).new(:name => 'urandom')) }
     it "clears the RC_SVCNAME environment variable" do
       Puppet::Util.withenv(:RC_SVCNAME => 'puppet') do
         Puppet::Util::Execution.expects(:execute).with(
           includes('/bin/rc-status'),
           has_entry(:custom_environment, {:RC_SVCNAME => nil})
-        ).returns ''
+        ).returns(Puppet::Util::Execution::ProcessOutput.new('', 0))
         subject.enabled?
       end
     end
@@ -83,46 +84,46 @@ describe Puppet::Type.type(:service).provider(:openrc) do
   describe "#enabled?" do
 
     before :each do
-      described_class.any_instance.stubs(:rcstatus).with('-C','-a').returns File.read(my_fixture('rcstatus'))
+      provider_class.any_instance.stubs(:rcstatus).with('-C','-a').returns File.read(my_fixture('rcstatus'))
     end
 
     it "should run rc-status to get a list of enabled services" do
-      provider = described_class.new(Puppet::Type.type(:service).new(:name => 'sshd'))
+      provider = provider_class.new(Puppet::Type.type(:service).new(:name => 'sshd'))
       provider.expects(:rcstatus).with('-C','-a').returns "\n"
       provider.enabled?
     end
 
     ['hwclock', 'modules', 'urandom'].each do |service|
       it "should consider service #{service} in runlevel boot as enabled" do
-        provider = described_class.new(Puppet::Type.type(:service).new(:name => service))
+        provider = provider_class.new(Puppet::Type.type(:service).new(:name => service))
         expect(provider.enabled?).to eq(:true)
       end
     end
 
     ['netmount', 'xdm', 'local', 'foo_with_very_very_long_servicename_no_still_not_the_end_wait_for_it_almost_there_almost_there_now_finally_the_end'].each do |service|
       it "should consider service #{service} in runlevel default as enabled" do
-        provider = described_class.new(Puppet::Type.type(:service).new(:name => service))
+        provider = provider_class.new(Puppet::Type.type(:service).new(:name => service))
         expect(provider.enabled?).to eq(:true)
       end
     end
 
     ['net.eth0', 'pcscd'].each do |service|
       it "should consider service #{service} in dynamic runlevel: hotplugged as disabled" do
-        provider = described_class.new(Puppet::Type.type(:service).new(:name => service))
+        provider = provider_class.new(Puppet::Type.type(:service).new(:name => service))
         expect(provider.enabled?).to eq(:false)
       end
     end
 
     ['sysfs', 'udev-mount'].each do |service|
       it "should consider service #{service} in dynamic runlevel: needed as disabled" do
-        provider = described_class.new(Puppet::Type.type(:service).new(:name => service))
+        provider = provider_class.new(Puppet::Type.type(:service).new(:name => service))
         expect(provider.enabled?).to eq(:false)
       end
     end
 
     ['sshd'].each do |service|
       it "should consider service #{service} in dynamic runlevel: manual as disabled" do
-        provider = described_class.new(Puppet::Type.type(:service).new(:name => service))
+        provider = provider_class.new(Puppet::Type.type(:service).new(:name => service))
         expect(provider.enabled?).to eq(:false)
       end
     end
@@ -131,7 +132,7 @@ describe Puppet::Type.type(:service).provider(:openrc) do
 
   describe "#enable" do
     it "should run rc-update add to enable a service" do
-      provider = described_class.new(Puppet::Type.type(:service).new(:name => 'sshd'))
+      provider = provider_class.new(Puppet::Type.type(:service).new(:name => 'sshd'))
       provider.expects(:rcupdate).with('-C', :add, 'sshd')
       provider.enable
     end
@@ -139,7 +140,7 @@ describe Puppet::Type.type(:service).provider(:openrc) do
 
   describe "#disable" do
     it "should run rc-update del to disable a service" do
-      provider = described_class.new(Puppet::Type.type(:service).new(:name => 'sshd'))
+      provider = provider_class.new(Puppet::Type.type(:service).new(:name => 'sshd'))
       provider.expects(:rcupdate).with('-C', :del, 'sshd')
       provider.disable
     end
@@ -149,7 +150,7 @@ describe Puppet::Type.type(:service).provider(:openrc) do
 
     describe "when a special status command if specified" do
       it "should use the status command from the resource" do
-        provider = described_class.new(Puppet::Type.type(:service).new(:name => 'sshd', :status => '/bin/foo'))
+        provider = provider_class.new(Puppet::Type.type(:service).new(:name => 'sshd', :status => '/bin/foo'))
         provider.expects(:execute).with(['/sbin/rc-service','sshd',:status], :failonfail => false, :override_locale => false, :squelch => false, :combine => true).never
         provider.expects(:execute).with(['/bin/foo'], :failonfail => false, :override_locale => false, :squelch => false, :combine => true)
         $CHILD_STATUS.stubs(:exitstatus).returns 0
@@ -157,7 +158,7 @@ describe Puppet::Type.type(:service).provider(:openrc) do
       end
 
       it "should return :stopped when status command returns with a non-zero exitcode" do
-        provider = described_class.new(Puppet::Type.type(:service).new(:name => 'sshd', :status => '/bin/foo'))
+        provider = provider_class.new(Puppet::Type.type(:service).new(:name => 'sshd', :status => '/bin/foo'))
         provider.expects(:execute).with(['/sbin/rc-service','sshd',:status], :failonfail => false, :override_locale => false, :squelch => false, :combine => true).never
         provider.expects(:execute).with(['/bin/foo'], :failonfail => false, :override_locale => false, :squelch => false, :combine => true)
         $CHILD_STATUS.stubs(:exitstatus).returns 3
@@ -165,7 +166,7 @@ describe Puppet::Type.type(:service).provider(:openrc) do
       end
 
       it "should return :running when status command returns with a zero exitcode" do
-        provider = described_class.new(Puppet::Type.type(:service).new(:name => 'sshd', :status => '/bin/foo'))
+        provider = provider_class.new(Puppet::Type.type(:service).new(:name => 'sshd', :status => '/bin/foo'))
         provider.expects(:execute).with(['/sbin/rc-service','sshd',:status], :failonfail => false, :override_locale => false, :squelch => false, :combine => true).never
         provider.expects(:execute).with(['/bin/foo'], :failonfail => false, :override_locale => false, :squelch => false, :combine => true)
         $CHILD_STATUS.stubs(:exitstatus).returns 0
@@ -175,14 +176,14 @@ describe Puppet::Type.type(:service).provider(:openrc) do
 
     describe "when hasstatus is false" do
       it "should return running if a pid can be found" do
-        provider = described_class.new(Puppet::Type.type(:service).new(:name => 'sshd', :hasstatus => false))
+        provider = provider_class.new(Puppet::Type.type(:service).new(:name => 'sshd', :hasstatus => false))
         provider.expects(:execute).with(['/sbin/rc-service','sshd',:status], :failonfail => false, :override_locale => false, :squelch => false, :combine => true).never
         provider.expects(:getpid).returns 1000
         expect(provider.status).to eq(:running)
       end
 
       it "should return stopped if no pid can be found" do
-        provider = described_class.new(Puppet::Type.type(:service).new(:name => 'sshd', :hasstatus => false))
+        provider = provider_class.new(Puppet::Type.type(:service).new(:name => 'sshd', :hasstatus => false))
         provider.expects(:execute).with(['/sbin/rc-service','sshd',:status], :failonfail => false, :override_locale => false, :squelch => false, :combine => true).never
         provider.expects(:getpid).returns nil
         expect(provider.status).to eq(:stopped)
@@ -191,14 +192,14 @@ describe Puppet::Type.type(:service).provider(:openrc) do
 
     describe "when hasstatus is true" do
       it "should return running if rc-service status exits with a zero exitcode" do
-        provider = described_class.new(Puppet::Type.type(:service).new(:name => 'sshd', :hasstatus => true))
+        provider = provider_class.new(Puppet::Type.type(:service).new(:name => 'sshd', :hasstatus => true))
         provider.expects(:execute).with(['/sbin/rc-service','sshd',:status], :failonfail => false, :override_locale => false, :squelch => false, :combine => true)
         $CHILD_STATUS.stubs(:exitstatus).returns 0
         expect(provider.status).to eq(:running)
       end
 
       it "should return stopped if rc-service status exits with a non-zero exitcode" do
-        provider = described_class.new(Puppet::Type.type(:service).new(:name => 'sshd', :hasstatus => true))
+        provider = provider_class.new(Puppet::Type.type(:service).new(:name => 'sshd', :hasstatus => true))
         provider.expects(:execute).with(['/sbin/rc-service','sshd',:status], :failonfail => false, :override_locale => false, :squelch => false, :combine => true)
         $CHILD_STATUS.stubs(:exitstatus).returns 3
         expect(provider.status).to eq(:stopped)
@@ -208,20 +209,20 @@ describe Puppet::Type.type(:service).provider(:openrc) do
 
   describe "#restart" do
     it "should use the supplied restart command if specified" do
-      provider = described_class.new(Puppet::Type.type(:service).new(:name => 'sshd', :restart => '/bin/foo'))
+      provider = provider_class.new(Puppet::Type.type(:service).new(:name => 'sshd', :restart => '/bin/foo'))
       provider.expects(:execute).with(['/sbin/rc-service','sshd',:restart], :failonfail => true, :override_locale => false, :squelch => false, :combine => true).never
       provider.expects(:execute).with(['/bin/foo'], :failonfail => true, :override_locale => false, :squelch => false, :combine => true)
       provider.restart
     end
 
     it "should restart the service with rc-service restart if hasrestart is true" do
-      provider = described_class.new(Puppet::Type.type(:service).new(:name => 'sshd', :hasrestart => true))
+      provider = provider_class.new(Puppet::Type.type(:service).new(:name => 'sshd', :hasrestart => true))
       provider.expects(:execute).with(['/sbin/rc-service','sshd',:restart], :failonfail => true, :override_locale => false, :squelch => false, :combine => true)
       provider.restart
     end
 
     it "should restart the service with rc-service stop/start if hasrestart is false" do
-      provider = described_class.new(Puppet::Type.type(:service).new(:name => 'sshd', :hasrestart => false))
+      provider = provider_class.new(Puppet::Type.type(:service).new(:name => 'sshd', :hasrestart => false))
       provider.expects(:execute).with(['/sbin/rc-service','sshd',:restart], :failonfail => true, :override_locale => false, :squelch => false, :combine => true).never
       provider.expects(:execute).with(['/sbin/rc-service','sshd',:stop], :failonfail => true, :override_locale => false, :squelch => false, :combine => true)
       provider.expects(:execute).with(['/sbin/rc-service','sshd',:start], :failonfail => true, :override_locale => false, :squelch => false, :combine => true)

data/spec/unit/provider/service/openwrt_spec.rb

@@ -4,7 +4,9 @@
 #
 require 'spec_helper'
 
-describe Puppet::Type.type(:service).provider(:openwrt), :if => Puppet.features.posix? do
+describe 'Puppet::Type::Service::Provider::Openwrt',
+    :if => Puppet.features.posix? && !Puppet::Util::Platform.jruby? do
+  let(:provider_class) { Puppet::Type.type(:service).provider(:openwrt) }
 
   let(:resource) do
     resource = stub 'resource'
@@ -16,7 +18,7 @@ describe Puppet::Type.type(:service).provider(:openwrt), :if => Puppet.features.
   end
 
   let(:provider) do
-    provider = described_class.new
+    provider = provider_class.new
     provider.stubs(:get).with(:hasstatus).returns false
 
     provider
@@ -41,7 +43,7 @@ describe Puppet::Type.type(:service).provider(:openwrt), :if => Puppet.features.
   operatingsystem = 'openwrt'
   it "should be the default provider on #{operatingsystem}" do
     Facter.expects(:value).with(:operatingsystem).returns(operatingsystem)
-    expect(described_class.default?).to be_truthy
+    expect(provider_class.default?).to be_truthy
   end
 
   # test self.instances
@@ -56,10 +58,10 @@ describe Puppet::Type.type(:service).provider(:openwrt), :if => Puppet.features.
     end
     it "should return instances for all services" do
       services.each do |inst|
-        described_class.expects(:new).with{|hash| hash[:name] == inst && hash[:path] == '/etc/init.d'}.returns("#{inst}_instance")
+        provider_class.expects(:new).with{|hash| hash[:name] == inst && hash[:path] == '/etc/init.d'}.returns("#{inst}_instance")
       end
       results = services.collect {|x| "#{x}_instance"}
-      expect(described_class.instances).to eq(results)
+      expect(provider_class.instances).to eq(results)
     end
   end
 

data/spec/unit/provider/service/rcng_spec.rb

@@ -2,14 +2,15 @@
 
 require 'spec_helper'
 
-provider_class = Puppet::Type.type(:service).provider(:rcng)
+describe 'Puppet::Type::Service::Provider::Rcng',
+    :unless => Puppet::Util::Platform.windows? || Puppet::Util::Platform.jruby? do
+  let(:provider_class) { Puppet::Type.type(:service).provider(:rcng) }
 
-describe provider_class, :unless => Puppet.features.microsoft_windows? do
   before :each do
-    Puppet::Type.type(:service).stubs(:defaultprovider).returns described_class
+    Puppet::Type.type(:service).stubs(:defaultprovider).returns provider_class
     Facter.stubs(:value).with(:operatingsystem).returns :netbsd
     Facter.stubs(:value).with(:osfamily).returns 'NetBSD'
-    described_class.stubs(:defpath).returns('/etc/rc.d')
+    provider_class.stubs(:defpath).returns('/etc/rc.d')
     @provider = provider_class.new
     @provider.stubs(:initscript)
   end
@@ -20,7 +21,7 @@ describe provider_class, :unless => Puppet.features.microsoft_windows? do
     end
 
     it "should set the proper contents to enable" do
-      provider = described_class.new(Puppet::Type.type(:service).new(:name => 'sshd'))
+      provider = provider_class.new(Puppet::Type.type(:service).new(:name => 'sshd'))
       Dir.stubs(:mkdir).with('/etc/rc.conf.d')
       fh = stub 'fh'
       Puppet::Util.expects(:replace_file).with('/etc/rc.conf.d/sshd', 0644).yields(fh)
@@ -29,7 +30,7 @@ describe provider_class, :unless => Puppet.features.microsoft_windows? do
     end
 
     it "should set the proper contents to enable when disabled" do
-      provider = described_class.new(Puppet::Type.type(:service).new(:name => 'sshd'))
+      provider = provider_class.new(Puppet::Type.type(:service).new(:name => 'sshd'))
       Dir.stubs(:mkdir).with('/etc/rc.conf.d')
       File.stubs(:read).with('/etc/rc.conf.d/sshd').returns("sshd_enable=\"NO\"\n")
       fh = stub 'fh'

data/spec/unit/provider/service/redhat_spec.rb

@@ -4,9 +4,9 @@
 #
 require 'spec_helper'
 
-provider_class = Puppet::Type.type(:service).provider(:redhat)
-
-describe provider_class, :if => Puppet.features.posix? do
+describe 'Puppet::Type::Service::Provider::Redhat',
+    :if => Puppet.features.posix? && !Puppet::Util::Platform.jruby? do
+  let(:provider_class) { Puppet::Type.type(:service).provider(:redhat) }
 
   before :each do
     @class = Puppet::Type.type(:service).provider(:redhat)
@@ -23,9 +23,9 @@ describe provider_class, :if => Puppet.features.posix? do
     Facter.stubs(:value).with(:osfamily).returns 'RedHat'
   end
 
-  osfamily = [ 'RedHat' ]
+  osfamilies = [ 'RedHat' ]
 
-  osfamily.each do |osfamily|
+  osfamilies.each do |osfamily|
     it "should be the default provider on #{osfamily}" do
       Facter.expects(:value).with(:osfamily).returns(osfamily)
       expect(provider_class.default?).to be_truthy
@@ -36,7 +36,7 @@ describe provider_class, :if => Puppet.features.posix? do
     Facter.stubs(:value).with(:osfamily).returns(:suse)
     Facter.stubs(:value).with(:operatingsystem).returns(:suse)
     Facter.stubs(:value).with(:operatingsystemmajrelease).returns("11")
-    expect(described_class.default?).to be_truthy
+    expect(provider_class.default?).to be_truthy
   end
 
   # test self.instances
@@ -88,6 +88,11 @@ describe provider_class, :if => Puppet.features.posix? do
       expect(@provider.enabled?).to eq(:true)
     end
 
+    it "should check for B" do
+      provider_class.stubs(:chkconfig).with(@resource[:name]).returns "#{@resource[:name]}  B"
+      expect(@provider.enabled?).to eq(:true)
+    end
+
     it "should check for off" do
       provider_class.stubs(:chkconfig).with(@resource[:name]).returns "#{@resource[:name]}  off"
       expect(@provider.enabled?).to eq(:false)

data/spec/unit/provider/service/runit_spec.rb

@@ -6,9 +6,8 @@
 #
 require 'spec_helper'
 
-provider_class = Puppet::Type.type(:service).provider(:runit)
-
-describe provider_class do
+describe 'Puppet::Type::Service::Provider::Runit', unless: Puppet::Util::Platform.jruby? do
+  let(:provider_class) { Puppet::Type.type(:service).provider(:runit) }
 
   before(:each) do
     # Create a mock resource

data/spec/unit/provider/service/smf_spec.rb

@@ -6,9 +6,9 @@
 #
 require 'spec_helper'
 
-provider_class = Puppet::Type.type(:service).provider(:smf)
-
-describe provider_class, :if => Puppet.features.posix? do
+describe 'Puppet::Type::Service::Provider::Smf',
+    if: Puppet.features.posix? && !Puppet::Util::Platform.jruby? do
+  let(:provider_class) { Puppet::Type.type(:service).provider(:smf) }
 
   before(:each) do
     # Create a mock resource
@@ -18,8 +18,6 @@ describe provider_class, :if => Puppet.features.posix? do
 
     FileTest.stubs(:file?).with('/usr/sbin/svcadm').returns true
     FileTest.stubs(:executable?).with('/usr/sbin/svcadm').returns true
-    FileTest.stubs(:file?).with('/usr/sbin/svccfg').returns true
-    FileTest.stubs(:executable?).with('/usr/sbin/svccfg').returns true
     FileTest.stubs(:file?).with('/usr/bin/svcs').returns true
     FileTest.stubs(:executable?).with('/usr/bin/svcs').returns true
     Facter.stubs(:value).with(:operatingsystem).returns('Solaris')
@@ -76,9 +74,9 @@ describe provider_class, :if => Puppet.features.posix? do
       @provider.expects(:svcs).with('-H', '-o', 'state,nstate', "/system/myservice").returns("online\t-")
       @provider.status
     end
-    it "should return absent if svcs can't find the service" do
+    it "should return stopped if svcs can't find the service" do
       @provider.stubs(:svcs).raises(Puppet::ExecutionFailure.new("no svc found"))
-      expect(@provider.status).to eq(:absent)
+      expect(@provider.status).to eq(:stopped)
     end
     it "should return running if online in svcs output" do
       @provider.stubs(:svcs).returns("online\t-")
@@ -151,6 +149,7 @@ describe provider_class, :if => Puppet.features.posix? do
     end
 
     it "should import the manifest if service is missing" do
+      @provider.expects(:svcs).with('-l', '/system/myservice').raises(Puppet::ExecutionFailure, "Exited 1")
       @provider.expects(:svccfg).with(:import, "/tmp/myservice.xml")
       @provider.expects(:texecute).with(:start, ["/usr/sbin/svcadm", :enable, '-rs', "/system/myservice"], true)
       @provider.expects(:wait).with('online')
@@ -159,6 +158,7 @@ describe provider_class, :if => Puppet.features.posix? do
     end
 
     it "should handle failures if importing a manifest" do
+      @provider.expects(:svcs).with('-l', '/system/myservice').raises(Puppet::ExecutionFailure, "Exited 1")
       @provider.expects(:svccfg).raises(Puppet::ExecutionFailure.new("can't svccfg import"))
       expect { @provider.start }.to raise_error(Puppet::Error, "Cannot config /system/myservice to enable it: can't svccfg import")
     end

data/spec/unit/provider/service/src_spec.rb

@@ -5,11 +5,10 @@
 
 require 'spec_helper'
 
-provider_class = Puppet::Type.type(:service).provider(:src)
+describe 'Puppet::Type::Service::Provider::Src', unless: Puppet::Util::Platform.jruby? do
+  let(:provider_class) { Puppet::Type.type(:service).provider(:src) }
 
-describe provider_class do
-
-  if Puppet.features.microsoft_windows?
+  if Puppet::Util::Platform.windows?
     # Get a pid for $CHILD_STATUS to latch on to
     command = "cmd.exe /c \"exit 0\""
     Puppet::Util::Execution.execute(command, {:failonfail => false})