chef 10.12.0 → 10.14.0.beta.1

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

@@ -0,0 +1,254 @@
+#
+# Author:: AJ Christensen (<aj@hjksolutions.com>)
+# Copyright:: Copyright (c) 2008 HJK Solutions, LLC
+# License:: Apache License, Version 2.0
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+# 
+#     http://www.apache.org/licenses/LICENSE-2.0
+# 
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+require 'spec_helper'
+
+describe Chef::Provider::Service::Debian, "load_current_resource" do
+  before(:each) do
+    @node = Chef::Node.new
+    @node[:command] = {:ps => 'fuuuu'}
+    @events = Chef::EventDispatch::Dispatcher.new
+    @run_context = Chef::RunContext.new(@node, {}, @events)
+
+    @new_resource = Chef::Resource::Service.new("chef")
+
+    @current_resource = Chef::Resource::Service.new("chef")
+
+    @provider = Chef::Provider::Service::Debian.new(@new_resource, @run_context)
+    @provider.current_resource = @current_resource
+
+    @pid, @stdin, @stdout, @stderr = nil, nil, nil, nil
+
+  end
+
+  it "ensures /usr/sbin/update-rc.d is available" do
+    File.should_receive(:exists?).with("/usr/sbin/update-rc.d").and_return(false)
+    @provider.define_resource_requirements
+    lambda { @provider.process_resource_requirements } .should raise_error(Chef::Exceptions::Service)
+  end
+
+  describe "when update-rc.d shows the init script linked to rc*.d/" do
+    before do
+      @provider.stub!(:assert_update_rcd_available)
+
+      result=<<-UPDATE_RC_D_SUCCESS
+Removing any system startup links for /etc/init.d/chef ...
+  /etc/rc0.d/K20chef
+  /etc/rc1.d/K20chef
+  /etc/rc2.d/S20chef
+  /etc/rc3.d/S20chef
+  /etc/rc4.d/S20chef
+  /etc/rc5.d/S20chef
+  /etc/rc6.d/K20chef
+  UPDATE_RC_D_SUCCESS
+      @stdout = StringIO.new(result)
+      @stderr = StringIO.new
+      @status = mock("Status", :exitstatus => 0, :stdout => @stdout)
+      @provider.stub!(:shell_out!).and_return(@status)
+      @provider.stub!(:popen4).and_yield(@pid, @stdin, @stdout, @stderr).and_return(@status)
+    end
+
+    it "says the service is enabled" do
+      @provider.service_currently_enabled?(@provider.get_priority).should be_true
+    end
+
+    it "stores the 'enabled' state" do
+      Chef::Resource::Service.stub!(:new).and_return(@current_resource)
+      @provider.load_current_resource.should equal(@current_resource)
+      @current_resource.enabled.should be_true
+    end
+  end
+
+  {"Debian/Lenny and older" => {
+      "linked" => {
+        "stdout" => " Removing any system startup links for /etc/init.d/chef ...
+   /etc/rc0.d/K20chef
+   /etc/rc1.d/K20chef
+   /etc/rc2.d/S20chef
+   /etc/rc3.d/S20chef
+   /etc/rc4.d/S20chef
+   /etc/rc5.d/S20chef
+   /etc/rc6.d/K20chef",
+        "stderr" => ""
+      },
+      "not linked" => {
+        "stdout" => " Removing any system startup links for /etc/init.d/chef ...",
+        "stderr" => ""
+      },
+    },
+    "Debian/Squeeze and earlier" => {
+      "linked" => {
+        "stdout" => "update-rc.d: using dependency based boot sequencing",
+        "stderr" => "insserv: remove service /etc/init.d/../rc0.d/K20chef-client
+insserv: remove service /etc/init.d/../rc1.d/K20chef-client
+insserv: remove service /etc/init.d/../rc2.d/S20chef-client
+insserv: remove service /etc/init.d/../rc3.d/S20chef-client
+insserv: remove service /etc/init.d/../rc4.d/S20chef-client
+insserv: remove service /etc/init.d/../rc5.d/S20chef-client
+insserv: remove service /etc/init.d/../rc6.d/K20chef-client
+insserv: dryrun, not creating .depend.boot, .depend.start, and .depend.stop"
+      },
+      "not linked" => {
+        "stdout" => "update-rc.d: using dependency based boot sequencing",
+        "stderr" => ""
+      }
+    }
+  }.each do |model, streams|
+    describe "when update-rc.d shows the init script linked to rc*.d/" do
+      before do
+        @provider.stub!(:assert_update_rcd_available)
+
+        @stdout = StringIO.new(streams["linked"]["stdout"])
+        @stderr = StringIO.new(streams["linked"]["stderr"])
+        @status = mock("Status", :exitstatus => 0, :stdout => @stdout)
+        @provider.stub!(:shell_out!).and_return(@status)
+        @provider.stub!(:popen4).and_yield(@pid, @stdin, @stdout, @stderr).and_return(@status)
+      end
+
+      it "says the service is enabled" do
+        @provider.service_currently_enabled?(@provider.get_priority).should be_true
+      end
+
+      it "stores the 'enabled' state" do
+        Chef::Resource::Service.stub!(:new).and_return(@current_resource)
+        @provider.load_current_resource.should equal(@current_resource)
+        @current_resource.enabled.should be_true
+      end
+
+      it "stores the start/stop priorities of the service" do
+        @provider.load_current_resource
+        expected_priorities = {"6"=>[:stop, "20"],
+          "0"=>[:stop, "20"],
+          "1"=>[:stop, "20"],
+          "2"=>[:start, "20"],
+          "3"=>[:start, "20"],
+          "4"=>[:start, "20"],
+          "5"=>[:start, "20"]}
+        @provider.current_resource.priority.should == expected_priorities
+      end
+    end
+
+    describe "when using squeeze/earlier and update-rc.d shows the init script isn't linked to rc*.d" do
+      before do
+        @provider.stub!(:assert_update_rcd_available)
+        @stdout = StringIO.new(streams["not linked"]["stdout"])
+        @stderr = StringIO.new(streams["not linked"]["stderr"])
+        @status = mock("Status", :exitstatus => 0, :stdout => @stdout)
+        @provider.stub!(:shell_out!).and_return(@status)
+        @provider.stub!(:popen4).and_yield(@pid, @stdin, @stdout, @stderr).and_return(@status)
+      end
+
+      it "says the service is disabled" do
+        @provider.service_currently_enabled?(@provider.get_priority).should be_false
+      end
+
+      it "stores the 'disabled' state" do
+        Chef::Resource::Service.stub!(:new).and_return(@current_resource)
+        @provider.load_current_resource.should equal(@current_resource)
+        @current_resource.enabled.should be_false
+      end
+    end
+  end
+
+  describe "when update-rc.d shows the init script isn't linked to rc*.d" do
+    before do
+      @provider.stub!(:assert_update_rcd_available)
+      @status = mock("Status", :exitstatus => 0)
+      @stdout = StringIO.new(" Removing any system startup links for /etc/init.d/chef ...")
+      @stderr = StringIO.new
+      @status = mock("Status", :exitstatus => 0, :stdout => @stdout)
+      @provider.stub!(:shell_out!).and_return(@status)
+      @provider.stub!(:popen4).and_yield(@pid, @stdin, @stdout, @stderr).and_return(@status)
+    end
+
+    it "says the service is disabled" do
+      @provider.service_currently_enabled?(@provider.get_priority).should be_false
+    end
+
+    it "stores the 'disabled' state" do
+      Chef::Resource::Service.stub!(:new).and_return(@current_resource)
+      @provider.load_current_resource.should equal(@current_resource)
+      @current_resource.enabled.should be_false
+    end
+  end
+
+  describe "when update-rc.d fails" do
+    before do
+      @status = mock("Status", :exitstatus => -1)
+      @provider.stub!(:popen4).and_return(@status)
+    end
+
+    it "raises an error" do
+      @provider.define_resource_requirements
+      lambda { @provider.process_resource_requirements }.should raise_error(Chef::Exceptions::Service)
+    end
+  end
+
+  describe "when enabling a service without priority" do
+    it "should call update-rc.d 'service_name' defaults" do
+      @provider.should_receive(:run_command).with({:command => "/usr/sbin/update-rc.d -f #{@new_resource.service_name} remove"})
+      @provider.should_receive(:run_command).with({:command => "/usr/sbin/update-rc.d #{@new_resource.service_name} defaults"})
+      @provider.enable_service()
+    end
+  end
+
+  describe "when enabling a service with simple priority" do
+    before do
+      @new_resource.priority(75)
+    end
+
+    it "should call update-rc.d 'service_name' defaults" do
+      @provider.should_receive(:run_command).with({:command => "/usr/sbin/update-rc.d -f #{@new_resource.service_name} remove"})
+      @provider.should_receive(:run_command).with({:command => "/usr/sbin/update-rc.d #{@new_resource.service_name} defaults 75 25"})
+      @provider.enable_service()
+    end
+  end
+
+  describe "when enabling a service with complex priorities" do
+    before do
+      @new_resource.priority(2 => [:start, 20], 3 => [:stop, 55])
+    end
+
+    it "should call update-rc.d 'service_name' defaults" do
+      @provider.should_receive(:run_command).with({:command => "/usr/sbin/update-rc.d -f #{@new_resource.service_name} remove"})
+      @provider.should_receive(:run_command).with({:command => "/usr/sbin/update-rc.d #{@new_resource.service_name} start 20 2 . stop 55 3 . "})
+      @provider.enable_service()
+    end
+  end
+
+  describe "when disabling a service without a priority" do
+
+    it "should call update-rc.d -f 'service_name' remove + stop with a default priority" do
+      @provider.should_receive(:run_command).with({:command => "/usr/sbin/update-rc.d -f #{@new_resource.service_name} remove"})
+      @provider.should_receive(:run_command).with({:command => "/usr/sbin/update-rc.d -f #{@new_resource.service_name} stop 80 2 3 4 5 ."})
+      @provider.disable_service()
+    end
+  end
+
+  describe "when disabling a service with simple priority" do
+    before do
+      @new_resource.priority(75)
+    end
+
+    it "should call update-rc.d -f 'service_name' remove + stop with a specified priority" do
+      @provider.should_receive(:run_command).with({:command => "/usr/sbin/update-rc.d -f #{@new_resource.service_name} remove"})
+      @provider.should_receive(:run_command).with({:command => "/usr/sbin/update-rc.d -f #{@new_resource.service_name} stop #{100 - @new_resource.priority} 2 3 4 5 ."})
+      @provider.disable_service()
+    end
+  end
+end

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

@@ -0,0 +1,351 @@
+#
+# Author:: Bryan McLellan (btm@loftninjas.org)
+# Copyright:: Copyright (c) 2009 Bryan McLellan
+# License:: Apache License, Version 2.0
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+# 
+#     http://www.apache.org/licenses/LICENSE-2.0
+# 
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+require 'spec_helper'
+
+describe Chef::Provider::Service::Freebsd do
+  before do
+    @node = Chef::Node.new
+    @node[:command] = {:ps => "ps -ax"}
+    @events = Chef::EventDispatch::Dispatcher.new
+    @run_context = Chef::RunContext.new(@node, {}, @events)
+
+    @new_resource = Chef::Resource::Service.new("apache22")
+    @new_resource.pattern("httpd")
+    @new_resource.supports({:status => false})
+
+    @current_resource = Chef::Resource::Service.new("apache22")
+
+    @provider = Chef::Provider::Service::Freebsd.new(@new_resource,@run_context)
+    Chef::Resource::Service.stub!(:new).and_return(@current_resource)
+  end
+
+  describe "load_current_resource" do
+    before(:each) do
+      @stdout = StringIO.new(<<-PS_SAMPLE)
+413  ??  Ss     0:02.51 /usr/sbin/syslogd -s
+539  ??  Is     0:00.14 /usr/sbin/sshd
+545  ??  Ss     0:17.53 sendmail: accepting connections (sendmail)
+PS_SAMPLE
+      @status = mock(:stdout => @stdout, :exitstatus => 0)
+      @provider.stub!(:shell_out!).with(@node[:command][:ps]).and_return(@status)
+
+      ::File.stub!(:exists?).and_return(false)
+      ::File.stub!(:exists?).with("/usr/local/etc/rc.d/#{@new_resource.service_name}").and_return(true)
+      @lines = mock("lines")
+      @lines.stub!(:each).and_yield("sshd_enable=\"YES\"").
+                          and_yield("#{@new_resource.name}_enable=\"YES\"")
+      ::File.stub!(:open).and_return(@lines)
+
+      @rc_with_name = StringIO.new(<<-RC_SAMPLE)
+name="apache22"
+rcvar=`set_rcvar`
+RC_SAMPLE
+      ::File.stub!(:open).with("/usr/local/etc/rc.d/#{@new_resource.service_name}").and_return(@rc_with_name)
+      @provider.stub(:service_enable_variable_name).and_return nil
+
+    end
+
+    it "should create a current resource with the name of the new resource" do
+      Chef::Resource::Service.should_receive(:new).and_return(@current_resource)
+      @provider.load_current_resource
+    end
+
+    it "should set the current resources service name to the new resources service name" do
+      @provider.load_current_resource
+      @current_resource.service_name.should == @new_resource.service_name
+    end
+
+    it "should not raise an exception if the rcscript have a name variable" do
+      @provider.load_current_resource
+      lambda { @provider.service_enable_variable_name }.should_not raise_error(Chef::Exceptions::Service)
+    end
+
+    describe "when the service supports status" do
+      before do
+        @new_resource.supports({:status => true})
+      end
+
+      it "should run '/etc/init.d/service_name status'" do
+        @provider.should_receive(:shell_out).with("/usr/local/etc/rc.d/#{@current_resource.service_name} status").and_return(@status)
+        @provider.load_current_resource
+      end
+
+      it "should set running to true if the the status command returns 0" do
+        @provider.should_receive(:shell_out).with("/usr/local/etc/rc.d/#{@current_resource.service_name} status").and_return(@status)
+        @current_resource.should_receive(:running).with(true)
+        @provider.load_current_resource
+      end
+
+      it "should set running to false if the status command returns anything except 0" do
+        @provider.should_receive(:shell_out).with("/usr/local/etc/rc.d/#{@current_resource.service_name} status").and_raise(Mixlib::ShellOut::ShellCommandFailed)
+        @current_resource.should_receive(:running).with(false)
+        @provider.load_current_resource
+       # @provider.current_resource.running.should be_false
+      end
+    end
+
+    describe "when a status command has been specified" do
+      before do
+        @new_resource.status_command("/bin/chefhasmonkeypants status")
+      end
+
+      it "should run the services status command if one has been specified" do
+        @provider.should_receive(:shell_out).with("/bin/chefhasmonkeypants status").and_return(@status)
+        @provider.load_current_resource
+      end
+
+    end
+
+    it "should raise error if the node has a nil ps attribute and no other means to get status" do
+      @node[:command] = {:ps => nil}
+      @provider.define_resource_requirements
+      lambda { @provider.process_resource_requirements }.should raise_error(Chef::Exceptions::Service)
+    end
+
+    it "should raise error if the node has an empty ps attribute and no other means to get status" do
+      @node[:command] = {:ps => ""}
+      @provider.define_resource_requirements
+      lambda { @provider.process_resource_requirements }.should raise_error(Chef::Exceptions::Service)
+    end
+    
+    describe "when executing assertions" do
+      it "should verify that /etc/rc.conf exists" do
+        ::File.should_receive(:exists?).with("/etc/rc.conf")
+        @provider.stub!(:service_enable_variable_name).and_return("#{@current_resource.service_name}_enable")
+        @provider.load_current_resource
+      end
+
+      it "should raise an exception when init script is not found" do
+        ::File.stub!(:exists?).and_return(false)
+        @provider.load_current_resource
+        @provider.define_resource_requirements 
+        @provider.instance_variable_get("@rcd_script_found").should be_false
+        lambda { @provider.process_resource_requirements }.should raise_error(Chef::Exceptions::Service)
+      end
+
+      it "update state when current resource enabled state could not be determined" do
+        ::File.should_receive(:exists?).with("/etc/rc.conf").and_return false
+        @provider.load_current_resource
+        @provider.instance_variable_get("@enabled_state_found").should be_false
+      end 
+
+      it "update state when current resource enabled state could be determined" do
+        ::File.should_receive(:exists?).with("/etc/rc.conf").and_return  true
+        @provider.load_current_resource
+        @provider.instance_variable_get("@enabled_state_found").should be_false
+        @provider.instance_variable_get("@rcd_script_found").should be_true
+        @provider.define_resource_requirements 
+        lambda { @provider.process_resource_requirements }.should raise_error(Chef::Exceptions::Service,
+          "Could not find the service name in /usr/local/etc/rc.d/#{@current_resource.service_name} and rcvar")
+      end 
+
+      it "should throw an exception if service line is missing from rc.d script" do
+          pending "not implemented" do
+            false.should be_true
+          end
+      end
+
+    end
+
+    describe "when we have a 'ps' attribute" do
+      before do
+        @node[:command] = {:ps => "ps -ax"}
+      end
+
+      it "should shell_out! the node's ps command" do
+        @provider.should_receive(:shell_out!).with(@node[:command][:ps]).and_return(@status)
+        @provider.load_current_resource
+      end
+
+      it "should read stdout of the ps command" do
+        @provider.stub!(:shell_out!).and_return(@status)
+        @stdout.should_receive(:each_line).and_return(true)
+        @provider.load_current_resource
+      end
+
+      it "should set running to true if the regex matches the output" do
+        @stdout.stub!(:each_line).and_yield("555  ??  Ss     0:05.16 /usr/sbin/cron -s").
+                                  and_yield(" 9881  ??  Ss     0:06.67 /usr/local/sbin/httpd -DNOHTTPACCEPT")
+        @provider.load_current_resource 
+        @current_resource.running.should be_true
+      end
+
+      it "should set running to false if the regex doesn't match" do
+        @provider.stub!(:shell_out!).and_return(@status)
+        @provider.load_current_resource
+        @current_resource.running.should be_false
+      end
+
+      it "should raise an exception if ps fails" do
+        @provider.stub!(:shell_out!).and_raise(Mixlib::ShellOut::ShellCommandFailed)
+        @provider.load_current_resource
+        @provider.define_resource_requirements
+        lambda { @provider.process_resource_requirements }.should raise_error(Chef::Exceptions::Service)
+      end
+    end
+
+    it "should return the current resource" do
+      @provider.load_current_resource.should eql(@current_resource)
+    end
+
+    describe "when starting the service" do
+      it "should call the start command if one is specified" do
+        @new_resource.start_command("/etc/rc.d/chef startyousillysally")
+        @provider.should_receive(:shell_out!).with("/etc/rc.d/chef startyousillysally")
+        @provider.load_current_resource
+        @provider.start_service()
+      end
+
+      it "should call '/usr/local/etc/rc.d/service_name faststart' if no start command is specified" do
+        @provider.should_receive(:shell_out!).with("/usr/local/etc/rc.d/#{@new_resource.service_name} faststart")
+        @provider.load_current_resource
+        @provider.start_service()
+      end
+    end
+
+    describe Chef::Provider::Service::Init, "stop_service" do
+      it "should call the stop command if one is specified" do
+        @new_resource.stop_command("/etc/init.d/chef itoldyoutostop")
+        @provider.should_receive(:shell_out!).with("/etc/init.d/chef itoldyoutostop")
+        @provider.load_current_resource
+        @provider.stop_service()
+      end
+
+      it "should call '/usr/local/etc/rc.d/service_name faststop' if no stop command is specified" do
+        @provider.should_receive(:shell_out!).with("/usr/local/etc/rc.d/#{@new_resource.service_name} faststop")
+        @provider.load_current_resource
+        @provider.stop_service()
+      end
+    end
+
+    describe "when restarting a service" do
+      it "should call 'restart' on the service_name if the resource supports it" do
+        @new_resource.supports({:restart => true})
+        @provider.should_receive(:shell_out!).with("/usr/local/etc/rc.d/#{@new_resource.service_name} fastrestart")
+        @provider.load_current_resource
+        @provider.restart_service()
+      end
+
+      it "should call the restart_command if one has been specified" do
+        @new_resource.restart_command("/etc/init.d/chef restartinafire")
+        @provider.should_receive(:shell_out!).with("/etc/init.d/chef restartinafire")
+        @provider.load_current_resource
+        @provider.restart_service()
+      end
+    end
+
+    describe "when the rcscript does not have a name variable" do
+      before do
+        @rc_without_name = StringIO.new(<<-RC_SAMPLE)
+rcvar=`set_rcvar`
+RC_SAMPLE
+        ::File.stub!(:open).with("/usr/local/etc/rc.d/#{@current_resource.service_name}").and_return(@rc_with_noname)
+        @provider.current_resource = @current_resource
+      end
+
+      describe "when rcvar returns foobar_enable" do
+        before do
+          @rcvar_stdout = <<RCVAR_SAMPLE
+# apache22
+#
+# #{@current_resource.service_name}_enable="YES"
+#   (default: "")
+RCVAR_SAMPLE
+          @status = mock(:stdout => @rcvar_stdout, :exitstatus => 0)
+          @provider.stub!(:shell_out!).with("/usr/local/etc/rc.d/#{@current_resource.service_name} rcvar").and_return(@status)
+        end
+
+        it "should get the service name from rcvar if the rcscript does not have a name variable" do
+          @provider.load_current_resource
+          @provider.unstub!(:service_enable_variable_name)
+          @provider.service_enable_variable_name.should == "#{@current_resource.service_name}_enable"
+        end
+
+        it "should not raise an exception if the rcscript does not have a name variable" do
+          @provider.load_current_resource
+          lambda { @provider.service_enable_variable_name }.should_not raise_error(Chef::Exceptions::Service)
+        end
+      end
+
+      describe "when rcvar does not return foobar_enable" do
+        before do
+          @rcvar_stdout = <<RCVAR_SAMPLE
+# service_with_noname
+#
+RCVAR_SAMPLE
+          @status = mock(:stdout => @rcvar_stdout, :exitstatus => 0)
+          @provider.stub!(:shell_out!).with("/usr/local/etc/rc.d/#{@current_resource.service_name} rcvar").and_return(@status)
+        end
+
+        it "should raise an exception if rcvar does not return foobar_enable" do
+          @provider.load_current_resource
+          @provider.define_resource_requirements
+          lambda { @provider.process_resource_requirements }.should raise_error(Chef::Exceptions::Service)
+        end
+      end
+    end
+  end
+
+  describe Chef::Provider::Service::Freebsd, "enable_service" do
+    before do
+      @provider.current_resource = @current_resource
+      @provider.stub!(:service_enable_variable_name).and_return("#{@current_resource.service_name}_enable")
+    end
+
+    it "should enable the service if it is not enabled" do
+      @current_resource.stub!(:enabled).and_return(false)
+      @provider.should_receive(:read_rc_conf).and_return([ "foo", "#{@current_resource.service_name}_enable=\"NO\"", "bar" ])
+      @provider.should_receive(:write_rc_conf).with(["foo", "bar", "#{@current_resource.service_name}_enable=\"YES\""])
+      @provider.enable_service()
+    end
+
+    it "should enable the service if it is not enabled and not already specified in the rc.conf file" do
+      @current_resource.stub!(:enabled).and_return(false)
+      @provider.should_receive(:read_rc_conf).and_return([ "foo", "bar" ])
+      @provider.should_receive(:write_rc_conf).with(["foo", "bar", "#{@current_resource.service_name}_enable=\"YES\""])
+      @provider.enable_service()
+    end
+
+    it "should not enable the service if it is already enabled" do
+      @current_resource.stub!(:enabled).and_return(true)
+      @provider.should_not_receive(:write_rc_conf)
+      @provider.enable_service
+    end
+  end
+
+  describe Chef::Provider::Service::Freebsd, "disable_service" do
+    before do
+      @provider.current_resource = @current_resource
+      @provider.stub!(:service_enable_variable_name).and_return("#{@current_resource.service_name}_enable")
+    end
+
+    it "should should disable the service if it is not disabled" do
+      @current_resource.stub!(:enabled).and_return(true)
+      @provider.should_receive(:read_rc_conf).and_return([ "foo", "#{@current_resource.service_name}_enable=\"YES\"", "bar" ])
+      @provider.should_receive(:write_rc_conf).with(["foo", "bar", "#{@current_resource.service_name}_enable=\"NO\""])
+      @provider.disable_service()
+    end
+
+    it "should not disable the service if it is already disabled" do
+      @current_resource.stub!(:enabled).and_return(false)
+      @provider.should_not_receive(:write_rc_conf)
+      @provider.disable_service()
+    end
+  end
+end