fixes chef-33, adds support for gentoo

Author: Lee Jensen

chef/lib/chef/platform.rb

@@ -38,7 +38,8 @@ class Platform
       :redhat   => {},
       :gentoo   => {
         :default => {
-          :package => Chef::Provider::Package::Portage
+          :package => Chef::Provider::Package::Portage,
+          :service => Chef::Provider::Service::Gentoo
         }
       },
       :solaris  => {},

chef/lib/chef/provider/service/gentoo.rb

@@ -0,0 +1,52 @@
+#
+# Author:: Lee Jensen (<ljensen@engineyard.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 'chef/provider/service'
+require 'chef/mixin/command'
+
+class Chef
+  class Provider
+    class Service
+      class Gentoo < Chef::Provider::Service::Init
+        def load_current_resource
+          super
+          
+          raise Chef::Exception::Service unless ::File.exists?("/sbin/rc-update")
+          
+          IO.popen('/sbin/rc-update -s default') do |io|
+            io.each_line do |line|
+              if line.match(/^\s*#{@current_resource.service_name}\s+/)
+                @current_resource.enabled true
+              end
+            end
+          end
+          
+          @current_resource
+        end
+        
+        def enable_service(name)
+          run_command(:command => "/sbin/rc-update add #{name} default")
+        end
+        
+        def disable_service(name)
+          run_command(:command => "/sbin/rc-update del #{name} default")
+        end
+      end
+    end
+  end
+end

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

@@ -0,0 +1,90 @@
+#
+# Author:: Lee Jensen (<ljensen@engineyard.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 File.expand_path(File.join(File.dirname(__FILE__), "..", "..", "..", "spec_helper"))
+
+describe Chef::Provider::Service::Gentoo do
+  before(:each) do
+    @node = mock("Chef::Node", :null_object => true)
+    
+    resource_opts     = { :null_object => true, :name => 'chef', :service_name => 'chef', :enabled => false, :running => nil }
+    @new_resource     = mock("Chef::Resource::Service", resource_opts)
+    @current_resource = mock("Chef::Resource::Service", resource_opts)
+    
+    @provider = Chef::Provider::Service::Gentoo.new(@node, @new_resource)
+    Chef::Resource::Service.stub!(:new).and_return(@current_resource)
+    File.stub!(:exists?).and_return(true)
+    
+    @io = mock("IO")
+    IO.stub!(:popen).and_return(@io)
+    @io.stub!(:each_line).and_yield("  gfs | default ")
+  end
+  
+  describe "load_current_resource" do  
+    it "should raise Chef::Exception::Service if /sbin/rc-update does not exist" do
+      File.should_receive(:exists?).with("/sbin/rc-update").and_return(false)
+      lambda { @provider.load_current_resource }.should raise_error(Chef::Exception::Service)
+    end
+  
+    it "should set enabled true if rc-update indicates service is in default runlevel" do
+      IO.should_receive(:popen).with("/sbin/rc-update -s default").and_yield(@io)
+      @io.should_receive(:each_line).
+          and_yield('  gfs | default ').
+          and_yield(' chef | default ').
+          and_yield('monit | default ').
+          and_yield('mysql | default ')
+    
+      @current_resource.should_receive(:enabled).with(true)
+      @provider.load_current_resource
+    end
+  
+    it "should set enabled false if rc-update indicates service is not in default runlevel" do
+      IO.should_receive(:popen).with("/sbin/rc-update -s default").and_yield(@io)
+      @io.should_receive(:each_line).
+          and_yield('    gfs | default ').
+          and_yield('notchef | default ').
+          and_yield('  monit | default ').
+          and_yield('  mysql | default ')
+    
+      @provider.load_current_resource
+      @current_resource.enabled.should be_false
+    end
+  
+    it "should return the current_resource" do
+      @provider.load_current_resource.should == @current_resource
+    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('chef')
+      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('chef')
+      end
+    end
+  end
+end