puppet 6.0.0 → 6.1.0

data/spec/unit/provider/package/yum_spec.rb

@@ -1,17 +1,15 @@
-#! /usr/bin/env ruby
 require 'spec_helper'
 
-provider_class = Puppet::Type.type(:package).provider(:yum)
-
-describe provider_class do
+describe Puppet::Type.type(:package).provider(:yum) do
   include PuppetSpec::Fixtures
-  it_behaves_like 'RHEL package provider', provider_class, 'yum'
+
+  it_behaves_like 'RHEL package provider', described_class, 'yum'
 
   it "should have lower specificity" do
     Facter.stubs(:value).with(:osfamily).returns(:redhat)
     Facter.stubs(:value).with(:operatingsystem).returns(:fedora)
     Facter.stubs(:value).with(:operatingsystemmajrelease).returns("22")
-    expect(provider_class.specificity).to be < 200
+    expect(described_class.specificity).to be < 200
   end
 
   describe "when supplied the source param" do
@@ -25,12 +23,12 @@ describe provider_class do
     end
 
     let(:provider) do
-      provider = provider_class.new
+      provider = described_class.new
       provider.resource = resource
       provider
     end
 
-    before { provider_class.stubs(:command).with(:cmd).returns("/usr/bin/yum") }
+    before { described_class.stubs(:command).with(:cmd).returns("/usr/bin/yum") }
 
     context "when installing" do
       it "should use the supplied source as the explicit path to a package to install" do
@@ -58,16 +56,18 @@ describe provider_class do
     end
   end
 
-  describe "parsing the output of check-update" do
-    describe "with no multiline entries" do
+  context "parsing the output of check-update" do
+    context "with no multiline entries" do
       let(:check_update) { File.read(my_fixture("yum-check-update-simple.txt")) }
       let(:output) { described_class.parse_updates(check_update) }
+
       it 'creates an entry for each package keyed on the package name' do
         expect(output['curl']).to eq([{:name => 'curl', :epoch => '0', :version => '7.32.0', :release => '10.fc20', :arch => 'i686'}, {:name => 'curl', :epoch => '0', :version => '7.32.0', :release => '10.fc20', :arch => 'x86_64'}])
         expect(output['gawk']).to eq([{:name => 'gawk', :epoch => '0', :version => '4.1.0', :release => '3.fc20', :arch => 'i686'}])
         expect(output['dhclient']).to eq([{:name => 'dhclient', :epoch => '12', :version => '4.1.1', :release => '38.P1.fc20', :arch => 'i686'}])
         expect(output['selinux-policy']).to eq([{:name => 'selinux-policy', :epoch => '0', :version => '3.12.1', :release => '163.fc20', :arch => 'noarch'}])
       end
+
       it 'creates an entry for each package keyed on the package name and package architecture' do
         expect(output['curl.i686']).to eq([{:name => 'curl', :epoch => '0', :version => '7.32.0', :release => '10.fc20', :arch => 'i686'}])
         expect(output['curl.x86_64']).to eq([{:name => 'curl', :epoch => '0', :version => '7.32.0', :release => '10.fc20', :arch => 'x86_64'}])
@@ -77,55 +77,69 @@ describe provider_class do
         expect(output['java-1.8.0-openjdk.x86_64']).to eq([{:name => 'java-1.8.0-openjdk', :epoch => '1', :version => '1.8.0.131', :release => '2.b11.el7_3', :arch => 'x86_64'}])
       end
     end
-    describe "with multiline entries" do
+
+    context "with multiline entries" do
       let(:check_update) { File.read(my_fixture("yum-check-update-multiline.txt")) }
       let(:output) { described_class.parse_updates(check_update) }
+
       it "parses multi-line values as a single package tuple" do
         expect(output['libpcap']).to eq([{:name => 'libpcap', :epoch => '14', :version => '1.4.0', :release => '1.20130826git2dbcaa1.el6', :arch => 'x86_64'}])
       end
     end
-    describe "with obsoleted packages" do
+
+    context "with obsoleted packages" do
       let(:check_update) { File.read(my_fixture("yum-check-update-obsoletes.txt")) }
       let(:output) { described_class.parse_updates(check_update) }
+
       it "ignores all entries including and after 'Obsoleting Packages'" do
         expect(output).not_to include("Obsoleting")
         expect(output).not_to include("NetworkManager-bluetooth.x86_64")
         expect(output).not_to include("1:1.0.0-14.git20150121.b4ea599c.el7")
       end
     end
-    describe "with security notifications" do
+
+    context "with security notifications" do
       let(:check_update) { File.read(my_fixture("yum-check-update-security.txt")) }
       let(:output) { described_class.parse_updates(check_update) }
+
       it "ignores all entries including and after 'Security'" do
         expect(output).not_to include("Security")
       end
+
       it "includes updates before 'Security'" do
         expect(output).to include("yum-plugin-fastestmirror.noarch")
       end
     end
-    describe "with broken update notices" do
+
+    context "with broken update notices" do
       let(:check_update) { File.read(my_fixture("yum-check-update-broken-notices.txt")) }
       let(:output) { described_class.parse_updates(check_update) }
+
       it "ignores all entries including and after 'Update'" do
         expect(output).not_to include("Update")
       end
+
       it "includes updates before 'Update'" do
         expect(output).to include("yum-plugin-fastestmirror.noarch")
       end
     end
-    describe "with improper package names in output" do
+
+    context "with improper package names in output" do
       it "raises an exception parsing package name" do
         expect {
           described_class.update_to_hash('badpackagename', '1')
         }.to raise_exception(Exception, /Failed to parse/)
       end
     end
-    describe "with trailing plugin output" do
+
+    context "with trailing plugin output" do
       let(:check_update) { File.read(my_fixture("yum-check-update-plugin-output.txt")) }
       let(:output) { described_class.parse_updates(check_update) }
+
       it "parses correctly formatted entries" do
         expect(output['bash']).to eq([{:name => 'bash', :epoch => '0', :version => '4.2.46', :release => '12.el7', :arch => 'x86_64'}])
       end
+
       it "ignores all mentions of plugin output" do
         expect(output).not_to include("Random plugin")
       end

data/spec/unit/provider/package/zypper_spec.rb

@@ -1,9 +1,6 @@
-#! /usr/bin/env ruby
 require 'spec_helper'
 
-provider_class = Puppet::Type.type(:package).provider(:zypper)
-
-describe provider_class do
+describe Puppet::Type.type(:package).provider(:zypper) do
   before(:each) do
     # Create a mock resource
     @resource = stub 'resource'
@@ -16,35 +13,35 @@ describe provider_class do
     @resource.stubs(:[]).with(:ensure).returns :installed
     @resource.stubs(:command).with(:zypper).returns "/usr/bin/zypper"
 
-    @provider = provider_class.new(@resource)
+    @provider = described_class.new(@resource)
   end
 
   it "should have an install method" do
-    @provider = provider_class.new
+    @provider = described_class.new
     expect(@provider).to respond_to(:install)
   end
 
   it "should have an uninstall method" do
-    @provider = provider_class.new
+    @provider = described_class.new
     expect(@provider).to respond_to(:uninstall)
   end
 
   it "should have an update method" do
-    @provider = provider_class.new
+    @provider = described_class.new
     expect(@provider).to respond_to(:update)
   end
 
   it "should have a latest method" do
-    @provider = provider_class.new
+    @provider = described_class.new
     expect(@provider).to respond_to(:latest)
   end
 
   it "should have a install_options method" do
-    @provider = provider_class.new
+    @provider = described_class.new
     expect(@provider).to respond_to(:install_options)
   end
 
-  describe "when installing with zypper version >= 1.0" do
+  context "when installing with zypper version >= 1.0" do
     it "should use a command-line with versioned package'" do
       @resource.stubs(:should).with(:ensure).returns "1.2.3-4.5.6"
       @resource.stubs(:allow_virtual?).returns false
@@ -65,7 +62,7 @@ describe provider_class do
     end
   end
 
-  describe "when installing with zypper version = 0.6.104" do
+  context "when installing with zypper version = 0.6.104" do
     it "should use a command-line with versioned package'" do
       @resource.stubs(:should).with(:ensure).returns "1.2.3-4.5.6"
       @resource.stubs(:allow_virtual?).returns false
@@ -86,7 +83,7 @@ describe provider_class do
     end
   end
 
-  describe "when installing with zypper version = 0.6.13" do
+  context "when installing with zypper version = 0.6.13" do
     it "should use a command-line with versioned package'" do
       @resource.stubs(:should).with(:ensure).returns "1.2.3-4.5.6"
       @resource.stubs(:allow_virtual?).returns false
@@ -107,15 +104,16 @@ describe provider_class do
     end
   end
 
-  describe "when updating" do
+  context "when updating" do
     it "should call install method of instance" do
       @provider.expects(:install)
       @provider.update
     end
   end
 
-  describe "when getting latest version" do
-    after do described_class.reset! end
+  context "when getting latest version" do
+    after { described_class.reset! }
+
     context "when the package has available update" do
       it "should return a version string with valid list-updates data from SLES11sp1" do
         fake_data = File.read(my_fixture('zypper-list-updates-SLES11sp1.out'))
@@ -144,7 +142,7 @@ describe provider_class do
     end
   end
 
-  describe "should install a virtual package" do
+  context "should install a virtual package" do
     it "when zypper version = 0.6.13" do
       @resource.stubs(:should).with(:ensure).returns :installed
       @resource.stubs(:allow_virtual?).returns true
@@ -164,7 +162,7 @@ describe provider_class do
     end
   end
 
-  describe "when installing with zypper install options" do
+  context "when installing with zypper install options" do
     it "should install the package without checking keys" do
       @resource.stubs(:[]).with(:name).returns "php5"
       @resource.stubs(:[]).with(:install_options).returns ['--no-gpg-check', {'-p' => '/vagrant/files/localrepo/'}]
@@ -215,7 +213,7 @@ describe provider_class do
     end
   end
 
-  describe 'when uninstalling' do
+  context 'when uninstalling' do
     it 'should use remove to uninstall on zypper version 1.6 and above' do
       @provider.stubs(:zypper_version).returns '1.6.308'
       @provider.expects(:zypper).with(:remove, '--no-confirm', 'mypackage')

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

@@ -1,11 +1,9 @@
-#!/usr/bin/env ruby
-
 require 'spec_helper'
 
 describe 'Puppet::Type::Service::Provider::Bsd',
          :unless => Puppet::Util::Platform.windows? || Puppet::Util::Platform.jruby? do
-
   let(:provider_class) { Puppet::Type.type(:service).provider(:bsd) }
+
   before :each do
     Puppet::Type.type(:service).stubs(:defaultprovider).returns provider_class
     Facter.stubs(:value).with(:operatingsystem).returns :netbsd
@@ -15,7 +13,7 @@ describe 'Puppet::Type::Service::Provider::Bsd',
     @provider.stubs(:initscript)
   end
 
-  describe "#instances" do
+  context "#instances" do
     it "should have an instances method" do
       expect(provider_class).to respond_to :instances
     end
@@ -25,7 +23,7 @@ describe 'Puppet::Type::Service::Provider::Bsd',
     end
   end
 
-  describe "#disable" do
+  context "#disable" do
     it "should have a disable method" do
       expect(@provider).to respond_to(:disable)
     end
@@ -46,7 +44,7 @@ describe 'Puppet::Type::Service::Provider::Bsd',
     end
   end
 
-  describe "#enable" do
+  context "#enable" do
     it "should have an enable method" do
       expect(@provider).to respond_to(:enable)
     end
@@ -71,7 +69,7 @@ describe 'Puppet::Type::Service::Provider::Bsd',
     end
   end
 
-  describe "#enabled?" do
+  context "#enabled?" do
     it "should have an enabled? method" do
       expect(@provider).to respond_to(:enabled?)
     end
@@ -91,7 +89,7 @@ describe 'Puppet::Type::Service::Provider::Bsd',
     end
   end
 
-  describe "#startcmd" do
+  context "#startcmd" do
     it "should have a startcmd method" do
       expect(@provider).to respond_to(:startcmd)
     end
@@ -110,7 +108,7 @@ describe 'Puppet::Type::Service::Provider::Bsd',
     end
   end
 
-  describe "#stopcmd" do
+  context "#stopcmd" do
     it "should have a stopcmd method" do
       expect(@provider).to respond_to(:stopcmd)
     end

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

@@ -1,12 +1,7 @@
-#! /usr/bin/env ruby
-#
-# Unit testing for the Daemontools service Provider
-#
-# author Brice Figureau
-#
 require 'spec_helper'
 
-describe 'Puppet::Type::Service::Provider::Daemontools', unless: Puppet::Util::Platform.jruby? do
+describe 'Puppet::Type::Service::Provider::Daemontools',
+         unless: Puppet::Util::Platform.jruby? do
   let(:provider_class) { Puppet::Type.type(:service).provider(:daemontools) }
 
   before(:each) do
@@ -62,7 +57,7 @@ describe 'Puppet::Type::Service::Provider::Daemontools', unless: Puppet::Util::P
     expect(@provider).to respond_to(:disable)
   end
 
-  describe "when starting" do
+  context "when starting" do
     it "should use 'svc' to start the service" do
       @provider.stubs(:enabled?).returns :true
       @provider.expects(:svc).with("-u", "/etc/service/myservice")
@@ -80,7 +75,7 @@ describe 'Puppet::Type::Service::Provider::Daemontools', unless: Puppet::Util::P
     end
   end
 
-  describe "when stopping" do
+  context "when stopping" do
     it "should use 'svc' to stop the service" do
       @provider.stubs(:disable)
       @provider.expects(:svc).with("-d", "/etc/service/myservice")
@@ -89,7 +84,7 @@ describe 'Puppet::Type::Service::Provider::Daemontools', unless: Puppet::Util::P
     end
   end
 
-  describe "when restarting" do
+  context "when restarting" do
     it "should use 'svc' to restart the service" do
       @provider.expects(:svc).with("-t", "/etc/service/myservice")
 
@@ -97,7 +92,7 @@ describe 'Puppet::Type::Service::Provider::Daemontools', unless: Puppet::Util::P
     end
   end
 
-  describe "when enabling" do
+  context "when enabling" do
     it "should create a symlink between daemon dir and service dir", :if => Puppet.features.manages_symlinks?  do
       daemon_path = File.join(@daemondir, "myservice")
       service_path = File.join(@servicedir, "myservice")
@@ -108,7 +103,7 @@ describe 'Puppet::Type::Service::Provider::Daemontools', unless: Puppet::Util::P
     end
   end
 
-  describe "when disabling" do
+  context "when disabling" do
     it "should remove the symlink between daemon dir and service dir" do
       FileTest.stubs(:directory?).returns(false)
       path = File.join(@servicedir,"myservice")
@@ -127,7 +122,7 @@ describe 'Puppet::Type::Service::Provider::Daemontools', unless: Puppet::Util::P
     end
   end
 
-  describe "when checking if the service is enabled?" do
+  context "when checking if the service is enabled?" do
     it "should return true if it is running" do
       @provider.stubs(:status).returns(:running)
 
@@ -145,26 +140,27 @@ describe 'Puppet::Type::Service::Provider::Daemontools', unless: Puppet::Util::P
     end
   end
 
-  describe "when checking status" do
+  context "when checking status" do
     it "should call the external command 'svstat /etc/service/myservice'" do
       @provider.expects(:svstat).with(File.join(@servicedir,"myservice"))
       @provider.status
     end
   end
 
-  describe "when checking status" do
+  context "when checking status" do
     it "and svstat fails, properly raise a Puppet::Error" do
       @provider.expects(:svstat).with(File.join(@servicedir,"myservice")).raises(Puppet::ExecutionFailure, "failure")
       expect { @provider.status }.to raise_error(Puppet::Error, 'Could not get status for service Service[myservice]: failure')
     end
+
     it "and svstat returns up, then return :running" do
       @provider.expects(:svstat).with(File.join(@servicedir,"myservice")).returns("/etc/service/myservice: up (pid 454) 954326 seconds")
       expect(@provider.status).to eq(:running)
     end
+
     it "and svstat returns not running, then return :stopped" do
       @provider.expects(:svstat).with(File.join(@servicedir,"myservice")).returns("/etc/service/myservice: supervise not running")
       expect(@provider.status).to  eq(:stopped)
     end
   end
-
 end

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

@@ -1,11 +1,7 @@
-#! /usr/bin/env ruby
-#
-# Unit testing for the debian service provider
-#
-
 require 'spec_helper'
 
-describe 'Puppet::Type::Service::Provider::Debian', unless: Puppet::Util::Platform.jruby? do
+describe 'Puppet::Type::Service::Provider::Debian',
+         unless: Puppet::Util::Platform.jruby? do
   let(:provider_class) { Puppet::Type.type(:service).provider(:debian) }
 
   if Puppet::Util::Platform.windows?
@@ -69,14 +65,14 @@ describe 'Puppet::Type::Service::Provider::Debian', unless: Puppet::Util::Platfo
     expect(@provider).to respond_to(:disable)
   end
 
-  describe "when enabling" do
+  context "when enabling" do
     it "should call update-rc.d twice" do
       @provider.expects(:update_rc).twice
       @provider.enable
     end
   end
 
-  describe "when disabling" do
+  context "when disabling" do
     it "should be able to disable services with newer sysv-rc versions" do
       @provider.stubs(:`).with("dpkg --compare-versions $(dpkg-query -W --showformat '${Version}' sysv-rc) ge 2.88 ; echo $?").returns "0"
 
@@ -95,7 +91,7 @@ describe 'Puppet::Type::Service::Provider::Debian', unless: Puppet::Util::Platfo
     end
   end
 
-  describe "when checking whether it is enabled" do
+  context "when checking whether it is enabled" do
     it "should call Kernel.system() with the appropriate parameters" do
       @provider.expects(:system).with("/usr/sbin/invoke-rc.d", "--quiet", "--query", @resource[:name], "start").once
       $CHILD_STATUS.stubs(:exitstatus).returns(0)
@@ -155,7 +151,7 @@ describe 'Puppet::Type::Service::Provider::Debian', unless: Puppet::Util::Platfo
     end
   end
 
-  describe "when checking service status" do
+  context "when checking service status" do
     it "should use the service command" do
       Facter.stubs(:value).with(:operatingsystem).returns('Debian')
       Facter.stubs(:value).with(:operatingsystemmajrelease).returns('8')

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

@@ -1,7 +1,7 @@
-#! /usr/bin/env ruby
 require 'spec_helper'
 
-describe 'Puppet::Type::Service::Provider::Freebsd', unless: Puppet::Util::Platform.jruby? do
+describe 'Puppet::Type::Service::Provider::Freebsd',
+         unless: Puppet::Util::Platform.jruby? do
   let(:provider_class) { Puppet::Type.type(:service).provider(:freebsd) }
 
   before :each do

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

@@ -345,4 +345,23 @@ describe 'Puppet::Type::Service::Provider::Launchd', unless: Puppet::Util::Platf
       expect { provider.jobsearch("NOSUCH") }.to raise_error(Puppet::Error)
     end
   end
+
+  describe "read_overrides" do
+    before do
+      Kernel.stubs(:sleep)
+    end
+    it "should read overrides" do
+      provider.expects(:read_plist).once.returns({})
+      expect(provider.read_overrides).to eq({})
+    end
+    it "should retry if read_plist fails" do
+      provider.expects(:read_plist).once.returns({})
+      provider.expects(:read_plist).once.returns(nil)
+      expect(provider.read_overrides).to eq({})
+    end
+    it "raises Puppet::Error after 20 attempts" do
+      provider.expects(:read_plist).times(20).returns(nil)
+      expect { provider.read_overrides }.to raise_error(Puppet::Error)
+    end
+ end
 end

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

@@ -1,11 +1,7 @@
-#!/usr/bin/env ruby
-#
-# Unit testing for the OpenBSD service provider
-
 require 'spec_helper'
 
 describe 'Puppet::Type::Service::Provider::Openbsd',
-    unless: Puppet::Util::Platform.windows? || Puppet::Util::Platform.jruby? do
+         unless: Puppet::Util::Platform.windows? || Puppet::Util::Platform.jruby? do
   let(:provider_class) { Puppet::Type.type(:service).provider(:openbsd) }
 
   before :each do
@@ -16,7 +12,7 @@ describe 'Puppet::Type::Service::Provider::Openbsd',
     FileTest.stubs(:executable?).with('/usr/sbin/rcctl').returns true
   end
 
-  describe "#instances" do
+  context "#instances" do
     it "should have an instances method" do
       expect(provider_class).to respond_to :instances
     end
@@ -29,7 +25,7 @@ describe 'Puppet::Type::Service::Provider::Openbsd',
     end
   end
 
-  describe "#start" do
+  context "#start" do
     it "should use the supplied start command if specified" do
       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)
@@ -43,7 +39,7 @@ describe 'Puppet::Type::Service::Provider::Openbsd',
     end
   end
 
-  describe "#stop" do
+  context "#stop" do
     it "should use the supplied stop command if specified" do
       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)
@@ -57,7 +53,7 @@ describe 'Puppet::Type::Service::Provider::Openbsd',
     end
   end
 
-  describe "#status" do
+  context "#status" do
     it "should use the status command from the resource" do
       provider = provider_class.new(Puppet::Type.type(:service).new(:name => 'sshd', :status => '/bin/foo'))
       provider.expects(:execute).with(['/usr/sbin/rcctl', :get, 'sshd', :status], :failonfail => true, :override_locale => false, :squelch => false, :combine => true).never
@@ -82,7 +78,7 @@ describe 'Puppet::Type::Service::Provider::Openbsd',
     end
   end
 
-  describe "#restart" do
+  context "#restart" do
     it "should use the supplied restart command if specified" do
       provider = provider_class.new(Puppet::Type.type(:service).new(:name => 'sshd', :restart => '/bin/foo'))
       provider.expects(:execute).with(['/usr/sbin/rcctl', '-f', :restart, 'sshd'], :failonfail => true, :override_locale => false, :squelch => false, :combine => true).never
@@ -105,7 +101,7 @@ describe 'Puppet::Type::Service::Provider::Openbsd',
     end
   end
 
-  describe "#enabled?" do
+  context "#enabled?" do
     it "should return :true if the service is enabled" do
       provider = provider_class.new(Puppet::Type.type(:service).new(:name => 'sshd'))
       provider_class.stubs(:rcctl).with(:get, 'sshd', :status)
@@ -121,7 +117,7 @@ describe 'Puppet::Type::Service::Provider::Openbsd',
     end
   end
 
-  describe "#enable" do
+  context "#enable" do
     it "should run rcctl enable to enable the service" do
       provider = provider_class.new(Puppet::Type.type(:service).new(:name => 'sshd'))
       provider_class.stubs(:rcctl).with(:enable, 'sshd').returns('')
@@ -139,7 +135,7 @@ describe 'Puppet::Type::Service::Provider::Openbsd',
     end
   end
 
-  describe "#disable" do
+  context "#disable" do
     it "should run rcctl disable to disable the service" do
       provider = provider_class.new(Puppet::Type.type(:service).new(:name => 'sshd'))
       provider_class.stubs(:rcctl).with(:disable, 'sshd').returns('')
@@ -148,7 +144,7 @@ describe 'Puppet::Type::Service::Provider::Openbsd',
     end
   end
 
-  describe "#running?" do
+  context "#running?" do
     it "should run rcctl check to check the service" do
       provider = provider_class.new(Puppet::Type.type(:service).new(:name => 'sshd'))
       provider_class.stubs(:rcctl).with(:check, 'sshd').returns('sshd(ok)')
@@ -171,7 +167,7 @@ describe 'Puppet::Type::Service::Provider::Openbsd',
     end
   end
 
-  describe "#flags" do
+  context "#flags" do
     it "should return flags when set" do
       provider = provider_class.new(Puppet::Type.type(:service).new(:name => 'sshd', :flags => '-6'))
       provider_class.stubs(:rcctl).with('get', 'sshd', 'flags').returns('-6')
@@ -194,8 +190,8 @@ describe 'Puppet::Type::Service::Provider::Openbsd',
     end
   end
 
-  describe "#flags=" do
-    it "should run rcctl to set flags", :unless => Puppet::Util::Platform.windows? || RUBY_PLATFORM == 'java' do
+  context "#flags=" do
+    it "should run rcctl to set flags", unless: Puppet::Util::Platform.windows? || RUBY_PLATFORM == 'java' do
       provider = provider_class.new(Puppet::Type.type(:service).new(:name => 'sshd'))
       provider_class.stubs(:rcctl).with(:set, 'sshd', :flags, '-4').returns('')
       provider.expects(:rcctl).with(:set, 'sshd', :flags, '-4')

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

@@ -1,9 +1,7 @@
-#!/usr/bin/env ruby
-
 require 'spec_helper'
 
 describe 'Puppet::Type::Service::Provider::Rcng',
-    :unless => Puppet::Util::Platform.windows? || Puppet::Util::Platform.jruby? do
+         unless: Puppet::Util::Platform.windows? || Puppet::Util::Platform.jruby? do
   let(:provider_class) { Puppet::Type.type(:service).provider(:rcng) }
 
   before :each do
@@ -15,7 +13,7 @@ describe 'Puppet::Type::Service::Provider::Rcng',
     @provider.stubs(:initscript)
   end
 
-  describe "#enable" do
+  context "#enable" do
     it "should have an enable method" do
       expect(@provider).to respond_to(:enable)
     end