chef 18.8.11 → 18.8.54

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

@@ -1,145 +0,0 @@
-#
-# Author:: Lee Jensen (<ljensen@engineyard.com>)
-# Author:: AJ Christensen (<aj@chef.io>)
-# Copyright:: Copyright (c) Chef Software 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)
-    allow(Chef::Resource::Service).to receive(:new).and_return(@current_resource)
-    @status = double("Status", exitstatus: 0, stdout: @stdout)
-    allow(@provider).to receive(:shell_out).and_return(@status)
-    allow(File).to receive(:exist?).with("/etc/init.d/chef").and_return(true)
-    allow(File).to receive(:exist?).with("/sbin/rc-update").and_return(true)
-    allow(File).to receive(:exist?).with("/etc/runlevels/default/chef").and_return(false)
-    allow(File).to receive(: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
-      expect(File).to receive(:exist?).with("/sbin/rc-update").and_return(false)
-      @provider.action = :start
-      @provider.define_resource_requirements
-      expect { @provider.process_resource_requirements }.to raise_error(Chef::Exceptions::Service)
-    end
-
-    it "should track when service file is not found in /etc/runlevels" do
-      @provider.load_current_resource
-      expect(@provider.instance_variable_get(:@found_script)).to be_falsey
-    end
-
-    it "should track when service file is found in /etc/runlevels/**/" do
-      allow(Dir).to receive(:glob).with("/etc/runlevels/**/chef").and_return(["/etc/runlevels/default/chef"])
-      @provider.load_current_resource
-      expect(@provider.instance_variable_get(:@found_script)).to be_truthy
-    end
-
-    describe "when detecting the service enable state" do
-      describe "and the glob returns a default service script file" do
-        before do
-          allow(Dir).to receive(:glob).with("/etc/runlevels/**/chef").and_return(["/etc/runlevels/default/chef"])
-        end
-
-        describe "and the file exists and is readable" do
-          before do
-            allow(File).to receive(:exist?).with("/etc/runlevels/default/chef").and_return(true)
-            allow(File).to receive(:readable?).with("/etc/runlevels/default/chef").and_return(true)
-          end
-          it "should set enabled to true" do
-            @provider.load_current_resource
-            expect(@current_resource.enabled).to be_truthy
-          end
-        end
-
-        describe "and the file exists but is not readable" do
-          before do
-            allow(File).to receive(:exist?).with("/etc/runlevels/default/chef").and_return(true)
-            allow(File).to receive(:readable?).with("/etc/runlevels/default/chef").and_return(false)
-          end
-
-          it "should set enabled to false" do
-            @provider.load_current_resource
-            expect(@current_resource.enabled).to be_falsey
-          end
-        end
-
-        describe "and the file does not exist" do
-          before do
-            allow(File).to receive(:exist?).with("/etc/runlevels/default/chef").and_return(false)
-            allow(File).to receive(:readable?).with("/etc/runlevels/default/chef").and_return("foobarbaz")
-          end
-
-          it "should set enabled to false" do
-            @provider.load_current_resource
-            expect(@current_resource.enabled).to be_falsey
-          end
-
-        end
-      end
-
-    end
-
-    it "should return the current_resource" do
-      expect(@provider.load_current_resource).to eq(@current_resource)
-    end
-
-    it "should support the status command automatically" do
-      @provider.load_current_resource
-      expect(@provider.supports[:status]).to be true
-    end
-
-    it "should support the restart command automatically" do
-      @provider.load_current_resource
-      expect(@provider.supports[:restart]).to be true
-    end
-
-    it "should not support the reload command automatically" do
-      @provider.load_current_resource
-      expect(@provider.supports[:reload]).to be_falsey
-    end
-
-  end
-
-  describe "action_methods" do
-    before(:each) { allow(@provider).to receive(:load_current_resource).and_return(@current_resource) }
-
-    describe Chef::Provider::Service::Gentoo, "enable_service" do
-      it "should call rc-update add *service* default" do
-        expect(@provider).to receive(:shell_out!).with("/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
-        expect(@provider).to receive(:shell_out!).with("/sbin/rc-update del chef default")
-        @provider.disable_service
-      end
-    end
-  end
-
-end

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

@@ -1,235 +0,0 @@
-#
-# Author:: AJ Christensen (<aj@hjksolutions.com>)
-# Copyright:: Copyright (c) Chef Software 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.automatic_attrs[: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)
-    allow(Chef::Resource::Service).to receive(: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 = double("Status", exitstatus: 0, stdout: @stdout)
-    allow(@provider).to receive(:shell_out!).and_return(@status)
-  end
-
-  it "should create a current resource with the name of the new resource" do
-    @provider.load_current_resource
-    expect(@provider.current_resource).to equal(@current_resource)
-  end
-
-  it "should set the current resources service name to the new resources service name" do
-    @provider.load_current_resource
-    expect(@current_resource.service_name).to eq("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
-      expect(@provider).to 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 status command returns 0" do
-      allow(@provider).to receive(:shell_out).with("/etc/init.d/#{@current_resource.service_name} status").and_return(@status)
-      @provider.load_current_resource
-      expect(@current_resource.running).to be_truthy
-    end
-
-    it "should set running to false if the status command returns anything except 0" do
-      allow(@status).to receive(:exitstatus).and_return(1)
-      allow(@provider).to receive(:shell_out).with("/etc/init.d/#{@current_resource.service_name} status").and_return(@status)
-      @provider.load_current_resource
-      expect(@current_resource.running).to be_falsey
-    end
-
-    it "should set running to false if the status command raises" do
-      allow(@provider).to receive(:shell_out).and_raise(Mixlib::ShellOut::ShellCommandFailed)
-      @provider.load_current_resource
-      expect(@current_resource.running).to be_falsey
-    end
-  end
-
-  describe "when a status command has been specified" do
-    before do
-      @new_resource.status_command("/etc/init.d/chefhasmonkeypants status")
-    end
-
-    it "should run the services status command if one has been specified" do
-      expect(@provider).to receive(:shell_out).with("/etc/init.d/chefhasmonkeypants status").and_return(@status)
-      @provider.load_current_resource
-    end
-
-  end
-
-  describe "when an init command has been specified" do
-    before do
-      @new_resource.init_command("/opt/chef-server/service/erchef")
-      @provider = Chef::Provider::Service::Init.new(@new_resource, @run_context)
-    end
-
-    it "should use the init_command if one has been specified" do
-      expect(@provider).to receive(:shell_out!).with("/opt/chef-server/service/erchef start", default_env: false)
-      @provider.start_service
-    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 property" do
-      @node.automatic_attrs[:command] = { ps: nil }
-      @provider.load_current_resource
-      @provider.action = :start
-      @provider.define_resource_requirements
-      expect { @provider.process_resource_requirements }.to raise_error(Chef::Exceptions::Service)
-    end
-
-    it "should raise an error if the node has an empty ps property" do
-      @node.automatic_attrs[:command] = { ps: "" }
-      @provider.load_current_resource
-      @provider.action = :start
-      @provider.define_resource_requirements
-      expect { @provider.process_resource_requirements }.to raise_error(Chef::Exceptions::Service)
-    end
-
-  end
-
-  describe "when we have a 'ps' property" do
-    it "should shell_out! the node's ps command" do
-      expect(@provider).to 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
-      allow(@status).to receive(:stdout).and_return(@stdout)
-      @provider.load_current_resource
-      expect(@current_resource.running).to be_truthy
-    end
-
-    it "should set running to false if the regex doesn't match" do
-      allow(@provider).to receive(:shell_out!).and_return(@status)
-      @provider.load_current_resource
-      expect(@current_resource.running).to be_falsey
-    end
-
-    it "should raise an exception if ps fails" do
-      allow(@provider).to receive(:shell_out!).and_raise(Mixlib::ShellOut::ShellCommandFailed)
-      @provider.load_current_resource
-      @provider.action = :start
-      @provider.define_resource_requirements
-      expect { @provider.process_resource_requirements }.to raise_error(Chef::Exceptions::Service)
-    end
-  end
-
-  it "should return the current resource" do
-    expect(@provider.load_current_resource).to 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")
-      expect(@provider).to receive(:shell_out!).with("/etc/init.d/chef startyousillysally", default_env: false)
-      @provider.start_service
-    end
-
-    it "should call '/etc/init.d/service_name start' if no start command is specified" do
-      expect(@provider).to receive(:shell_out!).with("/etc/init.d/#{@new_resource.service_name} start", default_env: false)
-      @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")
-      expect(@provider).to receive(:shell_out!).with("/etc/init.d/chef itoldyoutostop", default_env: false)
-      @provider.stop_service
-    end
-
-    it "should call '/etc/init.d/service_name stop' if no stop command is specified" do
-      expect(@provider).to receive(:shell_out!).with("/etc/init.d/#{@new_resource.service_name} stop", default_env: false)
-      @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 })
-      expect(@provider).to receive(:shell_out!).with("/etc/init.d/#{@new_resource.service_name} restart", default_env: false)
-      @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")
-      expect(@provider).to receive(:shell_out!).with("/etc/init.d/#{@new_resource.service_name} restartinafire", default_env: false)
-      @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
-      expect(@provider).to receive(:stop_service)
-      expect(@provider).to receive(:sleep).with(1)
-      expect(@provider).to 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 })
-      expect(@provider).to receive(:shell_out!).with("/etc/init.d/chef reload", default_env: false)
-      @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")
-      expect(@provider).to receive(:shell_out!).with("/etc/init.d/chef lollerpants", default_env: false)
-      @provider.reload_service
-    end
-  end
-
-  describe "when a custom command has been specified" do
-    before do
-      @new_resource.start_command("/etc/init.d/chef startyousillysally")
-      expect(@provider).to receive(:shell_out!).with("/etc/init.d/chef startyousillysally", default_env: false)
-    end
-
-    it "should still pass all why run assertions" do
-      expect { @provider.run_action(:start) }.not_to raise_error
-    end
-  end
-end

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

@@ -1,75 +0,0 @@
-#
-# Author:: Bryan McLellan <btm@loftninjas.org>
-# Copyright:: Copyright (c) Chef Software 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.automatic_attrs[: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 = double("Process::Status mock", exitstatus: 0, stdout: "")
-    allow(@provider).to receive(:shell_out!).and_return(@status)
-  end
-
-  describe "load_current_resource" do
-    describe "when startup links exist" do
-      before do
-        allow(Dir).to receive(:glob).with("/etc/rc*/**/S*initgrediant").and_return(["/etc/rc.d/rc5.d/S18initgrediant", "/etc/rc.d/rc2.d/S18initgrediant", "/etc/rc.d/rc4.d/S18initgrediant", "/etc/rc.d/rc3.d/S18initgrediant"])
-      end
-
-      it "sets the current enabled status to true" do
-        @provider.load_current_resource
-        expect(@provider.current_resource.enabled).to be_truthy
-      end
-    end
-
-    describe "when startup links do not exist" do
-      before do
-        allow(Dir).to receive(:glob).with("/etc/rc*/**/S*initgrediant").and_return([])
-      end
-
-      it "sets the current enabled status to false" do
-        @provider.load_current_resource
-        expect(@provider.current_resource.enabled).to be_falsey
-      end
-    end
-
-  end
-
-  describe "enable_service" do
-    it "should call insserv and create the default links" do
-      expect(@provider).to receive(:shell_out!).with("/sbin/insserv -r -f #{@new_resource.service_name}")
-      expect(@provider).to receive(:shell_out!).with("/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
-      expect(@provider).to receive(:shell_out!).with("/sbin/insserv -r -f #{@new_resource.service_name}")
-      @provider.disable_service
-    end
-  end
-end

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

@@ -1,211 +0,0 @@
-#
-# Author:: AJ Christensen (<aj@hjksolutions.com>)
-# Copyright:: Copyright (c) Chef Software 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.automatic_attrs[: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)
-    allow(Chef::Resource::Service).to receive(: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 = double("Status", exitstatus: 0, stdout: @stdout)
-    allow(@provider).to receive(:shell_out!).and_return(@status)
-  end
-
-  it "should create a current resource with the name of the new resource" do
-    @provider.load_current_resource
-    expect(@provider.current_resource).to equal(@current_resource)
-  end
-
-  it "should set the current resources service name to the new resources service name" do
-    @provider.load_current_resource
-    expect(@current_resource.service_name).to eq("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
-      expect(@provider).to 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 status command returns 0" do
-      allow(@provider).to receive(:shell_out).with("/usr/sbin/invoke-rc.d #{@current_resource.service_name} status").and_return(@status)
-      @provider.load_current_resource
-      expect(@current_resource.running).to be_truthy
-    end
-
-    it "should set running to false if the status command returns anything except 0" do
-      allow(@status).to receive(:exitstatus).and_return(1)
-      allow(@provider).to receive(:shell_out).with("/usr/sbin/invoke-rc.d #{@current_resource.service_name} status").and_return(@status)
-      @provider.load_current_resource
-      expect(@current_resource.running).to be_falsey
-    end
-
-    it "should set running to false if the status command raises" do
-      allow(@provider).to receive(:shell_out).with("/usr/sbin/invoke-rc.d #{@current_resource.service_name} status").and_raise(Mixlib::ShellOut::ShellCommandFailed)
-      @provider.load_current_resource
-      expect(@current_resource.running).to be_falsey
-    end
-  end
-
-  describe "when a status command has been specified" do
-    before do
-      @new_resource.status_command("/usr/sbin/invoke-rc.d chefhasmonkeypants status")
-    end
-
-    it "should run the services status command if one has been specified" do
-      expect(@provider).to 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 property and no other means to get status" do
-      @node.automatic_attrs[:command] = { ps: nil }
-      @provider.action = :start
-      @provider.define_resource_requirements
-      expect { @provider.process_resource_requirements }.to raise_error(Chef::Exceptions::Service)
-    end
-
-    it "should raise error if the node has an empty ps property and no other means to get status" do
-      @node.automatic_attrs[:command] = { ps: "" }
-      @provider.action = :start
-      @provider.define_resource_requirements
-      expect { @provider.process_resource_requirements }.to raise_error(Chef::Exceptions::Service)
-    end
-
-  end
-
-  describe "when we have a 'ps' property" do
-    it "should shell_out! the node's ps command" do
-      @status = double("Status", exitstatus: 0, stdout: @stdout)
-      expect(@provider).to 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 = double("Status", exitstatus: 0, stdout: @stdout)
-      expect(@provider).to receive(:shell_out!).and_return(@status)
-      @provider.load_current_resource
-      expect(@current_resource.running).to be_truthy
-    end
-
-    it "should set running to false if the regex doesn't match" do
-      @status = double("Status", exitstatus: 0, stdout: @stdout)
-      expect(@provider).to receive(:shell_out!).and_return(@status)
-      @provider.load_current_resource
-      expect(@current_resource.running).to be_falsey
-    end
-
-    it "should raise an exception if ps fails" do
-      allow(@provider).to receive(:shell_out!).and_raise(Mixlib::ShellOut::ShellCommandFailed)
-      @provider.action = :start
-      @provider.load_current_resource
-      @provider.define_resource_requirements
-      expect { @provider.process_resource_requirements }.to raise_error(Chef::Exceptions::Service)
-    end
-  end
-
-  it "should return the current resource" do
-    expect(@provider.load_current_resource).to 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")
-      expect(@provider).to receive(:shell_out!).with("/usr/sbin/invoke-rc.d chef startyousillysally", default_env: false)
-      @provider.start_service
-    end
-
-    it "should call '/usr/sbin/invoke-rc.d service_name start' if no start command is specified" do
-      expect(@provider).to receive(:shell_out!).with("/usr/sbin/invoke-rc.d #{@new_resource.service_name} start", default_env: false)
-      @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")
-      expect(@provider).to receive(:shell_out!).with("/usr/sbin/invoke-rc.d chef itoldyoutostop", default_env: false)
-      @provider.stop_service
-    end
-
-    it "should call '/usr/sbin/invoke-rc.d service_name stop' if no stop command is specified" do
-      expect(@provider).to receive(:shell_out!).with("/usr/sbin/invoke-rc.d #{@new_resource.service_name} stop", default_env: false)
-      @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 })
-      expect(@provider).to receive(:shell_out!).with("/usr/sbin/invoke-rc.d #{@new_resource.service_name} restart", default_env: false)
-      @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")
-      expect(@provider).to receive(:shell_out!).with("/usr/sbin/invoke-rc.d #{@new_resource.service_name} restartinafire", default_env: false)
-      @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
-      expect(@provider).to receive(:stop_service)
-      expect(@provider).to receive(:sleep).with(1)
-      expect(@provider).to 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 })
-      expect(@provider).to receive(:shell_out!).with("/usr/sbin/invoke-rc.d chef reload", default_env: false)
-      @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")
-      expect(@provider).to receive(:shell_out!).with("/usr/sbin/invoke-rc.d chef lollerpants", default_env: false)
-      @provider.reload_service
-    end
-  end
-end