microwave 0.1006.33 → 1.0.0

data/spec/unit/provider/http_request_spec.rb

@@ -0,0 +1,178 @@
+#
+# Author:: Adam Jacob (<adam@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::HttpRequest do
+  before(:each) do
+    @node = Chef::Node.new
+    @events = Chef::EventDispatch::Dispatcher.new
+    @run_context = Chef::RunContext.new(@node, {}, @events)
+
+    @new_resource = Chef::Resource::HttpRequest.new('adam')
+    @new_resource.name "adam"
+    @new_resource.url "http://www.opscode.com"
+    @new_resource.message "is cool"
+
+    @provider = Chef::Provider::HttpRequest.new(@new_resource, @run_context)
+  end
+
+  describe "load_current_resource" do  
+
+    it "should set up a Chef::REST client, with no authentication" do
+      Chef::REST.should_receive(:new).with(@new_resource.url, nil, nil)
+      @provider.load_current_resource
+    end
+  end
+
+  describe "when making REST calls" do
+    before(:each) do
+      # run_action(x) forces load_current_resource to run;
+      # that would overwrite our supplied mock Chef::Rest # object
+      @provider.stub!(:load_current_resource).and_return(true)
+      @rest = mock("Chef::REST", :create_url => "http://www.opscode.com", :run_request => "you made it!" )
+      @provider.rest = @rest
+    end
+
+    describe "action_get" do  
+      it "should create the url with a message argument" do
+        @rest.should_receive(:create_url).with("#{@new_resource.url}?message=#{@new_resource.message}")
+        @provider.run_action(:get)
+      end
+
+      it "should inflate a message block at runtime" do
+        @new_resource.stub!(:message).and_return(lambda { "return" })
+        @rest.should_receive(:create_url).with("#{@new_resource.url}?message=return")
+        @provider.run_action(:get)
+      end
+
+      it "should run a GET request" do
+        @rest.should_receive(:run_request).with(:GET, @rest.create_url, {}, false, 10, false)
+        @provider.run_action(:get)
+      end
+
+      it "should update the resource" do
+        @provider.run_action(:get)
+        @new_resource.should be_updated
+      end
+    end
+
+    describe "action_put" do  
+      it "should create the url" do
+        @rest.should_receive(:create_url).with("#{@new_resource.url}")
+        @provider.run_action(:put)
+      end
+
+      it "should run a PUT request with the message as the payload" do
+        @rest.should_receive(:run_request).with(:PUT, @rest.create_url, {}, @new_resource.message, 10, false)
+        @provider.run_action(:put)
+      end
+
+      it "should inflate a message block at runtime" do
+        @new_resource.stub!(:message).and_return(lambda { "return" })
+        @rest.should_receive(:run_request).with(:PUT, @rest.create_url, {}, "return", 10, false)    
+        @provider.run_action(:put)
+      end
+
+      it "should update the resource" do
+        @provider.run_action(:put)
+        @new_resource.should be_updated
+      end
+    end
+
+    describe "action_post" do  
+      it "should create the url" do
+        @rest.should_receive(:create_url).with("#{@new_resource.url}")
+        @provider.run_action(:post)
+      end
+  
+      it "should run a PUT request with the message as the payload" do
+        @rest.should_receive(:run_request).with(:POST, @rest.create_url, {}, @new_resource.message, 10, false)
+        @provider.run_action(:post)
+      end
+  
+      it "should inflate a message block at runtime" do
+        @new_resource.stub!(:message).and_return(lambda { "return" })
+        @rest.should_receive(:run_request).with(:POST, @rest.create_url, {}, "return", 10, false)    
+        @provider.run_action(:post)
+      end
+  
+      it "should update the resource" do
+        @provider.run_action(:post)
+        @new_resource.should be_updated
+      end
+    end
+
+    describe "action_delete" do  
+      it "should create the url" do
+        @rest.should_receive(:create_url).with("#{@new_resource.url}")
+        @provider.run_action(:delete)
+      end
+
+      it "should run a DELETE request" do
+        @rest.should_receive(:run_request).with(:DELETE, @rest.create_url, {}, false, 10, false)
+        @provider.run_action(:delete)
+      end
+
+      it "should update the resource" do
+        @provider.run_action(:delete)
+        @new_resource.should be_updated
+      end
+    end
+
+    describe "action_head" do
+      before do
+        @rest = mock("Chef::REST", :create_url => "http://www.opscode.com", :run_request => true)
+        @provider.rest = @rest
+      end
+
+      it "should create the url with a message argument" do
+        @rest.should_receive(:create_url).with("#{@new_resource.url}?message=#{@new_resource.message}")
+        @provider.run_action(:head)
+      end
+
+      it "should inflate a message block at runtime" do
+        @new_resource.stub!(:message).and_return(lambda { "return" })
+        @rest.should_receive(:create_url).with("#{@new_resource.url}?message=return")
+        @provider.run_action(:head)
+      end
+
+      it "should run a HEAD request" do
+        @rest.should_receive(:run_request).with(:HEAD, @rest.create_url, {}, false, 10, false)
+        @provider.run_action(:head)
+      end
+
+      it "should update the resource" do
+        @provider.run_action(:head)
+        @new_resource.should be_updated
+      end
+
+      it "should run a HEAD request with If-Modified-Since header" do
+        @new_resource.headers "If-Modified-Since" => File.mtime(__FILE__).httpdate
+        @rest.should_receive(:run_request).with(:HEAD, @rest.create_url, @new_resource.headers, false, 10, false)
+        @provider.run_action(:head)
+      end
+
+      it "doesn't call converge_by if HEAD does not return modified" do
+        @rest.should_receive(:run_request).and_return(false)
+        @provider.should_not_receive(:converge_by)
+        @provider.run_action(:head)
+      end
+    end
+  end
+end

data/spec/unit/provider/ifconfig_spec.rb

@@ -0,0 +1,213 @@
+#
+# Author:: Prajakta Purohit (prajakta@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 File.expand_path(File.join(File.dirname(__FILE__), "..", "..", "spec_helper"))
+require 'spec_helper'
+require 'chef/exceptions'
+
+describe Chef::Provider::Ifconfig do
+  before do
+    @node = Chef::Node.new
+    @cookbook_collection = Chef::CookbookCollection.new([])
+    @events = Chef::EventDispatch::Dispatcher.new
+    @run_context = Chef::RunContext.new(@node, @cookbook_collection, @events)
+    #This new_resource can be called anything --> it is not the same as in ifconfig.rb
+    @new_resource = Chef::Resource::Ifconfig.new("10.0.0.1", @run_context)
+    @new_resource.mask "255.255.254.0"
+    @new_resource.metric "1"
+    @new_resource.mtu "1500"
+    @new_resource.device "eth0"
+    @provider = Chef::Provider::Ifconfig.new(@new_resource, @run_context)
+    @current_resource = Chef::Resource::Ifconfig.new("10.0.0.1", @run_context)
+
+    status = mock("Status", :exitstatus => 0)
+    @provider.instance_variable_set("@status", status)
+    @provider.current_resource = @current_resource
+    
+ end
+  describe Chef::Provider::Ifconfig, "load_current_resource" do 
+    before do 
+      status = mock("Status", :exitstatus => 1)
+      @provider.should_receive(:popen4).and_return status 
+      @provider.load_current_resource
+    end
+    it "should track state of ifconfig failure." do
+      @provider.instance_variable_get("@status").exitstatus.should_not == 0
+    end
+    it "should thrown an exception when ifconfig fails" do 
+      @provider.define_resource_requirements
+      lambda { @provider.process_resource_requirements }.should raise_error Chef::Exceptions::Ifconfig 
+    end
+  end
+  describe Chef::Provider::Ifconfig, "action_add" do
+
+    it "should add an interface if it does not exist" do
+      #@provider.stub!(:run_command).and_return(true)
+      @provider.stub!(:load_current_resource)
+      @current_resource.inet_addr nil
+      command = "ifconfig eth0 10.0.0.1 netmask 255.255.254.0 metric 1 mtu 1500"
+      @provider.should_receive(:run_command).with(:command => command)
+      @provider.should_receive(:generate_config)
+
+      @provider.run_action(:add)
+      @new_resource.should be_updated
+    end
+
+    it "should not add an interface if it already exists" do
+      @provider.stub!(:load_current_resource)
+      @provider.should_not_receive(:run_command)
+      @current_resource.inet_addr "10.0.0.1"
+      @provider.should_receive(:generate_config)
+
+      @provider.run_action(:add)
+      @new_resource.should_not be_updated
+    end
+
+    #We are not testing this case with the assumption that anyone writing the cookbook would not make a typo == lo
+    #it "should add a blank command if the #{@new_resource.device} == lo" do
+    #end
+  end
+
+  describe Chef::Provider::Ifconfig, "action_enable" do
+    
+    it "should enable interface if does not exist" do
+      @provider.stub!(:load_current_resource)
+      @current_resource.inet_addr nil
+      command = "ifconfig eth0 10.0.0.1 netmask 255.255.254.0 metric 1 mtu 1500"
+      @provider.should_receive(:run_command).with(:command => command)
+      @provider.should_not_receive(:generate_config)
+
+      @provider.run_action(:enable)
+      @new_resource.should be_updated
+    end
+
+    it "should not enable interface if it already exists" do
+      @provider.stub!(:load_current_resource)
+      @provider.should_not_receive(:run_command)
+      @current_resource.inet_addr "10.0.0.1"
+      @provider.should_not_receive(:generate_config)
+
+      @provider.run_action(:enable)
+      @new_resource.should_not be_updated
+    end
+  end
+
+  describe Chef::Provider::Ifconfig, "action_delete" do
+
+    it "should delete interface if it exists" do
+      @provider.stub!(:load_current_resource)
+      @current_resource.device "eth0"
+      command = "ifconfig #{@new_resource.device} down"
+      @provider.should_receive(:run_command).with(:command => command)
+      @provider.should_receive(:delete_config)
+
+      @provider.run_action(:delete)
+      @new_resource.should be_updated
+    end
+
+    it "should not delete interface if it does not exist" do
+      @provider.stub!(:load_current_resource)
+      @provider.should_not_receive(:run_command)
+      @provider.should_not_receive(:delete_config)
+
+      @provider.run_action(:delete)
+      @new_resource.should_not be_updated
+    end
+  end
+  
+  describe Chef::Provider::Ifconfig, "action_disable" do
+
+    it "should disable interface if it exists" do
+      @provider.stub!(:load_current_resource)
+      @current_resource.device "eth0"
+      command = "ifconfig #{@new_resource.device} down"
+      @provider.should_receive(:run_command).with(:command => command)
+      @provider.should_not_receive(:delete_config)
+
+      @provider.run_action(:disable)
+      @new_resource.should be_updated
+    end
+
+    it "should not delete interface if it does not exist" do
+      @provider.stub!(:load_current_resource)
+      @provider.should_not_receive(:run_command)
+      @provider.should_not_receive(:delete_config)
+
+      @provider.run_action(:disable)
+      @new_resource.should_not be_updated
+    end
+  end
+
+  describe Chef::Provider::Ifconfig, "action_delete" do
+
+    it "should delete interface of it exists" do
+      @provider.stub!(:load_current_resource)
+      @current_resource.device "eth0"
+      command = "ifconfig #{@new_resource.device} down"
+      @provider.should_receive(:run_command).with(:command => command)
+      @provider.should_receive(:delete_config)
+
+      @provider.run_action(:delete)
+      @new_resource.should be_updated
+    end
+
+    it "should not delete interface if it does not exist" do
+      # This is so that our fake values do not get overwritten
+      @provider.stub!(:load_current_resource)
+      # This is so that nothing actually runs
+      @provider.should_not_receive(:run_command)
+      @provider.should_not_receive(:delete_config)
+
+      @provider.run_action(:delete)
+      @new_resource.should_not be_updated
+    end
+  end
+  
+  describe Chef::Provider::Ifconfig, "generate_config for action_add" do
+   #%w[ centos redhat fedora ].each do |platform|
+     
+     it "should write network-script for centos" do
+      @provider.stub!(:load_current_resource)
+      @node[:platform] = "centos"
+      @provider.stub!(:run_command)
+      config_filename = "/etc/sysconfig/network-scripts/ifcfg-#{@new_resource.device}"
+      config_file = StringIO.new
+      File.should_receive(:new).with(config_filename, "w").and_return(config_file)
+
+      @provider.run_action(:add)
+      config_file.string.should match(/^\s*DEVICE=eth0\s*$/)
+      config_file.string.should match(/^\s*IPADDR=10.0.0.1\s*$/)
+      config_file.string.should match(/^\s*NETMASK=255.255.254.0\s*$/)
+     end
+  end
+
+  describe Chef::Provider::Ifconfig, "delete_config for action_delete" do
+
+    it "should delete network-script if it exists for centos" do
+      @node[:platform] = "centos"
+      @current_resource.device "eth0"
+      @provider.stub!(:load_current_resource)
+      @provider.stub!(:run_command)
+      config_filename =  "/etc/sysconfig/network-scripts/ifcfg-#{@new_resource.device}"
+      File.should_receive(:exist?).with(config_filename).and_return(true)
+      FileUtils.should_receive(:rm_f).with(config_filename, :verbose => false)
+
+      @provider.run_action(:delete)
+    end
+  end
+end

data/spec/unit/provider/link_spec.rb

@@ -0,0 +1,252 @@
+#
+# 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)
+      lstat.stub!(:mode).and_return(0777)
+      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)
+      stat.stub!(:mode).and_return(0755)
+      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)
+        stat.stub!(:mode).and_return(0644)
+
+        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)
+        stat.stub!(:mode).and_return(0644)
+
+        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