chef 10.12.0 → 10.14.0.beta.1

data/spec/unit/provider/link_spec.rb

@@ -0,0 +1,249 @@
+#
+# Author:: AJ Christensen (<aj@junglist.gen.nz>)
+# Author:: John Keiser (<jkeiser@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 'ostruct'
+
+require 'spec_helper'
+
+if Chef::Platform.windows?
+  require 'chef/win32/file' #probably need this in spec_helper
+end
+
+describe Chef::Resource::Link do
+  let(:provider) do
+    node = Chef::Node.new
+    @events = Chef::EventDispatch::Dispatcher.new
+    run_context = Chef::RunContext.new(node, {}, @events)
+    Chef::Provider::Link.new(new_resource, run_context)
+  end
+  let(:new_resource) do
+    result = Chef::Resource::Link.new("#{CHEF_SPEC_DATA}/fofile-link")
+    result.to "#{CHEF_SPEC_DATA}/fofile"
+    result
+  end
+
+  def canonicalize(path)
+    Chef::Platform.windows? ? path.gsub('/', '\\') : path
+  end
+
+  describe "when the target is a symlink" do
+    before(:each) do
+      lstat = mock("stats", :ino => 5)
+      lstat.stub!(:uid).and_return(501)
+      lstat.stub!(:gid).and_return(501)
+
+      File.stub!(:lstat).with("#{CHEF_SPEC_DATA}/fofile-link").and_return(lstat)
+      provider.file_class.stub!(:symlink?).with("#{CHEF_SPEC_DATA}/fofile-link").and_return(true)
+      provider.file_class.stub!(:readlink).with("#{CHEF_SPEC_DATA}/fofile-link").and_return("#{CHEF_SPEC_DATA}/fofile")
+    end
+
+    describe "to a file that exists" do
+      before do
+        File.stub(:exist?).with("#{CHEF_SPEC_DATA}/fofile-link").and_return(true)
+        new_resource.owner 501 # only loaded in current_resource if present in new
+        new_resource.group 501
+        provider.load_current_resource
+      end
+
+      it "should set the symlink target" do
+        provider.current_resource.target_file.should == "#{CHEF_SPEC_DATA}/fofile-link"
+      end
+      it "should set the link type" do
+        provider.current_resource.link_type.should == :symbolic
+      end
+      it "should update the source of the existing link with the links target" do
+        provider.current_resource.to.should == canonicalize("#{CHEF_SPEC_DATA}/fofile")
+      end
+      it "should set the owner" do
+        provider.current_resource.owner.should == 501
+      end
+      it "should set the group" do
+        provider.current_resource.group.should == 501
+      end
+
+      # We test create in unit tests because there is no other way to ensure
+      # it does no work.  Other create and delete scenarios are covered in
+      # the functional tests for links.
+      context 'when the desired state is identical' do
+        let(:new_resource) do
+          result = Chef::Resource::Link.new("#{CHEF_SPEC_DATA}/fofile-link")
+          result.to "#{CHEF_SPEC_DATA}/fofile"
+          result
+        end
+        it 'create does no work' do
+          provider.access_controls.should_not_receive(:set_all)
+          provider.run_action(:create)
+        end
+      end
+    end
+
+    describe "to a file that doesn't exist" do
+      before do
+        File.stub!(:exist?).with("#{CHEF_SPEC_DATA}/fofile-link").and_return(false)
+        provider.file_class.stub!(:symlink?).with("#{CHEF_SPEC_DATA}/fofile-link").and_return(true)
+        provider.file_class.stub!(:readlink).with("#{CHEF_SPEC_DATA}/fofile-link").and_return("#{CHEF_SPEC_DATA}/fofile")
+        new_resource.owner "501" # only loaded in current_resource if present in new
+        new_resource.group "501"
+        provider.load_current_resource
+      end
+
+      it "should set the symlink target" do
+        provider.current_resource.target_file.should == "#{CHEF_SPEC_DATA}/fofile-link"
+      end
+      it "should set the link type" do
+        provider.current_resource.link_type.should == :symbolic
+      end
+      it "should update the source of the existing link to the link's target" do
+        provider.current_resource.to.should == canonicalize("#{CHEF_SPEC_DATA}/fofile")
+      end
+      it "should not set the owner" do
+        provider.current_resource.owner.should be_nil
+      end
+      it "should not set the group" do
+        provider.current_resource.group.should be_nil
+      end
+    end
+  end
+
+  describe "when the target doesn't exist" do
+    before do
+      File.stub!(:exists?).with("#{CHEF_SPEC_DATA}/fofile-link").and_return(false)
+      provider.file_class.stub!(:symlink?).with("#{CHEF_SPEC_DATA}/fofile-link").and_return(false)
+      provider.load_current_resource
+    end
+
+    it "should set the symlink target" do
+      provider.current_resource.target_file.should == "#{CHEF_SPEC_DATA}/fofile-link"
+    end
+    it "should update the source of the existing link to nil" do
+      provider.current_resource.to.should be_nil
+    end
+    it "should not set the owner" do
+      provider.current_resource.owner.should == nil
+    end
+    it "should not set the group" do
+      provider.current_resource.group.should == nil
+    end
+  end
+
+  describe "when the target is a regular old file" do
+    before do
+      stat = mock("stats", :ino => 5)
+      stat.stub!(:uid).and_return(501)
+      stat.stub!(:gid).and_return(501)
+      provider.file_class.stub!(:stat).with("#{CHEF_SPEC_DATA}/fofile-link").and_return(stat)
+
+      File.stub!(:exists?).with("#{CHEF_SPEC_DATA}/fofile-link").and_return(true)
+      provider.file_class.stub!(:symlink?).with("#{CHEF_SPEC_DATA}/fofile-link").and_return(false)
+    end
+
+    describe "and the source does not exist" do
+      before do
+        File.stub!(:exists?).with("#{CHEF_SPEC_DATA}/fofile").and_return(false)
+        provider.load_current_resource
+      end
+
+      it "should set the symlink target" do
+        provider.current_resource.target_file.should == "#{CHEF_SPEC_DATA}/fofile-link"
+      end
+      it "should update the current source of the existing link with an empty string" do
+        provider.current_resource.to.should == ''
+      end
+      it "should not set the owner" do
+        provider.current_resource.owner.should == nil
+      end
+      it "should not set the group" do
+        provider.current_resource.group.should == nil
+      end
+    end
+
+    describe "and the source exists" do
+      before do
+        stat = mock("stats", :ino => 6)
+        stat.stub!(:uid).and_return(502)
+        stat.stub!(:gid).and_return(502)
+
+        provider.file_class.stub!(:stat).with("#{CHEF_SPEC_DATA}/fofile").and_return(stat)
+
+        File.stub!(:exists?).with("#{CHEF_SPEC_DATA}/fofile").and_return(true)
+        provider.load_current_resource
+      end
+
+      it "should set the symlink target" do
+        provider.current_resource.target_file.should == "#{CHEF_SPEC_DATA}/fofile-link"
+      end
+      it "should update the current source of the existing link with an empty string" do
+        provider.current_resource.to.should == ''
+      end
+      it "should not set the owner" do
+        provider.current_resource.owner.should == nil
+      end
+      it "should not set the group" do
+        provider.current_resource.group.should == nil
+      end
+    end
+
+    describe "and is hardlinked to the source" do
+      before do
+        stat = mock("stats", :ino => 5)
+        stat.stub!(:uid).and_return(502)
+        stat.stub!(:gid).and_return(502)
+
+        provider.file_class.stub!(:stat).with("#{CHEF_SPEC_DATA}/fofile").and_return(stat)
+
+        File.stub!(:exists?).with("#{CHEF_SPEC_DATA}/fofile").and_return(true)
+        provider.load_current_resource
+      end
+
+      it "should set the symlink target" do
+        provider.current_resource.target_file.should == "#{CHEF_SPEC_DATA}/fofile-link"
+      end
+      it "should set the link type" do
+        provider.current_resource.link_type.should == :hard
+      end
+      it "should update the source of the existing link to the link's target" do
+        provider.current_resource.to.should == canonicalize("#{CHEF_SPEC_DATA}/fofile")
+      end
+      it "should not set the owner" do
+        provider.current_resource.owner.should == nil
+      end
+      it "should not set the group" do
+        provider.current_resource.group.should == nil
+      end
+
+      # We test create in unit tests because there is no other way to ensure
+      # it does no work.  Other create and delete scenarios are covered in
+      # the functional tests for links.
+      context 'when the desired state is identical' do
+        let(:new_resource) do
+          result = Chef::Resource::Link.new("#{CHEF_SPEC_DATA}/fofile-link")
+          result.to "#{CHEF_SPEC_DATA}/fofile"
+          result.link_type :hard
+          result
+        end
+        it 'create does no work' do
+          provider.file_class.should_not_receive(:symlink) 
+          provider.file_class.should_not_receive(:link) 
+          provider.access_controls.should_not_receive(:set_all)
+          provider.run_action(:create)
+        end
+      end
+    end
+  end
+end

data/spec/unit/provider/log_spec.rb

@@ -0,0 +1,81 @@
+#
+# Author:: Cary Penniman (<cary@rightscale.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::Log::ChefLog do
+
+  before(:each) do
+    @log_str = "this is my test string to log"
+    @node = Chef::Node.new
+    @events = Chef::EventDispatch::Dispatcher.new
+    @run_context = Chef::RunContext.new(@node, {}, @events)
+  end  
+
+  it "should be registered with the default platform hash" do
+    Chef::Platform.platforms[:default][:log].should_not be_nil
+  end
+
+  it "should write the string to the Chef::Log object at default level (info)" do
+      @new_resource = Chef::Resource::Log.new(@log_str)
+      @provider = Chef::Provider::Log::ChefLog.new(@new_resource, @run_context)
+      Chef::Log.should_receive(:info).with(@log_str).and_return(true)
+      @provider.action_write
+  end
+  
+  it "should write the string to the Chef::Log object at debug level" do
+      @new_resource = Chef::Resource::Log.new(@log_str)
+      @new_resource.level :debug
+      @provider = Chef::Provider::Log::ChefLog.new(@new_resource, @run_context)
+      Chef::Log.should_receive(:debug).with(@log_str).and_return(true)
+      @provider.action_write
+  end
+
+  it "should write the string to the Chef::Log object at info level" do
+      @new_resource = Chef::Resource::Log.new(@log_str)
+      @new_resource.level :info
+      @provider = Chef::Provider::Log::ChefLog.new(@new_resource, @run_context)
+      Chef::Log.should_receive(:info).with(@log_str).and_return(true)
+      @provider.action_write
+  end
+  
+  it "should write the string to the Chef::Log object at warn level" do
+      @new_resource = Chef::Resource::Log.new(@log_str)
+      @new_resource.level :warn
+      @provider = Chef::Provider::Log::ChefLog.new(@new_resource, @run_context)
+      Chef::Log.should_receive(:warn).with(@log_str).and_return(true)
+      @provider.action_write
+  end
+  
+  it "should write the string to the Chef::Log object at error level" do
+      @new_resource = Chef::Resource::Log.new(@log_str)
+      @new_resource.level :error
+      @provider = Chef::Provider::Log::ChefLog.new(@new_resource, @run_context)
+      Chef::Log.should_receive(:error).with(@log_str).and_return(true)
+      @provider.action_write
+  end
+  
+  it "should write the string to the Chef::Log object at fatal level" do
+      @new_resource = Chef::Resource::Log.new(@log_str)
+      @new_resource.level :fatal
+      @provider = Chef::Provider::Log::ChefLog.new(@new_resource, @run_context)
+      Chef::Log.should_receive(:fatal).with(@log_str).and_return(true)
+      @provider.action_write
+  end
+  
+end

data/spec/unit/provider/mdadm_spec.rb

@@ -0,0 +1,131 @@
+#
+# Author:: Joe Williams (<joe@joetify.com>)
+# Copyright:: Copyright (c) 2009 Joe Williams
+# 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'
+require 'ostruct'
+
+describe Chef::Provider::Mdadm do
+
+  before(:each) do
+    @node = Chef::Node.new
+    @events = Chef::EventDispatch::Dispatcher.new
+    @run_context = Chef::RunContext.new(@node, {}, @events)
+
+    @new_resource = Chef::Resource::Mdadm.new('/dev/md1')
+    @new_resource.devices ["/dev/sdz1","/dev/sdz2"]
+    @new_resource.level   1
+    @new_resource.chunk   256
+
+    @provider = Chef::Provider::Mdadm.new(@new_resource, @run_context)
+    #Chef::Resource::Mdadm.stub!(:new).and_return(@current_resource)
+
+    # @status = mock("Status", :exitstatus => 0)
+    # @provider.stub!(:popen4).and_return(@status)
+    # @stdin = mock("STDIN", :null_object => true)
+    # @stdout = mock("STDOUT", :null_object => true)
+    # @stderr = mock("STDERR", :null_object => true)
+    # @pid = mock("PID", :null_object => true)
+  end
+
+  describe "when determining the current metadevice status" do
+
+    it "should set the current resources mount point to the new resources mount point" do
+      @provider.stub!(:shell_out!).and_return(OpenStruct.new(:stdout => ''))
+      @provider.load_current_resource()
+      @provider.current_resource.name.should == '/dev/md1'
+      @provider.current_resource.raid_device.should == '/dev/md1'
+    end
+
+    it "determines that the metadevice exists when mdadm output shows the metadevice" do
+      @provider.stub!(:shell_out!).with("mdadm --detail --scan").and_return(OpenStruct.new(:stdout => '/dev/md1'))
+      @provider.load_current_resource
+      @provider.current_resource.exists.should be_true
+    end
+  end
+
+  describe "after the metadevice status is known" do
+    before(:each) do
+      @current_resource = Chef::Resource::Mdadm.new('/dev/md1')
+      @current_resource.devices ["/dev/sdz1","/dev/sdz2"]
+      @current_resource.level   1
+      @current_resource.chunk   256
+      @provider.stub!(:load_current_resource).and_return(true)
+      @provider.current_resource = @current_resource
+    end
+
+    describe "when creating the metadevice" do
+      it "should create the raid device if it doesnt exist" do
+        @current_resource.exists(false)
+        expected_command = "yes | mdadm --create /dev/md1 --chunk=256 --level 1 --metadata=0.90 --raid-devices 2 /dev/sdz1 /dev/sdz2"
+        @provider.should_receive(:shell_out!).with(expected_command)
+        @provider.run_action(:create)
+      end
+
+      it "should specify a bitmap only if set" do
+        @current_resource.exists(false)
+        @new_resource.bitmap('grow')
+        expected_command = "yes | mdadm --create /dev/md1 --chunk=256 --level 1 --metadata=0.90 --bitmap=grow --raid-devices 2 /dev/sdz1 /dev/sdz2"
+        @provider.should_receive(:shell_out!).with(expected_command)
+        @provider.run_action(:create)
+        @new_resource.should be_updated_by_last_action
+      end
+
+      it "should not create the raid device if it does exist" do
+        @current_resource.exists(true)
+        @provider.should_not_receive(:shell_out!)
+        @provider.run_action(:create)
+        @new_resource.should_not be_updated_by_last_action
+      end
+    end
+
+    describe "when asembling the metadevice" do
+      it "should assemble the raid device if it doesnt exist" do
+        @current_resource.exists(false)
+        expected_mdadm_cmd = "yes | mdadm --assemble /dev/md1 /dev/sdz1 /dev/sdz2"
+        @provider.should_receive(:shell_out!).with(expected_mdadm_cmd)
+        @provider.run_action(:assemble)
+        @new_resource.should be_updated_by_last_action
+      end
+
+        it "should not assemble the raid device if it doesnt exist" do
+        @current_resource.exists(true)
+        @provider.should_not_receive(:shell_out!)
+        @provider.run_action(:assemble)
+        @new_resource.should_not be_updated_by_last_action
+      end
+    end
+
+    describe "when stopping the metadevice" do
+
+      it "should stop the raid device if it exists" do
+        @current_resource.exists(true)
+        expected_mdadm_cmd = "yes | mdadm --stop /dev/md1"
+        @provider.should_receive(:shell_out!).with(expected_mdadm_cmd)
+        @provider.run_action(:stop)
+        @new_resource.should be_updated_by_last_action
+      end
+
+      it "should not attempt to stop the raid device if it does not exist" do
+        @current_resource.exists(false)
+        @provider.should_not_receive(:shell_out!)
+        @provider.run_action(:stop)
+        @new_resource.should_not be_updated_by_last_action
+      end
+    end
+  end
+end