chef 10.12.0 → 10.14.0.beta.1

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

@@ -0,0 +1,144 @@
+#
+# Author:: Lee Jensen (<ljensen@engineyard.com>)
+# Author:: AJ Christensen (<aj@opscode.com>)
+# Copyright:: Copyright (c) 2008 Opscode, Inc.
+# 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::Gentoo do
+  before(:each) do
+    @node = Chef::Node.new
+    @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::Gentoo.new(@new_resource, @run_context)
+    Chef::Resource::Service.stub!(:new).and_return(@current_resource)
+    @status = mock("Status", :exitstatus => 0, :stdout => @stdout)
+    @provider.stub!(:shell_out).and_return(@status)
+    File.stub!(:exists?).with("/etc/init.d/chef").and_return(true)
+    File.stub!(:exists?).with("/sbin/rc-update").and_return(true)
+    File.stub!(:exists?).with("/etc/runlevels/default/chef").and_return(false)
+    File.stub!(:readable?).with("/etc/runlevels/default/chef").and_return(false)
+  end
+ # new test: found_enabled state
+  #
+  describe "load_current_resource" do  
+    it "should raise Chef::Exceptions::Service if /sbin/rc-update does not exist" do
+      File.should_receive(:exists?).with("/sbin/rc-update").and_return(false)
+      @provider.define_resource_requirements
+      lambda { @provider.process_resource_requirements }.should raise_error(Chef::Exceptions::Service)
+    end
+
+    it "should track when service file is not found in /etc/runlevels" do
+      @provider.load_current_resource
+      @provider.instance_variable_get("@found_script").should be_false
+    end
+
+    it "should track when service file is found in /etc/runlevels/**/" do
+      Dir.stub!(:glob).with("/etc/runlevels/**/chef").and_return(["/etc/runlevels/default/chef"])
+      @provider.load_current_resource
+      @provider.instance_variable_get("@found_script").should be_true
+    end
+
+    describe "when detecting the service enable state" do
+      describe "and the glob returns a default service script file" do
+        before do
+          Dir.stub!(:glob).with("/etc/runlevels/**/chef").and_return(["/etc/runlevels/default/chef"])
+        end
+
+        describe "and the file exists and is readable" do
+          before do
+            File.stub!(:exists?).with("/etc/runlevels/default/chef").and_return(true)
+            File.stub!(:readable?).with("/etc/runlevels/default/chef").and_return(true)
+          end
+          it "should set enabled to true" do
+            @provider.load_current_resource
+            @current_resource.enabled.should be_true
+          end
+        end
+
+        describe "and the file exists but is not readable" do
+          before do
+            File.stub!(:exists?).with("/etc/runlevels/default/chef").and_return(true)
+            File.stub!(:readable?).with("/etc/runlevels/default/chef").and_return(false)
+          end
+          
+          it "should set enabled to false" do
+            @provider.load_current_resource
+            @current_resource.enabled.should be_false
+          end
+        end
+
+        describe "and the file does not exist" do
+          before do
+            File.stub!(:exists?).with("/etc/runlevels/default/chef").and_return(false)
+            File.stub!(:readable?).with("/etc/runlevels/default/chef").and_return("foobarbaz")
+          end
+
+          it "should set enabled to false" do
+            @provider.load_current_resource
+            @current_resource.enabled.should be_false
+          end
+
+        end
+      end
+
+  end
+ 
+    it "should return the current_resource" do
+      @provider.load_current_resource.should == @current_resource
+    end  
+
+    it "should support the status command automatically" do
+      @provider.load_current_resource
+      @new_resource.supports[:status].should be_true
+    end
+
+    it "should support the restart command automatically" do
+      @provider.load_current_resource
+      @new_resource.supports[:restart].should be_true
+    end
+
+    it "should not support the reload command automatically" do
+      @provider.load_current_resource
+      @new_resource.supports[:reload].should_not be_true
+    end
+
+  end
+  
+  describe "action_methods" do
+    before(:each) { @provider.stub!(:load_current_resource).and_return(@current_resource) }
+
+    describe Chef::Provider::Service::Gentoo, "enable_service" do
+      it "should call rc-update add *service* default" do
+        @provider.should_receive(:run_command).with({:command => "/sbin/rc-update add chef default"})
+        @provider.enable_service()
+      end
+    end
+
+    describe Chef::Provider::Service::Gentoo, "disable_service" do
+      it "should call rc-update del *service* default" do
+        @provider.should_receive(:run_command).with({:command => "/sbin/rc-update del chef default"})
+        @provider.disable_service()
+      end
+    end
+  end
+
+end

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

@@ -0,0 +1,212 @@
+#
+# Author:: AJ Christensen (<aj@hjksolutions.com>)
+# Copyright:: Copyright (c) 2008 Opscode, Inc.
+# 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::Init, "load_current_resource" do
+  before(:each) do
+    @node = Chef::Node.new
+    @node[:command] = {:ps => "ps -ef"}
+    @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::Init.new(@new_resource, @run_context)
+    Chef::Resource::Service.stub!(:new).and_return(@current_resource)
+
+    @stdout = StringIO.new(<<-PS)
+aj        7842  5057  0 21:26 pts/2    00:00:06 vi init.rb
+aj        7903  5016  0 21:26 pts/5    00:00:00 /bin/bash
+aj        8119  6041  0 21:34 pts/3    00:00:03 vi init_service_spec.rb
+PS
+    @status = mock("Status", :exitstatus => 0, :stdout => @stdout)
+    @provider.stub!(:shell_out!).and_return(@status)
+  end
+  
+  it "should create a current resource with the name of the new resource" do
+    @provider.load_current_resource
+    @provider.current_resource.should equal(@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 == 'chef'
+  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("/etc/init.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.stub!(:shell_out).with("/etc/init.d/#{@current_resource.service_name} status").and_return(@status)
+      @provider.load_current_resource
+      @current_resource.running.should be_true
+    end
+
+    it "should set running to false if the status command returns anything except 0" do
+      @status.stub!(:exitstatus).and_return(1)
+      @provider.stub!(:shell_out).with("/etc/init.d/#{@current_resource.service_name} status").and_return(@status)
+      @provider.load_current_resource
+      @current_resource.running.should be_false
+    end
+
+    it "should set running to false if the status command raises" do
+      @provider.stub!(:shell_out).and_raise(Mixlib::ShellOut::ShellCommandFailed)
+      @provider.load_current_resource
+      @current_resource.running.should be_false
+    end
+  end
+
+  describe "when a status command has been specified" do
+    before do
+      @new_resource.stub!(:status_command).and_return("/etc/init.d/chefhasmonkeypants status")
+    end
+
+    it "should run the services status command if one has been specified" do
+      @provider.should_receive(:shell_out).with("/etc/init.d/chefhasmonkeypants status").and_return(@status)
+      @provider.load_current_resource
+    end
+    
+  end
+  
+  describe "when the node has not specified a ps command" do
+    
+    it "should raise an error if the node has a nil ps attribute" do
+      @node[:command] = {:ps => nil}
+      @provider.load_current_resource
+      @provider.action = :start
+      @provider.define_resource_requirements
+      lambda { @provider.process_resource_requirements }.should raise_error(Chef::Exceptions::Service)
+    end
+
+    it "should raise an error if the node has an empty ps attribute" do
+      @node[:command] = {:ps => ""}
+      @provider.load_current_resource
+      @provider.action = :start
+      @provider.define_resource_requirements
+      lambda { @provider.process_resource_requirements }.should raise_error(Chef::Exceptions::Service)
+    end
+    
+  end
+
+
+  describe "when we have a 'ps' attribute" do
+    it "should shell_out! the node's ps command" do
+      @provider.should_receive(:shell_out!).and_return(@status)
+      @provider.load_current_resource
+    end
+
+    it "should set running to true if the regex matches the output" do
+      @stdout = StringIO.new(<<-RUNNING_PS)
+aj        7842  5057  0 21:26 pts/2    00:00:06 chef
+aj        7842  5057  0 21:26 pts/2    00:00:06 poos
+RUNNING_PS
+      @status.stub!(:stdout).and_return(@stdout)
+      @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.action = :start
+      @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/init.d/chef startyousillysally")
+      @provider.should_receive(:shell_out!).with("/etc/init.d/chef startyousillysally")
+      @provider.start_service()
+    end
+
+    it "should call '/etc/init.d/service_name start' if no start command is specified" do
+      @provider.should_receive(:shell_out!).with("/etc/init.d/#{@new_resource.service_name} start")
+      @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.stop_service()
+    end
+
+    it "should call '/etc/init.d/service_name stop' if no stop command is specified" do
+      @provider.should_receive(:shell_out!).with("/etc/init.d/#{@new_resource.service_name} stop")
+      @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("/etc/init.d/#{@new_resource.service_name} restart")
+      @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/#{@new_resource.service_name} restartinafire")
+      @provider.restart_service()
+    end
+
+    it "should just call stop, then start when the resource doesn't support restart and no restart_command is specified" do
+      @provider.should_receive(:stop_service)
+      @provider.should_receive(:sleep).with(1)
+      @provider.should_receive(:start_service)
+      @provider.restart_service()
+    end
+  end
+
+  describe "when reloading a service" do
+    it "should call 'reload' on the service if it supports it" do
+      @new_resource.supports({:reload => true})
+      @provider.should_receive(:shell_out!).with("/etc/init.d/chef reload")
+      @provider.reload_service()
+    end
+
+    it "should should run the user specified reload command if one is specified and the service doesn't support reload" do
+      @new_resource.reload_command("/etc/init.d/chef lollerpants")
+      @provider.should_receive(:shell_out!).with("/etc/init.d/chef lollerpants")
+      @provider.reload_service()
+    end
+  end
+end

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

@@ -0,0 +1,76 @@
+#
+# Author:: Bryan McLellan <btm@loftninjas.org>
+# Copyright:: Copyright (c) 2011 Opscode, Inc.
+# 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::Insserv do
+  before(:each) do
+    @node = Chef::Node.new
+    @events = Chef::EventDispatch::Dispatcher.new
+    @run_context = Chef::RunContext.new(@node, {}, @events)
+    @node[:command] = {:ps => "ps -ax"}
+
+    @new_resource = Chef::Resource::Service.new("initgrediant")
+    @current_resource = Chef::Resource::Service.new("initgrediant")
+
+    @provider = Chef::Provider::Service::Insserv.new(@new_resource, @run_context)
+    @status = mock("Process::Status mock", :exitstatus => 0, :stdout => "")
+    @provider.stub!(:shell_out!).and_return(@status)
+  end
+
+  describe "load_current_resource" do
+    describe "when startup links exist" do
+      before do 
+        Dir.stub!(:glob).with("/etc/rc**/S*initgrediant").and_return(["/etc/rc5.d/S18initgrediant", "/etc/rc2.d/S18initgrediant", "/etc/rc4.d/S18initgrediant", "/etc/rc3.d/S18initgrediant"])
+      end
+
+      it "sets the current enabled status to true" do
+        @provider.load_current_resource
+        @provider.current_resource.enabled.should be_true
+      end
+    end
+
+    describe "when startup links do not exist" do
+      before do 
+        Dir.stub!(:glob).with("/etc/rc**/S*initgrediant").and_return([])
+      end
+
+      it "sets the current enabled status to false" do
+        @provider.load_current_resource
+        @provider.current_resource.enabled.should be_false
+      end
+    end
+
+  end
+
+  describe "enable_service" do
+    it "should call insserv and create the default links" do
+      @provider.should_receive(:run_command).with({:command=>"/sbin/insserv -r -f #{@new_resource.service_name}"})
+      @provider.should_receive(:run_command).with({:command=>"/sbin/insserv -d -f #{@new_resource.service_name}"})
+      @provider.enable_service
+    end
+  end
+  
+  describe "disable_service" do
+    it "should call insserv and remove the links" do
+      @provider.should_receive(:run_command).with({:command=>"/sbin/insserv -r -f #{@new_resource.service_name}"})
+      @provider.disable_service
+    end
+  end
+end
+

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

@@ -0,0 +1,212 @@
+#
+# Author:: AJ Christensen (<aj@hjksolutions.com>)
+# Copyright:: Copyright (c) 2008 Opscode, Inc.
+# 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::Invokercd, "load_current_resource" do
+  before(:each) do
+    @node = Chef::Node.new
+    @node[:command] = {:ps => "ps -ef"}
+    @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::Invokercd.new(@new_resource, @run_context)
+    Chef::Resource::Service.stub!(:new).and_return(@current_resource)
+
+    @stdout = StringIO.new(<<-PS)
+aj        7842  5057  0 21:26 pts/2    00:00:06 vi init.rb
+aj        7903  5016  0 21:26 pts/5    00:00:00 /bin/bash
+aj        8119  6041  0 21:34 pts/3    00:00:03 vi init_service_spec.rb
+PS
+    @status = mock("Status", :exitstatus => 0, :stdout => @stdout)
+    @provider.stub!(:shell_out!).and_return(@status)
+  end
+  
+  it "should create a current resource with the name of the new resource" do
+    @provider.load_current_resource
+    @provider.current_resource.should equal(@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 == 'chef'
+  end
+
+  describe "when the service supports status" do
+    before do
+      @new_resource.supports({:status => true})
+    end
+
+    it "should run '/usr/sbin/invoke-rc.d service_name status'" do
+      @provider.should_receive(:shell_out).with("/usr/sbin/invoke-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.stub!(:shell_out).with("/usr/sbin/invoke-rc.d #{@current_resource.service_name} status").and_return(@status)
+      @provider.load_current_resource
+      @current_resource.running.should be_true
+    end
+
+    it "should set running to false if the status command returns anything except 0" do
+      @status.stub!(:exitstatus).and_return(1)
+      @provider.stub!(:shell_out).with("/usr/sbin/invoke-rc.d #{@current_resource.service_name} status").and_return(@status)
+      @provider.load_current_resource
+      @current_resource.running.should be_false
+    end
+
+    it "should set running to false if the status command raises" do
+      @provider.stub!(:shell_out).with("/usr/sbin/invoke-rc.d #{@current_resource.service_name} status").and_raise(Mixlib::ShellOut::ShellCommandFailed)
+      @provider.load_current_resource
+      @current_resource.running.should be_false
+    end
+  end
+
+  describe "when a status command has been specified" do
+    before do
+      @new_resource.stub!(:status_command).and_return("/usr/sbin/invoke-rc.d chefhasmonkeypants status")
+    end
+
+    it "should run the services status command if one has been specified" do
+      @provider.should_receive(:shell_out).with("/usr/sbin/invoke-rc.d chefhasmonkeypants status").and_return(@status)
+      @provider.load_current_resource
+    end
+    
+  end
+  
+  describe "when the node has not specified a ps command" do
+    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.action = :start
+      @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.action = :start
+      @provider.define_resource_requirements
+      lambda { @provider.process_resource_requirements }.should raise_error(Chef::Exceptions::Service)
+    end
+    
+  end
+
+
+  describe "when we have a 'ps' attribute" do
+    it "should shell_out! the node's ps command" do
+      @status = mock("Status", :exitstatus => 0, :stdout => @stdout)
+      @provider.should_receive(:shell_out!).with(@node[:command][:ps]).and_return(@status)
+      @provider.load_current_resource
+    end
+
+    it "should set running to true if the regex matches the output" do
+      @stdout = StringIO.new(<<-RUNNING_PS)
+aj        7842  5057  0 21:26 pts/2    00:00:06 chef
+aj        7842  5057  0 21:26 pts/2    00:00:06 poos
+RUNNING_PS
+      @status = mock("Status", :exitstatus => 0, :stdout => @stdout)
+      @provider.should_receive(:shell_out!).and_return(@status)
+      @provider.load_current_resource 
+      @current_resource.running.should be_true
+    end
+
+    it "should set running to false if the regex doesn't match" do
+      @status = mock("Status", :exitstatus => 0, :stdout => @stdout)
+      @provider.should_receive(: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.action = :start
+      @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("/usr/sbin/invoke-rc.d chef startyousillysally")
+      @provider.should_receive(:shell_out!).with("/usr/sbin/invoke-rc.d chef startyousillysally")
+      @provider.start_service()
+    end
+
+    it "should call '/usr/sbin/invoke-rc.d service_name start' if no start command is specified" do
+      @provider.should_receive(:shell_out!).with("/usr/sbin/invoke-rc.d #{@new_resource.service_name} start")
+      @provider.start_service()
+    end 
+  end
+
+  describe Chef::Provider::Service::Invokercd, "stop_service" do
+    it "should call the stop command if one is specified" do
+      @new_resource.stop_command("/usr/sbin/invoke-rc.d chef itoldyoutostop")
+      @provider.should_receive(:shell_out!).with("/usr/sbin/invoke-rc.d chef itoldyoutostop")
+      @provider.stop_service()
+    end
+
+    it "should call '/usr/sbin/invoke-rc.d service_name stop' if no stop command is specified" do
+      @provider.should_receive(:shell_out!).with("/usr/sbin/invoke-rc.d #{@new_resource.service_name} stop")
+      @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/sbin/invoke-rc.d #{@new_resource.service_name} restart")
+      @provider.restart_service()
+    end
+
+    it "should call the restart_command if one has been specified" do
+      @new_resource.restart_command("/usr/sbin/invoke-rc.d chef restartinafire")
+      @provider.should_receive(:shell_out!).with("/usr/sbin/invoke-rc.d #{@new_resource.service_name} restartinafire")
+      @provider.restart_service()
+    end
+
+    it "should just call stop, then start when the resource doesn't support restart and no restart_command is specified" do
+      @provider.should_receive(:stop_service)
+      @provider.should_receive(:sleep).with(1)
+      @provider.should_receive(:start_service)
+      @provider.restart_service()
+    end
+  end
+
+  describe "when reloading a service" do
+    it "should call 'reload' on the service if it supports it" do
+      @new_resource.supports({:reload => true})
+      @provider.should_receive(:shell_out!).with("/usr/sbin/invoke-rc.d chef reload")
+      @provider.reload_service()
+    end
+
+    it "should should run the user specified reload command if one is specified and the service doesn't support reload" do
+      @new_resource.reload_command("/usr/sbin/invoke-rc.d chef lollerpants")
+      @provider.should_receive(:shell_out!).with("/usr/sbin/invoke-rc.d chef lollerpants")
+      @provider.reload_service()
+    end
+  end
+end