diff options
| author | Nigel Kersten <nigelk@google.com> | 2009-08-14 13:35:24 -0700 |
|---|---|---|
| committer | James Turnbull <james@lovedthanlost.net> | 2009-08-18 08:31:01 +1000 |
| commit | c1967bb8d2e98d53182ea15fb13ac68d7bff7d84 (patch) | |
| tree | 8237a8dad9b31df5efb50ef521293296f332ebd6 /spec/unit/provider/service | |
| parent | 7e0924737d817644402b1c3d2d5abb2cf006e76b (diff) | |
| download | puppet-c1967bb8d2e98d53182ea15fb13ac68d7bff7d84.tar.gz puppet-c1967bb8d2e98d53182ea15fb13ac68d7bff7d84.tar.xz puppet-c1967bb8d2e98d53182ea15fb13ac68d7bff7d84.zip | |
Fixes #2513. debian service provider now uses invoke-rc.d to determine enabled? status
Diffstat (limited to 'spec/unit/provider/service')
| -rwxr-xr-x | spec/unit/provider/service/debian.rb | 89 |
1 files changed, 89 insertions, 0 deletions
diff --git a/spec/unit/provider/service/debian.rb b/spec/unit/provider/service/debian.rb new file mode 100755 index 000000000..87d627452 --- /dev/null +++ b/spec/unit/provider/service/debian.rb @@ -0,0 +1,89 @@ +#!/usr/bin/env ruby +# +# Unit testing for the debian service provider +# + +require File.dirname(__FILE__) + '/../../../spec_helper' + +provider_class = Puppet::Type.type(:service).provider(:debian) + +describe provider_class do + + before(:each) do + # Create a mock resource + @resource = stub 'resource' + + @provider = provider_class.new + + # A catch all; no parameters set + @resource.stubs(:[]).returns(nil) + + # But set name, source and path + @resource.stubs(:[]).with(:name).returns "myservice" + @resource.stubs(:[]).with(:ensure).returns :enabled + @resource.stubs(:ref).returns "Service[myservice]" + + @provider.resource = @resource + + @provider.stubs(:command).with(:update_rc).returns "update_rc" + @provider.stubs(:command).with(:invoke_rc).returns "invoke_rc" + + @provider.stubs(:update_rc) + @provider.stubs(:invoke_rc) + end + + it "should have an enabled? method" do + @provider.should respond_to(:enabled?) + end + + it "should have an enable method" do + @provider.should respond_to(:enable) + end + + it "should have a disable method" do + @provider.should respond_to(:disable) + end + + describe "when enabling" do + it "should call update-rc.d twice" do + @provider.expects(:update_rc).twice + @provider.enable + end + end + + describe "when disabling" do + it "should call update-rc.d twice" do + @provider.expects(:update_rc).twice + @provider.disable + end + end + + describe "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", "--query", @resource[:name], "start").once + @provider.enabled? + end + + it "should return true when invoke-rc.d exits with 104 status" do + @provider.stubs(:system) + $?.stubs(:exitstatus).returns(104) + @provider.enabled?.should == :true + end + + it "should return true when invoke-rc.d exits with 106 status" do + @provider.stubs(:system) + $?.stubs(:exitstatus).returns(106) + @provider.enabled?.should == :true + end + + # pick a range of non-[104.106] numbers, strings and booleans to test with. + [-100, -1, 0, 1, 100, "foo", "", :true, :false].each do |exitstatus| + it "should return false when invoke-rc.d exits with #{exitstatus} status" do + @provider.stubs(:system) + $?.stubs(:exitstatus).returns(exitstatus) + @provider.enabled?.should == :false + end + end + end + + end |
