smf_recursive_dependencies: Allow solaris services to start recursively.

Svcadm enable -s service only starts the named service. This commit adds the
recursive option to start service dependencies recursively.  The resulting command
"svcadm enable -sr service"  will try to start service dependencies before
starting the named service.

Signed-off-by: markgibbons <mark.gibbons@nordstrom.com>

2 files changed, 33 insertions, 0 deletions

diff --git a/spec/unit/provider/service/solaris_smf_service_spec.rb b/spec/unit/provider/service/solaris_smf_service_spec.rb
index c6835bed64..98b0ed34e0 100644
--- a/spec/unit/provider/service/solaris_smf_service_spec.rb
+++ b/spec/unit/provider/service/solaris_smf_service_spec.rb
@@ -198,6 +198,23 @@ describe Chef::Provider::Service::Solaris do
       end
     end
 
+    describe "when enabling the service recursively" do
+      before(:each) do
+        @new_resource.recursive(true)
+        @provider.current_resource = @current_resource
+      end
+
+      it "should call svcadm enable -sr chef" do
+        expect(@provider).to receive(:shell_out!).with("/bin/svcs", "-l", "chef", { :returns => [0, 1] }).and_return(@enabled_svc_status)
+        expect(@provider).not_to receive(:shell_out!).with("/usr/sbin/svcadm", "clear", @current_resource.service_name)
+        expect(@provider).to receive(:shell_out!).with("/usr/sbin/svcadm", "enable", "-sr", @current_resource.service_name).and_return(@success)
+        @provider.load_current_resource
+
+        expect(@provider.enable_service).to be_truthy
+        expect(@current_resource.enabled).to be_truthy
+      end
+    end
+
     describe "when disabling the service" do
       before(:each) do
         @provider.current_resource = @current_resource
diff --git a/spec/unit/resource/service_spec.rb b/spec/unit/resource/service_spec.rb
index 7aadc55532..4114a8ec3e 100644
--- a/spec/unit/resource/service_spec.rb
+++ b/spec/unit/resource/service_spec.rb
@@ -124,6 +124,22 @@ describe Chef::Resource::Service do
     end.to raise_error(ArgumentError)
   end
 
+  it "should accept true for recursive" do
+    @resource.recursive true
+    expect(@resource.send("recursive")).to eql(true)
+  end
+
+  it "should accept false for recursive" do
+    @resource.recursive false
+    expect(@resource.send("recursive")).to eql(false)
+  end
+
+  it "should not accept a string for recursive" do
+    expect do
+      @resource.recursive "poop"
+    end.to raise_error(ArgumentError)
+  end
+
   %w{enabled running}.each do |attrib|
     it "should accept true for #{attrib}" do
       @resource.send(attrib, true)