puppet 5.5.8-universal-darwin → 5.5.10-universal-darwin

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

@@ -1,14 +1,6 @@
-#! /usr/bin/env ruby
-#
-# Unit testing for the debian service provider
-#
-
 require 'spec_helper'
 
-provider_class = Puppet::Type.type(:service).provider(:debian)
-
-describe provider_class do
-
+describe Puppet::Type.type(:service).provider(:debian) do
   if Puppet.features.microsoft_windows?
     # Get a pid for $CHILD_STATUS to latch on to
     command = "cmd.exe /c \"exit 0\""
@@ -19,7 +11,7 @@ describe provider_class do
     # Create a mock resource
     @resource = stub 'resource'
 
-    @provider = provider_class.new
+    @provider = subject()
 
     # A catch all; no parameters set
     @resource.stubs(:[]).returns(nil)
@@ -43,14 +35,14 @@ describe provider_class do
     it "should be the default provider on CumulusLinux #{version}" do
       Facter.expects(:value).with(:operatingsystem).at_least_once.returns('CumulusLinux')
       Facter.expects(:value).with(:operatingsystemmajrelease).returns(version)
-      expect(provider_class.default?).to be_truthy
+      expect(described_class.default?).to be_truthy
     end
   end
 
   it "should be the default provider on Debian" do
     Facter.expects(:value).with(:operatingsystem).at_least_once.returns('Debian')
     Facter.expects(:value).with(:operatingsystemmajrelease).returns('7')
-    expect(provider_class.default?).to be_truthy
+    expect(described_class.default?).to be_truthy
   end
 
   it "should have an enabled? method" do
@@ -65,14 +57,14 @@ describe provider_class do
     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"
 
@@ -91,7 +83,7 @@ describe provider_class do
     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)
@@ -151,7 +143,7 @@ describe provider_class do
     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,11 +1,8 @@
-#! /usr/bin/env ruby
 require 'spec_helper'
 
-provider_class = Puppet::Type.type(:service).provider(:freebsd)
-
-describe provider_class do
+describe Puppet::Type.type(:service).provider(:freebsd) do
   before :each do
-    @provider = provider_class.new
+    @provider = subject()
     @provider.stubs(:initscript)
     Facter.stubs(:value).with(:osfamily).returns 'FreeBSD'
   end

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

@@ -1,12 +1,6 @@
-#!/usr/bin/env ruby
-#
-# Unit testing for the OpenBSD service provider
-
 require 'spec_helper'
 
-provider_class = Puppet::Type.type(:service).provider(:openbsd)
-
-describe provider_class, :unless => Puppet.features.microsoft_windows? do
+describe Puppet::Type.type(:service).provider(:openbsd), :unless => Puppet.features.microsoft_windows? do
   before :each do
     Puppet::Type.type(:service).stubs(:defaultprovider).returns described_class
     Facter.stubs(:value).with(:operatingsystem).returns :openbsd
@@ -15,7 +9,7 @@ describe provider_class, :unless => Puppet.features.microsoft_windows? do
     FileTest.stubs(:executable?).with('/usr/sbin/rcctl').returns true
   end
 
-  describe "#instances" do
+  context "#instances" do
     it "should have an instances method" do
       expect(described_class).to respond_to :instances
     end
@@ -28,7 +22,7 @@ describe provider_class, :unless => Puppet.features.microsoft_windows? do
     end
   end
 
-  describe "#start" do
+  context "#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.expects(:execute).with(['/bin/foo'], :failonfail => true, :override_locale => false, :squelch => false, :combine => true)
@@ -42,7 +36,7 @@ describe provider_class, :unless => Puppet.features.microsoft_windows? do
     end
   end
 
-  describe "#stop" do
+  context "#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.expects(:execute).with(['/bin/foo'], :failonfail => true, :override_locale => false, :squelch => false, :combine => true)
@@ -56,7 +50,7 @@ describe provider_class, :unless => Puppet.features.microsoft_windows? do
     end
   end
 
-  describe "#status" do
+  context "#status" 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.expects(:execute).with(['/usr/sbin/rcctl', :get, 'sshd', :status], :failonfail => true, :override_locale => false, :squelch => false, :combine => true).never
@@ -81,7 +75,7 @@ describe provider_class, :unless => Puppet.features.microsoft_windows? do
     end
   end
 
-  describe "#restart" do
+  context "#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.expects(:execute).with(['/usr/sbin/rcctl', '-f', :restart, 'sshd'], :failonfail => true, :override_locale => false, :squelch => false, :combine => true).never
@@ -104,7 +98,7 @@ describe provider_class, :unless => Puppet.features.microsoft_windows? do
     end
   end
 
-  describe "#enabled?" do
+  context "#enabled?" do
     it "should return :true if the service is enabled" do
       provider = described_class.new(Puppet::Type.type(:service).new(:name => 'sshd'))
       described_class.stubs(:rcctl).with(:get, 'sshd', :status)
@@ -120,7 +114,7 @@ describe provider_class, :unless => Puppet.features.microsoft_windows? do
     end
   end
 
-  describe "#enable" do
+  context "#enable" do
     it "should run rcctl enable to enable the service" do
       provider = described_class.new(Puppet::Type.type(:service).new(:name => 'sshd'))
       described_class.stubs(:rcctl).with(:enable, 'sshd').returns('')
@@ -138,7 +132,7 @@ describe provider_class, :unless => Puppet.features.microsoft_windows? do
     end
   end
 
-  describe "#disable" do
+  context "#disable" do
     it "should run rcctl disable to disable the service" do
       provider = described_class.new(Puppet::Type.type(:service).new(:name => 'sshd'))
       described_class.stubs(:rcctl).with(:disable, 'sshd').returns('')
@@ -147,7 +141,7 @@ describe provider_class, :unless => Puppet.features.microsoft_windows? do
     end
   end
 
-  describe "#running?" do
+  context "#running?" do
     it "should run rcctl check to check the service" do
       provider = described_class.new(Puppet::Type.type(:service).new(:name => 'sshd'))
       described_class.stubs(:rcctl).with(:check, 'sshd').returns('sshd(ok)')
@@ -170,7 +164,7 @@ describe provider_class, :unless => Puppet.features.microsoft_windows? do
     end
   end
 
-  describe "#flags" do
+  context "#flags" do
     it "should return flags when set" do
       provider = described_class.new(Puppet::Type.type(:service).new(:name => 'sshd', :flags => '-6'))
       described_class.stubs(:rcctl).with('get', 'sshd', 'flags').returns('-6')
@@ -193,7 +187,7 @@ describe provider_class, :unless => Puppet.features.microsoft_windows? do
     end
   end
 
-  describe "#flags=" do
+  context "#flags=" do
     it "should run rcctl to set flags" do
       provider = described_class.new(Puppet::Type.type(:service).new(:name => 'sshd'))
       described_class.stubs(:rcctl).with(:set, 'sshd', :flags, '-4').returns('')

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

@@ -1,20 +1,16 @@
-#!/usr/bin/env ruby
-
 require 'spec_helper'
 
-provider_class = Puppet::Type.type(:service).provider(:rcng)
-
-describe provider_class, :unless => Puppet.features.microsoft_windows? do
+describe Puppet::Type.type(:service).provider(:rcng), :unless => Puppet.features.microsoft_windows? do
   before :each do
     Puppet::Type.type(:service).stubs(:defaultprovider).returns described_class
     Facter.stubs(:value).with(:operatingsystem).returns :netbsd
     Facter.stubs(:value).with(:osfamily).returns 'NetBSD'
     described_class.stubs(:defpath).returns('/etc/rc.d')
-    @provider = provider_class.new
+    @provider = subject()
     @provider.stubs(:initscript)
   end
 
-  describe "#enable" do
+  context "#enable" do
     it "should have an enable method" do
       expect(@provider).to respond_to(:enable)
     end

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

@@ -1,19 +1,12 @@
-#! /usr/bin/env ruby
-#
-# Unit testing for the RedHat service Provider
-#
 require 'spec_helper'
 
-provider_class = Puppet::Type.type(:service).provider(:redhat)
-
-describe provider_class, :if => Puppet.features.posix? do
-
+describe Puppet::Type.type(:service).provider(:redhat), :if => Puppet.features.posix? do
   before :each do
     @class = Puppet::Type.type(:service).provider(:redhat)
     @resource = stub 'resource'
     @resource.stubs(:[]).returns(nil)
     @resource.stubs(:[]).with(:name).returns "myservice"
-    @provider = provider_class.new
+    @provider = subject()
     @resource.stubs(:provider).returns @provider
     @provider.resource = @resource
     @provider.stubs(:get).with(:hasstatus).returns false
@@ -28,7 +21,7 @@ describe provider_class, :if => Puppet.features.posix? do
   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
+      expect(described_class.default?).to be_truthy
     end
   end
 
@@ -40,7 +33,7 @@ describe provider_class, :if => Puppet.features.posix? do
   end
 
   # test self.instances
-  describe "when getting all service instances" do
+  context "when getting all service instances" do
     before :each do
       @services = ['one', 'two', 'three', 'four', 'kudzu', 'functions', 'halt', 'killall', 'single', 'linuxconf', 'boot', 'reboot']
       @not_services = ['functions', 'halt', 'killall', 'single', 'linuxconf', 'reboot', 'boot']
@@ -64,13 +57,13 @@ describe provider_class, :if => Puppet.features.posix? do
   end
 
   it "should use '--add' and 'on' when calling enable" do
-    provider_class.expects(:chkconfig).with("--add", @resource[:name])
-    provider_class.expects(:chkconfig).with(@resource[:name], :on)
+    described_class.expects(:chkconfig).with("--add", @resource[:name])
+    described_class.expects(:chkconfig).with(@resource[:name], :on)
     @provider.enable
   end
 
   it "(#15797) should explicitly turn off the service in all run levels" do
-    provider_class.expects(:chkconfig).with("--level", "0123456", @resource[:name], :off)
+    described_class.expects(:chkconfig).with("--level", "0123456", @resource[:name], :off)
     @provider.disable
   end
 
@@ -78,28 +71,28 @@ describe provider_class, :if => Puppet.features.posix? do
     expect(@provider).to respond_to(:enabled?)
   end
 
-  describe "when checking enabled? on Suse" do
+  context "when checking enabled? on Suse" do
     before :each do
       Facter.expects(:value).with(:osfamily).returns 'Suse'
     end
 
     it "should check for on" do
-      provider_class.stubs(:chkconfig).with(@resource[:name]).returns "#{@resource[:name]}  on"
+      described_class.stubs(:chkconfig).with(@resource[:name]).returns "#{@resource[:name]}  on"
       expect(@provider.enabled?).to eq(:true)
     end
 
     it "should check for B" do
-      provider_class.stubs(:chkconfig).with(@resource[:name]).returns "#{@resource[:name]}  B"
+      described_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"
+      described_class.stubs(:chkconfig).with(@resource[:name]).returns "#{@resource[:name]}  off"
       expect(@provider.enabled?).to eq(:false)
     end
 
     it "should check for unknown service" do
-      provider_class.stubs(:chkconfig).with(@resource[:name]).returns "#{@resource[:name]}: unknown service"
+      described_class.stubs(:chkconfig).with(@resource[:name]).returns "#{@resource[:name]}: unknown service"
       expect(@provider.enabled?).to eq(:false)
     end
   end
@@ -132,20 +125,23 @@ describe provider_class, :if => Puppet.features.posix? do
     end
   end
 
-  describe "when checking status" do
-    describe "when hasstatus is :true" do
+  context "when checking status" do
+    context "when hasstatus is :true" do
       before :each do
         @resource.stubs(:[]).with(:hasstatus).returns :true
       end
+
       it "should execute the service script with fail_on_failure false" do
         @provider.expects(:texecute).with(:status, ['/sbin/service', 'myservice', 'status'], false)
         @provider.status
       end
+
       it "should consider the process running if the command returns 0" do
         @provider.expects(:texecute).with(:status, ['/sbin/service', 'myservice', 'status'], false)
         $CHILD_STATUS.stubs(:exitstatus).returns(0)
         expect(@provider.status).to eq(:running)
       end
+
       [-10,-1,1,10].each { |ec|
         it "should consider the process stopped if the command returns something non-0" do
           @provider.expects(:texecute).with(:status, ['/sbin/service', 'myservice', 'status'], false)
@@ -154,11 +150,13 @@ describe provider_class, :if => Puppet.features.posix? do
         end
       }
     end
-    describe "when hasstatus is not :true" do
+
+    context "when hasstatus is not :true" do
       it "should consider the service :running if it has a pid" do
         @provider.expects(:getpid).returns "1234"
         expect(@provider.status).to eq(:running)
       end
+
       it "should consider the service :stopped if it doesn't have a pid" do
         @provider.expects(:getpid).returns nil
         expect(@provider.status).to eq(:stopped)
@@ -166,7 +164,7 @@ describe provider_class, :if => Puppet.features.posix? do
     end
   end
 
-  describe "when restarting and hasrestart is not :true" do
+  context "when restarting and hasrestart is not :true" do
     it "should stop and restart the process with the server script" do
       @provider.expects(:texecute).with(:stop,  ['/sbin/service', 'myservice', 'stop'],  true)
       @provider.expects(:texecute).with(:start, ['/sbin/service', 'myservice', 'start'], true)

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

@@ -1,20 +1,11 @@
-#! /usr/bin/env ruby
-#
-# Unit testing for the Runit service Provider
-#
-# author Brice Figureau
-#
 require 'spec_helper'
 
-provider_class = Puppet::Type.type(:service).provider(:runit)
-
-describe provider_class do
-
+describe Puppet::Type.type(:service).provider(:runit) do
   before(:each) do
     # Create a mock resource
     @resource = stub 'resource'
 
-    @provider = provider_class.new
+    @provider = subject()
     @servicedir = "/etc/service"
     @provider.servicedir=@servicedir
     @daemondir = "/etc/sv"
@@ -63,7 +54,7 @@ describe provider_class do
     expect(@provider).to respond_to(:disable)
   end
 
-  describe "when starting" do
+  context "when starting" do
     it "should enable the service if it is not enabled" do
       @provider.stubs(:sv)
 
@@ -81,21 +72,21 @@ describe provider_class do
     end
   end
 
-  describe "when stopping" do
+  context "when stopping" do
     it "should execute external command 'sv stop /etc/service/myservice'" do
       @provider.expects(:sv).with("stop", "/etc/service/myservice")
       @provider.stop
     end
   end
 
-  describe "when restarting" do
+  context "when restarting" do
     it "should call 'sv restart /etc/service/myservice'" do
       @provider.expects(:sv).with("restart","/etc/service/myservice")
       @provider.restart
     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")
@@ -105,7 +96,7 @@ describe provider_class do
     end
   end
 
-  describe "when disabling" do
+  context "when disabling" do
     it "should remove the '/etc/service/myservice' symlink" do
       path = File.join(@servicedir,"myservice")
 #      mocked_file = mock(path, :symlink? => true)
@@ -116,14 +107,14 @@ describe provider_class do
     end
   end
 
-  describe "when checking status" do
+  context "when checking status" do
     it "should call the external command 'sv status /etc/sv/myservice'" do
       @provider.expects(:sv).with('status',File.join(@daemondir,"myservice"))
       @provider.status
     end
   end
 
-  describe "when checking status" do
+  context "when checking status" do
     it "and sv status fails, properly raise a Puppet::Error" do
       @provider.expects(:sv).with('status',File.join(@daemondir,"myservice")).raises(Puppet::ExecutionFailure, "fail: /etc/sv/myservice: file not found")
       expect { @provider.status }.to raise_error(Puppet::Error, 'Could not get status for service Service[myservice]: fail: /etc/sv/myservice: file not found')
@@ -141,5 +132,4 @@ describe provider_class do
       expect(@provider.status).to eq(:stopped)
     end
   end
-
 end

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

@@ -1,20 +1,11 @@
-#! /usr/bin/env ruby
-#
-# Unit testing for the SMF service Provider
-#
-# author Dominic Cleal
-#
 require 'spec_helper'
 
-provider_class = Puppet::Type.type(:service).provider(:smf)
-
-describe provider_class, :if => Puppet.features.posix? do
-
+describe Puppet::Type.type(:service).provider(:smf), :if => Puppet.features.posix? do
   before(:each) do
     # Create a mock resource
     @resource = Puppet::Type.type(:service).new(
       :name => "/system/myservice", :ensure => :running, :enable => :true)
-    @provider = provider_class.new(@resource)
+    @provider = described_class.new(@resource)
 
     FileTest.stubs(:file?).with('/usr/sbin/svcadm').returns true
     FileTest.stubs(:executable?).with('/usr/sbin/svcadm').returns true
@@ -25,14 +16,14 @@ describe provider_class, :if => Puppet.features.posix? do
     Facter.stubs(:value).with(:operatingsystemrelease).returns '11.2'
   end
 
-  describe ".instances" do
+  context ".instances" do
     it "should have an instances method" do
-      expect(provider_class).to respond_to :instances
+      expect(described_class).to respond_to :instances
     end
 
     it "should get a list of services (excluding legacy)" do
-      provider_class.expects(:svcs).with('-H', '-o', 'state,fmri').returns File.read(my_fixture('svcs.out'))
-      instances = provider_class.instances.map { |p| {:name => p.get(:name), :ensure => p.get(:ensure)} }
+      described_class.expects(:svcs).with('-H', '-o', 'state,fmri').returns File.read(my_fixture('svcs.out'))
+      instances = described_class.instances.map { |p| {:name => p.get(:name), :ensure => p.get(:ensure)} }
       # we dont manage legacy
       expect(instances.size).to eq(3)
       expect(instances[0]).to eq({:name => 'svc:/system/svc/restarter:default', :ensure => :running })
@@ -70,6 +61,10 @@ describe provider_class, :if => Puppet.features.posix? do
   end
 
   describe "when checking status" do
+    before(:each) do
+      @provider.stubs(:complete_service?).returns(true)
+    end
+
     it "should call the external command 'svcs /system/myservice' once" do
       @provider.expects(:svcs).with('-H', '-o', 'state,nstate', "/system/myservice").returns("online\t-")
       @provider.status
@@ -78,6 +73,11 @@ describe provider_class, :if => Puppet.features.posix? do
       @provider.stubs(:svcs).raises(Puppet::ExecutionFailure.new("no svc found"))
       expect(@provider.status).to eq(:stopped)
     end
+    it "should return stopped for an incomplete service on Solaris 11" do
+      Facter.stubs(:value).with(:operatingsystemrelease).returns('11.3')
+      @provider.stubs(:complete_service?).returns(false)
+      expect(@provider.status).to eq(:stopped)
+    end
     it "should return running if online in svcs output" do
       @provider.stubs(:svcs).returns("online\t-")
       expect(@provider.status).to eq(:running)
@@ -104,7 +104,7 @@ describe provider_class, :if => Puppet.features.posix? do
     end
   end
 
-  describe "when starting" do
+  context "when starting" do
     it "should enable the service if it is not enabled" do
       @provider.expects(:status).returns :stopped
       @provider.expects(:texecute).with(:start, ['/usr/sbin/svcadm', :enable, '-rs', '/system/myservice'], true)
@@ -141,14 +141,15 @@ describe provider_class, :if => Puppet.features.posix? do
     end
   end
 
-  describe "when starting a service with a manifest" do
+  context "when starting a service with a manifest" do
     before(:each) do
       @resource = Puppet::Type.type(:service).new(:name => "/system/myservice", :ensure => :running, :enable => :true, :manifest => "/tmp/myservice.xml")
-      @provider = provider_class.new(@resource)
+      @provider = described_class.new(@resource)
       $CHILD_STATUS.stubs(:exitstatus).returns(1)
     end
 
     it "should import the manifest if service is missing" do
+      @provider.stubs(:complete_service?).returns(true)
       @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)
@@ -164,7 +165,7 @@ describe provider_class, :if => Puppet.features.posix? do
     end
   end
 
-  describe "when stopping" do
+  context "when stopping" do
     it "should execute external command 'svcadm disable /system/myservice'" do
       @provider.expects(:texecute).with(:stop, ["/usr/sbin/svcadm", :disable, '-s', "/system/myservice"], true)
       @provider.expects(:wait).with('offline', 'disabled', 'uninitialized')
@@ -178,8 +179,7 @@ describe provider_class, :if => Puppet.features.posix? do
     end
   end
 
-  describe "when restarting" do
-
+  context "when restarting" do
     it "should error if timeout occurs while restarting the service" do
       @provider.expects(:texecute).with(:restart, ["/usr/sbin/svcadm", :restart, '-s', "/system/myservice"], true)
       Timeout.expects(:timeout).with(60).raises(Timeout::Error)
@@ -212,6 +212,67 @@ describe provider_class, :if => Puppet.features.posix? do
         @provider.restart
       end
     end
+  end
+
+  describe '#service_fmri' do
+    it 'raises a Puppet::Error if the service resource matches multiple FMRIs' do
+      @provider.stubs(:svcs).with('-l', @provider.resource[:name]).returns(File.read(my_fixture('svcs_multiple_fmris.out')))
+
+      expect { @provider.service_fmri }.to raise_error do |error|
+        expect(error).to be_a(Puppet::Error)
+        expect(error.message).to match(@provider.resource[:name])
+        expect(error.message).to match('multiple')
+
+        matched_fmris = ["svc:/application/tstapp:one", "svc:/application/tstapp:two"]
+        expect(error.message).to match(matched_fmris.join(', '))
+      end
+    end
+
+    it 'raises a Puppet:ExecutionFailure if svcs fails' do
+      @provider.stubs(:svcs).with('-l', @provider.resource[:name]).raises(
+        Puppet::ExecutionFailure, 'svcs failed!'
+      )
+
+      expect { @provider.service_fmri }.to raise_error do |error|
+        expect(error).to be_a(Puppet::ExecutionFailure)
+        expect(error.message).to match('svcs failed!')
+      end
+    end
+
+    it "returns the service resource's fmri and memoizes it" do
+      @provider.stubs(:svcs).with('-l', @provider.resource[:name]).returns(File.read(my_fixture('svcs_fmri.out')))
+
+      expected_fmri = 'svc:/application/tstapp:default'
 
+      expect(@provider.service_fmri).to eql(expected_fmri)
+      expect(@provider.instance_variable_get(:@fmri)).to eql(expected_fmri)
+    end
+  end
+
+  describe '#complete_service?' do
+    let(:fmri) { 'service_fmri' }
+
+    before(:each) do
+      @provider.stubs(:service_fmri).returns(fmri)
+    end
+
+    it 'should raise a Puppet::Error if it is called on an older Solaris machine' do
+      Facter.stubs(:value).with(:operatingsystemrelease).returns('10.0')
+
+      expect { @provider.complete_service? }.to raise_error(Puppet::Error)
+    end
+
+    it 'should return false for an incomplete service' do
+      @provider.stubs(:svccfg).with('-s', fmri, 'listprop', 'general/complete').returns("")
+      expect(@provider.complete_service?).to be false
+    end
+
+    it 'should return true for a complete service' do
+      @provider.stubs(:svccfg)
+        .with('-s', fmri, 'listprop', 'general/complete')
+        .returns("general/complete astring")
+
+      expect(@provider.complete_service?).to be true
+    end
   end
 end